Project

General

Profile

Backport(バックポート) #2550

翻訳ファイルの優先順位がキャッシュ時に逆になっている

Added by Maki Takahashi over 7 years ago. Updated over 7 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2011-10-14
Due date:
% Done:

100%


Description

概要

i18n 翻訳用のメッセージファイルは複数ファイルで同じ語に対する翻訳を記述した場合、「モジュール → アプリケーション → 全体」の優先順に読み込まれる仕様であるが、翻訳ファイルのキャッシュを生成する過程の処理に誤りがあり、この優先順位が逆になってしまう問題がある。

これにより mobile_frontend 用の i18n ファイルに半角カタカナのエントリーを追加しているにもかかわらず、全角カタカナが表示されてしまうような現象が発生する。

なお、キャッシュファイルがない状態でアクセスした場合にはこの現象は発生しない。

詳細

opI18N::generateApplicationMessages() でキャッシュファイルを生成するが、この中で複数の i18n のメッセージファイルの内容をマージする処理で優先度の低いファイルの内容を優先してマージをしてしまっている。

opI18N::generateApplicationMessages() の引数 $dirs には優先度の高い順にディレクトリパスが格納された配列が渡されるが、foreach のループ内では後の値を前の値よりも優先させる形で array_merge() を使っているために本来の優先順位と逆の内容のキャッシュファイルが生成されていた。

修正前のコード:

  public function generateApplicationMessages($dirs)
  {
    $catalogues = array();
...
    foreach ($files as $file)
    {
...
      $data = $messageSource->loadData($file);

      $catalogues[$name] = array_merge($catalogues[$name], $data);
    }

確認バージョン

OpenPNE 3.7.0-dev (master)
OpenPNE 3.6.1 (stable-3.6.x)


Related issues

Related to OpenPNE 3 - Bug(バグ) #2490: 翻訳ファイルの優先順位がキャッシュ時に逆になっている Fixed(完了) 2011-10-14

Associated revisions

Revision d2a27414 (diff)
Added by Maki Takahashi over 7 years ago

(fixes #2550, BP from #2490) fixed merge order in i18n

Revision 75ac98e5 (diff)
Added by Maki Takahashi over 7 years ago

(refs #2550, BP from #2490) fixed to get correct order i18n dirs

Revision 53332913 (diff)
Added by Maki Takahashi over 7 years ago

(fixes #2550, BP from #2490) fixed to get correct order i18n dirs

History

#1 Updated by Maki Takahashi over 7 years ago

  • Target version changed from OpenPNE 3.6.1 to OpenPNE 3.6.x

#2 Updated by Minoru Takai over 7 years ago

  • Priority changed from Normal(通常) to High(高め)

親チケットに併せてステータスを変更します。

#3 Updated by Yuya Watanabe over 7 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.2

#4 Updated by Maki Takahashi over 7 years ago

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

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

#5 Updated by Kousuke Ebihara over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

http://redmine.openpne.jp/issues/2490#note-6 親チケットに従ってステータスを変更します。

#6 Updated by Maki Takahashi over 7 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 53332913db4a9743da6ce2684c3f3b6252da7d44 で適用されました。

#7 Updated by Kousuke Ebihara over 7 years ago

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

#8 Updated by Yuma Sakata over 7 years ago

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

テストOKです。

Also available in: Atom PDF