Bug（バグ） #4093
closed
api アプリケーションにおいて opI18n クラスが使用されていないため、SnsTerm が翻訳されない
100%
Description
Files
Updated by Shinichi Urabe about 8 years ago
- Related to Bug（バグ） #3596: フレンド申請をメッセージ空欄で送信すると当該メッセージへ遷移できなくなる added
Updated by Shinichi Urabe about 8 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
Updated by Shinichi Urabe about 8 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
Updated by Shinichi Urabe about 8 years ago
- Copied to Backport（バックポート） #4094: api アプリケーションにおいて opI18n クラスが使用されていないため、SnsTerm が翻訳されない added
Updated by Shinichi Urabe about 8 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
Updated by Shinichi Urabe about 8 years ago
SnsTerm を利用できるようにするには #3502 も取り込む必要がある
Updated by Shinichi Urabe about 8 years ago
- Copied to deleted (Backport（バックポート） #4094: api アプリケーションにおいて opI18n クラスが使用されていないため、SnsTerm が翻訳されない)
Updated by Shinichi Urabe about 8 years ago
- Related to Bug（バグ） #3502: apiのsettings.ymlのdefault_cultureが ja となっている added
Updated by Youichi Kimura almost 8 years ago
- Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
このバグは %friend% などを含むメッセージの翻訳を実際に api アプリケーション内で実行するコードが存在しないと動作確認が難しい (特に 3.6.x の場合) ため、修正の効果が分かるようなテストコードを追加して下さい。
例えば api アプリケーションかつカルチャが en の場合に %friend% が「friend」に置き換わるかを確認するテストコードであれば、 #3502 に影響されずにテストできるかと思います。
Updated by Shinichi Urabe almost 8 years ago
- Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
https://github.com/openpne/OpenPNE3/pull/404 に追加で、テストコードを追加しました
Updated by kaoru n almost 8 years ago
- Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
- Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0
対象バージョン変更により修正内容の確認が必要であるため差し戻します。
Updated by Youichi Kimura almost 8 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);
Updated by Shinichi Urabe almost 8 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 でプルリクエストしました
Updated by Youichi Kimura almost 8 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」になる）
Updated by Shinichi Urabe almost 8 years ago
- Status changed from Rejected（差し戻し） to Pending Merge（マージ待ち）
- % Done changed from 50 to 80
Updated by Shinichi Urabe almost 8 years ago
- Status changed from Pending Merge（マージ待ち） to Rejected（差し戻し）
- % Done changed from 80 to 50
master ブランチが 3.8.x ベースのソースとなったため 日本語のテストケースの考慮も可能、追加します
Updated by Shinichi Urabe almost 8 years ago
- Status changed from Rejected（差し戻し） to Pending Merge（マージ待ち）
- % Done changed from 50 to 80
https://github.com/openpne/OpenPNE3/pull/462 に日本語のテストケースも追加しました
Updated by Shinichi Urabe over 7 years ago
- Status changed from Pending Merge（マージ待ち） to Pending Review（レビュー待ち）
- % Done changed from 80 to 50
ステータス間違い
Updated by Youichi Kimura over 7 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
- 3.6 で発生するか changed from Unknown (未調査) to Yes (はい)
レビューOKです
Updated by kaoru n about 6 years ago
- Target version changed from OpenPNE 3.9.0 to OpenPNE-3.9.4
Updated by kaoru n about 6 years ago
- Target version changed from OpenPNE-3.9.4 to OpenPNE 3.10.x
Updated by kaoru n almost 6 years ago
- Target version changed from OpenPNE 3.10.x to OpenPNE-3.9.5
Updated by Chiharu Nakajima almost 6 years ago
- Status changed from Pending Testing（テスト待ち） to Pending Merge（マージ待ち）
- % Done changed from 70 to 80
動作確認OKです
Updated by kaoru n almost 6 years ago
- Status changed from Pending Merge（マージ待ち） to Fixed（完了）
- % Done changed from 80 to 100
マージしました