Opened 14 years ago
Closed 14 years ago
#2030 closed defect (fixed)
サイズ上限を超えた画像を添付して日記・トピックコメントをメール投稿すると、エラーメールが返されず本文のみが反映されている
Reported by: | imamura623 | Owned by: | ebihara |
---|---|---|---|
Priority: | critical | Milestone: | OpenPNE2.10.6 |
Component: | pne-ktai | Version: | 2.8.x & 2.10.x & 2.11.x |
Keywords: | OpenPNE2.8.11 OpenPNE2.12beta2 | Cc: |
Description
http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=15706 より転記
サイズ上限を超えた画像を添付して、携帯メールから日記を投稿すると、 画像の掲載は処理されず、日記の本文だけが掲載されます。 sns.phpを見ていると、コミュニティなどへのメール投稿は、 『写真が添付されていないか、ファイルサイズが大きすぎるため、登録できませんでした。』 といったエラーメッセージをユーザに返しており、 本来、日記に関してもこれと同様にすべきではないのでしょうか。 現状、get_images()が、 ・画像が添付されていない(場合によりエラー) ・画像ではない、あるいは処理できない画像(常にエラー) ・サイズオーバー(常にエラー) という別の判断に対して同じ結果(空の配列)を返していますが、 ここがどうも筋として違う気がしています。 いかがでしょうか? とりあえず、get_images()を書き換えてやりくりしようとは考えていますが・・・
追記 バージョン2.10.0 です
Change History (29)
comment:1 Changed 14 years ago by
Priority: | minor → major |
---|
comment:2 Changed 14 years ago by
Keywords: | 再現待ち removed |
---|---|
Summary: | サイズ上限を超えた画像を添付して日記をメール投稿すると、エラーメールが返されず本文のみが反映されている → サイズ上限を超えた画像を添付して日記・トピックコメントをメール投稿すると、エラーメールが返されず本文のみが反映されている |
comment:3 Changed 14 years ago by
Priority: | major → minor |
---|
comment:4 Changed 14 years ago by
Keywords: | OpenPNE2.8.10 OpenPNE2.11.5 added |
---|---|
Milestone: | → OpenPNE2.10.5 |
comment:5 Changed 14 years ago by
Priority: | minor → major |
---|
comment:6 Changed 14 years ago by
Priority: | major → critical |
---|
comment:7 Changed 14 years ago by
日記・コミュニティ書き込みでIMAGE_MAX_FILESIZEで設定した容量以上の添付ファイルのついたメール投稿が行われた場合、以下のエラーメールを返して、日記本文も反映させないようにしましょう。
ファイルサイズが大きすぎるため、登録できませんでした。
comment:8 Changed 14 years ago by
Priority: | critical → major |
---|
comment:9 Changed 14 years ago by
Keywords: | OpenPNE2.8.10 OpenPNE2.11.5 removed |
---|---|
Milestone: | OpenPNE2.10.5 |
Priority: | major → minor |
2.10.5では見送ります
comment:10 Changed 14 years ago by
Keywords: | OpenPNE2.8.11 OpenPNE2.12beta2 added |
---|---|
Milestone: | → OpenPNE2.10.6 |
comment:11 Changed 14 years ago by
Priority: | minor → critical |
---|
comment:12 Changed 14 years ago by
Owner: | changed from nobody to ebihara |
---|---|
Status: | new → assigned |
やります
comment:14 Changed 14 years ago by
Owner: | changed from ebihara to nobody |
---|---|
Status: | assigned → new |
comment:15 Changed 14 years ago by
Owner: | changed from nobody to ebihara |
---|---|
Status: | new → assigned |
対応方法を検討します。
comment:16 Changed 14 years ago by
条件分岐部分で型を考慮した比較をする必要がありますが、添付された画像のどれかにエラーがあった場合、OpenPNE_KtaiMail::get_images() は空配列ではなく false を返す形でとりあえずの対処をおこなおうと思います。
comment:17 Changed 14 years ago by
Keywords: | 確認中 added |
---|
comment:18 Changed 14 years ago by
Keywords: | 確認中 removed |
---|
OpenPNE_KtaiMail::_get_images()
// 画像が正しいかどうかチェック switch (strtolower($mail->ctype_secondary)) { ... }
の部分のチェックがエラーだった場合にfalseを返さないようになっているように見えます。
それとこのチケットとは直接関係ないですが、画像何枚目まで処理するかって引数で取ってもいいかもしれないですね。
comment:19 Changed 14 years ago by
Keywords: | 確認中 added |
---|
comment:20 Changed 14 years ago by
それとこのチケットとは直接関係ないですが、画像何枚目まで処理するかって引数で取ってもいいかもしれないですね。
については、 #2368 でおこなうようにしました。
comment:22 Changed 14 years ago by
Keywords: | テスト待ち added; 確認中 removed |
---|
comment:23 Changed 14 years ago by
Keywords: | テスト待ち removed |
---|
以下の現象の確認をお願いします。
■Version
- 2.8.x
■場所
- 携帯版プロフィール写真変更(ktai_page_h_config_image)
■現象
- プロフィール写真投稿メールをIMAGE_MAX_FILESIZE以上の画像を添付した状態で送信した場合、画像は反映されないがエラーメールが返ってこない
- プロフィール写真投稿メールを画像が1枚も添付されていない状態で送信してもエラーメールが返ってこない
- プロフィール写真投稿メールにJPEG・GIF・PNG以外のファイルを添付して送信した場合、ファイルは反映されないがエラーメールが返ってこない
※上記3つの現象は、2.10.xと2.12betaXではエラーメールが返ってきます
comment:24 Changed 14 years ago by
comment:26 Changed 14 years ago by
Keywords: | 差し戻し added; 確認中 removed |
---|
2.12系のみですが、add_diary_commentの画像エラー処理が抜けているように見えます。
comment:27 Changed 14 years ago by
Keywords: | 確認中 added; 差し戻し removed |
---|
comment:28 Changed 14 years ago by
Keywords: | テスト待ち added; 確認中 removed |
---|
comment:29 Changed 14 years ago by
Keywords: | テスト待ち removed |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
確認しました。開発版のテストはbetaでやるとして、こちらは閉じます。
Note: See
TracTickets for help on using
tickets.
以下のような結果になりました
■2.8.x
■2.10.x
■2.11.x