Backport（バックポート） #3459
opI18N::generateApplicationMessages() から呼ばれる sfMessageSource::factory() の引数の型が適切でない
Start date:
2013-10-07
Due date:
% Done:
100%
Description
Overview (現象)¶
opI18N::generateApplicationMessages() 内の下記コードで sfMessageSource::factory() の第 2 引数に array が渡されているが、sfMessageSource_File::__construct() にて引数をファイル名として string 型にキャストするため E_NOTICE レベルのエラーが発生する。
$messageSource = sfMessageSource::factory('OpenPNE', array());
$data = $messageSource->loadData($file);
Way to fix (修正内容)¶
sfMessageSource::factory() の第 2 引数は省略可能であるため array() の記述を削除する
Related issues
Associated revisions
Merge branches 't-3418', 't-3420', 't-3422', 't-3459', 't-3460', 't-3461', 't-3462', 't-3463', 't-3465' and 't-3471' into stable-3.8.x
History
#1
Updated by Akihiro KOBAYASHI almost 7 years ago
Updated by - Status changed from New（新規） to Accepted（着手）
- Assignee set to Akihiro KOBAYASHI
#2
Updated by Akihiro KOBAYASHI almost 7 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
pull request:https://github.com/openpne/OpenPNE3/pull/86
#3
Updated by Youichi Kimura almost 7 years ago
Updated by 更新履歴 ebf080f0d20e8a5a5376650639f98716e88cc2eb で適用されました。
#4
Updated by Youichi Kimura almost 7 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
#5
Updated by Mutsumi Imamura almost 7 years ago
Updated by - Status changed from Pending Testing（テスト待ち） to Fixed（完了）
- % Done changed from 70 to 100
./symfony test:all を実行し異常終了しないことを確認しました。
ブラウザからの動作テストも問題ありません。