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

Opened 12 years ago

Last modified 11 years ago

#1520 new enhancement

プロフィール画像の連投するとどちらもメイン写真になる

Reported by: yamaguchi Owned by: nobody
Priority: minor Milestone:
Component: その他 Version:
Keywords: Cc:

Description (last modified by yamaji)

プロフィール画像の連投すると一回に2枚の画像がアップロードされる上に、 どちらもメイン写真になる。

手順::
①画像が1枚以下の時に
「写真を編集」の「参照」から画像を選択する。
②一定以上の速度で(かなり素早く)、一回以上連続的に
「写真をアップロードする」をクリックする。

注意1:かなり早くクリックしなければ反応しません。
注意2:アップロード後片方を削除すると、もう一方も表示されなくなりますが
    メニュー(メイン写真にする等)は表示されたままです。
    ただし、さらにその状態からこの不具合手順をし、片方を削除すると
    片方は正しく削除され、もう片方は正しく表示されます。
注意3:三枚アップロードされている状態で、そのうち二枚がこの不具合のとき
    個別にメイン写真にするを選択できます。

■現象

プロフィール写真のアップロードの時に、1度目のアップロードが終了して、直ぐにまた写真をアップロードすると、メイン写真が見かけ上2個存在するようになったり、2枚目の写真が1枚目の写真に摩り替わっていることがある。

■原因

ファイル名に使用する画像番号の振り出し方法に問題があることが判明しました。 ファイル名は m_ユーザーID_ユニックスタイム.拡張子 となり、1秒以内の連続したファイルのアップロードで同じファイル名が出現します。

■修正内容

webapp/modules/pc/do/h_config_image.php

もし、アップロードされた画像ファイル名がc_member表に登録済みであれば、画像を登録しないようにプログラムを改定しました。

これにより、メイン画像が複数存在する状況はなくなる。

修正リビジョンは以下

2.10.x r4839


2.8.x r4840


trunk r4841

■関連情報

Change History (28)

comment:1 Changed 12 years ago by kiwa

Component: pne-adminpne-framework
Version: 2.8.x & 2.10.x & 2.11.x

Versionが入ってませんでした。

comment:2 Changed 12 years ago by kiwa

Keywords: 再現待ち added

comment:3 Changed 12 years ago by imoto

2.8.x & 2.10.x & 2.11.x にて再現確認しました。

再現手順
1.画像が1枚以下の時に「写真を編集」の「参照」から画像を選択する。
2.一回以上連続的に「写真をアップロードする」をクリックする。

上記の注意にある通り、すばやくクリックしなければアップロードされません。

comment:4 Changed 12 years ago by imoto

Keywords: 再現済み added; 再現待ち removed

comment:5 Changed 12 years ago by kiwa

Component: pne-frameworkその他
Priority: minormajor
Summary: 【バグ】2.8 プロフィール画像の連投するとどちらもメイン写真になるプロフィール画像の連投するとどちらもメイン写真になる

comment:6 Changed 12 years ago by kiwa

Description: modified (diff)
Milestone: OpenPNE2.10.2

comment:7 Changed 12 years ago by kiwa

Keywords: OpenPNE2.8.7 added

comment:8 Changed 12 years ago by ogawa

Keywords: OpenPNE2.8.7 removed
Milestone: OpenPNE2.10.2

comment:9 Changed 12 years ago by yamaji

Owner: changed from nobody to yamaji
Status: newassigned

comment:10 Changed 12 years ago by yamaji

Description: modified (diff)
Keywords: 確認待ち added; 再現済み removed

comment:11 Changed 12 years ago by yamaji

Keywords: 再現済み added; 確認待ち removed

セマフォを使うのは場合によってはPHPの再コンパイルが必要になるといった事情もあり没とします。

comment:12 Changed 12 years ago by yamaji

ファイル名に使用する画像番号の振り出し方法に問題があることが判明しました。 ファイル名は m_ユーザーID_ユニックスタイム.拡張子 となり、1秒以内の連続したファイルのアップロードで同じファイル名が出現します。

comment:13 Changed 12 years ago by yamaji

Description: modified (diff)
Keywords: 確認待ち added; 再現済み removed

プロフィール画像の連投すると一回に2枚の画像がアップロードされる上に、 どちらもメイン写真になる。

