Project

General

Profile

Backport(バックポート) #1761

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

Added by Masato Nagasawa almost 10 years ago. Updated over 9 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-11-08
Due date:
% Done:

100%


Description

再現バージョン

  • 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() も行わないように修正を行う。


Related issues

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

Associated revisions

Revision 5fa8dfcf (diff)
Added by Masato Nagasawa almost 10 years ago

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

Revision 4e2cbef4 (diff)
Added by Masato Nagasawa over 9 years ago

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

History

#1 Updated by Masato Nagasawa almost 10 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#2 Updated by Kousuke Ebihara almost 10 years ago

  • Target version changed from OpenPNE 3.6beta7 to OpenPNE 3.6beta8

#3 Updated by Kousuke Ebihara almost 10 years ago

  • Target version changed from OpenPNE 3.6beta8 to OpenPNE3.6beta9

#4 Updated by Kousuke Ebihara over 9 years ago

  • Target version changed from OpenPNE3.6beta9 to OpenPNE3.6beta11

#5 Updated by Masato Nagasawa over 9 years ago

  • Target version changed from OpenPNE3.6beta11 to OpenPNE 3.6.0

#6 Updated by Masato Nagasawa over 9 years ago

  • Target version changed from OpenPNE 3.6.0 to OpenPNE3.6beta11

#7 Updated by Kousuke Ebihara over 9 years ago

  • Target version changed from OpenPNE3.6beta11 to OpenPNE3.6beta12

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

#8 Updated by Kousuke Ebihara over 9 years ago

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

#9 Updated by Masato Nagasawa over 9 years ago

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

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

#10 Updated by Kousuke Ebihara over 9 years ago

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

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

#11 Updated by Minoru Takai over 9 years ago

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

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

#12 Updated by isao sano over 9 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

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

Also available in: Atom PDF