Project

General

Profile

Backport(バックポート) #3399

ActivityDataTable::updateActivity() メソッドで画像付きのアクティビティが作成できない

Added by Youichi Kimura about 6 years ago. Updated almost 6 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2012-09-14
Due date:
% Done:

100%


Description

Overview (現象)

ActivityDataTable::updateActivity() メソッドには、オプションとして file_id を含む配列を渡すことで画像付きのアクティビティを作成する機能があるが、このオプションを使用すると下記のようなエラーが発生してしまう。

SQLSTATE[23000]: Integrity constraint violation: 19 activity_image.mime_type may not be NULL, referer: http://sns.localhost:18080/sqlite/

Causes (原因)

$activityImage = new ActivityImage();
if (isset($image['file_id']))
{
  $activityImage->setFileId($image['file_id']);
}
elseif (isset($image['uri']) && isset($image['mime_type']))
{
  $activityImage->setUri($image['uri']);
  $activityImage->setMimeType($image['mime_type']);
}

ActivityDataTable::updateActivity() メソッド内の images オプションを扱う上記のコードで、file_id をオプションに指定した画像について activity_image.mime_type の値が設定されておらず、activity_image テーブルの NOT NULL 制約によってエラーが発生している。

Way to fix (修正内容)

添付される画像の MIME タイプを file テーブルから取得し activity_image.mime_type にセットする。


Related issues

Related to OpenPNE 3 - Bug(バグ) #3202: ActivityDataTable::updateActivity() メソッドで画像付きのアクティビティが作成できない Won't fix(対応せず) 2012-09-14

Associated revisions

Revision 94638894 (diff)
Added by Youichi Kimura about 6 years ago

fix not-null constraint violation in ActivityDataTable::updateActivity() (fixes #3399, BP from #3202)

(cherry picked from commit ae0f61b485afabdeac7a17c4c174c67eb93be789)

History

#1 Updated by Youichi Kimura about 6 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 9463889459c77ed74b29c5784007e914810ff378 で適用されました。

#2 Updated by Shinichi Urabe about 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです

#3 Updated by Mutsumi Imamura almost 6 years ago

本チケットの試験は #3075 の試験に含みます。
#3705 の試験が完了したら本チケットも完了と致します。

#4 Updated by Mutsumi Imamura almost 6 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

#3075 の試験が完了しましたので本チケットも完了と致します。

Also available in: Atom PDF