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

Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#2964 closed enhancement (fixed)

アルバム機能の携帯版対応

Reported by: imamura623 Owned by: yuri
Priority: major Milestone: OpenPNE2.13.7
Component: 指定しない Version:
Keywords: 2.13要望 Cc:

Description (last modified by kudo)

■概要

2.12に入ったフォトアルバム機能を携帯版に対応させる

■仕様

携帯版にPCと同等のアルバム機能を実装。メール投稿でアルバム・アルバム画像の追加を行える。

  • メンバーのアルバム一覧
  • アルバム作成
  • アルバム編集
  • アルバム削除
  • アルバム画像追加
  • アルバム画像編集
  • アルバム画像削除

また、PC版にない携帯独自の機能が追加される

  • アルバム画像容量制限が入っている場合、使用可能容量の表示
  • 登録したアルバム画像を表紙画像にする機能
  • 表示中のアルバムを使って日記を書く機能(PC版アルバム小窓挿入の代替版)

■関連情報

新機能を作る会:トピック http://sns.openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=4422

元のdescription

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

携帯からでもアルバムが使えるようにして欲しい。

mixiのようにPCと携帯では、表示順が異なる事のないようにしてくださいね。

アルバムは、FIFOですからね。

Change History (46)

comment:1 Changed 12 years ago by kiwa

Milestone: OpenPNE2.13.6

「新機能を作る会」で進んでいるようです。近日中に取り込めそうなので、とりあえずmilestoneを2.13.6にしておきます。

comment:2 Changed 12 years ago by kiwa

Milestone: OpenPNE2.13.6
Priority: minormajor

携帯版のアルバム対応を全体的にとりこむと考えるとすぐには無理そうなので、Milestone外します。

わかりやすくするためにPriorityあげておきます。

comment:3 Changed 12 years ago by kiwa

Milestone: OpenPNE2.14.0

comment:4 Changed 12 years ago by kiwa

Milestone: OpenPNE2.14.0OpenPNE2.13.7

できあがったみたいです。作成者のゆりさん、おつかれさまです。2.13.7で取り込みましょう。

comment:5 Changed 12 years ago by nakasone

Keywords: 確認中 added

r10986 にて取り込みを行いました。

comment:6 Changed 12 years ago by nakasone

作成者の方から文言の変更修正のメッセージを頂いたので以下のリビジョンで対応しました。

  • 修正内容
メール本文に写真の説明を記入してください。
を
メールの件名に写真の説明を記入してください。
に修正しました。

comment:7 Changed 12 years ago by kiwa

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

追加のパッチがあるので取り込みます。

comment:8 Changed 12 years ago by kiwa

Owner: changed from nobody to kiwa
Status: newassigned

comment:9 Changed 12 years ago by kiwa

Description: modified (diff)

comment:10 Changed 12 years ago by kiwa

Keywords: 確認中 added

r11112にて追加ファイルを取り込みました。ご確認ください。

comment:11 Changed 12 years ago by kiwa

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

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

  • アルバム・アルバム画像のメール投稿が反映されません (日記など既存の機能のメール投稿が反映される環境)
  • h_album_editにて、入力内容に不備があってリダイレクトした場合に値が保持されません
  • h_album_editにて、公開範囲のデフォルト値が日記の公開範囲のデフォルト値になっています
  • h_album_endにて、画像のファイルサイズがIMAGE_MAX_FILESIZEに問わず「300KB」と表示されます

また、アルバム作成・画像投稿時のメールに記載されているsubject・bodyを砕けすぎない文言に変更したほうが良いと思われます。

comment:12 in reply to:  11 ; Changed 12 years ago by fukamachi

修正しようと思いましたが、以下の件で問題が見つけられませんでした。

  • h_album_endにて、画像のファイルサイズがIMAGE_MAX_FILESIZEに問わず「300KB」と表示されます

それ以外の点はr11194で修正しました。

