ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 8 years ago

Closed 8 years ago

#4227 closed defect (fixed)

アルバム設定変更時に一部の環境でWarningが発生する

Reported by: imamura623 Owned by: kiwa
Priority: minor Milestone: OpenPNE2.14.1
Component: 指定しない Version: 2.14.x
Keywords: Cc:

Description (last modified by kiwa)

■現象

アルバムの設定変更をすると一部の環境でWarningが発生する。

携帯版の場合、画像の削除・アルバム画像編集・アルバム表紙変更などでもWarningが発生していた。

■再現環境

  • CentOS 4.7
  • PHP 4.3.9
  • Mysql 4.1.22

■原因

以下の関数を使用するとき、引数の指定が足りていなかった。

  • db_album_update_c_album
  • db_image_insert_c_image
  • db_album_image_data_delete

■修正内容

以下のように変更を行い、Warningを出さなくした

db_album_update_c_album
アルバム表紙画像の指定を任意にすることで、アルバム表紙画像を伴わない携帯での設定変更に対応した
db_image_insert_c_image
この関数を使用するdb_image_data_copy でも db_image_insert_c_image に合わせて引数を増やし、表紙画像の変更時に必須の値を指定できるようにした
db_album_image_data_delete
実行時にメンバーIDを指定していなかったものをメンバーIDを指定するよう変更、画像データの削除が正常に行われるようにした

■関連情報

http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=21251より転記

携帯からアルバムの公開範囲を変更したときに次のようなエラーが出ます。
ただし、公開範囲は変更されます。

これはバグでしょうか?

Warning: Missing argument 5 for db_album_update_c_album(), called in /hogehoge/webapp/modules/ktai/do/h_album_edit_insert_c_album.php on line 47 and defined in /hogehoge/webapp/lib/db/album.php on line 392

Warning: Cannot modify header information - headers already sent by (output started at /hogehoge/webapp/lib/db/album.php:392) in /hogehoge/webapp/

ちなみに、PHPのバージョンはPHP 5.2.10 です。 

Change History (22)

comment:1 Changed 8 years ago by kiwa

Description: modified (diff)
Keywords: 再現待ち removed
Milestone: OpenPNE2.14.1
Summary: 携帯からアルバム公開範囲を変更時にWarningが発生する携帯からのアルバム公開範囲を変更時に一部の環境でWarningが発生する
Version: 2.14.x

comment:2 Changed 8 years ago by imamura623

Description: modified (diff)

comment:3 Changed 8 years ago by mogi

以下で修正しました。確認お願いします。

comment:4 Changed 8 years ago by mogi

Keywords: 確認中 added

comment:5 Changed 8 years ago by mogi

Description: modified (diff)

comment:6 Changed 8 years ago by mogi

Description: modified (diff)

comment:7 Changed 8 years ago by shingo

未定義の変数「$image_filename」がいきなり使用されています。db_album_update_c_album()の定義側で、第5引数に初期値を設定することで「Warning」を回避してみてはどうでしょうか?

comment:8 Changed 8 years ago by kiwa

Keywords: 差し戻し added; 確認中 removed

comment:9 Changed 8 years ago by mogi

Owner: changed from nobody to mogi
Status: newassigned

comment:10 Changed 8 years ago by kiwa

Owner: changed from mogi to kiwa
Status: assignednew

引き継ぎます

comment:11 Changed 8 years ago by kiwa

Description: modified (diff)
Keywords: 確認中 added; 差し戻し removed

以下のリビジョンで未定義変数の使用からdb_album_update_c_album()にて$image_filenameにデフォルト値を指定する方法に変更しました。

comment:12 Changed 8 years ago by shingo

Keywords: 差し戻し added; 確認中 removed
  • 「,」の後ろに半角スペースを入れてください(1行が長すぎる場合はうまく改行して下さい)
    392	function db_album_update_c_album($c_album_id,$subject,$description,$public_flag,$image_filename = null) 
    
  • webapp/modules/pc/do/h_album_cover_edit_insert_c_album.php の変更は別チケットでお願いします。

comment:13 Changed 8 years ago by kiwa

Summary: 携帯からのアルバム公開範囲を変更時に一部の環境でWarningが発生する携帯からのアルバム設定変更時に一部の環境でWarningが発生する

comment:14 in reply to:  12 Changed 8 years ago by kiwa

Keywords: 確認中 added; 差し戻し removed

Replying to shingo:

以下のリビジョンで半角スペースを追加しました。

チケットについては、PC・携帯共にアルバム設定変更でWarningが発生していたので、summary/descriptionを変更する形で対応しました。

comment:15 Changed 8 years ago by imamura623

Keywords: 差し戻し added; 確認中 removed
  • PCでアルバム画像を編集を実施し、画像を変更するとWarningが発生する(画像は変更されている)

comment:16 Changed 8 years ago by kiwa

Description: modified (diff)
Keywords: 確認中 added; 差し戻し removed
Summary: 携帯からのアルバム設定変更時に一部の環境でWarningが発生するアルバム設定変更時に一部の環境でWarningが発生する

携帯版にて、アルバムの画像を伴うほぼ全ての箇所でWarningが出ていました。以下のリビジョンで対応しました。

comment:17 Changed 8 years ago by shingo

Keywords: 差し戻し added; 確認中 removed

変数に置き換える意図が分かりにくいです。981行目で「$c_albumc_member_id?」を使っているのでそれ以前で置き換えるか、もしくはそのまま関数に渡すかのどちらかにしてください。

webapp/lib/mail/sns.php

1005	            $c_member_id = $c_album['c_member_id']; 

comment:18 Changed 8 years ago by kiwa

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンでメンバーIDの指定を $c_album['c_member_id'] に変更しました

comment:19 Changed 8 years ago by shingo

Keywords: 差し戻し added; 確認中 removed
  • 引数が足りない
    webapp/lib/mail/sns.php
    
    693:            db_image_insert_c_image($filename, $image_data);
    
    742:        db_image_insert_c_image($filename, $image_data);
    
    1002:            db_image_insert_c_image($filename, $image_data);
    

comment:20 Changed 8 years ago by kiwa

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンでdb_image_insert_c_image で引数が足りていなかった箇所を修正しました。

comment:21 Changed 8 years ago by shingo

Keywords: テスト待ち added; 確認中 removed

確認しました。 #2964 のコードチェックがなされていない為、似たような不具合が無いとは言い切れませんが、 #3828 で確認することとします。

comment:22 Changed 8 years ago by imamura623

Keywords: テスト待ち removed
Resolution: fixed
Status: newclosed

動作テストし問題無いことを確認しました。

Note: See TracTickets for help on using tickets.