プロジェクト

全般

プロフィール

Bug(バグ) #4093

完了

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

Shinichi Urabe さんがほぼ8年前に追加. 5年以上前に更新.

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

100%

予定工数:
(合計: 0:00時間)
3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)

説明

現象

#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) i18n-opMessagePlugin.png Shinichi Urabe, 2016-12-27 19:46

子チケット 2 (0件未完了2件完了)

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

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

操作

関連するチケット 2 (0件未完了2件完了)

関連している opMessagePlugin - Bug(バグ) #3596: フレンド申請をメッセージ空欄で送信すると当該メッセージへ遷移できなくなるFixed(完了)Shinichi Urabe2014-04-16

操作
関連している OpenPNE 3 - Bug(バグ) #3502: apiのsettings.ymlのdefault_cultureが ja となっているWon't fix(対応せず)Akihiro KOBAYASHI2013-11-28

操作

Shinichi Urabe さんがほぼ8年前に更新

  • 関連している Bug(バグ) #3596: フレンド申請をメッセージ空欄で送信すると当該メッセージへ遷移できなくなる を追加

Shinichi Urabe さんがほぼ8年前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 担当者Shinichi Urabe にセット
  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.8.x に変更
  • 進捗率0 から 50 に変更

Shinichi Urabe さんがほぼ8年前に更新

  • ステータスPending Review(レビュー待ち) から New(新規) に変更
  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.9.0-old に変更
  • 進捗率50 から 0 に変更

Shinichi Urabe さんがほぼ8年前に更新

Shinichi Urabe さんがほぼ8年前に更新

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

note-2 は stable-3.8.x の PR のため修正

master への PR: https://github.com/openpne/OpenPNE3/pull/404

Shinichi Urabe さんがほぼ8年前に更新

SnsTerm を利用できるようにするには #3502 も取り込む必要がある

Shinichi Urabe さんがほぼ8年前に更新

Shinichi Urabe さんがほぼ8年前に更新

  • 関連している Bug(バグ) #3502: apiのsettings.ymlのdefault_cultureが ja となっている を追加

Youichi Kimura さんが7年以上前に更新

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

このバグは %friend% などを含むメッセージの翻訳を実際に api アプリケーション内で実行するコードが存在しないと動作確認が難しい (特に 3.6.x の場合) ため、修正の効果が分かるようなテストコードを追加して下さい。
例えば api アプリケーションかつカルチャが en の場合に %friend% が「friend」に置き換わるかを確認するテストコードであれば、 #3502 に影響されずにテストできるかと思います。

Shinichi Urabe さんが7年以上前に更新

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

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

kaoru n さんが7年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更
  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

Youichi Kimura さんが7年以上前に更新

opApplicationConfiguration::filterTemplateParameters() にも opI18N::initialize() と同様のコードが存在するため、この箇所の修正もお願いします。

source:lib/config/opApplicationConfiguration.class.php@183289a#L216:

    $table = Doctrine::getTable('SnsTerm');
    $application = sfConfig::get('sf_app');
    if($application == 'pc_backend')
    {   
        $application = 'pc_frontend';
    }
    $table->configure(sfContext::getInstance()->getUser()->getCulture(), $application);

Shinichi Urabe さんが7年以上前に更新

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

https://github.com/openpne/OpenPNE3/pull/404 は取り消しました
https://github.com/openpne/OpenPNE3/pull/462 でプルリクエストしました

Youichi Kimura さんが7年以上前に更新

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

テストコード内で friend titleize. のように説明が記述されていますが、%Friend% のように先頭を大文字にした場合に自動で行われる処理は titelize ではなく fronting です。
(例えば「my friend」という語は fronting では「My friend」に、titelize では「My Friend」になる)

Shinichi Urabe さんが7年以上前に更新

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

Shinichi Urabe さんが7年以上前に更新

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

master ブランチが 3.8.x ベースのソースとなったため 日本語のテストケースの考慮も可能、追加します

Shinichi Urabe さんが7年以上前に更新

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

https://github.com/openpne/OpenPNE3/pull/462 に日本語のテストケースも追加しました

Shinichi Urabe さんが7年以上前に更新

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

ステータス間違い

Youichi Kimura さんが約7年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更
  • 3.6 で発生するかUnknown (未調査) から Yes (はい) に変更

レビューOKです

Youichi Kimura さんが約7年前に更新

kaoru n さんがほぼ6年前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE-3.9.4 に変更

kaoru n さんがほぼ6年前に更新

  • 対象バージョンOpenPNE-3.9.4 から OpenPNE 3.10.x に変更

kaoru n さんが5年以上前に更新

  • 対象バージョンOpenPNE 3.10.x から OpenPNE-3.9.5 に変更

Chiharu Nakajima さんが5年以上前に更新

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

動作確認OKです

kaoru n さんが5年以上前に更新

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

マージしました

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