OpenPNE 3

API取得部分のjQueryTemplateが完成しました。
https://github.com/houou/HOUOU/commit/c374955abe402c0f5b73c2c53fd6a61c0993ac80
API整備ができ次第導入可能です。

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

差し戻し理由¶

• phpファイルでは __() ヘルパを直接呼び出すことはできません．

apps/api/lib/helper/opJsonApiHelper.php

186     'body' => __($notification['body']),

$i18n = sfContext::getInstance()->getI18N();
$i18n->__($notification['body']);

• 下記ビューでは $notifications 変数が用いられていますが，アクション側で $this->notifications に格納していないため，データがビュー側に渡っていません．

apps/api/modules/push/templates/arraySuccess.php

  1 <?php 
  2 
  3 $data = array();
  4 
  5 foreach ($notifications as $notification)
  6 {
  7   $data[] = op_api_notification($notification);
  8 }
  9 
 10 return array(
 11   'status' => 'success',
 12   'data' => $data
 13 );

apps/api/modules/push/actions/actions.class.php

 20   public function executeSearch(sfWebRequest $request)
 21   {
 22     $member = $this->getUser()->getMember();
 23     $notifications = opNotificationCenter::getNotifications($member);
 24 
 25     $this->setTemplate('array');
 26   }

• 下記翻訳カタログでプレースホルダーが一致していないため展開されずに%Frinend%のまま表示されます．

apps/pc_frontend/i18n/messages.ja.xml

1386       <trans-unit id="">
1387         <source>Do you accept %friend% link request?</source>
1388         <target>%Friend%申請を許可しますか？</target>
1389       </trans-unit>

• button タグがちゃんと終了していません

apps/pc_frontend/templates/_tosaka.php

 42             <button class="span2 btn btn-primary small friend-notify-button friend-accept"<?php echo __('Accept') ?></button>

"Notification Center" の翻訳カタログが存在しません．

apps/pc_frontend/templates/_tosaka.php

  7       <?php echo __('Notification Center') ?> 

ユーザビリティについて¶

• 上記を直した状態でフレンド申請が来たときにニックネームなどが表示されないため，だれからのフレンド申請かが画像を見るという方法しか取れず，noimage の場合はヒントがなく不親切だと思います．また，許可，拒否選択後の結果が表示されないため，どうなったかが不明瞭です．

• 通知センターに表示されるフレンド申請の履歴は残り続けたままなのでしょうか？フレンド申請の数が多いとスクロールする必要がある長さが大きくなり，操作が煩わしく感じます．

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

• 比較演算子が逆である
  現状だと 20 以下の場合のみ 20 にリサイズする処理が走ります．
• array_slice()関数は非破壊的であるため$notificationに対してなんの影響も与えていない
  array_slice()の返り値を用いましょう．
• sfConfig::get('op_notification_limit', 20) で返ってくる値は現状の実装だと常に一定
  sfConfig に設定される値はメモリ上でのみでしか変更，参照されないため，この値を変更するにはsfConfig::set('op_notification_limit', 値) と言った様なコードを埋め込むカスタマイズが必要になります．現象ではこの値を変更する手段はコード上で見られないため，今後の拡張として考えられているものと受け取ると問題はないとも言えます．ただし，sfConfig::set()を埋め込むようなカスタメイズを想定すると考えると，実装によっては「ある通知を行う場合のみ通知される数が増える/減る」といったような現象が発生することが考えられます．永続的なデータを参照するように変更することが望ましいのではないかと思います．追記: OpenPNE.yml で設定できることを確認しましたが，設定ファイルに記述するということなどが明確でないので OpenPNE.yml などでデフォルトで設定できるようにしておくことが望ましいと思います．
• 同じ値を返すであろう sfConfig::get() が二回呼び出されている

lib/util/opNotificationCenter.class.php

 52     if (sfConfig::get('op_notification_limit', 20) > count($notifications))
 53     {
 54       array_slice($notifications, 0, sfConfig::get('op_notification_limit', 20));
 55     }

別件ですが，コミットログはコミットログだけを見ても，コミットの内容がわかるような書き方をしておいたほうがいいと思います．

note-24 の指摘内容のコミットが含まれていないように見受けられますが，これの説明が何もない状態ですが大丈夫ですか？

表示に関するバグがありましたので、修正お願いいたします。

件数の表示位置(スマートフォン版)¶

通知センターの件数表示ですが、PC版と表示位置が異なります。
※追加した画像参照

件数が減らない場合がある(PC版)¶

以下の手順で、再現できます。
- インフォメーション欄から承認or拒否
- ホーム画面リロード
- 通知センターの件数

INFORMATION欄からのフレンドの承認or拒否を行うと、通知センターに表示される件数が変わりませんでした。

ユーザの画像が表示されない(PC・スマートフォン版)¶

フレンド申請したユーザの画像が表示されません。
デフォルト画像(No Image)は表示されます。
※追加した画像参照

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

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