OpenPNE 3

次のバージョンへ見送り

http://redmine.openpne.jp/projects/op3/repository/revisions/637776119102d6814b9865a850af42ebdf166551/diff/apps/pc_frontend/modules/friend/templates/_friendListBox.php

この変更でフレンドリストのmoreinfoのリンク先がプロフィール画面になってしまっています。

56ae0b11 で修正しました。

• 56ae0b11 の変更ですが、このリンクの生成に op_link_to_member() を用いる理由はありません（メンバーの存在如何によって文言を変えることが適切ではない）。これが何のリンクなのかをよく確認してください
• たとえば _partsNineTable や _partsPhotoTable のようにパーツ内で link_to() をコールしている箇所が置換されていません（単純には置換できないところであるということは理解できますが、共通部分が対応されていないというのは中途半端な対応のように感じます）
• op_link_to_member() は第一引数にメンバーIDのみを取りますが、これはコールする側で既に Member のインスタンスを取得済みであった場合などに無駄な DQL の生成がおこなわれるため、この関数の使用頻度が多くなればなるほど、パフォーマンスに影響を与えます。この関数は Member のインスタンスも受け入れられるようにしてください
• apps/mobile_frontend/modules/friend/templates/listSuccess.php に置換時のミスと見られる表記があります（@obj_member_profil）
• op_link_to_member() 第三引数を変更する機会が多く、第二引数を変更する機会が少ないのであれば、これらの立場を入れ替えるべきだと思います

コミットメッセージに refs の付け忘れです。。すみません。

f27e2c8d

Kousuke Ebihara は書きました:

• 56ae0b11 の変更ですが、このリンクの生成に op_link_to_member() を用いる理由はありません（メンバーの存在如何によって文言を変えることが適切ではない）。これが何のリンクなのかをよく確認してください

こちら、今回の修正の目的と異なる修正でしたので、元に戻しました。

• たとえば _partsNineTable や _partsPhotoTable のようにパーツ内で link_to() をコールしている箇所が置換されていません（単純には置換できないところであるということは理解できますが、共通部分が対応されていないというのは中途半端な対応のように感じます）

対処方法に悩んだのですが、use_op_link_to_member オプションを追加し、対処を行いました。一時的な対処である認識なので、別にチケットを作って、もっといい対処方法を検討したいと思います。

• op_link_to_member() は第一引数にメンバーIDのみを取りますが、これはコールする側で既に Member のインスタンスを取得済みであった場合などに無駄な DQL の生成がおこなわれるため、この関数の使用頻度が多くなればなるほど、パフォーマンスに影響を与えます。この関数は Member のインスタンスも受け入れられるようにしてください

Member インスタンスを受け入れるように対処しました。

• apps/mobile_frontend/modules/friend/templates/listSuccess.php に置換時のミスと見られる表記があります（@obj_member_profil）

こちらは、第二引数にオプションを設定できるように対処しましたので、ルーティング名自体は削除しています。

• op_link_to_member() 第三引数を変更する機会が多く、第二引数を変更する機会が少ないのであれば、これらの立場を入れ替えるべきだと思います

入れ替えで対処しました。

追加でunit test も修正しています。
確認をお願いします。

アクティビティの一覧にて、存在するメンバーのニックネームも退会メンバー扱いになっています。確認をお願いします。

213a96cb63cdf5fc15d9f9e98845f58e058f1cb8 で修正しました

• _partsNineTable などについては、 op_link_to_member() に特化したオプションを付け加えるのであれば第一引数に ID を渡す形でコールしなくてもよいのではないでしょうか。複数のメンバーを取って来るとなるとやはりパフォーマンスの観点から懸念があります
• op_link_to_member() がエスケープされた Member インスタンスを考慮していません。ヘルパーでクラスインスタンスを受け取るのであればその考慮も必要です

以下のリビジョンで修正しました。確認をお願いします。
acbcc63b8b668d229cb1f11ab69afb9e6cf31072
aec74fbdbbb3a02ba1ffa2e7dfc4b7341da4004a

携帯版フレンドホームにて以下のようなエラーが表示ます

( ! ) Parse error: syntax error, unexpected ',' in /OPENPNE_DIR/apps/mobile_frontend/modules/friend/templates/_friendListBox.php on line 6

修正しました。
89dd00c7f9e86845b7cc47bd470a5a7bbd7810eb

SNS全体を確認する必要がありますが今回はひとまず閉じます。