dinislage/PrestaShop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

// Fixed #PSTEST-260

Browse Source 
git-svn-id: http://dev.prestashop.com/svn/v1/branches/1.5.x@12093 b9a71923-0436-4b27-9f14-aed3839534dd
This commit is contained in:
bMancone
2012-01-03 17:41:40 +00:00
parent e155f70095
commit f33d42c0e5
2 changed files with 95 additions and 39 deletions
Download Patch File Download Diff File Expand all files Collapse all files
admin-dev/themes/default/template/addresses/form.tpl
+34
View File
@@ -37,6 +37,40 @@
<input type="hidden" name="id_customer" value="{$customer->id}" /> <input type="hidden" name="id_customer" value="{$customer->id}" />
<input type="hidden" name="email" value="{$customer->email}" /> <input type="hidden" name="email" value="{$customer->email}" />
{else} {else}
<script type="text/javascript">
$('input[name=email]').live('blur', function(e)
{
var email = $(this).val();
if (email.length > 5)
{
var data = {};
data.email = email;
data.token = "{$token}";
data.ajax = 1;
data.controller = "AdminAddresses";
data.action = "loadNames";
$.ajax({
type: "POST",
url: "ajax-tab.php",
data: data,
dataType: 'json',
async : true,
success: function(msg)
{
if (msg)
{
var infos = msg.infos.split('_');
$('input[name=firstname]').val(infos[0]);
$('input[name=lastname]').val(infos[1]);
}
},
error: function(msg)
{
}
});
}
});
</script>
<label>{l s='Customer e-mail'}</label> <label>{l s='Customer e-mail'}</label>
<div class="margin-form"> <div class="margin-form">
<input type="text" size="33" name="email" value="{$fields_value[$input.name]|escape:'htmlall':'UTF-8'}" style="text-transform: lowercase;" /> <sup>*</sup> <input type="text" size="33" name="email" value="{$fields_value[$input.name]|escape:'htmlall':'UTF-8'}" style="text-transform: lowercase;" /> <sup>*</sup>
controllers/admin/AdminAddressesController.php
+60 -38
View File
@@ -28,7 +28,7 @@
class AdminAddressesControllerCore extends AdminController class AdminAddressesControllerCore extends AdminController
{ {
/** @var array countries list */ /** @var array countries list */
private $countriesArray = array(); private $countries_array = array();
public function __construct() public function __construct()
{ {
@@ -55,7 +55,7 @@ class AdminAddressesControllerCore extends AdminController
'address1' => array('title' => $this->l('Address')), 'address1' => array('title' => $this->l('Address')),
'postcode' => array('title' => $this->l('Postcode/ Zip Code'), 'align' => 'right', 'width' => 80), 'postcode' => array('title' => $this->l('Postcode/ Zip Code'), 'align' => 'right', 'width' => 80),
'city' => array('title' => $this->l('City'), 'width' => 150), 'city' => array('title' => $this->l('City'), 'width' => 150),
'country' => array('title' => $this->l('Country'), 'width' => 100, 'type' => 'select', 'list' => $this->countriesArray, 'filter_key' => 'cl!id_country')); 'country' => array('title' => $this->l('Country'), 'width' => 100, 'type' => 'select', 'list' => $this->countries_array, 'filter_key' => 'cl!id_country'));
parent::__construct(); parent::__construct();
} }
@@ -63,13 +63,13 @@ class AdminAddressesControllerCore extends AdminController
public function renderList() public function renderList()
{ {
$this->_select = 'cl.`name` as country'; $this->_select = 'cl.`name` as country';
$this->_join = 'LEFT JOIN `'._DB_PREFIX_.'country_lang` cl ON $this->_join = '
(cl.`id_country` = a.`id_country` AND cl.`id_lang` = '.(int)$this->context->language->id.')'; LEFT JOIN `'._DB_PREFIX_.'country_lang` cl ON (cl.`id_country` = a.`id_country` AND cl.`id_lang` = '.(int)$this->context->language->id.')';
$this->_where = 'AND a.id_customer != 0'; $this->_where = 'AND a.id_customer != 0';
$countries = Country::getCountries($this->context->language->id); $countries = Country::getCountries($this->context->language->id);
foreach ($countries AS $country) foreach ($countries as $country)
$this->countriesArray[$country['id_country']] = $country['name']; $this->countries_array[$country['id_country']] = $country['name'];
return parent::renderList(); return parent::renderList();
} }
@@ -140,12 +140,12 @@ class AdminAddressesControllerCore extends AdminController
if ($id_customer) if ($id_customer)
{ {
$customer = new Customer((int)$id_customer); $customer = new Customer((int)$id_customer);
$tokenCustomer = Tools::getAdminToken('AdminCustomers'.(int)(Tab::getIdFromClassName('AdminCustomers')).(int)$this->context->employee->id); $token_customer = Tools::getAdminToken('AdminCustomers'.(int)(Tab::getIdFromClassName('AdminCustomers')).(int)$this->context->employee->id);
} }
// @todo in 1.4, this include was done before the class declaration // @todo in 1.4, this include was done before the class declaration
// We should use a hook now // We should use a hook now
if (Configuration::get('VATNUMBER_MANAGEMENT') AND file_exists(_PS_MODULE_DIR_.'vatnumber/vatnumber.php')) if (Configuration::get('VATNUMBER_MANAGEMENT') && file_exists(_PS_MODULE_DIR_.'vatnumber/vatnumber.php'))
include_once(_PS_MODULE_DIR_.'vatnumber/vatnumber.php'); include_once(_PS_MODULE_DIR_.'vatnumber/vatnumber.php');
if (Configuration::get('VATNUMBER_MANAGEMENT')) if (Configuration::get('VATNUMBER_MANAGEMENT'))
if (file_exists(_PS_MODULE_DIR_.'vatnumber/vatnumber.php') && VatNumber::isApplicable(Configuration::get('PS_COUNTRY_DEFAULT'))) if (file_exists(_PS_MODULE_DIR_.'vatnumber/vatnumber.php') && VatNumber::isApplicable(Configuration::get('PS_COUNTRY_DEFAULT')))
@@ -156,16 +156,17 @@ class AdminAddressesControllerCore extends AdminController
$this->tpl_form_vars = array( $this->tpl_form_vars = array(
'vat' => isset($vat) ? $vat : null, 'vat' => isset($vat) ? $vat : null,
'customer' => isset($customer) ? $customer : null, 'customer' => isset($customer) ? $customer : null,
'tokenCustomer' => isset ($tokenCustomer) ? $tokenCustomer : null 'tokenCustomer' => isset ($token_customer) ? $token_customer : null
); );
// Order address fields depending on country format // Order address fields depending on country format
$addresses_fields = $this->processAddressFormat(); $addresses_fields = $this->processAddressFormat();
$addresses_fields = $addresses_fields["dlv_all_fields"]; // we use delivery address // we use delivery address
$addresses_fields = $addresses_fields['dlv_all_fields'];
$temp_fields = array(); $temp_fields = array();
foreach($addresses_fields as $addr_field_item) foreach ($addresses_fields as $addr_field_item)
{ {
if ($addr_field_item == 'company') if ($addr_field_item == 'company')
{ {
@@ -184,7 +185,7 @@ class AdminAddressesControllerCore extends AdminController
'size' => 33, 'size' => 33,
); );
} }
elseif ($addr_field_item == 'lastname') else if ($addr_field_item == 'lastname')
{ {
$temp_fields[] = array( $temp_fields[] = array(
'type' => 'text', 'type' => 'text',
@@ -195,7 +196,7 @@ class AdminAddressesControllerCore extends AdminController
'hint' => $this->l('Invalid characters:').' 0-9!<>,;?=+()@#"�{}_$%:<span class="hint-pointer">&nbsp;</span>' 'hint' => $this->l('Invalid characters:').' 0-9!<>,;?=+()@#"�{}_$%:<span class="hint-pointer">&nbsp;</span>'
); );
} }
elseif ($addr_field_item == 'firstname') else if ($addr_field_item == 'firstname')
{ {
$temp_fields[] = array( $temp_fields[] = array(
'type' => 'text', 'type' => 'text',
@@ -206,7 +207,7 @@ class AdminAddressesControllerCore extends AdminController
'hint' => $this->l('Invalid characters:').' 0-9!<>,;?=+()@#"�{}_$%:<span class="hint-pointer">&nbsp;</span>' 'hint' => $this->l('Invalid characters:').' 0-9!<>,;?=+()@#"�{}_$%:<span class="hint-pointer">&nbsp;</span>'
); );
} }
elseif ($addr_field_item == 'address1') else if ($addr_field_item == 'address1')
{ {
$temp_fields[] = array( $temp_fields[] = array(
'type' => 'text', 'type' => 'text',
@@ -216,7 +217,7 @@ class AdminAddressesControllerCore extends AdminController
'required' => true, 'required' => true,
); );
} }
elseif ($addr_field_item == 'address2') else if ($addr_field_item == 'address2')
{ {
$temp_fields[] = array( $temp_fields[] = array(
'type' => 'text', 'type' => 'text',
@@ -236,7 +237,7 @@ class AdminAddressesControllerCore extends AdminController
'required' => false, 'required' => false,
); );
} }
elseif ($addr_field_item == 'city') else if ($addr_field_item == 'city')
{ {
$temp_fields[] = array( $temp_fields[] = array(
'type' => 'text', 'type' => 'text',
@@ -246,7 +247,7 @@ class AdminAddressesControllerCore extends AdminController
'required' => true, 'required' => true,
); );
} }
elseif ($addr_field_item == 'country' || $addr_field_item == 'Country:name') else if ($addr_field_item == 'country' || $addr_field_item == 'Country:name')
{ {
$temp_fields[] = array( $temp_fields[] = array(
'type' => 'select', 'type' => 'select',
@@ -274,7 +275,7 @@ class AdminAddressesControllerCore extends AdminController
} }
} }
if (!Tools::isSubmit('submit'.strtoupper($this->table)) && Validate::isLoadedObject($customer) && !Validate::isLoadedObject($this->object)) if (isset($customer) && !Tools::isSubmit('submit'.strtoupper($this->table)) && Validate::isLoadedObject($customer) && !Validate::isLoadedObject($this->object))
{ {
$this->fields_value['lastname'] = $customer->lastname; $this->fields_value['lastname'] = $customer->lastname;
$this->fields_value['firstname'] = $customer->firstname; $this->fields_value['firstname'] = $customer->firstname;
@@ -298,9 +299,9 @@ class AdminAddressesControllerCore extends AdminController
else else
$this->_errors[] = Tools::displayError('This e-mail address is not registered.'); $this->_errors[] = Tools::displayError('This e-mail address is not registered.');
} }
elseif ($id_customer = Tools::getValue('id_customer')) else if ($id_customer = Tools::getValue('id_customer'))
{ {
$customer = new Customer((int)($id_customer)); $customer = new Customer((int)$id_customer);
if (Validate::isLoadedObject($customer)) if (Validate::isLoadedObject($customer))
$_POST['id_customer'] = $customer->id; $_POST['id_customer'] = $customer->id;
else else
@@ -308,23 +309,25 @@ class AdminAddressesControllerCore extends AdminController
} }
else else
$this->_errors[] = Tools::displayError('Unknown customer'); $this->_errors[] = Tools::displayError('Unknown customer');
if (Country::isNeedDniByCountryId(Tools::getValue('id_country')) AND !Tools::getValue('dni')) if (Country::isNeedDniByCountryId(Tools::getValue('id_country')) && !Tools::getValue('dni'))
$this->_errors[] = Tools::displayError('Identification number is incorrect or has already been used.'); $this->_errors[] = Tools::displayError('Identification number is incorrect or has already been used.');
/* If the selected country does not contain states */ /* If the selected country does not contain states */
$id_state = (int)(Tools::getValue('id_state')); $id_state = (int)Tools::getValue('id_state');
if ($id_country = Tools::getValue('id_country') AND $country = new Country((int)($id_country)) AND !(int)($country->contains_states) AND $id_state) $id_country = (int)Tools::getValue('id_country');
$country = new Country((int)$id_country);
if ($country && !(int)$country->contains_states && $id_state)
$this->_errors[] = Tools::displayError('You have selected a state for a country that does not contain states.'); $this->_errors[] = Tools::displayError('You have selected a state for a country that does not contain states.');
/* If the selected country contains states, then a state have to be selected */ /* If the selected country contains states, then a state have to be selected */
if ((int)($country->contains_states) AND !$id_state) if ((int)$country->contains_states && !$id_state)
$this->_errors[] = Tools::displayError('An address located in a country containing states must have a state selected.'); $this->_errors[] = Tools::displayError('An address located in a country containing states must have a state selected.');
/* Check zip code */ /* Check zip code */
if ($country->need_zip_code) if ($country->need_zip_code)
{ {
$zip_code_format = $country->zip_code_format; $zip_code_format = $country->zip_code_format;
if (($postcode = Tools::getValue('postcode')) AND $zip_code_format) if (($postcode = Tools::getValue('postcode')) && $zip_code_format)
{ {
$zip_regexp = '/^'.$zip_code_format.'$/ui'; $zip_regexp = '/^'.$zip_code_format.'$/ui';
$zip_regexp = str_replace(' ', '( |)', $zip_regexp); $zip_regexp = str_replace(' ', '( |)', $zip_regexp);
@@ -333,17 +336,19 @@ class AdminAddressesControllerCore extends AdminController
$zip_regexp = str_replace('L', '[a-zA-Z]', $zip_regexp); $zip_regexp = str_replace('L', '[a-zA-Z]', $zip_regexp);
$zip_regexp = str_replace('C', $country->iso_code, $zip_regexp); $zip_regexp = str_replace('C', $country->iso_code, $zip_regexp);
if (!preg_match($zip_regexp, $postcode)) if (!preg_match($zip_regexp, $postcode))
$this->_errors[] = Tools::displayError('Your zip/postal code is incorrect.').'<br />'.Tools::displayError('Must be typed as follows:').' '.str_replace('C', $country->iso_code, str_replace('N', '0', str_replace('L', 'A', $zip_code_format))); $this->_errors[] = Tools::displayError('Your zip/postal code is incorrect.').'<br />'.
Tools::displayError('Must be typed as follows:').' '.
str_replace('C', $country->iso_code, str_replace('N', '0', str_replace('L', 'A', $zip_code_format)));
} }
elseif ($zip_code_format) else if ($zip_code_format)
$this->_errors[] = Tools::displayError('Postcode required.'); $this->_errors[] = Tools::displayError('Postcode required.');
elseif ($postcode AND !preg_match('/^[0-9a-zA-Z -]{4,9}$/ui', $postcode)) else if ($postcode && !preg_match('/^[0-9a-zA-Z -]{4,9}$/ui', $postcode))
$this->_errors[] = Tools::displayError('Your zip/postal code is incorrect.'); $this->_errors[] = Tools::displayError('Your zip/postal code is incorrect.');
} }
/* If this address come from order's edition and is the same as the other one (invoice or delivery one) /* If this address come from order's edition and is the same as the other one (invoice or delivery one)
** we delete its id_address to force the creation of a new one */ ** we delete its id_address to force the creation of a new one */
if ((int)(Tools::getValue('id_order'))) if ((int)Tools::getValue('id_order'))
{ {
$this->_redirect = false; $this->_redirect = false;
if (isset($_POST['address_type'])) if (isset($_POST['address_type']))
@@ -354,10 +359,10 @@ class AdminAddressesControllerCore extends AdminController
parent::processSave($token); parent::processSave($token);
/* Reassignation of the order's new (invoice or delivery) address */ /* Reassignation of the order's new (invoice or delivery) address */
$address_type = ((int)(Tools::getValue('address_type')) == 2 ? 'invoice' : ((int)(Tools::getValue('address_type')) == 1 ? 'delivery' : '')); $address_type = ((int)Tools::getValue('address_type') == 2 ? 'invoice' : ((int)Tools::getValue('address_type') == 1 ? 'delivery' : ''));
if ($this->action == 'save' AND ($id_order = (int)(Tools::getValue('id_order'))) AND !sizeof($this->_errors) AND !empty($address_type)) if ($this->action == 'save' && ($id_order = (int)Tools::getValue('id_order')) && !count($this->_errors) && !empty($address_type))
{ {
if(!Db::getInstance()->execute('UPDATE '._DB_PREFIX_.'orders SET `id_address_'.$address_type.'` = '.Db::getInstance()->Insert_ID().' WHERE `id_order` = '.$id_order)) if (!Db::getInstance()->execute('UPDATE '._DB_PREFIX_.'orders SET `id_address_'.$address_type.'` = '.Db::getInstance()->Insert_ID().' WHERE `id_order` = '.$id_order))
$this->_errors[] = Tools::displayError('An error occurred while linking this address to its order.'); $this->_errors[] = Tools::displayError('An error occurred while linking this address to its order.');
else else
Tools::redirectAdmin(Tools::getValue('back').'&conf=4'); Tools::redirectAdmin(Tools::getValue('back').'&conf=4');
@@ -371,13 +376,12 @@ class AdminAddressesControllerCore extends AdminController
*/ */
protected function processAddressFormat() protected function processAddressFormat()
{ {
$tmp_addr = new Address((int)Tools::getValue("id_address")); $tmp_addr = new Address((int)Tools::getValue('id_address'));
$selectedCountry = ($tmp_addr && $tmp_addr->id_country) ? $tmp_addr->id_country : $selected_country = ($tmp_addr && $tmp_addr->id_country) ? $tmp_addr->id_country : (int)Configuration::get('PS_COUNTRY_DEFAULT');
(int)(Configuration::get('PS_COUNTRY_DEFAULT'));
$inv_adr_fields = AddressFormat::getOrderedAddressFields($selectedCountry, false, true); $inv_adr_fields = AddressFormat::getOrderedAddressFields($selected_country, false, true);
$dlv_adr_fields = AddressFormat::getOrderedAddressFields($selectedCountry, false, true); $dlv_adr_fields = AddressFormat::getOrderedAddressFields($selected_country, false, true);
$inv_all_fields = array(); $inv_all_fields = array();
$dlv_all_fields = array(); $dlv_all_fields = array();
@@ -390,11 +394,29 @@ class AdminAddressesControllerCore extends AdminController
foreach(explode(' ',$fields_line) as $field_item) foreach(explode(' ',$fields_line) as $field_item)
${$adr_type.'_all_fields'}[] = trim($field_item); ${$adr_type.'_all_fields'}[] = trim($field_item);
$out[$adr_type.'_adr_fields'] = ${$adr_type.'_adr_fields'}; $out[$adr_type.'_adr_fields'] = ${$adr_type.'_adr_fields'};
$out[$adr_type.'_all_fields'] = ${$adr_type.'_all_fields'}; $out[$adr_type.'_all_fields'] = ${$adr_type.'_all_fields'};
} }
return $out; return $out;
} }
/**
* Method called when an ajax request is made
* @see AdminController::postProcess()
*/
public function ajaxProcess()
{
if (Tools::isSubmit('email'))
{
$email = pSQL(Tools::getValue('email'));
$customer = Customer::searchByName($email);
if (!empty($customer))
{
$customer = $customer['0'];
echo Tools::jsonEncode(array('infos' => pSQL($customer['firstname']).'_'.pSQL($customer['lastname'])));
}
}
die;
}
} }