Skip to content

Conversation

@arafubeatbox
Copy link
Contributor

@arafubeatbox arafubeatbox commented Nov 30, 2025

What

ページ取得部分でデグレが生じたことで revert された #10532 を修正し、再適用するための PR。

デグレの内容

どのページ取得結果も 404 となる。

デグレの原因

45e8b16 のコミットでページ取得 API 内の pages 変数にデフォルト値として空の配列を設定したが、pages が undefined ではなく空の配列の場合は「ページが見つからなかった」という結果にする仕様だった。

修正内容

https://github.com/growilabs/growi/pull/10555/files/fe1dfce0a16f81b3b7d65bf4a545bf33ed2ff4e4..225dd23a7a50bf1fa98dace730708d0b02c6fd2e
ページ検索が findAll かそうでない場合かの処理を分離し、pages パラメータの定義やその結果による処理を findAll: true の文脈で完結するように修正。
これにより、findAll: false の文脈のために pages を undefined で定義しておく必要がなくなる。

動作確認

  • 通常のページを表示できることを確認
  • findAll の検索が正常に実行されることを確認
    • ただし、findAll の実行はパスの重複がある場合に指定されたパスのページを全て取得するためのオプションであり、dev/7.4.x では isIdenticalPathAtom をセットしている箇所がなく、そもそも重複がある場合でも検索が行われないようになっていた (動作確認は無理やり isIdenticalPathAtom をセットして行った)
  • isSharedPage が指定された場合の shareLinkId によるページ取得が正常に動作することを確認
    • ただし dev/7.4.x でこれが指定されているリクエスト箇所は見つけることができなかった。

task

https://redmine.weseek.co.jp/issues/175049

…82-app-apiv3-routes-biome-2"

This reverts commit a0f5481.
@arafubeatbox arafubeatbox changed the title Reapply "Merge pull request #10532 from growilabs/support/156162-1740… support: Reapply biome configuration for app apiv3 routes (app-settings, page) Dec 1, 2025
@arafubeatbox arafubeatbox marked this pull request as ready for review December 2, 2025 14:00
@yuki-takei yuki-takei merged commit 94a6b55 into dev/7.4.x Dec 4, 2025
48 of 50 checks passed
@yuki-takei yuki-takei deleted the support/156162-175049-app-apiv3-routes-biome-2-fixed branch December 4, 2025 10:38
@github-actions github-actions bot mentioned this pull request Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants