プロジェクト

全般

プロフィール

Backport(バックポート) #4094

Bug(バグ) #4093: api アプリケーションにおいて opI18n クラスが使用されていないため、SnsTerm が翻訳されない

api アプリケーションにおいて opI18n クラスが使用されていないため、SnsTerm が翻訳されない

Shinichi Urabe7年以上前に追加. 約5年前に更新.

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

100%


説明

現象

#3596 の修正中に発覚

原因

apps/api/config/factories.yml において opI18n クラスが使用される設定となっていない

修正内容

apps/api/config/factories.yml において opI18n クラスを使用する設定にし、
lib/i18n/opI18N.class.php において、 api アプリケーションの場合は、 pc_frontend の SnsTerm を使用するように設定

i18n-opMessagePlugin.png 表示 (113 KB) Shinichi Urabe, 2016-12-27 19:46


関連するチケット

コピー先 OpenPNE 3 - Backport(バックポート) #4095: api アプリケーションにおいて opI18n クラスが使用されていないため、SnsTerm が翻訳されない Fixed(完了) 2016-12-27

関係しているリビジョン

リビジョン 1592af98 (差分)
Shinichi Urabe約7年前に追加

(refs #4094, BP from #4093) use opI18n class in api application.

リビジョン 8ff6d7cb (差分)
Shinichi Urabe約7年前に追加

(refs #4094, BP from #4093) add opI18N test case for api application.

リビジョン 11ee00f8 (差分)
Shinichi Urabe約7年前に追加

(refs #4094, BP from #4093) Fix cording standard. - Control statements must have a single space before the opening parenthesis of the conditional.

リビジョン 52558cf6 (差分)
Shinichi Urabe約7年前に追加

(refs #4094, BP from #4093) use opI18n class in api application in template.

リビジョン de6a33d9 (差分)
Shinichi Urabe約7年前に追加

(refs #4094, BP from #4093) Fix test case comment titleize (NG) -> fronting (OK)

リビジョン 59d3dfd3 (差分)
Shinichi Urabe約7年前に追加

(refs #4094, BP from #4093) Add ja_JP test case.

リビジョン ced3d594
kaoru n約5年前に追加

Merge pull request #405 from ShinichiU/t-4094

(refs #4094, BP from #4093) use opI18n class in api application.

履歴

#1 Shinichi Urabe7年以上前に更新

  • コピー元 Bug(バグ) #4093: api アプリケーションにおいて opI18n クラスが使用されていないため、SnsTerm が翻訳されない を追加

#2 Shinichi Urabe7年以上前に更新

#3 Shinichi Urabe7年以上前に更新

  • コピー元 を削除 (Bug(バグ) #4093: api アプリケーションにおいて opI18n クラスが使用されていないため、SnsTerm が翻訳されない)

#4 Shinichi Urabe7年以上前に更新

  • 説明 を更新 (diff)
  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

#6 Youichi Kimura約7年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

#4093-10 と同様の理由で差し戻します。

#7 Shinichi Urabe約7年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

テストコードを追加しました
https://github.com/openpne/OpenPNE3/pull/405

#8 Youichi Kimura約7年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

#4093-13 と同様の理由により差し戻します。

#9 Shinichi Urabe約7年前に更新

  • ステータスRejected(差し戻し) から Pending Merge(マージ待ち) に変更
  • 進捗率50 から 80 に変更

#10 Youichi Kimura約7年前に更新

  • ステータスPending Merge(マージ待ち) から Rejected(差し戻し) に変更
  • 進捗率80 から 50 に変更

#4093-15 と同様の理由で差し戻します。

#11 Shinichi Urabe約7年前に更新

  • ステータスRejected(差し戻し) から Pending Merge(マージ待ち) に変更
  • 進捗率50 から 80 に変更

#12 Shinichi Urabe約7年前に更新

  • ステータスPending Merge(マージ待ち) から Rejected(差し戻し) に変更
  • 進捗率80 から 50 に変更

#4093-17 に併せて日本語のテストケースも追加します

#13 Shinichi Urabe約7年前に更新

  • ステータスRejected(差し戻し) から Pending Merge(マージ待ち) に変更
  • 進捗率50 から 80 に変更

同じプルリクエストに日本語のテストケースを追加しました

#14 kaoru nほぼ7年前に更新

  • ステータスPending Merge(マージ待ち) から Pending Review(レビュー待ち) に変更
  • 進捗率80 から 50 に変更

#15 Youichi Kimura6年以上前に更新

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

レビューOKです

#16 kaoru n6年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.25 に変更

#17 kaoru n6年以上前に更新

  • 対象バージョンOpenPNE 3.8.25 から OpenPNE 3.8.x に変更

#18 kaoru n6年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.26 に変更

#19 Chiharu Nakajima6年以上前に更新

  • ステータスPending Testing(テスト待ち) から Rejected(差し戻し) に変更
  • 進捗率70 から 50 に変更

opMessagePlugin2.0.0.3で、以下のエラーが発生します

500 | Internal Server Error | InvalidArgumentException
Unable to load "i18nHelper.php" helper in: SF_ROOT_DIR/apps/pc_frontend/lib/helper, SF_ROOT_DIR/lib/helper, SF_ROOT_DIR/plugins/opMessagePlugin/lib/helper, SF_ROOT_DIR/lib/plugins/sfImageHandlerPlugin/lib/helper, SF_ROOT_DIR/lib/plugins/sfProtoculousPlugin/lib/helper, SF_ROOT_DIR/lib/vendor/symfony/lib/helper.
stack trace

    at ()
    in SF_ROOT_DIR/lib/vendor/symfony/lib/config/sfApplicationConfiguration.class.php line 663 ...

                    if (!$included)

                    {

                      throw new InvalidArgumentException(sprintf('Unable to load "%sHelper.php" helper in: %s.', $helperName, implode(', ', array_map(array('sfDebug', 'shortenFilePath'), $dirs))));

                    }

                  }

#20 kaoru n6年以上前に更新

  • 対象バージョンOpenPNE 3.8.26 から OpenPNE 3.8.x に変更

#21 kaoru n5年以上前に更新

  • 担当者Shinichi Urabe から kaoru n に変更

#4094-19 再現しません。
opMessagePlugin-2.0.0.4-alpha4 でも再現しません。

もし、https://github.com/openpne/OpenPNE3/pull/405 を checkout しているようであれば、merge して再試験お願いします。

#22 kaoru n5年以上前に更新

  • ステータスRejected(差し戻し) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#23 kaoru n5年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.30 に変更

#24 Chiharu Nakajima5年以上前に更新

  • ステータスPending Testing(テスト待ち) から Rejected(差し戻し) に変更
  • 進捗率70 から 50 に変更

#4094-19 ですが、以下の組み合わせで発生しました。

・OpnePNE 3.8.30-dev + opMessagePlugin 2.0.0.3
・OpnePNE 3.8.23 + opMessagePlugin 2.0.0.3

また、以下の組み合わせでは設定しませんでした。
・OpnePNE 3.8.30-dev + opMessagePlugin 2.0.0
・OpnePNE 3.8.23 + opMessagePlugin 2.0.0

※プラグインの問題かどうかは未調査

#25 kaoru n5年以上前に更新

  • 対象バージョンOpenPNE 3.8.30 から OpenPNE 3.8.x に変更

#26 kaoru n5年以上前に更新

  • ステータスRejected(差し戻し) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

https://github.com/tejimaya/opMessagePlugin/blob/master/lib/util/opMessageSender.class.php#L110

$context->getConfiguration()->loadHelpers('i18n');

にて

lib/vendor/symfony/lib/helper/I18NHelper.php をロードしようとしていますが、小文字であるため

Unable to load "i18nHelper.php" helper

になっているようです。

opMessagePlugin 向けに #4295 にてチケット作成しました。
このチケットには直接関係のないバグであったため、このチケットのテスト時は #4295 にマージリクエストされた内容を適用してテストしてください。

#27 kaoru n約5年前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.31 に変更

#28 Chiharu Nakajima約5年前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

動作確認OKです

#29 kaoru n約5年前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

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