プロジェクト

全般

プロフィール

Backport(バックポート) #1761

occurs "warning" when second argument of the __() i18n translation method is non-array (i18nの翻訳メソッド__の第二引数に配列以外を渡した場合に warning が発生する)

Masato Nagasawa13年以上前に追加. 12年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2010-11-08
期日:
進捗率:

100%


説明

再現バージョン

  • OpenPNE3: OpenPNE3.6beta7-dev

概要

この現象は以下のURIにて確認可能です。

pc_frontend_dev.php/connection/new

opToolkit::retrieveAPIList() にて下記のように第二引数に null を指定している箇所がある。

if ($isWithI18n)
{
  $caption = $i18n->__($caption, null, 'api');
}

opI18N::__() の問題の箇所で foreach() をしているため warning となっている。

foreach ($args as $k => $v)
{
  if ($v instanceof SnsTerm)
  {
    $args[$k] = (string)$v;
  }
}

さらに return 時に array_merge を行っているため、この対応も必要である。

return parent::__($string, array_merge($this->parsed[$string], $args), $catalogue);

修正方針

第二引数が配列でない場合に foreach() を処理しないようにし、
同様に array_merge() も行わないように修正を行う。


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #1759: occurs "warning" when second argument of the __() i18n translation method is non-array (i18nの翻訳メソッド__の第二引数に配列以外を渡した場合に warning が発生する) Fixed(完了) 2010-11-08

関係しているリビジョン

リビジョン 5fa8dfcf (差分)
Masato Nagasawa13年以上前に追加

fixed non-array (fixes #1761, BP from #1759)

リビジョン 4e2cbef4 (差分)
Masato Nagasawa12年以上前に追加

fixed non-array (fixes #1761, BP from #1759)

履歴

#1 Masato Nagasawa13年以上前に更新

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

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

#2 Kousuke Ebihara13年以上前に更新

  • 対象バージョンOpenPNE 3.6beta7 から OpenPNE 3.6beta8 に変更

#3 Kousuke Ebihara約13年前に更新

  • 対象バージョンOpenPNE 3.6beta8 から OpenPNE3.6beta9 に変更

#4 Kousuke Ebiharaほぼ13年前に更新

  • 対象バージョンOpenPNE3.6beta9 から OpenPNE3.6beta11 に変更

#5 Masato Nagasawaほぼ13年前に更新

  • 対象バージョンOpenPNE3.6beta11 から OpenPNE 3.6.0 に変更

#6 Masato Nagasawaほぼ13年前に更新

  • 対象バージョンOpenPNE 3.6.0 から OpenPNE3.6beta11 に変更

#7 Kousuke Ebiharaほぼ13年前に更新

  • 対象バージョンOpenPNE3.6beta11 から OpenPNE3.6beta12 に変更

親チケットが「テスト待ち」「完了」でないものをすべて beta12 以降での対応とします。

#8 Kousuke Ebihara12年以上前に更新

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

#9 Masato Nagasawa12年以上前に更新

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

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

#10 Kousuke Ebihara12年以上前に更新

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

http://redmine.openpne.jp/issues/1761#note-9 のリリースブランチへの取り込みでステータスが「レビュー待ち」に巻き戻ってしまいましたので、もとに戻します。

#11 Minoru Takai12年以上前に更新

  • 題名occurs "waning" when second arguments of the __ i18n translation method is non-array (i18nの翻訳メソッド__の第二引数に配列以外を渡した場合に warning が発生する) から occurs "warning" when second argument of the __() i18n translation method is non-array (i18nの翻訳メソッド__の第二引数に配列以外を渡した場合に warning が発生する) に変更

チケットタイトルの文字列を一部修正しました。

#12 isao sano12年以上前に更新

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

テスト完了しました。
問題は発見されませんでしたので、このチケットをFixed(完了)とします。

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