ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 13 years ago

Closed 13 years ago

#3812 closed enhancement (fixed)

フレンド管理ページのフレンド一覧表示用関数でc_member_id_fromとc_member_id_toが逆に指定されている

Reported by: shingo Owned by: toraneko
Priority: minor Milestone: OpenPNE2.14beta1
Component: 指定しない Version: 2.12.x & 2.14.x
Keywords: Cc:

Description (last modified by imamura623)

■現象

関数db_friend_c_friend_list_disp4c_member_id() 内のSQLでc_member_id_fromとc_member_id_toが逆に指定されている

    source   : webapp/lib/db/friend.php
    function : db_friend_c_friend_list_disp4c_member_id
    内容 :
      フレンド一覧を取得するSQLで、c_member_id_from と c_member_id_to を逆に指定している
      (自分を友達に設定しているフレンド一覧を取得している)
      また、SQL発行以降で他の関数をコールする時に、c_member_id_fromとc_member_id_toを逆に指定している
      ( (*)が該当箇所。fromとtoが逆 )
    修正内容:
      該当箇所(*)のc_member_id_fromとc_member_id_toを逆にする

---------------------------------------------------
    $sql =  "SELECT c_member_id,".
                "image_filename,".
                "nickname,".
                "intro,".
                "c_member_id_from,".
                "c_member_id_to".
            " FROM c_member,".
                "c_friend".
(*)         " WHERE c_member.c_member_id = c_friend.c_member_id_from".
(*)             " AND c_friend.c_member_id_to = ?".
            " ORDER BY c_friend.r_datetime DESC";
    $params = array(intval($c_member_id));
    $c_friend_list = db_get_all_page($sql, $page, $size, $params);

    foreach ($c_friend_list as $key => $value) {
(*)     $c_friend = db_friend_c_friend4c_member_id_from4c_member_id_to($value['c_member_id_to'], $value['c_member_id_from']);
        $c_friend_list[$key]['intro'] = $c_friend['intro'];

(*)     $c_friend_list[$key]['friend_count'] = db_friend_count_friends($value['c_member_id_from']);
    }
---------------------------------------------------

■原因

■修正内容

■関連情報

#3678 で見つかった不具合
上記チケットでの変更を取り消し、このチケットにてまとめて修正することとする。

提供コードのリビジョンr10894

http://trac.openpne.jp/ticket/3941
コードチェックは上記のチケットで実施する

Change History (23)

comment:1 Changed 13 years ago by kiwa

Keywords: OpenPNE2.13.8 added
Milestone: OpenPNE2.13.8OpenPNE2.12.10
Summary: フレンド管理ページのフレンドいたらん表示用関数に不具合があるフレンド管理ページのフレンド一覧表示用関数に不具合がある

comment:2 Changed 13 years ago by kiwa

Summary: フレンド管理ページのフレンド一覧表示用関数に不具合があるフレンド管理ページのフレンド一覧表示用関数でc_member_id_fromとc_member_id_toが逆に指定されている

comment:3 Changed 13 years ago by kiwa

Description: modified (diff)

comment:4 Changed 13 years ago by kiwa

Description: modified (diff)

comment:5 Changed 13 years ago by kiwa

Milestone: OpenPNE2.12.10OpenPNE2.12.11

余裕をもって確認を行いたい項目なので、安定版は2.12.11で対応しましょう。

comment:6 Changed 13 years ago by nagasawa

Keywords: 確認中 added
Owner: changed from nobody to nagasawa

以下のリビジョンでr10894をマージしました。

comment:7 Changed 13 years ago by kiwa

Keywords: 差し戻し added; 確認中 removed

質問です。

  1. db_friend_c_friend_list_disp4c_member_idのみ修正されていますが、pc_page_h_manage_friendやそのページにある機能(紹介文を書く、フレンドから外す)などにの影響はないですか?
  2. db_friend_c_friend_list_disp4c_member_idはpc_page_h_manage_friendにしか使用されていないようですが、それ以外のフレンド取得系の関数は問題ないですか?(fh_friend_listに使用する関数・携帯版など)

comment:8 Changed 13 years ago by nagasawa

  1. 相手メンバーの情報を取得する「db_friend_c_friend4c_member_id_from4c_member_id_to」と相手メンバーのフレンド数を取得する「db_friend_count_friends」に関しても同様に逆なっているため結果的に関数の戻り値は変わらないので他の箇所に影響がでる事はありません

comment:9 Changed 13 years ago by nagasawa

r11412,r11413 で2.12への変更を取り消しました。

comment:10 Changed 13 years ago by nagasawa

Keywords: 確認中 added; 差し戻し removed
  1. webapp/lib/db/friend.php内を確認しましたが問題なさそうです

comment:11 Changed 13 years ago by kiwa

Keywords: 差し戻し added; 確認中 removed

レポートからぬけてました・・・確認が必要なものなので2.13.8取り込み内容から外しましょう。

comment:12 Changed 13 years ago by nagasawa

r11567 で取り消しました。

comment:13 Changed 13 years ago by kiwa

Keywords: OpenPNE2.14beta1 added; OpenPNE2.13.8 差し戻し removed

確認しました。取り消されています。

comment:14 Changed 13 years ago by kiwa

Keywords: OpenPNE2.14beta1 removed
Milestone: OpenPNE2.12.11OpenPNE2.14beta1
Type: defectenhancement

2.14beta1で取り込み、問題なければ2.12にもとりこみましょう。

comment:15 Changed 13 years ago by nagasawa

r11598 で前回取り消した r11567 を逆マージしました。

comment:16 Changed 13 years ago by nagasawa

Keywords: 確認中 added

comment:17 Changed 13 years ago by imamura623

以下のような項目を動作確認しましたが問題はありませんでした。

  • マイフレンド管理画面にマイフレンドリストが表示される
  • マイフレンドのトップ画像が表示される
  • 紹介文の作成、編集、削除ができる
  • 紹介文が表示される
  • マイフレンドから外すが機能する
  • マイフレンドの最新書き込みを表示する・しないが選択できる
  • 次を表示が機能する(リストの最終のページは次を表示は非表示)
  • 前を表示が機能する(リストの最初のページは前を表示は非表示)

comment:18 in reply to:  17 Changed 13 years ago by imamura623

Replying to imamura623:

以下のような項目を動作確認しましたが問題はありませんでした。

  • マイフレンド管理画面にマイフレンドリストが表示される
  • マイフレンドのトップ画像が表示される
  • 紹介文の作成、編集、削除ができる
  • 紹介文が表示される
  • マイフレンドから外すが機能する
  • マイフレンドの最新書き込みを表示する・しないが選択できる
  • 次を表示が機能する(リストの最終のページは次を表示は非表示)
  • 前を表示が機能する(リストの最初のページは前を表示は非表示)

これらの項目をマイフレンド双方より検証し問題無いことを確認しました。
また

  • マイフレンド登録が無い場合はマイフレンド登録がありませんと表示される

も確認いたしました。

comment:19 Changed 13 years ago by imamura623

Description: modified (diff)

コードチェックについては別チケットにて対応する
http://trac.openpne.jp/ticket/3941

comment:20 Changed 13 years ago by imamura623

Keywords: 確認中 removed
Resolution: fixed
Status: newclosed

問題無いようなので、修正完了としこのチケットは閉じます。
お疲れ様でした。

comment:21 Changed 13 years ago by imamura623

Resolution: fixed
Status: closedreopened

assign変更のためreopenします

comment:22 Changed 13 years ago by imamura623

Owner: changed from nagasawa to toraneko
Status: reopenednew

comment:23 Changed 13 years ago by imamura623

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.