From imai ＠ glatts.com Fri May 14 16:56:06 2004 From: imai ＠ glatts.com (Katsunori IMAI) Date: Fri, 14 May 2004 16:56:06 +0900 Subject: [Tep-j-develop] =?iso-2022-jp?b?Mi4ybXMxahskQiUiJUklbCU5RCIbKEI=?= =?iso-2022-jp?b?GyRCNFhPIklUNnE5Z0pzOXAbKEI=?= Message-ID: <20040514163237.7339.IMAI@glatts.com> 今井と申します。はじめまして。 Tep-j-develop MLには、初めて投稿させていただきます。 2.2ms1jのアドレス帳に項目を追加するカスタマイズを行って いるときに、電話番号のチェックで不具合を発見しました。 diff形式で末尾につけておきます。ただ、私自身は項目追加の カスタマイズと同時に直したので、末尾添付の変更だけを行っ たコードでは未テストです。すみません。 宜しくお願いいたします。 余談 たぶん、account関連への電話番号追加改造後に、それを元に address_book関連の改造をされたのですね。実は私も項目追加 したときに、同様のバグを作りこんでいました。 通常はJavaScriptでチェックされ、サーバ側のチェックで引っ かかることはないので、見つけにくいですよね。 ======== diff -rc ../2.2ms1j/catalog/address_book_process.php ./catalog/address_book_process.php *** ../2.2ms1j/catalog/address_book_process.php 2003-09-21 11:38:24.000000000 +0900 --- ./catalog/address_book_process.php 2004-05-14 16:21:15.000000000 +0900 *************** *** 115,124 **** // 2003-06-06 add_telephone if (strlen($telephone) < ENTRY_TELEPHONE_MIN_LENGTH) { $error = true; - $entry_telephone_error = true; } else { ! $entry_telephone_error = false; } if (ACCOUNT_STATE == 'true') { --- 115,124 ---- // 2003-06-06 add_telephone if (strlen($telephone) < ENTRY_TELEPHONE_MIN_LENGTH) { + $telephone_error = true; $error = true; } else { ! $telephone_error = false; } if (ACCOUNT_STATE == 'true') { diff -rc ../2.2ms1j/catalog/includes/modules/address_book_details.php ./catalog/includes/modules/address_book_details.php *** ../2.2ms1j/catalog/includes/modules/address_book_details.php 2003-09-21 11:38:29.000000000 +0900 --- ./catalog/includes/modules/address_book_details.php 2004-05-14 16:13:48.000000000 +0900 *************** *** 150,163 **** // 2003-06-06 add_telephone // telephone ! if ($is_read_only == true) { ! $a_value = $account['entry_telephone']; ! } elseif ($error) { ! if ($entry_telephone_error == true) { ! $a_value = tep_draw_input_field('telephone') . ' ' . ENTRY_TELEPHONE_NUMBER_ERROR; ! } else { ! $a_value = $telephone . tep_draw_hidden_field('telephone'); ! } } else { $a_value = tep_draw_input_field('telephone', $entry['entry_telephone']) . ' ' . ENTRY_TELEPHONE_NUMBER_TEXT; } --- 150,161 ---- // 2003-06-06 add_telephone // telephone ! if ($process == true) { ! if ($telephone_error == true) { ! $a_value = tep_draw_input_field('telephone') . ' ' . ENTRY_TELEPHONE_NUMBER_ERROR; ! } else { ! $a_value = $telephone . tep_draw_hidden_field('telephone'); ! } } else { $a_value = tep_draw_input_field('telephone', $entry['entry_telephone']) . ' ' . ENTRY_TELEPHONE_NUMBER_TEXT; } diff -rc ../2.2ms1j/catalog/includes/modules/checkout_new_address.php ./catalog/includes/modules/checkout_new_address.php *** ../2.2ms1j/catalog/includes/modules/checkout_new_address.php 2003-09-21 11:38:29.000000000 +0900 --- ./catalog/includes/modules/checkout_new_address.php 2004-05-14 16:06:15.000000000 +0900 *************** *** 151,157 **** // 2003-06-06 add_telephone // telephone ! if ($telephone == true) { if ($telephone_error == true) { $a_value = tep_draw_input_field('telephone') . ' ' . ENTRY_TELEPHONE_NUMBER_ERROR; } else { --- 151,157 ---- // 2003-06-06 add_telephone // telephone ! if ($process == true) { if ($telephone_error == true) { $a_value = tep_draw_input_field('telephone') . ' ' . ENTRY_TELEPHONE_NUMBER_ERROR; } else { ======== From suzukawa ＠ bitscope.co.jp Sun May 16 13:55:43 2004 From: suzukawa ＠ bitscope.co.jp (Naomi Suzukawa) Date: Sun, 16 May 2004 13:55:43 +0900 Subject: [Tep-j-develop] =?iso-2022-jp?b?UmU6IDIuMm1zMWobJEIlIiVJJWwbKEI=?= =?iso-2022-jp?b?GyRCJTlEIjRYTyJJVDZxOWdKczlwGyhC?= In-Reply-To: <20040514163237.7339.IMAI@glatts.com> References: <20040514163237.7339.IMAI@glatts.com> Message-ID: <20040516115810.10D8.SUZUKAWA@bitscope.co.jp> 今井さん、はじめまして鈴川です。 ご指摘ありがとうございます。 ソースを修正し cvs commit しました。 # 条件判定の箇所で無意味なif文がありましたね。 # JavaScript、サーバ側の両方ともテストして、 # 意図した動作を行なうので気づきませんでした。 -- 鈴川直美 / 株式会社ビットスコープ E-mail: suzukawa ＠ bitscope.co.jp http://www.bitscope.co.jp/