プロジェクト

全般

プロフィール

Backport(バックポート) #2296

/default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される

Yuma Sakataほぼ13年前に追加. 12年以上前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
担当者:
対象バージョン:
開始日:
2011-03-30
期日:
進捗率:

100%


説明

Overview (現象)

/default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される。
動作として特に問題があるわけではないが、OpenPNEのアプリケーションとして想定していない画面が表示されてしまうのはあまり適切ではないと思われる。

再現バージョン

  • OpenPNE3.4.15-dev

再現手順

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/secure
    該当なし。
  • default/customizingCss
    該当なし。

関連するチケット

関連している OpenPNE 3 - Bug(バグ) #1987: /default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される Fixed(完了) 2011-03-30
関連している OpenPNE 3 - Backport(バックポート) #2088: /default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される Fixed(完了) 2011-03-30 2011-06-24

関係しているリビジョン

リビジョン a28d9bad (差分)
Yuya Watanabe12年以上前に追加

fixed had access the not defined the default modules in the routing rules(fixes #2296, BP from #1987)

履歴

#1 Yuma Sakataほぼ13年前に更新

  • 担当者 を削除 (Masato Nagasawa)

#2 Mutsumi Imamura12年以上前に更新

  • 担当者Mutsumi Imamura にセット

#3 Mutsumi Imamura12年以上前に更新

  • 担当者 を削除 (Mutsumi Imamura)

#4 Yuya Watanabe12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Yuya Watanabe にセット

#5 Yuya Watanabe12年以上前に更新

本チケットの対象バージョンにおいてはバックポート元の修正方針では不備が生じる可能性があるため再調査を行う必要があります。

#6 Yuya Watanabe12年以上前に更新

バックポート元の調査方法が不明だったため、バックポート元の調査結果に基づいた調査としてforwardやリンクが作成されそうな場所についてマッチするような検索を行いました。以下に調査結果を示します。しかし、下記の調査方法ではバックポート元で示されるような結果が得られなかったため調査方法について検討する必要があると思います。

ackによる調査

  • default/index
    $ ack "'default', *'index'|default/index" -l
    lib/vendor/symfony/test/unit/routing/sfPatternRoutingTest.php
    lib/vendor/symfony/test/functional/genericTest.php
    test/functional/pc_backend/defaultActionsTest.php
    test/functional/mobile_frontend/defaultActionsTest.php
    

    主にユニットテストでの使用のため問題なし。
  • default/module
    $ ack "'default', *'module'|default/module" -l
    lib/task/skeleton/module/module/actions/actions.class.php
    lib/vendor/symfony/lib/task/generator/skeleton/module/module/actions/actions.class.php
    

    スケルトン作成タスク用のテンプレートで用いられているのみのため動作には問題なし。
  • default/error404
    $ ack "'default', *'error404'|default/error404" -l
    lib/vendor/symfony/test/functional/prodTest.php
    

    主にユニットテストでの使用のため問題なし。
  • 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で用いられているように見えるが、バックポート元と同じ対処を行った場合でも動作には問題がないようにみえた。(要検証)
  • default/disabled
    $ ack "'default', *'disabled'|default/disabled" -l
    lib/vendor/symfony/test/functional/genericTest.php
    

    ユニットテストでの使用のため問題なし。
  • default/secure
    $ ack "'default', *'secure'|default/secure" -l
    

    該当なし。
  • default/customizingCss
    $ ack "'default', *'customizingCss'|default/customizingCss" -l
    

    該当なし。

#7 wa ta12年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 a28d9bad753f83710a2051bdb598cebcd49c320f で適用されました。

#8 Mutsumi Imamura12年以上前に更新

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 アクセスエラーページに遷移する

#9 Minoru Takai12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

レビューしました。

バージョン 3.4.x に限らず、この修正内容は修正方針として妥当だと思います。仮にデフォルトルートを用いている箇所があったとしても、この修正内容を変更するのではなく、デフォルトルートを用いている側を修正することが適切でしょう。 stable-3.4.x への修正内容の取り込みも問題なく行なわれていることを確認しました。修正 OK です。

#10 Mutsumi Imamura12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

http://redmine.openpne.jp/issues/2296#note-8 でテスト済みです。
問題ありあませんので完了とします。

他の形式にエクスポート: Atom PDF