プロジェクト

全般

プロフィール

Bug(バグ) #2649

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

Yuma Sakata12年以上前に追加. ほぼ7年前に更新.

ステータス:
Won't fix(対応せず)
優先度:
Low(低め)
担当者:
対象バージョン:
開始日:
2011-12-01
期日:
進捗率:

0%

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

説明

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

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


関連するチケット

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

関係しているリビジョン

リビジョン 9c0841b5 (差分)
Yuya Watanabe11年以上前に追加

(fixes #2649) fixed i18n

リビジョン 29355a40 (差分)
Yuya Watanabe11年以上前に追加

(refs #2649) removed trailing whitespaces

リビジョン e8399868 (差分)
Yuya Watanabe11年以上前に追加

(refs #2649) fixed for coding standards

リビジョン 344523f3 (差分)
Yuya Watanabe11年以上前に追加

(refs #2649) add license

リビジョン e1a2ed8a (差分)
Yuya Watanabe11年以上前に追加

(refs #2649) removed duplicated tranlatinos

リビジョン f7f9b53f (差分)
Yuya Watanabe11年以上前に追加

(refs #2649) removed unnesessary setLabel

リビジョン 0ea51599 (差分)
Yuya Watanabe11年以上前に追加

(refs #2649) fixed to capitalize second word

リビジョン ec35ad51 (差分)
Yuya Watanabe11年以上前に追加

(refs #2649) fixed english

リビジョン 70c70f7f (差分)
Yuya Watanabe11年以上前に追加

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

リビジョン 5334e829 (差分)
Yuya Watanabe11年以上前に追加

(fixes #2649) add translation to community sub administorator

リビジョン 6cd368aa (差分)
Yuya Watanabe11年以上前に追加

(fixes #2649) fixed english followed to translation file

履歴

#1 Fumie Toyooka12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

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

#2 Fumie Toyooka12年以上前に更新

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

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

#3 Fumie Toyooka12年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 担当者Fumie Toyooka にセット
  • 対象バージョンOpenPNE 3.7.0 にセット
  • 進捗率0 から 50 に変更

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

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

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

#4 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 としておきます.

#5 Fumie Toyooka約12年前に更新

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

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

#7 Fumie Toyooka約12年前に更新

  • ステータスRejected(差し戻し) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

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

#8 Fumie Toyooka約12年前に更新

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

#9 Shouta Kashiwagi約12年前に更新

  • 対象バージョンOpenPNE 3.7.0 から 252 に変更

#10 Shouta Kashiwagiほぼ12年前に更新

  • 対象バージョン252 から OpenPNE 3.8.x に変更

#11 Yuma Sakata11年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.2 に変更

#12 Yuma Sakata11年以上前に更新

  • 対象バージョンOpenPNE 3.8.2 から OpenPNE 3.9.0-old に変更
  • 3.8 で発生するかUnknown (未調査) にセット

#13 Yuma Sakata11年以上前に更新

  • ステータスPending Testing(テスト待ち) から Pending Review(レビュー待ち) に変更
  • 進捗率70 から 50 に変更

#14 Yuya Watanabe11年以上前に更新

  • ステータス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(

#15 Yuya Watanabe11年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 担当者Yuya Watanabe にセット
  • 進捗率50 から 0 に変更

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

#16 Yuya Watanabe11年以上前に更新

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 Yuya Watanabe11年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#18 Rimpei Ogawa11年以上前に更新

  • ステータスPending Review(レビュー待ち) から 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 開 石切山11年以上前に更新

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

#20 Yuya Watanabe11年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#21 Yuya Watanabe11年以上前に更新

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

#22 Yuya Watanabe11年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#23 Rimpei Ogawa11年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

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

#24 Yuya Watanabe11年以上前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#25 Yuya Watanabe11年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#26 Rimpei Ogawa11年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#27 Rimpei Ogawaほぼ10年前に更新

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

#28 誠二 天重9年以上前に更新

#29 Chiharu Nakajima9年以上前に更新

  • 優先度Normal(通常) から Low(低め) に変更

#31 isao sanoほぼ7年前に更新

  • ステータスPending Testing(テスト待ち) から Won't fix(対応せず) に変更
  • 進捗率70 から 0 に変更

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

他の形式にエクスポート: Atom PDF