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

Opened 10 years ago

Last modified 10 years ago

[フロント]商品

Reported by: kotani Owned by: tao
Priority: Milestone: EC-CUBE2.11.0 リファクタリングProject
Component: フロント Version: 2.11.0 β
Keywords: Cc:
修正済み:

Description

▼リファクタリング対象ファイル
/products/LC_Page_Products_CategoryList.php
/products/LC_Page_Products_Detail.php
/products/LC_Page_Products_List.php
/products/LC_Page_Products_Review.php
/products/LC_Page_Products_ReviewComplete.php
/products/LC_Page_Products_Search.php
/cart/LC_Page_Cart.php


リファクタリングガイドライン
http://svn.ec-cube.net/open_trac/wiki/リファクタリングガイドライン


Change History

comment:1 Changed 10 years ago by tao

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

やります。

comment:2 Changed 10 years ago by kotani

tao様

株式会社ロックオン小谷です。お世話になっております。
リファクタリング開発にご協力頂きありがとうございます。

r20187 でコミット頂きました /products/LC_Page_Products_List.php につきまして、
エラーが発生しページが表示できないことと、エラー部分を修正してもデータが取得できていない状態でありましたので、リビジョンを戻させて頂きました。

恐れ入りますがご確認の程よろしくお願い致します。

comment:3 Changed 10 years ago by tao

r20240 で一覧、詳細は対応しました。

comment:4 Changed 10 years ago by shutta

r20270 にて、(ある程度削除されていたが)トラックバック関連部分の削除しました。

comment:5 Changed 10 years ago by kotani

参照渡しとなっている引数に対して、呼び出し側で参照渡ししないよう修正しました。

r20170

comment:6 Changed 10 years ago by kotani

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

comment:7 Changed 10 years ago by tao

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

comment:8 follow-up: ↓ 9 Changed 10 years ago by shutta

/products/LC_Page_Products_Review.php をSC_FormParamを利用するようにリファクタリング(r20426)

comment:9 in reply to: ↑ 8 Changed 10 years ago by tao

shutta への返信

/products/LC_Page_Products_Review.php をSC_FormParamを利用するようにリファクタリング(r20426)

すいません、なかなか時間が取れなくて...

comment:10 Changed 10 years ago by kotani

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

comment:11 Changed 10 years ago by kotani

r20457
branches/version-2_5-dev/data/class/pages/products/LC_Page_Products_Detail.php

  • 不要な関数を削除(今は存在しないdtb_customer_readingへINSERTする関数)
Note: See TracTickets for help on using tickets.