Ticket #1862 (closed 改善提案: 修正済)

Opened 11 years ago

Last modified 10 years ago

E_WARNINGエラーが出る部分を修正

Reported by: shutta Owned by: shutta
Priority: 低 Milestone: EC-CUBE2.12.2
Component: 管理画面 Version: 2.12.0
Keywords: Cc:
修正済み: yes

Description

error.logにE_WARNINGエラーが記録されるが、システムエラー時の解析等に邪魔になったりするので、修正可能なものはぼちぼちと修正するようにする。 そもそもエラーが出ない方が良いでしょ。

Change History

comment:1 Changed 11 years ago by shutta

  • Owner changed from somebody to shutta
  • Status changed from new to assigned

r21905 

[/version-2_12-dev/html/admin/products/product_class.php] Warning(E_WARNING): Invalid argument supplied for foreach() on [/var/www/version-2_12-dev/data/class/pages/admin/products/LC_Page_Admin_Products_ProductClass.php(641)]

のようにエラーになっていたのを修正。

r21906 

[/version-2_12-dev/html/admin/products/product.php] Warning(E_WARNING): htmlspecialchars() expects parameter 1 to be string, array given on [/var/www/version-2_12-dev/data/smarty_extends/modifier.h.php(22)]

のようにエラーになっていたのを修正。
htmlspecialcharsに配列が渡ってしまっていたのが原因。
hiddenに埋めなくても良いパラメーターだったので、該当キーの場合は除外するようにした。
いっそのこと、/data/smarty_extends/modifier.h.php側で、配列が渡ってきた場合は処理しないようにしても良いかも。

comment:2 Changed 11 years ago by shutta

r21909 

[/version-2_12-dev/html/admin/products/class.php] Warning(E_WARNING): Missing argument 1 for LC_Page_Admin_Products_Class::lfGetClass(), called in /var/www/version-2_12-dev/data/class/pages/admin/products/LC_Page_Admin_Products_Class.php on line 129 and defined on [/var/www/version-2_12-dev/data/class/pages/admin/products/LC_Page_Admin_Products_Class.php(162)]

のようにエラーになっていたのを修正。
lfGetClassメソッドに引数が定義されているが、実際は何も渡していないのが原因だった。

comment:3 Changed 11 years ago by adachi

  • Milestone changed from EC-CUBE2.12.2 to EC-CUBE 2.12.3

comment:4 Changed 10 years ago by kim

  • Status changed from assigned to closed
  • 修正済み set
  • Resolution set to 修正済
  • Milestone changed from EC-CUBE2.12.3 to EC-CUBE2.12.2

shutta様 ありがとうございます。 こちらは、2.12.2でクローズとさせていただきます。 同様の問題がまだございましたら、お手数ですが、再度チケットを作成の上、コミットいただけますと幸いです。

Note: See TracTickets for help on using tickets.