Project

General

Profile

Actions

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

closed

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

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

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

100%

Estimated time:
opCommunityTopicPlugin 絞込条件:

Description

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

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

Actions #1

Updated by pnetan   almost 15 years ago

  • Project changed from OpenPNE3 Plugins to opCommunityTopicPlugin
Actions #2

Updated by Kousuke Ebihara over 14 years ago

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

Updated by Shingo Yamada over 14 years ago

引き受けます

Actions #5

Updated by Shingo Yamada over 14 years ago

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

Updated by Shingo Yamada over 14 years ago

現状認識している問題点

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

Updated by tu nguyen ngoc over 14 years ago

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

Actions #8

Updated by Minoru Takai over 14 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
のソースを追ってみましたがこの不具合を生じさせている箇所を特定できませんでした。
ここまでの内容を残しておきます。

Actions #9

Updated by tu nguyen ngoc over 14 years ago

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

Actions #10

Updated by tu nguyen ngoc over 14 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

Actions #11

Updated by Shingo Yamada over 14 years ago

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

Actions #12

Updated by Shingo Yamada over 14 years ago

r13586 でmigrationを追加しました

Actions #13

Updated by Kousuke Ebihara over 14 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
Actions

Also available in: Atom PDF