Bug(バグ) #2649
完了管理画面の「ログアウト」「管理画面設定」「パスワード変更」「コミュニティ管理」などが日本語化されていない
0%
説明
Overview (現象)¶
管理画面に日本語化されていない文言がある
Environment (再現バージョン)¶
OpenPNE 3.6.0
Way to repro (再現手順)¶
管理画面にある以下のワードが日本語に翻訳されない
- ログアウト Logged out.
- 管理画面設定 Account Management、Change Password、Change Language
- パスワード変更(Old password、New password)
- コミュニティ管理
- コミュニティリスト(Sub Administrator)
- 初期コミュニティ設定(This community is already the default.)
Way to fix (修正内容)¶
日本語で表示されるように修正お願いします。
Fumie Toyooka さんがほぼ13年前に更新
「管理画面設定」の修正が終了し、プルリクエストしました。
https://github.com/openpne/OpenPNE3/pull/28
b90395f76050c0d39de3e8c86a3528ef0a9bf277
次はコミュニティ設定の方を修正します。
Fumie Toyooka さんがほぼ13年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 担当者 を Fumie Toyooka にセット
- 対象バージョン を OpenPNE 3.7.0 にセット
- 進捗率 を 0 から 50 に変更
「コミュニティ管理」の方も修正終了しました。
管理画面の「ログアウト」に関しては、
3.7.0 に対して、http://redmine.openpne.jp/issues/2450 にて修正済みですが、3.6.x へはバックポートされていないようです。
以上よろしくお願いします。
Yuya Watanabe さんが12年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
- 3.6 で発生するか を Unknown (未調査) にセット
- 3.4 で発生するか を Unknown (未調査) にセット
詳しく見ていませんが,下記差分はチケットのタイトルの意図とは違うと思われます.(詳しく見ていないので他の部分も指摘がある可能性あり)
https://github.com/fumicos/OpenPNE3/commit/b90395f76050c0d39de3e8c86a3528ef0a9bf277#diff-9
https://github.com/fumicos/OpenPNE3/commit/b90395f76050c0d39de3e8c86a3528ef0a9bf277#diff-10
また,現時点のチケットのタイトルだとカバーするバグの範囲が広く,取り込み完了の基準が不明瞭になりがちで対応しづらいと思われます.
とりあえず不要な差分が存在するという点について Rejected としておきます.
Fumie Toyooka さんが12年以上前に更新
- 題名 を 管理画面に日本語化されていない文言がある から 管理画面の「ログアウト」「管理画面設定」「パスワード変更」「コミュニティ管理」などが日本語化されていない に変更
Fumie Toyooka さんが12年以上前に更新
Yuya Watanabe は書きました:
詳しく見ていませんが,下記差分はチケットのタイトルの意図とは違うと思われます.(詳しく見ていないので他の部分も指摘がある可能性あり)
https://github.com/fumicos/OpenPNE3/commit/b90395f76050c0d39de3e8c86a3528ef0a9bf277#diff-9
https://github.com/fumicos/OpenPNE3/commit/b90395f76050c0d39de3e8c86a3528ef0a9bf277#diff-10また,現時点のチケットのタイトルだとカバーするバグの範囲が広く,取り込み完了の基準が不明瞭になりがちで対応しづらいと思われます.
とりあえず不要な差分が存在するという点について Rejected としておきます.
diff-10 の変更ですがこの変更は必要です。
翻訳ファイル内には、小文字から始まる英語表記が存在します。
この差分は、英語版のページの項目やナビゲーションなどでこれらの単語が使用される場合、
先頭の文字を大文字にして表示するための処理をしています。
よって、修正方針はdiff-9の差分を無かったこととします。
Fumie Toyooka さんが12年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
なぜかstatusをレビュー待ちにできませんでした。
新しいブランチを切って修正しなおしました。確認お願いします。
https://github.com/openpne/OpenPNE3/pull/29
Yuma Sakata さんが約12年前に更新
- 対象バージョン を OpenPNE 3.8.2 から OpenPNE 3.9.0-old に変更
- 3.8 で発生するか を Unknown (未調査) にセット
Yuma Sakata さんが約12年前に更新
- ステータス を Pending Testing(テスト待ち) から Pending Review(レビュー待ち) に変更
- 進捗率 を 70 から 50 に変更
Yuya Watanabe さんが約12年前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
- 担当者 を削除 (
Fumie Toyooka)
差し戻し内容¶
"Change language"が翻訳されておらず,下記URLで表示されるページが全体的に翻訳されていません.ただし,このあたりの文言を表示する画面はむしろ非日本語圏の人のためのものと考えられ,最初から英語のみを表示するという方針でも問題ないような気もします.このあたりが明確でないため一旦差し戻しとします.
/pc_backend.php/admin/changeLanguage
備考¶
note-8 で頂いた pull request では note-4 の指摘を十分に満たしていないため下記パッチを適用したものをレビュー対象としています.
diff --git a/apps/pc_backend/i18n/messages.ja.xml b/apps/pc_backend/i18n/messages.ja.xml index e418da3..7de090a 100644 --- a/apps/pc_backend/i18n/messages.ja.xml +++ b/apps/pc_backend/i18n/messages.ja.xml @@ -1433,6 +1433,46 @@ <source>Advanced Settings</source> <target>上級者向け設定</target> </trans-unit> + <trans-unit id=""> + <source>Old password</source> + <target>古いパスワード</target> + </trans-unit> + <trans-unit id=""> + <source>New password</source> + <target>新しいパスワード</target> + </trans-unit> + <trans-unit id=""> + <source>Account management</source> + <target>アカウント管理</target> + </trans-unit> + <trans-unit id=""> + <source>Set administrator account.</source> + <target>管理用アカウントを設定します。</target> + </trans-unit> + <trans-unit id=""> + <source>Registering a new administrator account</source> + <target>アカウント追加</target> + </trans-unit> + <trans-unit id=""> + <source>Add</source> + <target>追加</target> + </trans-unit> + <trans-unit id=""> + <source>User name</source> + <target>ユーザ名</target> + </trans-unit> + <trans-unit id=""> + <source>Delete account</source> + <target>アカウント削除</target> + </trans-unit> + <trans-unit id=""> + <source>Are you sure you want to delete this account?</source> + <target>このアカウントを本当に削除しますか?</target> + </trans-unit> + <trans-unit id=""> + <source>Delete account</source> + <target>アカウント削除</target> + </trans-unit> </body> </file> </xliff> diff --git a/apps/pc_backend/modules/admin/lib/AdminUserEditPasswordForm.class.php b/apps/pc_backend/modules/admin/lib/AdminUserEditPasswordForm.class.php index 86c08ee..1261e94 100644 --- a/apps/pc_backend/modules/admin/lib/AdminUserEditPasswordForm.class.php +++ b/apps/pc_backend/modules/admin/lib/AdminUserEditPasswordForm.class.php @@ -29,6 +29,11 @@ class AdminUserEditPasswordForm extends AdminUserForm 'callback' => array('AdminUserEditPasswordForm', 'validate'), 'arguments' => array('object' => $this->getObject()), ))); + + $this->widgetSchema->setLabels(( + 'old_password' => 'Old password'; + 'new_password' => 'New password'; + )); } public static function validate($validator, $values, $arguments = array()) diff --git a/apps/pc_backend/modules/admin/templates/_submenu.php b/apps/pc_backend/modules/admin/templates/_submenu.php index 90c1a54..c9c7356 100644 --- a/apps/pc_backend/modules/admin/templates/_submenu.php +++ b/apps/pc_backend/modules/admin/templates/_submenu.php @@ -1,3 +1,3 @@ -<li><?php echo link_to(__('Account Management'), 'admin/manageUser') ?></li> -<li><?php echo link_to(__('Change Password'), 'admin/editPassword') ?></li> -<li><?php echo link_to(__('Change Language'), 'admin/changeLanguage') ?></li> +<li><?php echo link_to(__('Account management'), 'admin/manageUser') ?></li> +<li><?php echo link_to(__('Change password'), 'admin/editPassword') ?></li> +<li><?php echo link_to(__('Change language'), 'admin/changeLanguage') ?></li> diff --git a/apps/pc_backend/modules/admin/templates/addUserSuccess.php b/apps/pc_backend/modules/admin/templates/addUserSuccess.php index efb8fa5..0d101f6 100644 --- a/apps/pc_backend/modules/admin/templates/addUserSuccess.php +++ b/apps/pc_backend/modules/admin/templates/addUserSuccess.php @@ -2,13 +2,13 @@ <?php include_partial('submenu') ?> <?php end_slot(); ?> -<h2><?php echo __('アカウント追加') ?></h2> +<h2><?php echo __('Registering a new administrator account') ?></h2> <form action="<?php url_for('admin/addUser') ?>" method="post"> <table> <?php echo $form ?> <tr> -<td colspan="2"><input type="submit" value="<?php echo __('追加') ?>" /></td> +<td colspan="2"><input type="submit" value="<?php echo __('Add') ?>" /></td> </tr> </table> </form> diff --git a/apps/pc_backend/modules/admin/templates/changeLanguageSuccess.php b/apps/pc_backend/modules/admin/templates/changeLanguageSuccess.php index 9798255..650d66b 100644 --- a/apps/pc_backend/modules/admin/templates/changeLanguageSuccess.php +++ b/apps/pc_backend/modules/admin/templates/changeLanguageSuccess.php @@ -2,10 +2,10 @@ <?php include_partial('submenu') ?> <?php end_slot(); ?> -<?php slot('title', __('Change Language')) ?> +<?php slot('title', __('Change language')) ?> <form action="<?php echo url_for('admin/changeLanguage') ?>" method="post"> <?php echo $form ?> <br /> -<input type="submit" value="<?php echo __('Setting') ?>" /> +<input type="submit" value="<?php echo __('Submit') ?>" /> </form> diff --git a/apps/pc_backend/modules/admin/templates/deleteUserSuccess.php b/apps/pc_backend/modules/admin/templates/deleteUserSuccess.php index 0262807..39e83e8 100644 --- a/apps/pc_backend/modules/admin/templates/deleteUserSuccess.php +++ b/apps/pc_backend/modules/admin/templates/deleteUserSuccess.php @@ -2,9 +2,9 @@ <?php include_partial('submenu') ?> <?php end_slot(); ?> -<h2><?php echo __('アカウント削除') ?></h2> +<h2><?php echo __('Delete account') ?></h2> -<p><?php echo __('このアカウントを本当に削除しますか?') ?></p> +<p><?php echo __('Are you sure you want to delete this account?') ?></p> <form action="<?php url_for('admin/deleteUser') ?>" method="post"> <table> @@ -12,12 +12,12 @@ <th><?php echo __('ID') ?></th><td><?php echo $user->getId() ?></td> </tr> <tr> -<th><?php echo __('ユーザ名') ?></th><td><?php echo $user->getUsername() ?></td> +<th><?php echo __('User name') ?></th><td><?php echo $user->getUsername() ?></td> </tr> <tr> <td colspan="2"> <?php echo $form ?> -<input type="submit" value="<?php echo __('削除') ?>" /> +<input type="submit" value="<?php echo __('Delete') ?>" /> </td> </tr> </table> diff --git a/apps/pc_backend/modules/admin/templates/editPasswordSuccess.php b/apps/pc_backend/modules/admin/templates/editPasswordSuccess.php index e33187c..0dc9d50 100644 --- a/apps/pc_backend/modules/admin/templates/editPasswordSuccess.php +++ b/apps/pc_backend/modules/admin/templates/editPasswordSuccess.php @@ -2,13 +2,13 @@ <?php include_partial('submenu') ?> <?php end_slot(); ?> -<h2><?php echo __('パスワード変更') ?></h2> +<h2><?php echo __('Change password') ?></h2> <form action="<?php url_for('admin/editPassword') ?>" method="post"> <table> <?php echo $form ?> <tr> -<td colspan="2"><input type="submit" value="<?php echo __('変更') ?>" /></td> +<td colspan="2"><input type="submit" value="<?php echo __('Setting') ?>" /></td> </tr> </table> </form> diff --git a/apps/pc_backend/modules/admin/templates/manageUserSuccess.php b/apps/pc_backend/modules/admin/templates/manageUserSuccess.php index 6c15829..1345882 100644 --- a/apps/pc_backend/modules/admin/templates/manageUserSuccess.php +++ b/apps/pc_backend/modules/admin/templates/manageUserSuccess.php @@ -2,23 +2,23 @@ <?php include_partial('submenu') ?> <?php end_slot(); ?> -<h2><?php echo __('アカウント管理') ?></h2> +<h2><?php echo __('Account management') ?></h2> -<p><?php echo __('管理用アカウントを設定します。') ?></p> -<p><?php echo link_to(__('アカウントを追加する'), 'admin/addUser') ?></p> +<p><?php echo __('Set administrator account.') ?></p> +<p><?php echo link_to(__('Registering a new administrator account'), 'admin/addUser') ?></p> <table> <tr> <th><?php echo __('ID') ?></th> -<th><?php echo __('ユーザ名') ?></th> -<th><?php echo __('操作') ?></th> +<th><?php echo __('User name') ?></th> +<th><?php echo __('Operation') ?></th> </tr> <?php foreach ($users as $user) : ?> <tr> <th><?php echo $user->getId() ?></th> <td><?php echo $user->getUsername() ?></td> <td><?php if ($user->getId() != 1 && $user->getId() != $sf_user->getId()) : ?> -<?php echo link_to(__('削除'), 'admin/deleteUser?id='.$user->getId()) ?> +<?php echo link_to(__('Delete'), 'admin/deleteUser?id='.$user->getId()) ?> <?php endif; ?></td> </tr> <?php endforeach; ?> diff --git a/lib/form/doctrine/AdminUserForm.class.php b/lib/form/doctrine/AdminUserForm.class.php index 1a2fba0..74e162f 100644 --- a/lib/form/doctrine/AdminUserForm.class.php +++ b/lib/form/doctrine/AdminUserForm.class.php @@ -34,8 +34,8 @@ class AdminUserForm extends BaseAdminUserForm $this->errorSchema = new sfValidatorErrorSchema($this->validatorSchema); $this->widgetSchema->setLabels(array( - 'username' => 'アカウント名', - 'password' => 'パスワード', + 'username' => 'User name', + 'password' => 'Password', )); $this->mergePostValidator(new sfValidatorDoctrineUnique(array( </pre.
Yuya Watanabe さんが約12年前に更新
- ステータス を Rejected(差し戻し) から Accepted(着手) に変更
- 担当者 を Yuya Watanabe にセット
- 進捗率 を 50 から 0 に変更
本チケットで言及されているページのすべての文言を日本語で表示する方針で実装を担当します.
Yuya Watanabe さんが約12年前に更新
note-14 のものから下記を修正
- apps/pc_backend/modules/admin/lib/AdminUserEditPasswordForm.class.php の構文エラー
- apps/pc_backend/i18n/messages.ja.xml に必要な辞書を追加
- apps/pc_backend/i18n/messages.ja.xml で空白終わり部分を削除
- コーディング規約を修正
Yuya Watanabe さんが約12年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 9c0841b5ca210dbbecb20a32e3fbb86142cc0f95 で適用されました。
Rimpei Ogawa さんが約12年前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
以下の内容について修正を検討してください。
(1) ナビゲーションや見出しなどのタイトルとして使用している部分で敢えて2語目以降を小文字に変えるのは不自然に思えます。
参考- http://en.wikipedia.org/wiki/Letter_case#Usage
- OpenPNE の SnsTerm::doTitleize()
- http://api.rubyonrails.org/classes/ActiveSupport/Inflector.html#method-i-titleize
OpenPNE 内でのルールは明確ではないかもしれないですが、管理画面だけを見ても2語目以降が大文字になっている箇所が多いように見えます。
また、note-6 で触れられている CSS で text-transform: capitalize するという以前の修正は、すべての単語の先頭を大文字表示しようという意図だったと思いますが、HTML 上で大文字にした方が自然ですし、前置詞などは小文字にしておきたい場合が多いので一律に大文字にしてしまうのも考えものです。
(2) apps/pc_backend/i18n/messages.ja.xml 内で重複しているエントリーがあります。
$ grep '<source>' apps/pc_backend/i18n/messages.ja.xml | sort | uniq -cd 2 <source>Add</source> 2 <source>Birthday</source> 2 <source>Country</source> 2 <source>Delete account</source> 2 <source>Delete</source> 2 <source>Edit</source> 2 <source>Image</source> 2 <source>Operation</source> 2 <source>Postal Code</source> 2 <source>Region in Japan</source> 2 <source>Region</source> 2 <source>Self Introduction</source> 2 <source>Sex</source> 2 <source>Telephone Number</source>
(3) AdminUserEditPasswordForm に追加された setLabels は特に意味がないように見えます。
sfWidgetFormSchemaFormatter::generateLabelName() で生成される標準のラベル名と同じ結果。
(4) 'Registering a new administrator account' は単純に動詞で Register とした方がよいと思いました。
(5) 言語設定変更後に表示される 'Changed configuration for your culture.' が翻訳されていません。
Yuya Watanabe さんが約12年前に更新
- ステータス を Rejected(差し戻し) から Accepted(着手) に変更
- 進捗率 を 50 から 0 に変更
Yuya Watanabe さんが約12年前に更新
- サブメニューやタイトルのような場所の英単語のキャピタライズ
- apps/pc_backend/i18n/messages.ja.xml の重複辞書を削除
- 副管理者の翻訳を追加
- setLabels() を削除
- 言語変更時のメッセージの翻訳を追加
- Registering a new administrator account を Register new account に変更
- もともとが「アカウント追加」で,単に「Register」として「アカウント追加」と翻訳すると他の翻訳に影響が出そうだったので文にしました
Yuya Watanabe さんが約12年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 5334e829bd56b6eaf3d234684adb0a00f78a751e で適用されました。
Rimpei Ogawa さんが約12年前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
apps/pc_backend/modules/admin/templates/addUserSuccess.php に 'Registering a new administrator account' が残っています。
(note-18 の (4) の指摘は言葉足らずでした。文にしたままで Registering → Register でよいのではないかと思っていたのでこの修正方針で問題ないです。)
Yuya Watanabe さんが約12年前に更新
- ステータス を Rejected(差し戻し) から Accepted(着手) に変更
- 進捗率 を 50 から 0 に変更
Yuya Watanabe さんが約12年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 6cd368aaee7f379610e2a58012767e2eef332352 で適用されました。
Rimpei Ogawa さんが約12年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
Rimpei Ogawa さんが10年以上前に更新
- 関連している Backport(バックポート) #3487: 管理画面の「ログアウト」「管理画面設定」「パスワード変更」「コミュニティ管理」などが日本語化されていない を追加