Project

General

Profile

Enhancement(機能追加・改善) #158

トピック・トピックコメントでの画像投稿に対応する

Added by Kousuke Ebihara about 14 years ago. Updated over 13 years ago.

Status:
Fixed(完了)
Priority:
Low(低め)
Target version:
Start date:
2009-03-18
Due date:
% Done:

100%

opCommunityTopicPlugin 絞込条件:

Description

以前のチケットは http://trac.openpne.jp/ticket/3744 です

トピック・トピックコメントで画像投稿に対応させる。

Associated revisions

Revision 13560 (diff)
Added by Shingo Yamada over 13 years ago

refs #158:コミュニティ画像投稿機能開発の差分の取り込み(opCommunityTopicPlugin-100628[git diff fc81fbb569d1a5f12a43^.])

Revision 13570 (diff)
Added by tu nguyen ngoc over 13 years ago

(refs #158) create a new topic comment by mobile mail

Revision 13571 (diff)
Added by tu nguyen ngoc over 13 years ago

(refs #158)post a topic comment with images

Revision 13572 (diff)
Added by tu nguyen ngoc over 13 years ago

(refs #158) fixed bugs: the topic's images are not displayed correctly

Revision 13573 (diff)
Added by tu nguyen ngoc over 13 years ago

(refs #158) deleted unnecessary uploading forms in mobile mode

Revision 13578 (diff)
Added by Shingo Yamada over 13 years ago

refs #158:コミュニティ画像投稿機能開発の差分の取り込み(opCommunityTopicPlugin-100628[git diff fc81fbb569d1a5f12a43.])

Revision 13580 (diff)
Added by Shingo Yamada over 13 years ago

(refs #158)post a topic comment with images

Revision 13581 (diff)
Added by Shingo Yamada over 13 years ago

(refs #158) fixed bugs: the topic's images are not displayed correctly

Revision 13582 (diff)
Added by Shingo Yamada over 13 years ago

(refs #158) deleted unnecessary uploading forms in mobile mode

Revision 13586 (diff)
Added by Shingo Yamada over 13 years ago

(refs #158)add migrate source

Revision 13589 (diff)
Added by Kousuke Ebihara over 13 years ago

fixes a form of a community topic is not displayed correctly (refs #158)

Revision 13590 (diff)
Added by Kousuke Ebihara over 13 years ago

added missing files (refs #158)

Revision 13591 (diff)
Added by Kousuke Ebihara over 13 years ago

added the auto-generated files (refs #158)

Revision 13593 (diff)
Added by Kousuke Ebihara over 13 years ago

added ability to attach photo to community event (refs #158)

Revision 13594 (diff)
Added by Kousuke Ebihara over 13 years ago

added ability to attach photo to community event comment (refs #158)

Revision 13596 (diff)
Added by Kousuke Ebihara over 13 years ago

changed migration file for the file_id column to allow NULL (refs #158)

Revision 13601 (diff)
Added by Kousuke Ebihara over 13 years ago

replaced getCommunity*ImagesJoinFile() to getImages() (refs #158)

Revision 13602 (diff)
Added by Kousuke Ebihara over 13 years ago

設定値に app_ というプレフィックスがついていなかったのを修正 (refs #158)

Revision 13603 (diff)
Added by Kousuke Ebihara over 13 years ago

r13579 で削除されていた opCommunityTopicPluginTopicCommentActions の @author を戻した (refs #158)

Revision 13604 (diff)
Added by Kousuke Ebihara over 13 years ago

ライセンス表記やクラスレベルの PHPDoc を追加、修正 (refs #158)

Revision 13605 (diff)
Added by Kousuke Ebihara over 13 years ago

使用していないフォームクラスの記述を削除 (refs #158)

Revision 13606 (diff)
Added by Kousuke Ebihara over 13 years ago

携帯版でもコミュニティトピックコメントの画像投稿フォームフィールドが表示されていたのを修正 (refs #158)

Revision 13607 (diff)
Added by Kousuke Ebihara over 13 years ago

携帯版での画像の表示が適切ではなかったのを修正 (refs #158)

History

#1 Updated by pnetan   almost 14 years ago

  • Project changed from OpenPNE3 Plugins to opCommunityTopicPlugin

#2 Updated by Kousuke Ebihara over 13 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Kousuke Ebihara
  • Target version set to 1.0.2

#3 Updated by Shingo Yamada over 13 years ago

引き受けます

#5 Updated by Shingo Yamada over 13 years ago

以下のリビジョンで別リポジトリでの開発分を取り込みました
  • r13560 :コミュニティ画像投稿機能開発の差分の取り込み(opCommunityTopicPlugin-100628[git diff fc81fbb569d1a5f12a43
  • 元リポジトリ:http://github.com/niryuu/opCommunityTopicPlugin-100628/commits/master

#6 Updated by Shingo Yamada over 13 years ago

現状認識している問題点

  • 編集時に画像投稿が出来ない
  • コミュニティトピックコメントの画像投稿機能がない

#7 Updated by tu nguyen ngoc over 13 years ago

携帯メールからコミュニティトピックコメントの画像投稿機能
元リポジトリ:http://github.com/tunn/opCommunityTopicPlugin-100628
追加した部分:git diff 09033be5c1ea8b88e680

#8 Updated by Minoru Takai over 13 years ago

git clone git://github.com/openpne/OpenPNE3.git
のmasterブランチのソースをインストールして、opCommunityTopicPlugin をアンインストールし、
note-5 にある http://github.com/niryuu/opCommunityTopicPlugin-100628/commits/master
のソースを plugins 下に入れて再インストールした環境のSNSで確認しました。

現状の挙動をPC版で確認してみましたが、トピック作成時に画像を複数追加すると意図しない挙動となったため原因を追っていました。

mysql> select * from community_topic_image;
+----+---------+---------+--------+
| id | post_id | file_id | number |
+----+---------+---------+--------+
|  1 |       2 |       1 |      1 |
|  2 |       2 |       2 |      2 |
|  3 |       3 |       3 |      1 |
|  4 |       3 |       4 |      2 |
|  5 |       3 |       5 |      3 |

画像投稿をすると上記のテーブルに画像の追加分だけレコードが追加され、このレコードであれば、

  • topic_id=2 に 画像1,2
  • topic_id=3 に 画像3,4,5

が表示されるのが正しいはずですが、動作を確認した限りでは、

  • topic_id=1 に 画像1
  • topic_id=2 に 画像2
  • topic_id=3 に 画像3
  • topic_id=4 に 画像4
  • topic_id=5 に 画像5

のようになっているようです。
plugins/opCommunityTopicPlugin/apps/pc_frontend/modules/communityTopic/templates/showSuccess.php
のソースを追ってみましたがこの不具合を生じさせている箇所を特定できませんでした。
ここまでの内容を残しておきます。

#9 Updated by tu nguyen ngoc over 13 years ago

PC版コミュニティトピックコメントの画像投稿機能
http://github.com/tunn/opCommunityTopicPlugin-100628/commit/700df9d6b3cb0dd4094b9b41ad4cd358e2527cf3
元リポジトリ:http://github.com/tunn/opCommunityTopicPlugin-100628
追加した部分:git diff 1726cbd9ee015c21da0f

#10 Updated by tu nguyen ngoc over 13 years ago

Minoru Takai は書きました:

git clone git://github.com/openpne/OpenPNE3.git
のmasterブランチのソースをインストールして、opCommunityTopicPlugin をアンインストールし、
note-5 にある http://github.com/niryuu/opCommunityTopicPlugin-100628/commits/master
のソースを plugins 下に入れて再インストールした環境のSNSで確認しました。

現状の挙動をPC版で確認してみましたが、トピック作成時に画像を複数追加すると意図しない挙動となったため原因を追っていました。

[...]

画像投稿をすると上記のテーブルに画像の追加分だけレコードが追加され、このレコードであれば、

  • topic_id=2 に 画像1,2
  • topic_id=3 に 画像3,4,5

が表示されるのが正しいはずですが、動作を確認した限りでは、

  • topic_id=1 に 画像1
  • topic_id=2 に 画像2
  • topic_id=3 に 画像3
  • topic_id=4 に 画像4
  • topic_id=5 に 画像5

のようになっているようです。
plugins/opCommunityTopicPlugin/apps/pc_frontend/modules/communityTopic/templates/showSuccess.php
のソースを追ってみましたがこの不具合を生じさせている箇所を特定できませんでした。
ここまでの内容を残しておきます。

バグ修正:
http://github.com/tunn/opCommunityTopicPlugin-100628/commit/47e1590f2c090f75fb2be0d89a7355f64319d302
元リポジトリ:http://github.com/tunn/opCommunityTopicPlugin-100628
追加した部分:git diff 700df9d6b3cb0dd4094b

#11 Updated by Shingo Yamada over 13 years ago

以下のリビジョンでtrunkに取り込みました
r13578 / r13580 - r13582

#12 Updated by Shingo Yamada over 13 years ago

r13586 でmigrationを追加しました

#13 Updated by Kousuke Ebihara over 13 years ago

  • Status changed from Accepted(着手) to Fixed(完了)
  • Target version changed from 1.0.2 to 1.0.0
  • % Done changed from 0 to 100

Also available in: Atom PDF