comment:13 Changed 12 years ago by fukamachi

r11195

インデントがタブになっていたので半角スペースに修正

comment:14 in reply to:  12 Changed 12 years ago by kiwa

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

Replying to fukamachi:

修正しようと思いましたが、以下の件で問題が見つけられませんでした。

  • h_album_endにて、画像のファイルサイズがIMAGE_MAX_FILESIZEに問わず「300KB」と表示されます

再確認したところ問題ありませんでした。お騒がせしました。

差し戻した内容は修正されたとみて確認中にします。

comment:15 Changed 12 years ago by imamura623

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

アルバムのメール投稿機能について動作確認いたしました。以下の点を再確認お願いします。

  • IMAGE_MAX_FILESIZE以上のjpeg画像をアップすると画像無しアルバムが作成される。
  • 添付画像無しでアルバムを作成すると、エラーメールが送信される。
  • 添付画像:jpeg/png/gif以外のファイルを添付するとアルバムが作成されてしまう。
  • subject:空欄でもアルバムが作成できる。(タイトルが無題のアルバムが作成される)
  • body:空欄でもアルバムが作成できる。(説明文が空欄のアルバムが作成される)

メール投稿時のエラーメッセージに”写真が添付されていないか、ファイルサイズが大きすぎるため、アルバム表紙を登録できませんでした。”と書かれているが、添付画像無しの投稿はエラーではないので文言の修正が必要。

comment:16 Changed 12 years ago by imamura623

追加で確認お願いします。

  • h_album_editにて、入力内容に不備があってリダイレクトした場合に値が保持されません

まだ修正されていないようです。

comment:17 Changed 12 years ago by fukamachi

自分は修正方法がよく分からないので、assignできずにいるのですが、 ソースを確認したところ、webapp/lib/mail/sns.php に問題がありそうです。

  • 拡張子の確認をしていない
  • サイズがIMAGE_MAX_FILESIZEを越えていないかのチェックをしていない
  • アルバム表紙変更の正規表現が間違っている("+"が必要)? [327行目]

comment:18 Changed 12 years ago by fukamachi

Owner: changed from kiwa to fukamachi
Status: assignednew

修正してみます

comment:19 Changed 12 years ago by fukamachi

r11235で以下の内容を修正しました。

  • 添付画像無しでアルバムを作成すると、エラーメールが送信される。
  • subject:空欄でもアルバムが作成できる。(タイトルが無題のアルバムが作成される)
  • body:空欄でもアルバムが作成できる。(説明文が空欄のアルバムが作成される)

以下の件については調査中です。

  • IMAGE_MAX_FILESIZE以上のjpeg画像をアップすると画像無しアルバムが作成される。
  • 添付画像:jpeg/png/gif以外のファイルを添付するとアルバムが作成されてしまう。

comment:20 Changed 12 years ago by fukamachi

インデントがタブになっていたのでr11236で修正

comment:21 in reply to:  19 Changed 12 years ago by fukamachi

Replying to fukamachi:

以下の件については調査中です。

  • IMAGE_MAX_FILESIZE以上のjpeg画像をアップすると画像無しアルバムが作成される。
  • 添付画像:jpeg/png/gif以外のファイルを添付するとアルバムが作成されてしまう。

原因がわかりました。 $this->decoder->get_images()が返す値が、「添付画像がない場合」でも「画像サイズが大きい場合」でもfalseを返すことが原因のようです。この2つでは動作が異なるため、webapp/lib/OpenPNE/KtaiMail.phpの、get_images()の動作を変更する必要があります。

  • 添付画像がない場合 -> アルバム作成
  • 画像サイズが大きい -> エラーメール(アルバムは作成しない)

comment:22 Changed 12 years ago by fukamachi

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

r11237で修正しました。

