Project

General

Profile

Bug(バグ) #3424

Task(タスク) #3403: OpenPNE の既存のテストコードが正常に動作するように修正を行う

opI18N::generateApplicationMessages() から呼ばれる sfMessageSource::factory() の引数の型が適切でない

Added by Youichi Kimura about 6 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Target version:
Start date:
2013-10-07
Due date:
% Done:

0%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

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

Related to OpenPNE 3 - Backport(バックポート) #3433: opI18N::generateApplicationMessages() から呼ばれる sfMessageSource::factory() の引数の型が適切でない Fixed(完了) 2013-10-07
Related to OpenPNE 3 - Backport(バックポート) #3459: opI18N::generateApplicationMessages() から呼ばれる sfMessageSource::factory() の引数の型が適切でない Fixed(完了) 2013-10-07

Associated revisions

Revision 015897d3 (diff)
Added by Youichi Kimura about 6 years ago

fix `Array to string conversion' error in opI18N (fixes #3424)

History

#1 Updated by Youichi Kimura about 6 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#2 Updated by Shinichi Urabe about 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです。

#4 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • % Done changed from 70 to 0

OpenPNE 3.8.9 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF