Bug（バグ） #4084
完了
PC版にてSNS内名称設定に対応していない箇所が存在する
100%
説明
ファイル
| pc_menu_search_community.png (34.2 KB) pc_menu_search_community.png | kaoru n, 2016-12-27 15:04 | ||
| loca_nav_myfriend.png (19.4 KB) loca_nav_myfriend.png | kaoru n, 2016-12-27 15:05 | ||
| nav_community_top.png (20.4 KB) nav_community_top.png | kaoru n, 2016-12-27 15:08 |
kaoru n さんがほぼ8年前に更新
- ステータス を New（新規） から Pending Review（レビュー待ち） に変更
- 担当者 を kaoru n にセット
- 対象バージョン を OpenPNE 3.9.0-old にセット
- 進捗率 を 0 から 50 に変更
https://github.com/openpne/OpenPNE3/pull/401
にてプルリクエストしました。
レビューをお願いします。
Shinichi Urabe さんがほぼ8年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
- ナビの説明において <?php echo __('Friend') ?> のように記載していますが、あえて Friend を翻訳せずとも
のように SnsTerm で使われている名称を使えばよいように思いますが、どうでしょうか。- 'My Friend' 'Community' 'Nickname' 'Activity' 'Post Activity' も同様
opI18n::__($nav->caption)という処理について caption は navigation_translation の表示すべき言語に翻訳済みの言葉で、それを二重に翻訳しているような動作となるので、本来であれば Navigation モデル内で、SnsTerm を呼び出して文言定義を使うのが適切かもしれません。- %community% %friend% が元々大文字で始まっているものは大文字にしておいてください。
- Search %community% となっていますが、こちらは元々複数形です、元々と単語が変わっています。
SnsTerm::pluralize()を使えば複数形にできますが、対応方法は難しいかもしれません。
kaoru n さんがほぼ7年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
https://github.com/openpne/OpenPNE3/pull/401 いったんcloseし、
https://github.com/openpne/OpenPNE3/pull/491 でプルリクエストし直しました。
ナビの説明において のように記載していますが、あえて Friend を翻訳せずとも のように SnsTerm で使われている名称を使えばよいように思いますが、どうでしょうか。
修正しました。
community friend が元々大文字で始まっているものは大文字にしておいてください。
修正しました。
opI18n::__($nav->caption) という処理について caption は navigation_translation の表示すべき言語に翻訳済みの言葉で、それを二重に翻訳しているような動作となるので、本来であれば Navigation モデル内で、SnsTerm を呼び出して文言定義を使うのが適切かもしれません。
https://github.com/openpne/OpenPNE3/blob/master/lib/model/doctrine/NavigationTable.class.php#L13-L19
のクエリ実行時点で
caption は navigation_translation の表示すべき言語に翻訳済みの言葉
になっています。
このあと、SnsTerm を呼び出すのであれば、テンプレートで変換するのとあまり変わらないのではと思いますので変更していません。
より良い対処方法があれば具体的にご教示いただければと思います。
Search community となっていますが、こちらは元々複数形です、元々と単語が変わっています。 SnsTerm::pluralize() を使えば複数形にできますが、対応方法は難しいかもしれません。
yml 内で SnsTerm::pluralize() を行うのは難しかったので、en 向け設定は、SnsTerm を使用しない設定のままにしておきます。
kaoru n さんがほぼ7年前に更新
- 関連している Enhancement（機能追加・改善） #4267: #4084 修正適用後、SNS内名称設定を行った際、キャッシュクリアをしないと反映されない を追加
Rimpei Ogawa さんが6年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
「友人」と「フレンド」は別扱いなので、
https://github.com/openpne/OpenPNE3/pull/491/files#diff-d87e351215b0e64189d87a7c73e612f0R48
この変更は誤りです。
Rimpei Ogawa さんが6年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
apps/pc_backend/modules/navigation/templates/listSuccess.php
において、SNS内名称設定の説明にベタ書きで項目が並んでいますが、SNS内名称設定フォーム opSnsTermForm では、設定可能項目を設定ファイルから動的に取得しているため、これと似たような処理が必要です。
また、
+ <tr>
+ <th><?php echo __('%Friend%') ?></th>
+ <td>%friend%</td>
+ </tr>
のように、大文字小文字の表記揺れがあり、日本語の場合には影響がないですが、英語で表示した場合にはおかしい表示になると思います。
Rimpei Ogawa さんが6年以上前に更新
そもそも caption の出力部分を単純な __() の翻訳にかけてしまってよいのか、という部分も検討が必要かもしれません。
https://github.com/openpne/OpenPNE3/pull/491/files#diff-e2928f1e67149b64dcdb8b83a84d35cdR12
例えば、ナビゲーション部分は英語表記で「Search」と表示したいのに、apps/pc_frontend/i18n/messages.ja.xml にエントリーがあるため「検索」と翻訳されて表示されてしまったりするのが問題になるケースがあると思います。
Rimpei Ogawa さんが約5年前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
レビューOKです。