仕様を以下のように変更しました。

  • 添付画像がない場合 -> エラーメール(アルバムは作成しない)
  • 画像サイズが大きい -> エラーメール(アルバムは作成しない)
  • jpeg/png/gif以外のファイルを添付 -> エラーメール(アルバムは作成しない)

上記のどの場合も「写真が添付されていないか、ファイルサイズが大きすぎるため、ファイル表紙を登録できませんでした。」というエラーメールを返すようにしました。

comment:23 Changed 12 years ago by fukamachi

たびたび申し訳ないです。

インデントがタブになっていたのでr11238で修正しました。

comment:24 Changed 12 years ago by ShogoKawahara

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

以下の点の確認お願いします。

  • 携帯版でアルバムを新規作成するときに、初期状態で全員に公開にチェックされていない。(日記などではデフォルトで全員に公開にチェックされています)
  • 携帯版でアルバム情報を編集するときに、デフォルトで現在のアルバムの情報(タイトル・説明文・公開範囲)がセットされていません。

comment:25 Changed 12 years ago by imamura623

  • 添付画像がない場合 -> エラーメール(アルバムは作成しない)
  • 画像サイズが大きい -> エラーメール(アルバムは作成しない)
  • jpeg/png/gif以外のファイルを添付 -> エラーメール(アルバムは作成しない)

上記確認しました。

結果、アルバムが作成されてしまうので差し戻します。エラーメールはOKです。

comment:26 Changed 12 years ago by ShogoKawahara

  • 携帯版で、アルバムの表紙画像を削除するとき以下のWarningが出ます。
Warning: in_array() [function.in-array]: Wrong datatype for second argument in ~~/webapp/lib/util/util.php on line 1117

これは、#3731 に関連しているように思えます。

comment:27 Changed 12 years ago by imamura623

以下の修正点を確認しました。

  • subject:空欄でもアルバムが作成できる。(タイトルが無題のアルバムが作成される)
  • body:空欄でもアルバムが作成できる。(説明文が空欄のアルバムが作成される)

OKです。

comment:28 in reply to:  24 Changed 12 years ago by fukamachi

r11245, r11246でそれぞれ修正しました。

Replying to kawahara:

以下の点の確認お願いします。

  • 携帯版でアルバムを新規作成するときに、初期状態で全員に公開にチェックされていない。(日記などではデフォルトで全員に公開にチェックされています)
  • 携帯版でアルバム情報を編集するときに、デフォルトで現在のアルバムの情報(タイトル・説明文・公開範囲)がセットされていません。

comment:29 in reply to:  25 Changed 12 years ago by fukamachi

r11243, r11244で修正しました。

Replying to imamura623:

  • 添付画像がない場合 -> エラーメール(アルバムは作成しない)
  • 画像サイズが大きい -> エラーメール(アルバムは作成しない)
  • jpeg/png/gif以外のファイルを添付 -> エラーメール(アルバムは作成しない)

上記確認しました。

結果、アルバムが作成されてしまうので差し戻します。エラーメールはOKです。

comment:30 Changed 12 years ago by imamura623

以下の点、修正お願いします

  • 携帯メールで既存のアルバムに画像を追加する際、添付画像無しで投稿してもエラーメールが返ってこない。
  • 携帯メールで既存のアルバムに画像を追加する際、jpeg/png/gif 以外のファイルを投稿してもエラーメールが来ない。
  • h_album_edit画面で、サブタイトル、フォーム部分のアルバムタイトル、説明文、公開範囲が保持されていない。
  • h_album_edit画面で、タイトル空欄または説明文を空欄で編集しようとするとエラーで編集画面に戻るが、内容が保持されていない。

よろしくお願いします。

comment:31 in reply to:  26 Changed 12 years ago by ShogoKawahara

Replying to kawahara:

  • 携帯版で、アルバムの表紙画像を削除するとき以下のWarningが出ます。
Warning: in_array() [function.in-array]: Wrong datatype for second argument in ~~/webapp/lib/util/util.php on line 1117

