OpenPNE 3

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

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

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

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

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

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

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

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

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

差し戻し内容¶

"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.

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

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

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

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

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

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

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

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