diff --git a/controllers/admin/AdminCarriersController.php b/controllers/admin/AdminCarriersController.php index 126134938..8f4ad9a35 100644 --- a/controllers/admin/AdminCarriersController.php +++ b/controllers/admin/AdminCarriersController.php @@ -101,54 +101,6 @@ class AdminCarriersControllerCore extends AdminController ) ); - $carrier_default_sort = array( - array('value' => Carrier::SORT_BY_PRICE, 'name' => $this->l('Price')), - array('value' => Carrier::SORT_BY_POSITION, 'name' => $this->l('Position')) - ); - - $carrier_default_order = array( - array('value' => Carrier::SORT_BY_ASC, 'name' => $this->l('Ascending')), - array('value' => Carrier::SORT_BY_DESC, 'name' => $this->l('Descending')) - ); - - $this->fields_options = array( - 'general' => array( - 'title' => $this->l('Carrier options'), - 'fields' => array( - 'PS_CARRIER_DEFAULT' => array( - 'title' => $this->l('Default carrier:'), - 'desc' => $this->l('Your shop\'s default carrier'), - 'cast' => 'intval', - 'type' => 'select', - 'identifier' => 'id_carrier', - 'list' => array_merge( - array( - -1 => array('id_carrier' => -1, 'name' => $this->l('Best price')), - -2 => array('id_carrier' => -2, 'name' => $this->l('Best grade')) - ), - Carrier::getCarriers((int)Configuration::get('PS_LANG_DEFAULT'), true, false, false, null, Carrier::ALL_CARRIERS)) - ), - 'PS_CARRIER_DEFAULT_SORT' => array( - 'title' => $this->l('Sort by:'), - 'desc' => $this->l('This will only be visible in the Front Office'), - 'cast' => 'intval', - 'type' => 'select', - 'identifier' => 'value', - 'list' => $carrier_default_sort - ), - 'PS_CARRIER_DEFAULT_ORDER' => array( - 'title' => $this->l('Order by:'), - 'desc' => $this->l('This will only be visible in the Front Office'), - 'cast' => 'intval', - 'type' => 'select', - 'identifier' => 'value', - 'list' => $carrier_default_order - ), - ), - 'submit' => array() - ) - ); - parent::__construct(); } diff --git a/controllers/admin/AdminShippingController.php b/controllers/admin/AdminShippingController.php index 58a48a4ce..ec4d01129 100644 --- a/controllers/admin/AdminShippingController.php +++ b/controllers/admin/AdminShippingController.php @@ -37,7 +37,17 @@ class AdminShippingControllerCore extends AdminController foreach ($carriers as $key => $carrier) if ($carrier['is_free']) unset($carriers[$key]); + + $carrier_default_sort = array( + array('value' => Carrier::SORT_BY_PRICE, 'name' => $this->l('Price')), + array('value' => Carrier::SORT_BY_POSITION, 'name' => $this->l('Position')) + ); + $carrier_default_order = array( + array('value' => Carrier::SORT_BY_ASC, 'name' => $this->l('Ascending')), + array('value' => Carrier::SORT_BY_DESC, 'name' => $this->l('Descending')) + ); + $this->fields_options = array( 'handling' => array( 'title' => $this->l('Handling'), @@ -68,12 +78,45 @@ class AdminShippingControllerCore extends AdminController
  • '.$this->l('Coupons are not taken into account when calculating free shipping').'
  • ', 'submit' => array() + ), + 'general' => array( + 'title' => $this->l('Carrier options'), + 'fields' => array( + 'PS_CARRIER_DEFAULT' => array( + 'title' => $this->l('Default carrier:'), + 'desc' => $this->l('Your shop\'s default carrier'), + 'cast' => 'intval', + 'type' => 'select', + 'identifier' => 'id_carrier', + 'list' => array_merge( + array( + -1 => array('id_carrier' => -1, 'name' => $this->l('Best price')), + -2 => array('id_carrier' => -2, 'name' => $this->l('Best grade')) + ), + Carrier::getCarriers((int)Configuration::get('PS_LANG_DEFAULT'), true, false, false, null, Carrier::ALL_CARRIERS)) + ), + 'PS_CARRIER_DEFAULT_SORT' => array( + 'title' => $this->l('Sort by:'), + 'desc' => $this->l('This will only be visible in the Front Office'), + 'cast' => 'intval', + 'type' => 'select', + 'identifier' => 'value', + 'list' => $carrier_default_sort + ), + 'PS_CARRIER_DEFAULT_ORDER' => array( + 'title' => $this->l('Order by:'), + 'desc' => $this->l('This will only be visible in the Front Office'), + 'cast' => 'intval', + 'type' => 'select', + 'identifier' => 'value', + 'list' => $carrier_default_order + ), + ), + 'submit' => array() ) - ); + ); } - - public function postProcess() { /* Shipping fees */ diff --git a/install-dev/data/xml/tab.xml b/install-dev/data/xml/tab.xml index 477d81b3d..f0c8d992f 100644 --- a/install-dev/data/xml/tab.xml +++ b/install-dev/data/xml/tab.xml @@ -42,9 +42,6 @@ AdminPriceRule - - AdminParentShipping - AdminParentLocalization @@ -150,12 +147,6 @@ AdminMarketing - - AdminShipping - - - AdminCarriers - AdminCarrierWizard @@ -198,6 +189,9 @@ AdminPayment + + AdminCarriers + AdminPreferences @@ -210,6 +204,9 @@ AdminCustomerPreferences + + AdminShipping + AdminThemes diff --git a/install-dev/upgrade/sql/1.5.5.0.sql b/install-dev/upgrade/sql/1.5.5.0.sql index 27118c735..c95942e20 100644 --- a/install-dev/upgrade/sql/1.5.5.0.sql +++ b/install-dev/upgrade/sql/1.5.5.0.sql @@ -22,6 +22,14 @@ CHANGE `module_name` `module_name` VARCHAR(64) NULL DEFAULT NULL; /* PHP:remove_tab(AdminRangePrice); */; /* PHP:remove_tab(AdminRangeWeight); */; +/* PHP:remove_tab(AdminShipping); */; +/* PHP:remove_tab(AdminParentShipping); */; +/* PHP:remove_tab(AdminCarriers); */; + +/* PHP:add_new_tab(AdminShipping, es:Shipping|it:Shipping|en:Shipping|de:Shipping|fr:Transport, 0, false, AdminParentPreferences); */; +/* PHP:add_new_tab(AdminCarriers, es:Carriers|it:Carriers|en:Carriers|de:Carriers|fr:Tarsporteurs, 0, false, AdminParentModules); */; + + ALTER TABLE `PREFIX_log` ADD `id_employee` INT(10) UNSIGNED NULL DEFAULT NULL AFTER `object_id`; SET @id_parent = (SELECT IFNULL(id_tab, 1) FROM `PREFIX_tab` WHERE `class_name` = 'AdminPriceRule' LIMIT 1);