Opened 13 years ago
Closed 12 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 )
■概要
ファイルアップロード機能をコミュニティイベントに対応させる。
■仕様
コミュニティイベント・コミュニティイベントコメントにトピックと同じようにファイルアップロードのフォームを追加。
■関連情報
元のDescription
http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=17408 より転記
現在トピックにはファイル添付ができますが、 同じようにイベントにもファイル添付ができると便利だと思います。 たとえばイベント当日に印刷して持ってきてもらう書類がある場合、 PDFをアップしておけば別に用意する必要が無いので助かります 【追記】 むしろ画像もファイルも同様に扱い、画像の場合プレビューを、ファイルの場合ファイル形式に沿ったアイコンを表示出来るようになれば大変うれしいです。 そうなれば現在のアルバム機能も、ブリーフケースやマイドキュメント的な使い方ができます。 (ファイルに関しては、DBに書き込まない方式をデフォルトにしていただけると助かりますが・・・)
Attachments (4)
Change History (17)
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
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
- 添付ファイルのDB登録の処理を追加
comment:4 Changed 12 years ago by
Owner: | changed from nobody to kiwa |
---|---|
Status: | new → assigned |
取り込みやります。
comment:5 Changed 12 years ago by
Description: | modified (diff) |
---|---|
Keywords: | 確認中 added |
Summary: | イベントにもファイル添付をできるようにしたい → ファイルアップロードのイベント対応 |
r10787にて取り込みました。
comment:6 follow-up: 7 Changed 12 years ago by
Keywords: | 差し戻し added; 確認中 removed |
---|
コードチェック前ですが、テンプレート側に軽く問題があったので差し戻します(動作に影響はないレベルです)。
症状1
IE7にて、ファイルが添付されているイベント詳細画面(pc_page_c_event_detail)の「タイトル」部分のレイアウトが崩れる。
確認ブラウザ
- 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&a=do_c_file_download&target_c_commu_topic_id=XX&sessid=XXXXXXXX">ファイル名</a></td></div> 119: </tr><tr>
確認ツール
- Firefox3 プラグイン 「HTML Validator」
comment:7 follow-up: 8 Changed 12 years ago by
Keywords: | 確認中 added; 差し戻し removed |
---|
Replying to kiwa:
comment:6についてr10826で修正しました。
comment:8 Changed 12 years ago by
comment:9 Changed 12 years ago by
Keywords: | テスト待ち added; 確認中 removed |
---|
comment:10 Changed 12 years ago by
Keywords: | テスト待ち removed |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
確認しました。問題ありません。
comment:12 Changed 12 years ago by
Owner: | changed from kiwa to ruikubo |
---|---|
Status: | reopened → new |
comment:13 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=18122 こちらからも同じ要望があがっています