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

Opened 9 years ago

Closed 8 years ago

#2673 closed enhancement (fixed)

ファイルアップロードのイベント対応

Reported by: imamura623 Owned by: ruikubo
Priority: minor Milestone: OpenPNE2.13.6
Component: その他 Version:
Keywords: 2.13要望 Cc:

Description (last modified by kiwa)

■概要

ファイルアップロード機能をコミュニティイベントに対応させる。

■仕様

コミュニティイベント・コミュニティイベントコメントにトピックと同じようにファイルアップロードのフォームを追加。

■関連情報

元のDescription

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

現在トピックにはファイル添付ができますが、
同じようにイベントにもファイル添付ができると便利だと思います。

たとえばイベント当日に印刷して持ってきてもらう書類がある場合、
PDFをアップしておけば別に用意する必要が無いので助かります

【追記】

むしろ画像もファイルも同様に扱い、画像の場合プレビューを、ファイルの場合ファイル形式に沿ったアイコンを表示出来るようになれば大変うれしいです。
そうなれば現在のアルバム機能も、ブリーフケースやマイドキュメント的な使い方ができます。

(ファイルに関しては、DBに書き込まない方式をデフォルトにしていただけると助かりますが・・・)



Attachments (4)

trunk.patch (24.0 KB) - added by ruikubo 9 years ago.
2系trunk版のパッチを作成したので添付します
event_edit.PNG (172.0 KB) - added by ruikubo 9 years ago.
画面キャプチャ(2/2)
event_detail.PNG (99.5 KB) - added by ruikubo 9 years ago.
画面キャプチャ(1/2)
20090223_pc_page_c_event_detail_ie7.png (5.5 KB) - added by kiwa 9 years ago.
IE7でのレイアウト崩れ

Download all attachments as: .zip

Change History (17)

comment:1 Changed 9 years ago by imamura623

http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=18122 こちらからも同じ要望があがっています

ファイルアップロードがコメント投稿時にもできるようになりました。

しかし、
コミュニティのトピックに対するコメントでは可能な一方
コミュニティのイベントに対するコメントでは不可能な状態です。

同じテーブルを使ってるので、技術的問題はない
(&特に仕様が違う意味がない)と思いますので
仕様を揃えていただけると助かります。



*会員からの指摘をいただいて気付きました


追記
【要望】イベントのコメントでファイルアップロード
から
【要望】イベントでファイルアップロード

Changed 9 years ago by ruikubo

Attachment: trunk.patch added

2系trunk版のパッチを作成したので添付します

Changed 9 years ago by ruikubo

Attachment: event_edit.PNG added

画面キャプチャ(2/2)

Changed 9 years ago by ruikubo

Attachment: event_detail.PNG added

画面キャプチャ(1/2)

comment:2 Changed 9 years ago by ruikubo

Description: modified (diff)

パッチ補足

コミュニティイベントでファイルを添付できるようにしました。
コミュニティトピックと同じ動作にしています。

ちょっとサイズが大きくなりましたが、パッチの概要は次の通りです。

  • 表示周り
    • 添付ファイルの入力欄の追加
      • /templates/c_event_add.tpl, /page/c_event_add.php
      • /templates/c_event_edit.tpl, /page/c_event_edit.php
      • /templates/c_event_write_confirm.tpl, /page/c_event_write_confirm.php
    • 添付ファイルのダウンロードリンクの追加
      • /templates/c_event_detail.tpl, /page/c_event_detail.php
    • 添付ファイルのファイル名の表示
      • /templates/c_event_add_confirm.tpl, /page/c_event_add_confirm.php
      • /templates/c_event_edit.tpl, /page/c_event_edit.php
    • 添付ファイルの削除リンクの追加
      • /templates/c_event_edit.tpl
  • DB周り
    • 添付ファイルのDB登録の処理を追加
      • /do/c_event_add_insert_c_commu_topic.php
      • /do/c_event_edit_update_c_commu_topic.php
      • /do/c_event_write_insert_c_commu_topic_comment.php
    • 添付ファイルのDB削除の処理を追加
      • /do/c_event_edit_delete_c_commu_topic_comment_file.php(新規)
    • 添付ファイルの一時保存の処理を追加
      • /page/c_event_add_confirm.php
      • /page/c_event_write_confirm.php
    • valudationルールの追加
      • /validate/do/c_event_add_insert_c_commu_topic.ini
      • /validate/do/c_event_edit_delete_c_commu_topic_comment_file.ini(新規)
      • /validate/do/c_event_write_insert_c_commu_topic_comment.ini

comment:3 Changed 9 years ago by kiwa

Milestone: OpenPNE2.13.6

おお、ありがとうございます!2.13に取り込みたいと思いますー

comment:4 Changed 9 years ago by kiwa

Owner: changed from nobody to kiwa
Status: newassigned

取り込みやります。

comment:5 Changed 9 years ago by kiwa

Description: modified (diff)
Keywords: 確認中 added
Summary: イベントにもファイル添付をできるようにしたいファイルアップロードのイベント対応

r10787にて取り込みました。

Changed 9 years ago by kiwa

IE7でのレイアウト崩れ

comment:6 Changed 9 years ago by kiwa

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

コードチェック前ですが、テンプレート側に軽く問題があったので差し戻します(動作に影響はないレベルです)。

症状1

IE7にて、ファイルが添付されているイベント詳細画面(pc_page_c_event_detail)の「タイトル」部分のレイアウトが崩れる。 IE7でのレイアウト崩れ

確認ブラウザ

  • Windows XP SP3
    • IE7

同OSのIE6, Firefox3, Safari3, Opera9では発生せず

症状2

ファイルが添付されているイベント詳細画面(pc_page_c_event_detail)でHMTL構文の警告が出る。

警告と該当行のソース

line 116 column 1 - 警告: <div> isn't allowed in <tr> elements

115: <th>ファイル</th>
116: <div class="block attachFile">
117: <td>

line 116 column 1 - 警告: missing </div> before <td>

115: <th>ファイル</th>
116: <div class="block attachFile">
117: <td>

line 118 column 153 - 警告: discarding unexpected </div>

117: <td>
118: <a href="./?m=pc&amp;a=do_c_file_download&amp;target_c_commu_topic_id=XX&amp;sessid=XXXXXXXX">ファイル名</a></td></div>
119: </tr><tr>

確認ツール

  • Firefox3 プラグイン 「HTML Validator」

comment:7 in reply to:  6 ; Changed 8 years ago by kiwa

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

Replying to kiwa:

comment:6についてr10826で修正しました。

comment:8 in reply to:  7 Changed 8 years ago by kiwa

Replying to kiwa:

Replying to kiwa:

comment:6についてr10826で修正しました。

リンクがうまくいってませんでした。r10826です。

comment:9 Changed 8 years ago by ShogoKawahara

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

comment:10 Changed 8 years ago by kiwa

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

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

comment:11 Changed 8 years ago by kiwa

Resolution: fixed
Status: closedreopened

owner変更のためreopenします。

comment:12 Changed 8 years ago by kiwa

Owner: changed from kiwa to ruikubo
Status: reopenednew

comment:13 Changed 8 years ago by kiwa

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.