Project

General

Profile

Backport(バックポート) #4094

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

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

Added by Shinichi Urabe almost 8 years ago. Updated over 5 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2016-12-27
Due date:
% Done:

100%


Description

現象

#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 View (113 KB) Shinichi Urabe, 2016-12-27 19:46


Related issues

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

Associated revisions

Revision 1592af98 (diff)
Added by Shinichi Urabe over 7 years ago

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

Revision 8ff6d7cb (diff)
Added by Shinichi Urabe over 7 years ago

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

Revision 11ee00f8 (diff)
Added by Shinichi Urabe over 7 years ago

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

Revision 52558cf6 (diff)
Added by Shinichi Urabe over 7 years ago

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

Revision de6a33d9 (diff)
Added by Shinichi Urabe over 7 years ago

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

Revision 59d3dfd3 (diff)
Added by Shinichi Urabe over 7 years ago

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

Revision ced3d594
Added by kaoru n over 5 years ago

Merge pull request #405 from ShinichiU/t-4094

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

History

#1 Updated by Shinichi Urabe almost 8 years ago

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

#2 Updated by Shinichi Urabe almost 8 years ago

#3 Updated by Shinichi Urabe almost 8 years ago

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

#4 Updated by Shinichi Urabe almost 8 years ago

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

#6 Updated by Youichi Kimura over 7 years ago

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

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

#7 Updated by Shinichi Urabe over 7 years ago

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

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

#8 Updated by Youichi Kimura over 7 years ago

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

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

#9 Updated by Shinichi Urabe over 7 years ago

  • Status changed from Rejected(差し戻し) to Pending Merge(マージ待ち)
  • % Done changed from 50 to 80

#10 Updated by Youichi Kimura over 7 years ago

  • Status changed from Pending Merge(マージ待ち) to Rejected(差し戻し)
  • % Done changed from 80 to 50

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

#11 Updated by Shinichi Urabe over 7 years ago

  • Status changed from Rejected(差し戻し) to Pending Merge(マージ待ち)
  • % Done changed from 50 to 80

#12 Updated by Shinichi Urabe over 7 years ago

  • Status changed from Pending Merge(マージ待ち) to Rejected(差し戻し)
  • % Done changed from 80 to 50

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

#13 Updated by Shinichi Urabe over 7 years ago

  • Status changed from Rejected(差し戻し) to Pending Merge(マージ待ち)
  • % Done changed from 50 to 80

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

#14 Updated by kaoru n over 7 years ago

  • Status changed from Pending Merge(マージ待ち) to Pending Review(レビュー待ち)
  • % Done changed from 80 to 50

#15 Updated by Youichi Kimura about 7 years ago

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

レビューOKです

#16 Updated by kaoru n about 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.25

#17 Updated by kaoru n about 7 years ago

  • Target version changed from OpenPNE 3.8.25 to OpenPNE 3.8.x

#18 Updated by kaoru n almost 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.26

#19 Updated by Chiharu Nakajima almost 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 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 Updated by kaoru n almost 7 years ago

  • Target version changed from OpenPNE 3.8.26 to OpenPNE 3.8.x

#21 Updated by kaoru n about 6 years ago

  • Assignee changed from Shinichi Urabe to kaoru n

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

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

#22 Updated by kaoru n about 6 years ago

  • Status changed from Rejected(差し戻し) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#23 Updated by kaoru n almost 6 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.30

#24 Updated by Chiharu Nakajima almost 6 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 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 Updated by kaoru n almost 6 years ago

  • Target version changed from OpenPNE 3.8.30 to OpenPNE 3.8.x

#26 Updated by kaoru n almost 6 years ago

  • Status changed from Rejected(差し戻し) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 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 Updated by kaoru n over 5 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.31

#28 Updated by Chiharu Nakajima over 5 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

動作確認OKです

#29 Updated by kaoru n over 5 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF