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

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#2802 closed defect (fixed)

Amazon画像のURL形式変更の影響で、一部のレビュー画像がリンク切れになっている

Reported by: imamura623 Owned by: shingo
Priority: critical Milestone: OpenPNE2.12.4
Component: 指定しない Version: 2.10.x & 2.12.x & 2.14.x
Keywords: OpenPNE2.10.10 OpenPNE2.13.3 Cc:

Description (last modified by kudo)

■原因

Amazonのデータに変更があっても、レビューデータ(c_review)は変更されない為

■修正内容

c_reviewをh_review_list_product、h_review_add_writeで表示する時、1ヶ月以上更新が無い場合、最新のデータに更新するように修正

■関連情報

元のDescription

http://sns.openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=3817 より転記

OpenPNE2.12.1にアップした後、ユーザーからいろいろと報告があがってきているのですが、その中で今まで登録していたレビューの中で画像のリンクが切れているものがいくつか見つかりました。

これって何が原因なのでしょうか?思い当たる方はいらっしゃいますか?

また解決策は・・・どうすればよい?
まさか、レビューの再登録・・・なーんてことはないですよね?

Change History (22)

comment:1 Changed 11 years ago by kiwa

Priority: minormajor

comment:2 Changed 11 years ago by kiwa

Keywords: 再現待ち removed
Summary: レビュー画像のリンク切れレビューの画像URLが変更された場合、画像がNoImage画像にならずにリンク切れ状態になる
Version: 2.12.x & 2.13.x2.10.x & 2.12.x & 2.13.x

Amazonの画像URL形式の変更により、レビューの画像も一部表示できなくなった模様です。

Amazon側に画像がない場合のレビュー画像のNoImage置き換えの機能(#1546)は、元からAmazon側に画像がなかったレビューに対してはNoImage画像に置き換えられても、途中でAmazon側の画像URLが変更されるなどして画像がなくなった場合には対応できない仕様なので、NoImage画像が表示されずにリンク切れになってしまったのだと思われます。

対応できるのなら対応したいので、Summaryを変更してバグチケットのまま通します。

comment:3 Changed 11 years ago by kiwa

Priority: majorminor

comment:4 Changed 11 years ago by kiwa

http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=17988 こちらからも報告があがっています

あと、レビューを編集したのは画像が表示されていなかったので更新すれば表示されるかなぁと思ったからで、でも更新しても画像は表示されませんでした。
amazonに画像はあるので、なければもう一度取得しに行って欲しい気もします。

comment:5 Changed 11 years ago by asma

Owner: changed from nobody to asma

comment:6 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.10 OpenPNE2.13.2 added
Milestone: OpenPNE2.12.4
Priority: minorcritical

comment:7 Changed 11 years ago by ebihara

Owner: changed from asma to nobody

comment:8 Changed 11 years ago by kiwa

Keywords: OpenPNE2.13.3 added; OpenPNE2.13.2 removed

開発版は2.13.2では見送ります。

comment:9 Changed 11 years ago by shingo

Owner: changed from nobody to shingo
Status: newassigned

作業します。

comment:10 Changed 11 years ago by shingo

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

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

comment:11 Changed 11 years ago by shingo

Description: modified (diff)

comment:12 Changed 11 years ago by ogawa

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

レビュー登録時に実行される関数 do_c_review_add_insert_c_review() にある、値がNULLだった場合に空文字列に変換するという処理が抜けています。DB側でNOT NULL制約が付いているのでNULLがあった場合にUPDATEのSQLがおそらくエラーになります。

do_c_review_add_insert_c_review() と同様の処理を追加するか、もしくは string 型への変換などで対処する必要があると思います。

comment:13 Changed 11 years ago by ogawa

それと、この新しく追加されたスクリプトの使用方法を説明するためのドキュメントが何かしら必要ではないでしょうか。

comment:14 Changed 11 years ago by shingo

Description: modified (diff)
Keywords: 確認中 added; 差し戻し removed

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

i#2802:関数do_c_review_add_insert_c_review() 内の不適切な表現を修正。

直接はこのチケットとは関係ないのですが、今回参考にした関数 do_c_review_add_insert_c_review() 内に不適切なコーディングを見つけましたので ついでに修正しました。 ご確認ください。

i#2802:今回追加分のスクリプトの使用方法を説明するためのドキュメントを作成

以下のリビジョンでドキュメントを作成しました。 ご確認ください。

comment:15 Changed 11 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed
  • README.txt の一行目の、「Amazon」となるべきところが「mazon」となっています
    1 	mazonのデータ変更に対応するスクリプトを格納しています。
    
  • bin/README.txt というファイルには、 bin ディレクトリ以下に置かれた各スクリプトについての説明を期待してしまいます。また一行目の「Amazonのデータ変更に対応するスクリプトを格納しています。」という説明は適切でないように思います(この説明では、 bin ディレクトリは Amazon のデータ変更に対応するスクリプトが格納されたディレクトリであると誤読してしまいます)
  • データベースの接続情報など、 config.php でおこなわれる設定を必要としているスクリプトであることから、「config.phpの設定を行ったのちにスクリプトを実行してください。」というような説明は明記しておくべきです
  • このスクリプトの実行は必ずしも必要でない旨、実行しない場合 Amazon 側のデータに変更があった場合に一部の画像が表示できなくなるなどの不具合は残るが重大な影響はないという旨も、できれば記述したいところです

また、今回の修正とは関係ありませんが、以下についても検討いただければと思います。

comment:16 Changed 11 years ago by kiwa

Summary: レビューの画像URLが変更された場合、画像がNoImage画像にならずにリンク切れ状態になるAmazonの画像URL形式の変更の影響で、一部の画像がリンク切れになる

Summaryを変更しました

comment:17 Changed 11 years ago by kiwa

Summary: Amazonの画像URL形式の変更の影響で、一部の画像がリンク切れになるAmazonの画像URL形式の変更の影響で、一部の画像がリンク切れになっている

comment:18 Changed 11 years ago by kiwa

Summary: Amazonの画像URL形式の変更の影響で、一部の画像がリンク切れになっているAmazon画像のURL形式変更の影響で、一部のレビュー画像がリンク切れになっている

comment:19 Changed 11 years ago by shingo

Description: modified (diff)
Keywords:  確認中 added; 差し戻し removed

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

以下のリビジョンで修正しました。(不要な関数・ファイルを削除) ご確認ください。

以下のリビジョンで修正しました。(変数名を分かり易い名前に変更) ご確認ください。

comment:20 Changed 11 years ago by ogawa

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

comment:21 Changed 11 years ago by kiwa

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

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

comment:22 Changed 11 years ago by kudo

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