手順::
①画像が1枚以下の時に
「写真を編集」の「参照」から画像を選択する。
②一定以上の速度で(かなり素早く)、一回以上連続的に
「写真をアップロードする」をクリックする。

注意1:かなり早くクリックしなければ反応しません。
注意2:アップロード後片方を削除すると、もう一方も表示されなくなりますが
    メニュー(メイン写真にする等)は表示されたままです。
    ただし、さらにその状態からこの不具合手順をし、片方を削除すると
    片方は正しく削除され、もう片方は正しく表示されます。
注意3:三枚アップロードされている状態で、そのうち二枚がこの不具合のとき
    個別にメイン写真にするを選択できます。

■現象

プロフィール写真のアップロードの時に、1度目のアップロードが終了して、直ぐにまた写真をアップロードすると、メイン写真が見かけ上2個存在するようになったり、2枚目の写真が1枚目の写真に摩り替わっていることがある。

■原因

ファイル名に使用する画像番号の振り出し方法に問題があることが判明しました。 ファイル名は m_ユーザーID_ユニックスタイム.拡張子 となり、1秒以内の連続したファイルのアップロードで同じファイル名が出現します。

■修正内容

webapp/modules/pc/do/h_config_image.php

もし、アップロードされた画像ファイル名がc_member表に登録済みであれば、画像を登録しないようにプログラムを改定しました。

これにより、メイン画像が複数存在する状況はなくなる。

修正リビジョンは以下

2.10.x r4839 2.8.x r4840 trunk r4841

■関連情報

comment:14 Changed 12 years ago by kunitada

2,5

comment:15 Changed 12 years ago by ebihara

2.8.8と2.10.3では対応しません。

comment:16 Changed 12 years ago by ebihara

Keywords: 確認待ち removed

以下のリビジョンで安定版から変更を取り消しました。

comment:17 Changed 12 years ago by ebihara

以下の理由から、安定版からは見送りました。

  • ハードタブを使っている箇所がある
  • 連続投稿可能な時間は短くなったが、根本的な解決にはなっていない
    • 投稿1回目の投稿処理が完了していない状態で、投稿2回目以降のプロセスが走った場合、2回目以降のプロセスで取得できる $c_member にも投稿しようとしているファイル名が入っていない状態となるため、どちらもメイン写真になるという状態は起こりうる

comment:18 Changed 12 years ago by ebihara

Keywords: OpenPNE2.8.9 OpenPNE2.11.5 added
Milestone: OpenPNE2.10.4

リベンジしたいところ

comment:19 Changed 12 years ago by ebihara

Priority: majorminor

comment:20 Changed 12 years ago by ogawa

Owner: changed from yamaji to nobody
Status: assignednew

comment:21 Changed 12 years ago by kiwa

Keywords: OpenPNE2.8.9 OpenPNE2.11.5 removed
Milestone: OpenPNE2.10.4

2.10.4では見送りになりました。

comment:22 Changed 12 years ago by kiwa

Keywords: OpenPNE2.8.10 OpenPNE2.11.5 added
Milestone: OpenPNE2.10.5

comment:23 Changed 12 years ago by ebihara

Keywords: OpenPNE2.8.10 OpenPNE2.11.5 removed
Milestone: OpenPNE2.10.5OpenPNE2.11.5

意図しない連続投稿の防止策として、ボタンクリック後、JavaScriptでボタンを無効にするという方法を採りたいと思います。

影響範囲が大きくなりそうなので、ひとまず開発版にて対応しましょう。問題がなければ安定版にマージするという方向でいきます。

comment:24 Changed 11 years ago by ebihara

Milestone: OpenPNE2.11.5OpenPNE2.11.6

comment:25 Changed 11 years ago by imamura623

Milestone: OpenPNE2.11.6

2.11.6では見送ります。

comment:26 Changed 11 years ago by ogawa

Version: 2.8.x & 2.10.x & 2.11.x2.10.x & 2.12.x & 2.13.x

comment:27 Changed 11 years ago by ebihara

Milestone: OpenPNE2.12RC1

comment:28 Changed 11 years ago by ogawa

Milestone: OpenPNE2.12RC1
Type: defectenhancement
Version: 2.10.x & 2.12.x & 2.13.x
Note: See TracTickets for help on using tickets.