Actions
Backport(バックポート) #3814
closedデフォルトの OpenPNE.yml によって sf_factory_storage が設定されるため、 test 環境用のセッションストレージである sfSessionTestStorage が利用されず、セッションが発行されなくなる
Start date:
2013-09-04
Due date:
% Done:
100%
Estimated time:
Description
現象¶
セッションを database に保持する設定とした場合、デフォルトの OpenPNE.yml 設定によって sf_factory_storage として opPDODatabaseSessionStorage が設定される。これによって、 factories.yml の設定を変更することなく、このセッションストレージクラスが使用されることになる。
だが、これには、 test 環境でもこのストレージクラスが使用されてしまうという問題がある。 symfony デフォルトの factories.yml においては、 test 環境用の sfSessionTestStorage を使用するような配慮がなされているが、 OpenPNE.yml のデフォルト設定によって無力化してしまう。これによって、
- 機能テストの出力の存在をストレージクラスが考慮せず、セッションの開始が拒否される
- セッションの継続ができない
などといった問題が生じる。
原因¶
バグが発生した原因を記入
修正案¶
解決策としては「sf_factory_storage の設定内容に応じた test 環境用のラッパークラスを作成する」「test 環境では OpenPNE.yml のセッションストレージの上書きを無効にする」のどちらかを選択することになるが、実装の容易さ等を考慮し、後者を選択することが適当と思われる。
Files
Updated by Chiharu Nakajima over 9 years ago
- Copied from Bug(バグ) #3391: デフォルトの OpenPNE.yml によって sf_factory_storage が設定されるため、 test 環境用のセッションストレージである sfSessionTestStorage が利用されず、セッションが発行されなくなる added
Updated by Chiharu Nakajima over 9 years ago
- Status changed from New(新規) to Pending Testing(テスト待ち)
- % Done changed from 0 to 70
プルリクエストしました。
https://github.com/openpne/OpenPNE3/pull/259
Updated by Shinichi Urabe over 9 years ago
- Status changed from Pending Testing(テスト待ち) to Pending Review(レビュー待ち)
- % Done changed from 70 to 50
ステータスを多分間違えているようです
Updated by Shinichi Urabe over 9 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
レビューOK
Updated by isao sano over 9 years ago
- Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
- % Done changed from 70 to 80
単体試験完了しました。
問題ありません。
Updated by Shinichi Urabe over 9 years ago
- Status changed from Pending Merge(マージ待ち) to Fixed(完了)
- % Done changed from 80 to 100
merged
Actions