Backport（バックポート） #2296
完了
/default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される
Yuma Sakata さんが13年以上前に追加.
13年以上前に更新.
説明
Overview (現象)¶
/default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される。
動作として特に問題があるわけではないが、OpenPNEのアプリケーションとして想定していない画面が表示されてしまうのはあまり適切ではないと思われる。
再現バージョン¶
再現手順¶
1.pc_frontendにログインする
2.下記のURLにアクセスする
default/index
default/module
default/error404
default/login
default/disabled
default/customizingCss ← このURLはホワイトアウトする(devモードでも通常でも)
3.symfonyのデフォルト画面が表示される
Causes (原因)¶
ルーティングルールでdefaultモジュールがすべて有効な状態になっている。
Way to fix (修正内容)¶
調査結果¶
ackによる調査
- default/index
主にユニットテストでの使用のため問題なし。
- default/module
スケルトン作成タスク用のテンプレートで用いられているのみのため動作には問題なし。
- default/error404
主にユニットテストでの使用のため問題なし。
- default/login
pc_backendで用いられているように見えるが、バックポート元と同じ対処を行った場合でも動作には問題がないように思われる。
- default/disabled
ユニットテストでの使用のため問題なし。
- default/customizingCss
該当なし。
関連するチケット
2 (0件未完了 — 2件完了)
- 担当者 を削除 (
Masato Nagasawa)
- 担当者 を Mutsumi Imamura にセット
- 担当者 を削除 (
Mutsumi Imamura)
- ステータス を New（新規） から Accepted（着手） に変更
- 担当者 を Yuya Watanabe にセット
本チケットの対象バージョンにおいてはバックポート元の修正方針では不備が生じる可能性があるため再調査を行う必要があります。
バックポート元の調査方法が不明だったため、バックポート元の調査結果に基づいた調査としてforwardやリンクが作成されそうな場所についてマッチするような検索を行いました。以下に調査結果を示します。しかし、下記の調査方法ではバックポート元で示されるような結果が得られなかったため調査方法について検討する必要があると思います。
ackによる調査¶
- default/login
$ ack "'default', *'login'|default/login" -l
lib/vendor/symfony/test/functional/genericTest.php
apps/pc_backend/modules/default/templates/loginSuccess.php
apps/pc_backend/modules/default/actions/logoutAction.class.php
test/functional/pc_backend/snsActionsTest.php
test/functional/pc_backend/designActionsTest.php
pc_backendで用いられているように見えるが、バックポート元と同じ対処を行った場合でも動作には問題がないようにみえた。（要検証）
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
Seleniumにてテストケースを作成し、対象のページにアクセスした際の動作結果は以下の通りです。
■mobile_frontend¶
| No. |
確認したページ |
期待結果 |
結果 |
| 1 |
default/csrfError |
アクセスエラーページに遷移する |
○ |
| 2 |
default/disabled |
アクセスエラーページに遷移する |
○ |
| 3 |
default/error404 |
アクセスエラーページに遷移する |
○ |
| 4 |
default/error |
アクセスエラーページに遷移する |
○ |
| 5 |
default/index |
アクセスエラーページに遷移する |
○ |
| 6 |
default/login |
アクセスエラーページに遷移する |
○ |
| 7 |
default/module |
アクセスエラーページに遷移する |
○ |
| 8 |
default/nonMobileError |
アクセスエラーページに遷移する |
○ |
| 9 |
default/privacyPolicy |
アクセスエラーページに遷移する |
○ |
| 10 |
default/secure |
アクセスエラーページに遷移する |
○ |
| 11 |
default/urlProxy |
アクセスエラーページに遷移する |
○ |
| 12 |
default/userAgreement |
アクセスエラーページに遷移する |
○ |
■pc_frontend¶
| No. |
確認したページ |
期待結果 |
結果 |
| 1 |
default/csrfError |
アクセスエラーページに遷移する |
○ |
| 2 |
default/customizingCss |
アクセスエラーページに遷移する |
○ |
| 3 |
default/disabled |
アクセスエラーページに遷移する |
○ |
| 4 |
default/error404 |
アクセスエラーページに遷移する |
○ |
| 5 |
default/index |
アクセスエラーページに遷移する |
○ |
| 6 |
default/login |
アクセスエラーページに遷移する |
○ |
| 7 |
default/module |
アクセスエラーページに遷移する |
○ |
| 8 |
default/search |
アクセスエラーページに遷移する |
○ |
| 9 |
default/secure |
アクセスエラーページに遷移する |
○ |
| 10 |
default/privacyPolicy |
プライバシーポリシーのページに遷移する |
○ |
| 11 |
default/userAgreement |
利用規約のページ遷移するに遷移する |
○ |
■pc_backend¶
| No. |
確認したページ |
期待結果 |
結果 |
| 1 |
csrfError |
アクセスエラーページに遷移する |
○ |
| 2 |
default/customizingCss |
アクセスエラーページに遷移する |
○ |
| 3 |
default/disabled |
アクセスエラーページに遷移する |
○ |
| 4 |
default/error404 |
アクセスエラーページに遷移する |
○ |
| 5 |
default/index |
アクセスエラーページに遷移する |
○ |
| 6 |
default/login |
アクセスエラーページに遷移する |
○ |
| 7 |
default/module |
アクセスエラーページに遷移する |
○ |
| 8 |
default/search |
アクセスエラーページに遷移する |
○ |
| 9 |
default/secure |
アクセスエラーページに遷移する |
○ |
| 10 |
default/top |
アクセスエラーページに遷移する |
○ |
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
レビューしました。
バージョン 3.4.x に限らず、この修正内容は修正方針として妥当だと思います。仮にデフォルトルートを用いている箇所があったとしても、この修正内容を変更するのではなく、デフォルトルートを用いている側を修正することが適切でしょう。 stable-3.4.x への修正内容の取り込みも問題なく行なわれていることを確認しました。修正 OK です。
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
他の形式にエクスポート: Atom
PDF
