BIZで使用されている画像を管理画面から削除するとユーザ画面で画像がリンク切れの状態になる

[kiwa]

■現象

BIZで使用している画像（グループ画像・施設画像）を管理画面（admin_page_list_c_image / admin_page_edit_c_image）から削除すると、ユーザ画面から確認したときに画像がNoImageではなく画像がリンク切れした状態で表示される

■原因

管理画面からの削除実行時、画像データがデータベースの画像テーブル（c_image）からは削除されているがグループテーブル（biz_group）・施設テーブル（biz_shisetsu）からは削除されていない為。

■修正内容

管理画面からの削除実行時、画像データをグループテーブル（biz_group）・施設テーブル（biz_shisetsu）から削除するように修正。 biz_group・biz_shisetsuのimage_filenameに0がinsertされているのを（空文字列）に変更。

■関連情報

[shingo]

修正します。

[shingo]

以下のリビジョンで修正しました。ご確認ください。

• 2.10.x - r8262
• 2.12.x - r8260

[shingo]

修正リビジョンの追加です。

• trunk - r8261

[ebihara]

biz_shisetsu にも image_filename が存在しますが、こちらについても削除の必要があるのではないでしょうか。

mysql> DESC biz_shisetsu;
+-----------------+---------+------+-----+---------+----------------+
| Field           | Type    | Null | Key | Default | Extra          |
+-----------------+---------+------+-----+---------+----------------+
| biz_shisetsu_id | int(11) | NO   | PRI | NULL    | auto_increment | 
| name            | text    | NO   |     | NULL    |                | 
| image_filename  | text    | NO   |     | NULL    |                | 
| info            | text    | NO   |     | NULL    |                | 
+-----------------+---------+------+-----+---------+----------------+
4 rows in set (0.00 sec)

また PHP においては、特別な事情(変数の展開を文字列内でおこなう必要がある・可読性が著しく低下するなど)がない限り文字列をダブルクオートではなくシングルクオートで囲むことをお勧めします。

[ebihara]

Replying to ebihara:

また PHP においては、特別な事情(変数の展開を文字列内でおこなう必要がある・可読性が著しく低下するなど)がない限り文字列をダブルクオートではなくシングルクオートで囲むことをお勧めします。

失礼しました、この指摘は別のチケットについてのものでした。

[shingo]

以下のリビジョンで修正しました。ご確認ください。

• 2.10.x - r8287
• 2.12.x - r8285
• trunk - r8286

→「ダブルクオート・シングルクオート」の件

了解です。以後気をつけます。

[ebihara]

• これでは、 admin_do_delete_c_image から画像削除をおこなった場合に biz_group や biz_shisetsu の image_filename が空になりません。 db_admin_delete_c_image_link4image_filename() 内で削除処理をおこなうべきです
• addslashes() は不要ではないでしょうか
• UPDATE 後の image_filename の値は空文字列にするのが望ましいです
• biz_group の画像削除処理に対してはコメントを記述しているのに、 biz_shisetsu の画像削除処理についてはコメントを記述していないのはなぜでしょうか
• Desription を最新の情報に書き換えてください

[shingo]

以下のリビジョンで修正しました。ご確認ください。

• 2.10.x - r8302
• 2.12.x - r8300
• trunk - r8301

[shingo]

◆biz_group・biz_shisetsuのimage_filenameに0がinsertされている

上記のバグを発見しましたので、 以下のリビジョンで修正しました。ご確認ください。

• 2.10.x - r8305
• 2.12.x - r8303
• trunk - r8304

[ebihara]

• db_admin_delete_c_image_link4image_filename() 内の他の画像削除処理に従ってください
• biz_deleteShisetsuImage() と biz_deleteGroupImage() の UPDATE 文で image_filename を 0 にしてしまっています

[shingo]

以下のリビジョンで修正しました。ご確認ください。

• 2.10.x - r8318
• 2.12.x - r8316
• trunk - r8317

[ebihara]

ついでなので biz_deleteShisetsuImage() と biz_deleteGroupImage() は db_update() を使う形に変更してください。

[shingo]

以下のリビジョンで修正しました。ご確認ください。

• 2.10.x - r8327
• 2.12.x - r8325
• trunk - r8326

[kiwa]

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