Ticket #1015 (closed 新規開発: 修正済)

Opened 10 years ago

Last modified 10 years ago

[管理画面]コンテンツ管理(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:1 Changed 10 years ago by nakanishi

  • Owner changed from somebody to nakanishi

comment:2 Changed 10 years ago by nakanishi

  • Status changed from new to assigned

comment:3 Changed 10 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 10 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行目

comment:5 Changed 10 years ago by nakanishi

  • Status changed from reopened to closed
  • Resolution set to 修正済

r20296 で不具合に対して修正。

Note: See TracTickets for help on using tickets.