Repository navigation
Conversation
dumpError()がheader.inc.php(完全なHTML・メニュー・ツリー)を#mainPane内へ 挿入し、画面崩れと「読み込み中」残りを起こしていた。断片要求では出力を省略し、 通知後の遷移もEvoShell.navigateで行う。 Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FVRJfjddiH9DeArnkyHvEt
|
Warning Review limit reachedNext included review available in 12 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Repository: modxcms-jp/evolution-jp/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (3)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1aa09f6c57
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FVRJfjddiH9DeArnkyHvEt
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
pane判定前のエラーとモーダル内エラーで画面崩れやオーバーレイ残留が発生する。
Review effort: Balanced
Findings: 2
Open (2)
What changed in this PR
AJAX断片応答時のエラー表示によるHTML入れ子化を防止する変更。
- pane request識別用定数を追加
- エラー応答から完全HTMLを除外
- EvoShell経由のエラー後遷移を追加
| File | Description |
|---|---|
manager/index.php |
pane request識別定数を定義 |
manager/includes/error.class.inc.php |
pane向け断片エラー応答を生成 |
manager/media/style/common/dump_error.tpl |
EvoShellナビゲーションを追加 |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FVRJfjddiH9DeArnkyHvEt
|
#488 で作り直したため、クローズします。 #483 は
#488 は、header.inc.php / footer.inc.php の側で、応答ヘッダー |
概要
AJAX断片応答（pane request）でエラーが発生した場合、header/footerを含む完全なHTMLが出力されることで、
#mainPane内に<html>タグやメニュー・ツリーが入れ子で挿入され、画面崩れとローディング残りが発生していた問題を修正します。変更内容
EVO_PANE_REQUEST定数を定義し、エラーハンドラが断片応答であることを認識できるようにしましたEVO_PANE_REQUEST定数の有無で、header/footerの出力を制御evoRenderPaneFooterExtras()を呼び出しEvoShell.navigate()を使用してナビゲーションtop.main.document.location.hrefを保持確認手順
備考
EVO_PANE_REQUEST定数はpane request時のみ定義され、エラーハンドラ内で条件分岐の判定に使用されますisEvoPaneRequest()関数との重複を避けるため、定数として別途定義していますhttps://claude.ai/code/session_01FVRJfjddiH9DeArnkyHvEt