OpenPNE 3

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

差し戻し理由（#2756 の note-21 とは別の指摘）¶

下記文言の翻訳カタログが存在せず，日本語版でも英語で表示されます．また #2756 では "%friend% link" となっていますが，文言としては別の意味を持つという状態でしょうか？

apps/pc_frontend/templates/_header.php

  8       <?php echo __('Notification Center') ?>
...
 27         <?php echo __('Friend request') ?>
...
 29         <?php echo __('Do you accept friend request?') ?>

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

差し戻し理由¶

• 下記部分がコードに記述があるが動作上どこだかわからなかったので正常に動作していないように感じました．また，正常に動作している場合でも言語を英語にした状態で日本語が表示されるのではないでしょうか？

web/js/jquery.notify.js

 42               $(this).parent().find('.ncfriendresultmessage').text('リクエストを承認しました');
 43               $(this).parent().find('.ncfriendresultmessage').show();
...
 50             $(this).parent().find('.ncfriendresultmessage').text('既に承認済みです');
 51             $(this).parent().find('.ncfriendresultmessage').show();
...
 75               $('.ncfrinedresultmessage', $(this).parent()).text('リクエストを拒否しました');
 76               $('.ncfrinedresultmessage', $(this).parent()).show();

リクエスト拒否については下記のように記述するととりあえず動作しているように見えました．ただ，このあたりの記述がどういう意図の実装なのかがわからなかったため，下記記述が正しいものかどうかについては言及できません．

diff --git a/web/js/jquery.notify.js b/web/js/jquery.notify.js
index 77198bd..ff34d7a 100755
--- a/web/js/jquery.notify.js
+++ b/web/js/jquery.notify.js
@@ -72,8 +72,8 @@
           success: function(data) {
             if(data.status=='success'){
               $('.ncfriendloading').hide();
-              $('.ncfrinedresultmessage', $(this).parent()).text('リクエストを拒否しました');
-              $('.ncfrinedresultmessage', $(this).parent()).show();
+              $('.push_yesno .ncfriendresultmessage').text('リクエストを拒否しました');
+              $('.push_yesno .ncfriendresultmessage').show();
              }else{
               alert(data.message);
             }   

• 20行目のみタブ文字が入っています．

web/js/jquery.notify.js

 19           window.location = linkUrl;
 20 >.......}
 21       });

• インデントのための空白がひとつ多いです．

web/js/jquery.notify.js

 77              }else{

更新履歴 622a2bc2d025bfb11bdea581d18820936b0c70ae で適用されました。

note-17 が release-3.8beta1 ブランチに含まれるもののようです．

レビューOKです．ついでに実装者を担当者を設定しました．

https://docs.google.com/a/openpne.jp/spreadsheet/ccc?key=0AsdDZCDQSqiYdDFCSGdTZzlYQWV6bDFkTmI2N2p2emc#gid=0
上のテスト表にてテストが完了しているようなので閉じます。