Katsunori IMAI
imai****@glatt*****
2004年 5月 14日 (金) 16:56:06 JST
今井と申します。はじめまして。
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 {
========