Project

General

Profile

Actions

Backport(バックポート) #4094

closed

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

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

Added by Shinichi Urabe about 8 years ago. Updated almost 6 years ago.

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

100%

Estimated time:

Description

現象

#3596 の修正中に発覚

原因

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

修正内容

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


Files

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

Related issues 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Shinichi Urabe about 8 years ago

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

Updated by Shinichi Urabe about 8 years ago

Actions #3

Updated by Shinichi Urabe about 8 years ago

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

Updated by Shinichi Urabe about 8 years ago

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

Updated by Youichi Kimura almost 8 years ago

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

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

Actions #7

Updated by Shinichi Urabe almost 8 years ago

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

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

Actions #8

Updated by Youichi Kimura almost 8 years ago

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

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

Actions #9

Updated by Shinichi Urabe almost 8 years ago

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

Updated by Youichi Kimura almost 8 years ago

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

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

Actions #11

Updated by Shinichi Urabe almost 8 years ago

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

Updated by Shinichi Urabe almost 8 years ago

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

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

Actions #13

Updated by Shinichi Urabe almost 8 years ago

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

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

Actions #14

Updated by kaoru n over 7 years ago

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

Updated by Youichi Kimura over 7 years ago

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

レビューOKです

Actions #16

Updated by kaoru n over 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.25
Actions #17

Updated by kaoru n over 7 years ago

  • Target version changed from OpenPNE 3.8.25 to OpenPNE 3.8.x
Actions #18

Updated by kaoru n about 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.26
Actions #19

Updated by Chiharu Nakajima about 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))));

                    }

                  }

Actions #20

Updated by kaoru n about 7 years ago

  • Target version changed from OpenPNE 3.8.26 to OpenPNE 3.8.x
Actions #21

Updated by kaoru n over 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 して再試験お願いします。

Actions #22

Updated by kaoru n over 6 years ago

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

Updated by kaoru n about 6 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.30
Actions #24

Updated by Chiharu Nakajima about 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

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

Actions #25

Updated by kaoru n about 6 years ago

  • Target version changed from OpenPNE 3.8.30 to OpenPNE 3.8.x
Actions #26

Updated by kaoru n about 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 にマージリクエストされた内容を適用してテストしてください。

Actions #27

Updated by kaoru n almost 6 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.31
Actions #28

Updated by Chiharu Nakajima almost 6 years ago

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

動作確認OKです

Actions #29

Updated by kaoru n almost 6 years ago

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

マージしました

Actions

Also available in: Atom PDF