Project

General

Profile

Bug(バグ) #2649

管理画面の「ログアウト」「管理画面設定」「パスワード変更」「コミュニティ管理」などが日本語化されていない

Added by Yuma Sakata over 7 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Low(低め)
Assignee:
Target version:
Start date:
2011-12-01
Due date:
% Done:

0%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

Description

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 (修正内容)

日本語で表示されるように修正お願いします。


Related issues

Related to OpenPNE 3 - Bug(バグ) #2450: 管理画面からログアウトしたときの文言が英語で表示される Fixed(完了)
Related to OpenPNE 3 - Bug(バグ) #1370: #1179 におけるコミュニティ名称設定に関する変更が不充分 Fixed(完了) 2010-07-18
Related to OpenPNE 3 - Bug(バグ) #2528: 管理画面側の翻訳カタログにプラグインの機能へのカタログがある Fixed(完了) 2011-10-19
Related to OpenPNE 3 - Backport(バックポート) #3175: 管理画面の「ログアウト」「管理画面設定」「パスワード変更」「コミュニティ管理」などが日本語化されていない Fixed(完了) 2011-12-01
Related to OpenPNE 3 - Bug(バグ) #2492: pc_backend 用の i18n ファイルにプリセットプロフィールの Caption に相当する項目が重複して含まれている Works for me(再現せず) 2011-10-14
Related to OpenPNE 3 - Backport(バックポート) #3487: 管理画面の「ログアウト」「管理画面設定」「パスワード変更」「コミュニティ管理」などが日本語化されていない New(新規) 2011-12-01
Related to OpenPNE 3 - Backport(バックポート) #3687: 管理画面の翻訳カタログで「操作」と「削除」が重複して存在している Invalid(無効) 2011-10-25

Associated revisions

Revision 9c0841b5 (diff)
Added by Yuya Watanabe almost 7 years ago

