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

Opened 11 years ago

Last modified 11 years ago

[フロント]お問い合わせ

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

Description

▼リファクタリング対象ファイル
/contact/LC_Page_Contact.php
/contact/LC_Page_Contact_Complete.php


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


Change History

comment:1 Changed 11 years ago by yomoro

  • Owner changed from somebody to yomoro

comment:2 Changed 11 years ago by yomoro

  • Status changed from new to assigned

comment:3 Changed 11 years ago by yomoro

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

comment:4 Changed 11 years ago by kotani

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

yomoro様

株式会社ロックオン小谷です。お世話になっております。
チェックが遅くなり大変恐縮です。
/contact/LC_Page_Contact.php
に関しまして、SC_CheckError を使っている部分がございます。
http://svn.ec-cube.net/open_trac/wiki/リファクタリングガイドライン でお願いしております通り、SC_FromParam を使うようにリファクタリングして頂くことは可能でしょうか。
ご検討頂ければ幸いです。

comment:5 Changed 11 years ago by yomoro

了解しました! チェック有り難う御座います!

comment:6 Changed 11 years ago by yomoro

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

修正完了しました! お手数おかけしますが、ご確認の程、何卒よろしくお願い申し上げます。

comment:7 Changed 11 years ago by kotani

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

yomoro様

株式会社ロックオン小谷です。お世話になっております。
お忙しい中ご対応誠にありがとうございます!
早速ソースチェックしましたのでご確認お願い致します。

/contact/LC_Page_Contact.php
タブとスペースが混在し、インデントが崩れているようです。
スペース4文字のインデントに統一頂けないでしょうか。

73行目
$this->CONF = $objDb->sfGetBasisData(); 店舗基本情報
この行は不要です。

161行目
関数コメントお願いします!

お忙しい中大変恐縮ですが何卒よろしくお願い致します。

comment:8 Changed 11 years ago by yomoro

kotaniさん

すすすみません(汗 修正しました。。 何度も申し訳ないです。。。

comment:9 Changed 11 years ago by nanasess

r20190 若干修正しました.

SC_Helper_Mail::sfSendTemplateMail() 自体が冗長な処理をしているので, 余力があればこちらも修正したいですね...

comment:10 Changed 11 years ago by yomoro

おああ レベルの低いリファクタリングしててすみません。。 nanasessさん有り難う御座います!

comment:11 Changed 11 years ago by yomoro

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

取り急ぎ、こちらclosedで大丈夫でしょうか?? 修正あれば仰ってください!

Note: See TracTickets for help on using tickets.