これは、#3731 に関連しているように思えます。

上の件は、config.phpの更新忘れでした。

失礼しました。

comment:32 in reply to:  30 Changed 12 years ago by fukamachi

r11247で修正しました。

Replying to imamura623:

以下の点、修正お願いします

  • 携帯メールで既存のアルバムに画像を追加する際、添付画像無しで投稿してもエラーメールが返ってこない。
  • 携帯メールで既存のアルバムに画像を追加する際、jpeg/png/gif 以外のファイルを投稿してもエラーメールが来ない。
  • h_album_edit画面で、サブタイトル、フォーム部分のアルバムタイトル、説明文、公開範囲が保持されていない。
  • h_album_edit画面で、タイトル空欄または説明文を空欄で編集しようとするとエラーで編集画面に戻るが、内容が保持されていない。

よろしくお願いします。

comment:33 Changed 12 years ago by fukamachi

下記の2点で問題が見つけられませんでした。現在原因を調査中です。

  • 携帯メールで既存のアルバムに画像を追加する際、添付画像無しで投稿してもエラーメールが返ってこない。
  • 携帯メールで既存のアルバムに画像を追加する際、jpeg/png/gif 以外のファイルを投稿してもエラーメールが来ない。

comment:34 Changed 12 years ago by fukamachi

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

r11256で修正しました。

comment:35 Changed 12 years ago by imamura623

Keywords: 確認中 removed

以下、確認しました。

  • 携帯メールで既存のアルバムに画像を追加する際、添付画像無しで投稿してもエラーメールが返ってこない。
  • 携帯メールで既存のアルバムに画像を追加する際、jpeg/png/gif 以外のファイルを投稿してもエラーメールが来ない。
  • h_album_edit画面で、サブタイトル、フォーム部分のアルバムタイトル、説明文、公開範囲が保持されていない。
  • h_album_edit画面で、タイトル空欄または説明文を空欄で編集しようとするとエラーで編集画面に戻るが、内容が保持されていない。

修正OKです!

comment:36 Changed 12 years ago by fukamachi

アルバムの新規作成・編集フォームで、エラー発生時にリダイレクトされた際、直前の値が保持されない問題は、#3784で対応します。

comment:37 Changed 12 years ago by kiwa

Description: modified (diff)

description修正しました。

comment:38 Changed 12 years ago by kiwa

Keywords: 確認中 added

comment:39 Changed 12 years ago by kiwa

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

h_album_image_edit.tplにて

  2 <table width="100%"><tr><td align="center" bgcolor="#({$ktai_color_config.bg_02})">
  3 <font color="#({$ktai_color_config.font_05})"><a name="top">
  4 写真の編集</a></font><br>
  5 </td></tr></table>
  6 <tr><td bgcolor="#({$ktai_color_config.bg_03})" align="center">
  7 <font color="#({$ktai_color_config.color_24})">({$album_info.subject})</font><br>
  8 </td></tr></table>

5行目に不要な</table>が入っているため、サブタイトル部分のレイアウトが崩れます。

comment:40 Changed 12 years ago by fukamachi

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

r11271で修正しました。

comment:41 Changed 12 years ago by kiwa

Keywords: 確認中 removed
Resolution: fixed
Status: newclosed

#3828:アルバム機能の携帯版対応 コードチェック

残りのコードチェックは上のチケットで行います。動作を確認したところ致命的な問題はなさそうなので、今回はこれで完了とします。

comment:42 Changed 12 years ago by kiwa

Resolution: fixed
Status: closedreopened

owner変更のためreopenします

comment:43 Changed 12 years ago by kiwa

Owner: changed from fukamachi to yuri
Status: reopenednew

comment:44 Changed 12 years ago by kiwa

Resolution: fixed
Status: newclosed

comment:45 Changed 12 years ago by kudo

Description: modified (diff)

comment:46 Changed 12 years ago by kudo

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