(fixes #2649) fixed i18n

Revision 29355a40 (diff)
Added by Yuya Watanabe almost 7 years ago

(refs #2649) removed trailing whitespaces

Revision e8399868 (diff)
Added by Yuya Watanabe almost 7 years ago

(refs #2649) fixed for coding standards

Revision 344523f3 (diff)
Added by Yuya Watanabe almost 7 years ago

(refs #2649) add license

Revision e1a2ed8a (diff)
Added by Yuya Watanabe almost 7 years ago

(refs #2649) removed duplicated tranlatinos

Revision f7f9b53f (diff)
Added by Yuya Watanabe almost 7 years ago

(refs #2649) removed unnesessary setLabel

Revision 0ea51599 (diff)
Added by Yuya Watanabe almost 7 years ago

(refs #2649) fixed to capitalize second word

Revision ec35ad51 (diff)
Added by Yuya Watanabe almost 7 years ago

(refs #2649) fixed english

Revision 70c70f7f (diff)
Added by Yuya Watanabe almost 7 years ago

(refs #2649) add transtaion for when the culture setting was changed

Revision 5334e829 (diff)
Added by Yuya Watanabe almost 7 years ago

(fixes #2649) add translation to community sub administorator

Revision 6cd368aa (diff)
Added by Yuya Watanabe almost 7 years ago

(fixes #2649) fixed english followed to translation file

History

#1 Updated by Fumie Toyooka over 7 years ago

  • Status changed from New(新規) to Accepted(着手)

まず、管理画面から修正します。

#2 Updated by Fumie Toyooka over 7 years ago

「管理画面設定」の修正が終了し、プルリクエストしました。
https://github.com/openpne/OpenPNE3/pull/28
b90395f76050c0d39de3e8c86a3528ef0a9bf277

次はコミュニティ設定の方を修正します。

#3 Updated by Fumie Toyooka over 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • Assignee set to Fumie Toyooka
  • Target version set to OpenPNE 3.7.0
  • % Done changed from 0 to 50

「コミュニティ管理」の方も修正終了しました。

管理画面の「ログアウト」に関しては、
3.7.0 に対して、http://redmine.openpne.jp/issues/2450 にて修正済みですが、3.6.x へはバックポートされていないようです。

以上よろしくお願いします。

#4 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • 3.6 で発生するか set to Unknown (未調査)
  • 3.4 で発生するか set to Unknown (未調査)

詳しく見ていませんが,下記差分はチケットのタイトルの意図とは違うと思われます.(詳しく見ていないので他の部分も指摘がある可能性あり)

https://github.com/fumicos/OpenPNE3/commit/b90395f76050c0d39de3e8c86a3528ef0a9bf277#diff-9
https://github.com/fumicos/OpenPNE3/commit/b90395f76050c0d39de3e8c86a3528ef0a9bf277#diff-10

また,現時点のチケットのタイトルだとカバーするバグの範囲が広く,取り込み完了の基準が不明瞭になりがちで対応しづらいと思われます.

とりあえず不要な差分が存在するという点について Rejected としておきます.

#5 Updated by Fumie Toyooka over 7 years ago

  • Subject changed from 管理画面に日本語化されていない文言がある to 管理画面の「ログアウト」「管理画面設定」「パスワード変更」「コミュニティ管理」などが日本語化されていない

#6 Updated by Fumie Toyooka over 7 years ago

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の差分を無かったこととします。

#7 Updated by Fumie Toyooka over 7 years ago

  • Status changed from Rejected(差し戻し) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

なぜかstatusをレビュー待ちにできませんでした。
新しいブランチを切って修正しなおしました。確認お願いします。
https://github.com/openpne/OpenPNE3/pull/29

#8 Updated by Fumie Toyooka over 7 years ago

新しくプルリクエストしました。
https://github.com/openpne/OpenPNE3/pull/30

#9 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from OpenPNE 3.7.0 to 252

#10 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from 252 to OpenPNE 3.8.x

#11 Updated by Yuma Sakata almost 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.2

#12 Updated by Yuma Sakata almost 7 years ago

  • Target version changed from OpenPNE 3.8.2 to OpenPNE 3.9.0-old
  • 3.8 で発生するか set to Unknown (未調査)

#13 Updated by Yuma Sakata almost 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Review(レビュー待ち)
  • % Done changed from 70 to 50

#14 Updated by Yuya Watanabe almost 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Assignee deleted (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(

#15 Updated by Yuya Watanabe almost 7 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • Assignee set to Yuya Watanabe
  • % Done changed from 50 to 0

本チケットで言及されているページのすべての文言を日本語で表示する方針で実装を担当します.

#16 Updated by Yuya Watanabe almost 7 years ago

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 で空白終わり部分を削除
  • コーディング規約を修正

#17 Updated by Yuya Watanabe almost 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 9c0841b5ca210dbbecb20a32e3fbb86142cc0f95 で適用されました。

#18 Updated by Rimpei Ogawa almost 7 years ago

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

以下の内容について修正を検討してください。

(1) ナビゲーションや見出しなどのタイトルとして使用している部分で敢えて2語目以降を小文字に変えるのは不自然に思えます。

参考

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.' が翻訳されていません。

#19 Updated by 開 石切山 almost 7 years ago

管理画面の言語設定が「日本語」に設定されている際、コミュニティ管理ページ(/pc_backend.php/community)の「副管理者」の項目が翻訳されず「Sub Administrator」と表示さてしまう。

#20 Updated by Yuya Watanabe almost 7 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#21 Updated by Yuya Watanabe almost 7 years ago

  • サブメニューやタイトルのような場所の英単語のキャピタライズ
  • apps/pc_backend/i18n/messages.ja.xml の重複辞書を削除
  • 副管理者の翻訳を追加
  • setLabels() を削除
  • 言語変更時のメッセージの翻訳を追加
  • Registering a new administrator account を Register new account に変更
    • もともとが「アカウント追加」で,単に「Register」として「アカウント追加」と翻訳すると他の翻訳に影響が出そうだったので文にしました

#22 Updated by Yuya Watanabe almost 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 5334e829bd56b6eaf3d234684adb0a00f78a751e で適用されました。

#23 Updated by Rimpei Ogawa almost 7 years ago

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

apps/pc_backend/modules/admin/templates/addUserSuccess.php に 'Registering a new administrator account' が残っています。
(note-18 の (4) の指摘は言葉足らずでした。文にしたままで Registering → Register でよいのではないかと思っていたのでこの修正方針で問題ないです。)

#24 Updated by Yuya Watanabe almost 7 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#25 Updated by Yuya Watanabe almost 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 6cd368aaee7f379610e2a58012767e2eef332352 で適用されました。

#26 Updated by Rimpei Ogawa almost 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#27 Updated by Rimpei Ogawa about 5 years ago

  • Related to Backport(バックポート) #3487: 管理画面の「ログアウト」「管理画面設定」「パスワード変更」「コミュニティ管理」などが日本語化されていない added

#28 Updated by 誠二 天重 almost 5 years ago

#29 Updated by Chiharu Nakajima over 4 years ago

  • Priority changed from Normal(通常) to Low(低め)

#31 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • % Done changed from 70 to 0

OpenPNE 3.8.2 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF