Project

General

Profile

Bug（バグ） #4093

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

Added by Shinichi Urabe over 2 years ago. Updated 4 months ago.

Status:
Fixed（完了）
Priority:
Normal（通常）
Assignee:
Shinichi Urabe
Target version:
OpenPNE-3.9.5
Start date:
2016-12-27
Due date:
% Done:

100%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)

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

Subtasks

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

Backport（バックポート） #4095: api アプリケーションにおいて opI18n クラスが使用されていないため、SnsTerm が翻訳されないFixed（完了）Shinichi Urabe

Related issues

Related to opMessagePlugin - Bug（バグ） #3596: フレンド申請をメッセージ空欄で送信すると当該メッセージへ遷移できなくなる Fixed（完了） 2014-04-16
Related to OpenPNE 3 - Bug（バグ） #3502: apiのsettings.ymlのdefault_cultureが ja となっている Won't fix（対応せず） 2013-11-28

History

#1 Updated by Shinichi Urabe over 2 years ago

  • Related to Bug（バグ） #3596: フレンド申請をメッセージ空欄で送信すると当該メッセージへ遷移できなくなる added

#2 Updated by Shinichi Urabe over 2 years ago

  • Status changed from New（新規） to Pending Review（レビュー待ち）
  • Assignee set to Shinichi Urabe
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.8.x
  • % Done changed from 0 to 50

#3 Updated by Shinichi Urabe over 2 years ago

  • Status changed from Pending Review（レビュー待ち） to New（新規）
  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.9.0-old
  • % Done changed from 50 to 0

#4 Updated by Shinichi Urabe over 2 years ago

#5 Updated by Shinichi Urabe over 2 years ago

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

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

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

#6 Updated by Shinichi Urabe over 2 years ago

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

#7 Updated by Shinichi Urabe over 2 years ago

#8 Updated by Shinichi Urabe over 2 years ago

#10 Updated by Youichi Kimura over 2 years ago

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

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

#11 Updated by Shinichi Urabe over 2 years ago

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

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

#12 Updated by kaoru n over 2 years ago

  • Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

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

#13 Updated by Youichi Kimura over 2 years ago

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);

#14 Updated by Shinichi Urabe over 2 years ago

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

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

#15 Updated by Youichi Kimura over 2 years ago

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

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

#16 Updated by Shinichi Urabe over 2 years ago

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

#17 Updated by Shinichi Urabe over 2 years ago

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

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

#18 Updated by Shinichi Urabe over 2 years ago

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

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

#19 Updated by Shinichi Urabe about 2 years ago

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

ステータス間違い

#20 Updated by Youichi Kimura almost 2 years ago

  • Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
  • % Done changed from 50 to 70
  • 3.6 で発生するか changed from Unknown (未調査) to Yes (はい)

レビューOKです

#21 Updated by Youichi Kimura almost 2 years ago

  • Description updated (diff)

#22 Updated by kaoru n 8 months ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE-3.9.4

#23 Updated by kaoru n 8 months ago

  • Target version changed from OpenPNE-3.9.4 to OpenPNE 3.9.x

#24 Updated by kaoru n 5 months ago

  • Target version changed from OpenPNE 3.9.x to OpenPNE-3.9.5

#25 Updated by Chiharu Nakajima 4 months ago

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

動作確認OKです

#26 Updated by kaoru n 4 months ago

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

マージしました

Also available in: Atom PDF