サイズ上限を超えた画像を添付して日記・トピックコメントをメール投稿すると、エラーメールが返されず本文のみが反映されている

[imamura623]

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

サイズ上限を超えた画像を添付して、携帯メールから日記を投稿すると、
画像の掲載は処理されず、日記の本文だけが掲載されます。

sns.phpを見ていると、コミュニティなどへのメール投稿は、

　『写真が添付されていないか、ファイルサイズが大きすぎるため、登録できませんでした。』

といったエラーメッセージをユーザに返しており、
本来、日記に関してもこれと同様にすべきではないのでしょうか。

現状、get_images()が、

　・画像が添付されていない（場合によりエラー）
　・画像ではない、あるいは処理できない画像（常にエラー）
　・サイズオーバー（常にエラー）

という別の判断に対して同じ結果（空の配列）を返していますが、
ここがどうも筋として違う気がしています。

いかがでしょうか？

とりあえず、get_images()を書き換えてやりくりしようとは考えていますが・・・

追記
バージョン2.10.0　です

[kiwa]

以下のような結果になりました

■2.8.x

	本文なし	画像容量オーバー
日記	エラーメールあり	エラーメール無し
トピック	エラーメールあり	エラーメールなし
プロフィール画像	-	-
トピック画像	-	-
イベント画像	-	-
コミュニティ画像	-	-

■2.10.x

	本文なし	画像容量オーバー
日記	エラーメールあり	エラーメール無し
トピック	エラーメールあり	エラーメールなし
プロフィール画像	-	○
トピック画像	-	○
イベント画像	-	○
コミュニティ画像	-	○

■2.11.x

	本文なし	画像容量オーバー
日記	エラーメールあり	エラーメール無し
トピック	エラーメールあり	エラーメールなし
プロフィール画像	-	○
トピック画像	-	○
イベント画像	-	○
コミュニティ画像	-	○

[kiwa]

日記・コミュニティ書き込みでIMAGE_MAX_FILESIZEで設定した容量以上の添付ファイルのついたメール投稿が行われた場合、以下のエラーメールを返して、日記本文も反映させないようにしましょう。

ファイルサイズが大きすぎるため、登録できませんでした。

[kiwa]

2.10.5では見送ります

[ebihara]

やります

[ebihara]

後回しにします。おそらく OpenPNE_KtaiMail::get_images() 周りの処理を書き換える必要があります。

[ebihara]

対応方法を検討します。

[ebihara]

条件分岐部分で型を考慮した比較をする必要がありますが、添付された画像のどれかにエラーがあった場合、OpenPNE_KtaiMail::get_images() は空配列ではなく false を返す形でとりあえずの対処をおこなおうと思います。

[ebihara]

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

• 2.8.x - r6932, r6933
• 2.10.x - r6930, r6934
• 2.12.x - r6931, r6935

[ogawa]

OpenPNE_KtaiMail::_get_images()

// 画像が正しいかどうかチェック
switch (strtolower($mail->ctype_secondary)) {
    ...
}

の部分のチェックがエラーだった場合にfalseを返さないようになっているように見えます。

それとこのチケットとは直接関係ないですが、画像何枚目まで処理するかって引数で取ってもいいかもしれないですね。

[ebihara]

以下のリビジョンで画像の形式チェックに失敗した場合にエラーを返すようにしました。

• 28x - r7003
• 210x - r7000
• 212x - r7004
• trunk - r7005

それとこのチケットとは直接関係ないですが、画像何枚目まで処理するかって引数で取ってもいいかもしれないですね。

については、別チケットで開発版のみの改善項目として対応したと思います。

[ebihara]

それとこのチケットとは直接関係ないですが、画像何枚目まで処理するかって引数で取ってもいいかもしれないですね。

については、 #2368 でおこなうようにしました。

[ogawa]

テストに回します。なるべく多くの実機でメール投稿試したいところですね。

[kiwa]

以下の現象の確認をお願いします。

■Version

• 2.8.x

■場所

• 携帯版プロフィール写真変更(ktai_page_h_config_image)

■現象

1. プロフィール写真投稿メールをIMAGE_MAX_FILESIZE以上の画像を添付した状態で送信した場合、画像は反映されないがエラーメールが返ってこない
2. プロフィール写真投稿メールを画像が１枚も添付されていない状態で送信してもエラーメールが返ってこない
3. プロフィール写真投稿メールにJPEG・GIF・PNG以外のファイルを添付して送信した場合、ファイルは反映されないがエラーメールが返ってこない

※上記３つの現象は、2.10.xと2.12betaXではエラーメールが返ってきます

[ebihara]

プロフィール写真変更で登録できない写真を投稿した際にエラーメールを返す機能は 2.10beta1 で #1367 にて実装されたものであり、2.8.x ではこの機能が存在せず、コードレベルでは正しい動作であるといえます。

ただ変更が微小であり、また 2.10.x でこの変更についての不具合等が発生した事例がないことから、本チケットの範囲から外れますが #1367 でおこなった変更を取り込む形で対処しようと思います。

[ebihara]

r7165で対応しました。ご確認ください。

[ogawa]

2.12系のみですが、add_diary_commentの画像エラー処理が抜けているように見えます。

[ebihara]

以下のリビジョンで add_diary_comment にも画像エラー処理を追加しました。

• 2.12.x - r7171
• trunk - r7172

[kiwa]

確認しました。開発版のテストはbetaでやるとして、こちらは閉じます。