Ticket #1015 (closed 新規開発: 修正済)
[管理画面]コンテンツ管理（CSV出力項目設定以外）
| Reported by: | kotani | Owned by: | nakanishi |
|---|---|---|---|
| Priority: | 中 | Milestone: | EC-CUBE2.11.0 リファクタリングProject |
| Component: | 管理画面 | Version: | 2.11.0 β |
| Keywords: | Cc: | ||
| 修正済み: |
Description
▼リファクタリング対象ファイル
/admin/contents/LC_Page_Admin_Contents.php
/admin/contents/LC_Page_Admin_Contents_FileManager.php
/admin/contents/LC_Page_Admin_Contents_FileView.php
/admin/contents/LC_Page_Admin_Contents_Recommend.php
/admin/contents/LC_Page_Admin_Contents_RecommendSearch.php
リファクタリングガイドライン
http://svn.ec-cube.net/open_trac/wiki/リファクタリングガイドライン
Change History
comment:3 Changed 12 years ago by nakanishi
- Status changed from assigned to closed
- Resolution set to 修正済
リファクタリングしたものをコミットしたので確認お願いします。 下記の3ファイルです。
/admin/contents/LC_Page_Admin_Contents.php /admin/contents/LC_Page_Admin_Contents_Recommend.php /admin/contents/LC_Page_Admin_Contents_RecommendSearch.php
comment:4 Changed 12 years ago by kotani
- Status changed from closed to reopened
- Resolution 修正済 deleted
/admin/contents/LC_Page_Admin_Contents.php
参照渡しとなっている引数に対して、呼び出し側で参照渡しにすると下記警告が出ます。
呼び出し側のアンパサンドを削除してください。
Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of [runtime function name](). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in /mnt/working_copy/eccube211/data/class/pages/admin/contents/LC_Page_Admin_Contents.php on line 93
順位が「1 - 1+1」と表示されています。
「」番目へ移動 が動作しません。
/admin/contents/LC_Page_Admin_Contents_Recommend.php
参照渡しとなっている引数に対して、呼び出し側で参照渡しにしないでください。
80行目、94行目、103行目
/admin/contents/LC_Page_Admin_Contents_RecommendSearch.php
参照渡しとなっている引数に対して、呼び出し側で参照渡しにしないでください。
80行目、87行目
