Ticket #978 (closed 新規開発: 修正済)
[フロント]お問い合わせ
| 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:3 Changed 13 years ago by yomoro
- Status changed from assigned to closed
- Resolution set to 修正済
comment:4 Changed 13 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:6 Changed 13 years ago by yomoro
- Status changed from reopened to closed
- Resolution set to 修正済
修正完了しました！ お手数おかけしますが、ご確認の程、何卒よろしくお願い申し上げます。
comment:7 Changed 13 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:9 Changed 13 years ago by nanasess
r20190 若干修正しました.
SC_Helper_Mail::sfSendTemplateMail() 自体が冗長な処理をしているので, 余力があればこちらも修正したいですね...
comment:10 Changed 13 years ago by yomoro
おああ レベルの低いリファクタリングしててすみません。。 nanasessさん有り難う御座います！
comment:11 Changed 13 years ago by yomoro
- Status changed from reopened to closed
- Resolution set to 修正済
取り急ぎ、こちらclosedで大丈夫でしょうか？？ 修正あれば仰ってください！
