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

Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#2901 closed defect (fixed)

レビューの二重投稿が可能

Reported by: imamura623 Owned by: nakasone
Priority: minor Milestone: OpenPNE2.12.3
Component: その他 Version: 2.10.x & 2.12.x & 2.14.x
Keywords: OpenPNE2.10.9 OpenPNE2.13.2 Cc:

Description (last modified by kudo)

■現象

商品レビューは、同一商品に対して、1メンバー1レビューであるが、1メンバー2レビュー以上の状態にできる。

■原因

pc_do_h_review_addで、書き込みが可能かの判断がされていないため。

■修正内容

商品レビューが二重投稿できないようにする。

■関連情報

元のDescription

http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=18195 より転記

OpenPNE-2.10.8にて確認
(tracでコードを見ただけでですが、OpenPNE-2.12.2においても同じことが再現できると思われます)

[現象]
商品レビューは、同一商品に対して、1メンバー1レビューであるが、1メンバー2レビュー以上の状態にできる
かつ(?!)、1商品に対して複数メンバーの個々のレビューとなるはずが、同一商品が複数表示できる状態になる

[再現方法]
1.レビューコメント確認画面において、「作成」ボタンを押下し、商品レビューを作成する。
2.ブラウザの戻るボタンを押下
3.再度レビューコメント確認画面において、「作成」ボタンを押下

[原因]
クラスpc_do_h_review_addにおいて、書込済みであるかどうかの判断がなされていないため。

[対応]
webapp/modules/pc/do/h_review_add.php行19に以下を追加

$c_review_comment = db_review_add_write_c_review_comment4asin_c_member_id($asin, $u);
if($c_review_comment) {
$p = array('c_review_id' => $c_review_comment['c_review_id']);
openpne_redirect('pc', 'page_h_review_list_product', $p);
}

「登録済みであった場合には、商品レビューリストの画面に遷移する」とできる。

Change History (10)

comment:1 Changed 12 years ago by kiwa

Keywords: 再現待ち removed

再現できました。

■version

  • 2.10.8
  • 2.12.2

■再現手順

  1. レビュー新規作成(pc_page_h_review_add_write)にて必要な内容を入力し、レビュー作成確認(pc_page_h_review_add_write_confirm)に遷移
  2. レビュー作成確認(pc_page_h_review_add_write_confirm)にて、「作成」を押下
  3. レビューが作成されるので、ブラウザバックでレビュー作成確認(pc_page_h_review_add_write_confirm)に戻り、再度「作成」を押下
  4. 同じレビューが複数作成されてしまう

comment:2 Changed 12 years ago by kiwa

Keywords: OpenPNE2.10.9 OpenPNE2.13.2 added
Milestone: OpenPNE2.12.3

修正方法も入っているので、次の安定版に入れられないでしょうか。

comment:3 Changed 12 years ago by nakasone

やります。

comment:4 Changed 12 years ago by nakasone

Owner: changed from nobody to nakasone
Status: newassigned

comment:5 Changed 12 years ago by nakasone

Description: modified (diff)
Keywords: 確認中 added

以下のリビジョンで修正しました。ご確認ください。

comment:6 Changed 12 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed
  • pc_page_h_review_add_write_confirm もリダイレクトさせるようにしてください
    • pc_page_h_review_add_write は pc_page_h_review_edit へ forward するような記述になっています。これを考慮したうえでリダイレクト先などを決めるとよいでしょう
  • 制御構造のインデントやスペースの取り方については PEAR標準コーディング規約に準拠してください(参考: http://pear.php.net/manual/ja/standards.control.php
    • 21行目の if と開き括弧の間に半角スペースを入れてください
    • if 文は適切なインデントをおこなって記述してください(22行目、23行目)

comment:7 Changed 12 years ago by nakasone

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

以下のリビジョンで修正しなおしました。ご確認ください。

comment:8 Changed 12 years ago by ebihara

Keywords: テスト待ち added; 確認中 removed

comment:9 Changed 12 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

確認しました。問題ありません。

comment:10 Changed 11 years ago by kudo

Description: modified (diff)
Note: See TracTickets for help on using tickets.