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

// Small fixes on AdminProduct tab

Browse Source
This commit is contained in:
rGaillard committed 2012-01-26 12:50:41 +00:00
1 parent 75e18e0cf5
commit dc99f52f25
6 files changed
+176 -185
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

admin-dev/themes/default/template/controllers/products/prices.tpl
+10 -10
View File
@@ -25,7 +25,7 @@
*} *}
{* BEGIN CUSTOMER AUTO-COMPLETE / TO REFACTO *} {* BEGIN CUSTOMER AUTO-COMPLETE / TO REFACTO *}
{literal}
<script type="text/javascript"> <script type="text/javascript">
var Customer = { var Customer = {
"hiddenField": jQuery('#id_customer'), "hiddenField": jQuery('#id_customer'),
@@ -41,13 +41,13 @@ var Customer = {
}).focus(Customer.placeholderIn).blur(Customer.placeholderOut); }).focus(Customer.placeholderIn).blur(Customer.placeholderOut);
}, },
"placeholderIn": function() { "placeholderIn": function() {
if (this.value == '{/literal}{l s='All customers'}{literal}') { if (this.value == '{l s='All customers'}') {
this.value = ''; this.value = '';
} }
}, },
"placeholderOut": function() { "placeholderOut": function() {
if (this.value == '') { if (this.value == '') {
this.value = '{/literal}{l s='All customers'}{literal}'; this.value = '{l s='All customers'}';
} }
}, },
"search": function() "search": function()
@@ -55,13 +55,13 @@ var Customer = {
Customer.showLoader(); Customer.showLoader();
jQuery.ajax({ jQuery.ajax({
"type": "POST", "type": "POST",
"url": "{/literal}{$link->getAdminLink('AdminProducts')}{literal}", "url": "{$link->getAdminLink('AdminCustomers')}",
"async": true, "async": true,
"dataType": "json", "dataType": "json",
"data": { "data": {
"ajax": "1", "ajax": "1",
"token": "{/literal}{$token}{literal}", "token": "{getAdminToken tab='AdminCustomers'}",
"tab": "AdminProducts", "tab": "AdminCustomers",
"action": "searchCustomers", "action": "searchCustomers",
"customer_search": Customer.field.val() "customer_search": Customer.field.val()
}, },
@@ -73,14 +73,14 @@ var Customer = {
if(result.found) { if(result.found) {
var html = '<ul class="clearfix">'; var html = '<ul class="clearfix">';
jQuery.each(result.customers, function() { jQuery.each(result.customers, function() {
html += '<li><a class="fancybox" href="{/literal}{$link->getAdminLink('AdminCustomers')}{literal}&id_customer='+this.id_customer+'&viewcustomer&liteDisplaying=1">'+this.firstname+' '+this.lastname+'</a>'+(this.birthday ? ' - '+this.birthday:'')+'<br/>'; html += '<li><a class="fancybox" href="{$link->getAdminLink('AdminCustomers')}&id_customer='+this.id_customer+'&viewcustomer&liteDisplaying=1">'+this.firstname+' '+this.lastname+'</a>'+(this.birthday ? ' - '+this.birthday:'')+'<br/>';
html += '<a href="mailto:'+this.email+'">'+this.email+'</a><br />'; html += '<a href="mailto:'+this.email+'">'+this.email+'</a><br />';
html += '<a onclick="Customer.select('+this.id_customer+', \''+this.firstname+' '+this.lastname+'\'); return false;" href="#" class="button">{/literal}{l s='Choose'}{literal}</a></li>'; html += '<a onclick="Customer.select('+this.id_customer+', \''+this.firstname+' '+this.lastname+'\'); return false;" href="#" class="button">{l s='Choose'}</a></li>';
}); });
html += '</ul>'; html += '</ul>';
} }
else else
html = '<div class="warn">{/literal}{l s='No customers found'}{literal}</div>'; html = '<div class="warn">{l s='No customers found'}</div>';
Customer.hideLoader(); Customer.hideLoader();
Customer.container.html(html); Customer.container.html(html);
jQuery('.fancybox', Customer.container).fancybox(); jQuery('.fancybox', Customer.container).fancybox();
@@ -101,7 +101,7 @@ var Customer = {
}; };
jQuery(document).ready(Customer.init); jQuery(document).ready(Customer.init);
</script> </script>
{/literal}
{* END CUSTOMER AUTO-COMPLETE / TO REFACTO *} {* END CUSTOMER AUTO-COMPLETE / TO REFACTO *}
<h4>{l s='Product price'}</h4> <h4>{l s='Product price'}</h4>
admin-dev/themes/default/template/form_submit_ajax.tpl
+3 -3
View File
@@ -26,12 +26,12 @@
<script type="text/javascript"> <script type="text/javascript">
$(document).ready(function() { $(document).ready(function() {
$('#formAdd{$table}').submit(function(e) { $('#{$table}_form').submit(function(e) {
e.preventDefault(); e.preventDefault();
var form_datas = new Object; var form_datas = new Object;
form_datas['liteDisplaying'] = 1; form_datas['liteDisplaying'] = 1;
var form_inputs = $('#formAdd{$table} input, #formAdd{$table} textarea, #formAdd{$table} button'); var form_inputs = $('#{$table}_form input, #{$table}_form textarea, #{$table}_form button');
var form_selects = $('#formAdd{$table} select'); var form_selects = $('#{$table}_form select');
$.each(form_inputs, function() { $.each(form_inputs, function() {
if (this.type == 'radio' || this.type == 'checkbox') if (this.type == 'radio' || this.type == 'checkbox')
if (!this.checked) if (!this.checked)
classes/Product.php
+10 -9
View File
@@ -671,11 +671,12 @@ class ProductCore extends ObjectModel
public function deleteSelection($products) public function deleteSelection($products)
{ {
$return = 1; $return = 1;
foreach ($products as $id_product) if (is_array($products) && count($products))
{ foreach ($products as $id_product)
$product = new Product((int)$id_product); {
$return &= $product->delete(); $product = new Product((int)$id_product);
} $return &= $product->delete();
}
return $return; return $return;
} }
@@ -1500,16 +1501,16 @@ class ProductCore extends ObjectModel
); );
$sql = 'DELETE FROM `'._DB_PREFIX_.'product_attribute` $sql = 'DELETE FROM `'._DB_PREFIX_.'product_attribute`
WHERE `id_product_attribute` = '.$id_product_attribute.' WHERE `id_product_attribute` = '.(int)$id_product_attribute.'
AND `id_product` = '.$this->id; AND `id_product` = '.(int)$this->id;
$result = Db::getInstance()->execute($sql); $result = Db::getInstance()->execute($sql);
$sql = 'DELETE FROM `'._DB_PREFIX_.'product_attribute_combination` $sql = 'DELETE FROM `'._DB_PREFIX_.'product_attribute_combination`
WHERE `id_product_attribute` = '.$id_product_attribute; WHERE `id_product_attribute` = '.(int)$id_product_attribute;
$result2 = Db::getInstance()->execute($sql); $result2 = Db::getInstance()->execute($sql);
$sql = 'DELETE FROM `'._DB_PREFIX_.'cart_product` $sql = 'DELETE FROM `'._DB_PREFIX_.'cart_product`
WHERE `id_product_attribute` = '.$id_product_attribute; WHERE `id_product_attribute` = '.(int)$id_product_attribute;
$result3 = Db::getInstance()->execute($sql); $result3 = Db::getInstance()->execute($sql);
return ($result && $result2 && $result3); return ($result && $result2 && $result3);
} }
classes/SpecificPrice.php
+3 -2
View File
@@ -219,8 +219,9 @@ class SpecificPriceCore extends ObjectModel
public static function setPriorities($priorities) public static function setPriorities($priorities)
{ {
$value = ''; $value = '';
foreach ($priorities as $priority) if (is_array($priorities))
$value .= pSQL($priority).';'; foreach ($priorities as $priority)
$value .= pSQL($priority).';';
SpecificPrice::deletePriorities(); SpecificPrice::deletePriorities();
controllers/admin/AdminCustomersController.php
+13
View File
@@ -878,6 +878,19 @@ class AdminCustomersControllerCore extends AdminController
return $tpl->fetch(); return $tpl->fetch();
} }
/**
* Search customers
*/
public function ajaxProcessSearchCustomers()
{
if ($customers = Customer::searchByName(pSQL(Tools::getValue('customer_search'))))
$to_return = array('customers' => $customers, 'found' => true);
else
$to_return = array('found' => false);
$this->content = Tools::jsonEncode($to_return);
}
} }
controllers/admin/AdminProductsController.php
+137 -161
View File
@@ -33,7 +33,7 @@ class AdminProductsControllerCore extends AdminController
protected $max_file_size = null; protected $max_file_size = null;
protected $max_image_size = null; protected $max_image_size = null;
private $_category; protected $_category;
/** /**
* @var string name of the tab to display * @var string name of the tab to display
*/ */
@@ -65,7 +65,7 @@ class AdminProductsControllerCore extends AdminController
protected $default_tab = 'Informations'; protected $default_tab = 'Informations';
protected $available_tabs_lang = array (); protected $available_tabs_lang = array();
public function __construct() public function __construct()
{ {
@@ -163,14 +163,13 @@ class AdminProductsControllerCore extends AdminController
); );
/* Join categories table */ /* Join categories table */
if ($id_category = (int)Tools::getValue('productFilter_cl!name')) if ($id_category = (int)Tools::getValue('productFilter_cl!name'))
{ {
$this->_category = new Category($id_category); $this->_category = new Category((int)$id_category);
$_POST['productFilter_cl!name'] = $this->_category->name[$this->context->language->id]; $_POST['productFilter_cl!name'] = $this->_category->name[$this->context->language->id];
} }
else if ($id_category = Tools::getvalue('id_category')) elseif ($id_category = Tools::getvalue('id_category'))
$this->_category = new Category($id_category); $this->_category = new Category((int)$id_category);
else else
$this->_category = new Category(); $this->_category = new Category();
@@ -230,13 +229,13 @@ class AdminProductsControllerCore extends AdminController
$_POST['weight'] = empty($_POST['weight']) ? '0' : str_replace(',', '.', $_POST['weight']); $_POST['weight'] = empty($_POST['weight']) ? '0' : str_replace(',', '.', $_POST['weight']);
if (Tools::getIsset('unit_price') != null) if (Tools::getIsset('unit_price') != null)
$object->unit_price = str_replace(',', '.', $_POST['unit_price']); $object->unit_price = str_replace(',', '.', Tools::getValue('unit_price'));
if (array_key_exists('ecotax', $_POST) && $_POST['ecotax'] != null) if (Tools::getIsset('unit_price') != null)
$object->ecotax = str_replace(',', '.', $_POST['ecotax']); $object->ecotax = str_replace(',', '.', Tools::getValue('ecotax'));
$object->available_for_order = (int)Tools::isSubmit('available_for_order'); $object->available_for_order = (int)Tools::getValue('available_for_order');
$object->show_price = $object->available_for_order ? 1 : (int)Tools::isSubmit('show_price'); $object->show_price = $object->available_for_order ? 1 : (int)Tools::getValue('show_price');
$object->on_sale = Tools::isSubmit('on_sale'); $object->on_sale = (int)Tools::getValue('on_sale');
$object->online_only = Tools::isSubmit('online_only'); $object->online_only = (int)Tools::getValue('online_only');
} }
public function getList($id_lang, $orderBy = null, $orderWay = null, $start = 0, $limit = null, $id_lang_shop = null) public function getList($id_lang, $orderBy = null, $orderWay = null, $start = 0, $limit = null, $id_lang_shop = null)
@@ -302,12 +301,12 @@ class AdminProductsControllerCore extends AdminController
foreach ($languages as $language) foreach ($languages as $language)
{ {
$attachment_name_lang = Tools::getValue('attachment_name_'.(int)($language['id_lang'])); $attachment_name_lang = Tools::getValue('attachment_name_'.(int)($language['id_lang']));
if (strlen($attachment_name_lang ) > 0) if (Tools::strlen($attachment_name_lang ) > 0)
$is_attachment_name_valid = true; $is_attachment_name_valid = true;
if (!Validate::isGenericName(Tools::getValue('attachment_name_'.(int)($language['id_lang'])))) if (!Validate::isGenericName(Tools::getValue('attachment_name_'.(int)($language['id_lang']))))
$this->errors[] = Tools::displayError('Invalid Name'); $this->errors[] = Tools::displayError('Invalid Name');
else if (Tools::strlen(Tools::getValue('attachment_name_'.(int)($language['id_lang']))) > 32) elseif (Tools::strlen(Tools::getValue('attachment_name_'.(int)($language['id_lang']))) > 32)
$this->errors[] = Tools::displayError('Name is too long.').' '.'(32 '.Tools::displayError('chars max').')'; $this->errors[] = Tools::displayError('Name is too long.').' '.'(32 '.Tools::displayError('chars max').')';
if (!Validate::isCleanHtml(Tools::getValue('attachment_description_'.(int)($language['id_lang'])))) if (!Validate::isCleanHtml(Tools::getValue('attachment_description_'.(int)($language['id_lang']))))
$this->errors[] = Tools::displayError('Invalid description'); $this->errors[] = Tools::displayError('Invalid description');
@@ -330,7 +329,7 @@ class AdminProductsControllerCore extends AdminController
@unlink($_FILES['attachment_file']['tmp_name']); @unlink($_FILES['attachment_file']['tmp_name']);
} }
} }
else if ((int)$_FILES['attachment_file']['error'] === 1) elseif ((int)$_FILES['attachment_file']['error'] === 1)
{ {
$max_upload = (int)ini_get('upload_max_filesize'); $max_upload = (int)ini_get('upload_max_filesize');
$max_post = (int)ini_get('post_max_size'); $max_post = (int)ini_get('post_max_size');
@@ -345,14 +344,14 @@ class AdminProductsControllerCore extends AdminController
$attachment = new Attachment(); $attachment = new Attachment();
foreach ($languages as $language) foreach ($languages as $language)
{ {
if (isset($_POST['attachment_name_'.(int)$language['id_lang']])) if (Tools::getIsset('attachment_name_'.(int)$language['id_lang']))
$attachment->name[(int)$language['id_lang']] = pSQL($_POST['attachment_name_'.(int)$language['id_lang']]); $attachment->name[(int)$language['id_lang']] = Tools::getValue('attachment_name_'.(int)$language['id_lang']);
if (isset($_POST['attachment_description_'.(int)$language['id_lang']])) if (Tools::getIsset('attachment_description_'.(int)$language['id_lang']))
$attachment->description[(int)$language['id_lang']] = pSQL($_POST['attachment_description_'.(int)$language['id_lang']]); $attachment->description[(int)$language['id_lang']] = Tools::getValue('attachment_description_'.(int)$language['id_lang']);
} }
$attachment->file = $uniqid; $attachment->file = $uniqid;
$attachment->mime = $_FILES['attachment_file']['type']; $attachment->mime = $_FILES['attachment_file']['type'];
$attachment->file_name = pSQL($_FILES['attachment_file']['name']); $attachment->file_name = $_FILES['attachment_file']['name'];
if (empty($attachment->mime) || Tools::strlen($attachment->mime) > 128) if (empty($attachment->mime) || Tools::strlen($attachment->mime) > 128)
$this->errors[] = Tools::displayError('Invalid file extension'); $this->errors[] = Tools::displayError('Invalid file extension');
if (!Validate::isGenericName($attachment->file_name)) if (!Validate::isGenericName($attachment->file_name))
@@ -363,6 +362,8 @@ class AdminProductsControllerCore extends AdminController
$attachment->add(); $attachment->add();
else else
$this->errors[] = Tools::displayError('Invalid file'); $this->errors[] = Tools::displayError('Invalid file');
if (Validate::isLoadedObject($attachment))
$this->redirect_after = self::$currentIndex.'&id_product='.Tools::getValue('id_product').(isset($_POST['id_category']) ? '&id_category='.(int)$_POST['id_category'] : '').'&conf=4&add'.$this->table.'&action=Attachments&token='.($token ? $token : $this->token);
} }
} }
} }
@@ -381,13 +382,13 @@ class AdminProductsControllerCore extends AdminController
$attachments = trim(Tools::getValue('arrayAttachments'), ','); $attachments = trim(Tools::getValue('arrayAttachments'), ',');
$attachments = explode(',', $attachments); $attachments = explode(',', $attachments);
if (Attachment::attachToProduct($id, $attachments)) if (Attachment::attachToProduct($id, $attachments))
$this->redirect_after = self::$currentIndex.'&id_product='.(int)$id.(isset($_POST['id_category']) ? '&id_category='.(int)$_POST['id_category'] : '').'&conf=4&add'.$this->table.'&action=Attachments&token='.($token ? $token : $this->token); $this->redirect_after = self::$currentIndex.'&id_product='.(int)$id.(Tools::getIsset('id_category') ? '&id_category='.(int)Tools::getValue('id_category') : '').'&conf=4&add'.$this->table.'&action=Attachments&token='.($token ? $token : $this->token);
} }
} }
public function processDuplicate($token) public function processDuplicate($token)
{ {
if (Validate::isLoadedObject($product = new Product((int)(Tools::getValue('id_product'))))) if (Validate::isLoadedObject($product = new Product((int)Tools::getValue('id_product'))))
{ {
$id_product_old = $product->id; $id_product_old = $product->id;
unset($product->id); unset($product->id);
@@ -416,7 +417,7 @@ class AdminProductsControllerCore extends AdminController
{ {
Hook::exec('actionProductAdd', array('product' => $product)); Hook::exec('actionProductAdd', array('product' => $product));
Search::indexation(false, $product->id); Search::indexation(false, $product->id);
$this->redirect_after = self::$currentIndex.'&id_category='.(!empty($_REQUEST['id_category'])?$_REQUEST['id_category']:'1').'&conf=19&token='.($token ? $token : $this->token); $this->redirect_after = self::$currentIndex.'&id_category='.(Tools::getIsset('id_category') ? Tools::getValue('id_category') : '1').'&conf=19&token='.($token ? $token : $this->token);
} }
} }
else else
@@ -433,8 +434,8 @@ class AdminProductsControllerCore extends AdminController
$this->errors[] = Tools::displayError('You need at least one object.').' <b>'.$this->table.'</b><br />'.Tools::displayError('You cannot delete all of the items.'); $this->errors[] = Tools::displayError('You need at least one object.').' <b>'.$this->table.'</b><br />'.Tools::displayError('You cannot delete all of the items.');
else else
{ {
$id_category = Tools::getValue('id_category'); $id_category = (int)Tools::getValue('id_category');
$category_url = empty($id_category) ? '' : '&id_category='.$id_category; $category_url = empty($id_category) ? '' : '&id_category='.(int)$id_category;
if ($this->deleted) if ($this->deleted)
{ {
@@ -443,7 +444,7 @@ class AdminProductsControllerCore extends AdminController
if ($object->update()) if ($object->update())
$this->redirect_after = self::$currentIndex.'&conf=1&token='.($token ? $token : $this->token).$category_url; $this->redirect_after = self::$currentIndex.'&conf=1&token='.($token ? $token : $this->token).$category_url;
} }
else if ($object->delete()) elseif ($object->delete())
$this->redirect_after = self::$currentIndex.'&conf=1&token='.($token ? $token : $this->token).$category_url; $this->redirect_after = self::$currentIndex.'&conf=1&token='.($token ? $token : $this->token).$category_url;
$this->errors[] = Tools::displayError('An error occurred during deletion.'); $this->errors[] = Tools::displayError('An error occurred during deletion.');
} }
@@ -454,13 +455,13 @@ class AdminProductsControllerCore extends AdminController
public function processImage($token) public function processImage($token)
{ {
$id_image = Tools::getValue('id_image'); $id_image = (int)Tools::getValue('id_image');
$image = new Image($id_image); $image = new Image((int)$id_image);
if (Validate::isLoadedObject($image)) if (Validate::isLoadedObject($image))
{ {
/* Update product image/legend */ /* Update product image/legend */
// @todo : move in processEditProductImage // @todo : move in processEditProductImage
if (isset($_GET['editImage'])) if (Tools::getIsset('editImage'))
{ {
if ($image->cover) if ($image->cover)
$_POST['cover'] = 1; $_POST['cover'] = 1;
@@ -469,7 +470,7 @@ class AdminProductsControllerCore extends AdminController
} }
/* Choose product cover image */ /* Choose product cover image */
else if (isset($_GET['coverImage'])) elseif (Tools::getIsset('coverImage'))
{ {
Image::deleteCover($image->id_product); Image::deleteCover($image->id_product);
$image->cover = 1; $image->cover = 1;
@@ -477,18 +478,18 @@ class AdminProductsControllerCore extends AdminController
$this->errors[] = Tools::displayError('Cannot change the product cover'); $this->errors[] = Tools::displayError('Cannot change the product cover');
else else
{ {
$productId = (int)(Tools::getValue('id_product')); $productId = (int)Tools::getValue('id_product');
@unlink(_PS_TMP_IMG_DIR_.'product_'.$productId.'.jpg'); @unlink(_PS_TMP_IMG_DIR_.'product_'.$productId.'.jpg');
@unlink(_PS_TMP_IMG_DIR_.'product_mini_'.$productId.'.jpg'); @unlink(_PS_TMP_IMG_DIR_.'product_mini_'.$productId.'.jpg');
$this->redirect_after = self::$currentIndex.'&id_product='.$image->id_product.'&id_category='.(!empty($_REQUEST['id_category'])?$_REQUEST['id_category']:'1').'&action=Images&addproduct'.'&token='.($token ? $token : $this->token); $this->redirect_after = self::$currentIndex.'&id_product='.$image->id_product.'&id_category='.(Tools::getIsset('id_category') ? (int)Tools::getValue('id_category') : '1').'&action=Images&addproduct'.'&token='.($token ? $token : $this->token);
} }
} }
/* Choose product image position */ /* Choose product image position */
else if (isset($_GET['imgPosition']) && isset($_GET['imgDirection'])) elseif (Tools::getIsset('imgPosition') && Tools::getIsset('imgDirection'))
{ {
$image->updatePosition(Tools::getValue('imgDirection'), Tools::getValue('imgPosition')); $image->updatePosition(Tools::getValue('imgDirection'), Tools::getValue('imgPosition'));
$this->redirect_after = self::$currentIndex.'&id_product='.$image->id_product.'&id_category='.(!empty($_REQUEST['id_category'])?$_REQUEST['id_category']:'1').'&add'.$this->table.'&action=Images&token='.($token ? $token : $this->token); $this->redirect_after = self::$currentIndex.'&id_product='.$image->id_product.'&id_category='.(Tools::getIsset('id_category') ? (int)Tools::getValue('id_category') : '1').'&add'.$this->table.'&action=Images&token='.($token ? $token : $this->token);
} }
} }
else else
@@ -512,7 +513,7 @@ class AdminProductsControllerCore extends AdminController
{ {
foreach (Tools::getValue($this->table.'Box') as $id) foreach (Tools::getValue($this->table.'Box') as $id)
{ {
$toDelete = new $this->className($id); $toDelete = new $this->className((int)$id);
$toDelete->deleted = 1; $toDelete->deleted = 1;
$result = $result && $toDelete->update(); $result = $result && $toDelete->update();
} }
@@ -522,8 +523,8 @@ class AdminProductsControllerCore extends AdminController
if ($result) if ($result)
{ {
$id_category = Tools::getValue('id_category'); $id_category = (int)Tools::getValue('id_category');
$category_url = empty($id_category) ? '' : '&id_category='.$id_category; $category_url = empty($id_category) ? '' : '&id_category='.(int)$id_category;
$this->redirect_after = self::$currentIndex.'&conf=2&token='.$token.$category_url; $this->redirect_after = self::$currentIndex.'&conf=2&token='.$token.$category_url;
} }
@@ -537,16 +538,16 @@ class AdminProductsControllerCore extends AdminController
public function processProductAttribute($token) public function processProductAttribute($token)
{ {
// Don't process if the combination fields have not been submitted // Don't process if the combination fields have not been submitted
if (!Combination::isFeatureActive() || !isset($_POST['attribute'])) if (!Combination::isFeatureActive() || !Tools::getIsset('attribute'))
return; return;
$is_virtual = (int)Tools::getValue('is_virtual'); $is_virtual = (int)Tools::getValue('is_virtual');
if (Validate::isLoadedObject($product = $this->object)) if (Validate::isLoadedObject($product = $this->object))
{ {
if (!isset($_POST['attribute_price']) || $_POST['attribute_price'] == null) if (!Tools::getIsset('attribute_price') || Tools::getIsset('attribute_price') == null)
$this->errors[] = Tools::displayError('Attribute price required.'); $this->errors[] = Tools::displayError('Attribute price required.');
if (!isset($_POST['attribute_combinaison_list']) || empty($_POST['attribute_combinaison_list'])) if (!Tools::getIsset('attribute_combinaison_list') || Tools::isEmpty(Tools::getValue('attribute_combinaison_list')))
$this->errors[] = Tools::displayError('You must add at least one attribute.'); $this->errors[] = Tools::displayError('You must add at least one attribute.');
if (!count($this->errors)) if (!count($this->errors))
@@ -562,13 +563,13 @@ class AdminProductsControllerCore extends AdminController
{ {
if ($this->tabAccess['edit'] === '1') if ($this->tabAccess['edit'] === '1')
{ {
if ($product->productAttributeExists($_POST['attribute_combinaison_list'], $id_product_attribute)) if ($product->productAttributeExists(Tools::getValue('attribute_combinaison_list'), (int)$id_product_attribute))
$this->errors[] = Tools::displayError('This attribute already exists.'); $this->errors[] = Tools::displayError('This attribute already exists.');
else else
{ {
if (Validate::isDateFormat(Tools::getValue('available_date_attribute'))) if (Validate::isDateFormat(Tools::getValue('available_date_attribute')))
{ {
$product->updateAttribute($id_product_attribute, $product->updateAttribute((int)$id_product_attribute,
Tools::getValue('attribute_wholesale_price'), Tools::getValue('attribute_wholesale_price'),
Tools::getValue('attribute_price') * Tools::getValue('attribute_price_impact'), Tools::getValue('attribute_price') * Tools::getValue('attribute_price_impact'),
Tools::getValue('attribute_weight') * Tools::getValue('attribute_weight_impact'), Tools::getValue('attribute_weight') * Tools::getValue('attribute_weight_impact'),
@@ -583,11 +584,6 @@ class AdminProductsControllerCore extends AdminController
Tools::getValue('minimal_quantity'), Tools::getValue('minimal_quantity'),
Tools::getValue('available_date_attribute')); Tools::getValue('available_date_attribute'));
if ($id_reason = (int)Tools::getValue('id_mvt_reason') && (int)Tools::getValue('attribute_mvt_quantity') > 0 && $id_reason > 0)
{
if (!$product->addStockMvt(Tools::getValue('attribute_mvt_quantity'), $id_reason, $id_product_attribute, null, $this->context->employee->id))
$this->errors[] = Tools::displayError('An error occurred while updating qty.');
}
Hook::exec('actionProductAttributeUpdate', array('id_product_attribute' => (int)$id_product_attribute)); Hook::exec('actionProductAttributeUpdate', array('id_product_attribute' => (int)$id_product_attribute));
$this->updateDownloadProduct($product, 1, $id_product_attribute); $this->updateDownloadProduct($product, 1, $id_product_attribute);
} }
@@ -603,7 +599,7 @@ class AdminProductsControllerCore extends AdminController
{ {
if ($this->tabAccess['add'] === '1') if ($this->tabAccess['add'] === '1')
{ {
if ($product->productAttributeExists($_POST['attribute_combinaison_list'])) if ($product->productAttributeExists(Tools::getValue('attribute_combinaison_list')))
$this->errors[] = Tools::displayError('This combination already exists.'); $this->errors[] = Tools::displayError('This combination already exists.');
else else
$id_product_attribute = $product->addCombinationEntity( $id_product_attribute = $product->addCombinationEntity(
@@ -639,7 +635,7 @@ class AdminProductsControllerCore extends AdminController
if (!empty($is_virtual)) if (!empty($is_virtual))
Product::updateIsVirtual($product->id); Product::updateIsVirtual($product->id);
$this->redirect_after = self::$currentIndex.'&id_product='.$product->id.'&id_category='.(!empty($_REQUEST['id_category'])?$_REQUEST['id_category']:'1').'&add'.$this->table.'&conf=4&action=Combinations&token='.($token ? $token : $this->token); $this->redirect_after = self::$currentIndex.'&id_product='.$product->id.'&id_category='.(Tools::getIsset('id_category') ? (int)Tools::getValue('id_category') : '1').'&add'.$this->table.'&conf=4&action=Combinations&token='.($token ? $token : $this->token);
} }
} }
} }
@@ -650,7 +646,7 @@ class AdminProductsControllerCore extends AdminController
if (!Feature::isFeatureActive()) if (!Feature::isFeatureActive())
return; return;
if (Validate::isLoadedObject($product = new Product((int)(Tools::getValue('id_product'))))) if (Validate::isLoadedObject($product = new Product((int)Tools::getValue('id_product'))))
{ {
// delete all objects // delete all objects
$product->deleteFeatures(); $product->deleteFeatures();
@@ -720,13 +716,13 @@ class AdminProductsControllerCore extends AdminController
$this->errors = Tools::displayError('An error occurred while updating the specific price.'); $this->errors = Tools::displayError('An error occurred while updating the specific price.');
} }
if (!count($this->errors)) if (!count($this->errors))
$this->redirect_after = self::$currentIndex.'&id_product='.(int)(Tools::getValue('id_product')).'&id_category='.(!empty($_REQUEST['id_category'])?$_REQUEST['id_category']:'1').'&update'.$this->table.'&action=Prices&token='.($token ? $token : $this->token); $this->redirect_after = self::$currentIndex.'&id_product='.(int)(Tools::getValue('id_product')).'&id_category='.(Tools::getIsset('id_category') ? (int)Tools::getValue('id_category') : '1').'&update'.$this->table.'&action=Prices&token='.($token ? $token : $this->token);
} }
public function processPriceAddition($token) public function processPriceAddition($token)
{ {
$id_product = (int)(Tools::getValue('id_product')); $id_product = Tools::getValue('id_product');
$id_shop = Tools::getValue('sp_id_shop'); $id_shop = Tools::getValue('sp_id_shop');
$id_currency = Tools::getValue('sp_id_currency'); $id_currency = Tools::getValue('sp_id_currency');
$id_country = Tools::getValue('sp_id_country'); $id_country = Tools::getValue('sp_id_country');
@@ -741,7 +737,7 @@ class AdminProductsControllerCore extends AdminController
if ($this->_validateSpecificPrice($id_shop, $id_currency, $id_country, $id_group, $id_customer, $price, $from_quantity, $reduction, $reduction_type, $from, $to)) if ($this->_validateSpecificPrice($id_shop, $id_currency, $id_country, $id_group, $id_customer, $price, $from_quantity, $reduction, $reduction_type, $from, $to))
{ {
$specificPrice = new SpecificPrice(); $specificPrice = new SpecificPrice();
$specificPrice->id_product = $id_product; $specificPrice->id_product = (int)$id_product;
$specificPrice->id_product_attribute = (int)Tools::getValue('id_product_attribute'); $specificPrice->id_product_attribute = (int)Tools::getValue('id_product_attribute');
$specificPrice->id_shop = (int)$id_shop; $specificPrice->id_shop = (int)$id_shop;
$specificPrice->id_currency = (int)($id_currency); $specificPrice->id_currency = (int)($id_currency);
@@ -765,12 +761,12 @@ class AdminProductsControllerCore extends AdminController
{ {
if (!($obj = $this->loadObject())) if (!($obj = $this->loadObject()))
return; return;
$id_specific_price = Tools::getValue('id_specific_price'); $id_specific_price = (int)Tools::getValue('id_specific_price');
if (!$id_specific_price || !Validate::isUnsignedId($id_specific_price)) if (!$id_specific_price || !Validate::isUnsignedId($id_specific_price))
$this->errors[] = Tools::displayError('Invalid specific price ID'); $this->errors[] = Tools::displayError('Invalid specific price ID');
else else
{ {
$specificPrice = new SpecificPrice((int)($id_specific_price)); $specificPrice = new SpecificPrice((int)$id_specific_price);
if (!$specificPrice->delete()) if (!$specificPrice->delete())
$this->errors[] = Tools::displayError('An error occurred while deleting the specific price'); $this->errors[] = Tools::displayError('An error occurred while deleting the specific price');
} }
@@ -782,14 +778,14 @@ class AdminProductsControllerCore extends AdminController
return; return;
if (!$priorities = Tools::getValue('specificPricePriority')) if (!$priorities = Tools::getValue('specificPricePriority'))
$this->errors[] = Tools::displayError('Please specify priorities'); $this->errors[] = Tools::displayError('Please specify priorities');
else if (Tools::isSubmit('specificPricePriorityToAll')) elseif (Tools::isSubmit('specificPricePriorityToAll'))
{ {
if (!SpecificPrice::setPriorities($priorities)) if (!SpecificPrice::setPriorities($priorities))
$this->errors[] = Tools::displayError('An error occurred while updating priorities.'); $this->errors[] = Tools::displayError('An error occurred while updating priorities.');
else else
$this->confirmations[] = $this->l('Price rule successfully updated'); $this->confirmations[] = $this->l('Price rule successfully updated');
} }
else if (!SpecificPrice::setSpecificPriority((int)($obj->id), $priorities)) elseif (!SpecificPrice::setSpecificPriority((int)$obj->id, $priorities))
$this->errors[] = Tools::displayError('An error occurred while setting priorities.'); $this->errors[] = Tools::displayError('An error occurred while setting priorities.');
else else
$this->confirmations[] = $this->l('Price priorities successfully updated'); $this->confirmations[] = $this->l('Price priorities successfully updated');
@@ -797,19 +793,19 @@ class AdminProductsControllerCore extends AdminController
public function processCustomizationConfiguration($token) public function processCustomizationConfiguration($token)
{ {
if (Validate::isLoadedObject($product = new Product(Tools::getValue('id_product')))) if (Validate::isLoadedObject($product = new Product((int)Tools::getValue('id_product'))))
{ {
if (!$product->createLabels((int)($_POST['uploadable_files']) - (int)($product->uploadable_files), (int)($_POST['text_fields']) - (int)($product->text_fields))) if (!$product->createLabels((int)Tools::getValue('uploadable_files') - (int)$product->uploadable_files, (int)Tools::getValue('text_fields') - (int)$product->text_fields))
$this->errors[] = Tools::displayError('An error occurred while creating customization fields.'); $this->errors[] = Tools::displayError('An error occurred while creating customization fields.');
if (!count($this->errors) && !$product->updateLabels()) if (!count($this->errors) && !$product->updateLabels())
$this->errors[] = Tools::displayError('An error occurred while updating customization.'); $this->errors[] = Tools::displayError('An error occurred while updating customization.');
$product->uploadable_files = (int)($_POST['uploadable_files']); $product->uploadable_files = (int)Tools::getValue('uploadable_files');
$product->text_fields = (int)($_POST['text_fields']); $product->text_fields = (int)Tools::getValue('text_fields');
$product->customizable = ((int)($_POST['uploadable_files']) > 0 || (int)($_POST['text_fields']) > 0) ? 1 : 0; $product->customizable = ((int)Tools::getValue('uploadable_files') > 0 || (int)Tools::getValue('text_fields') > 0) ? 1 : 0;
if (!count($this->errors) && !$product->update()) if (!count($this->errors) && !$product->update())
$this->errors[] = Tools::displayError('An error occurred while updating customization configuration.'); $this->errors[] = Tools::displayError('An error occurred while updating customization configuration.');
if (!count($this->errors)) if (!count($this->errors))
$this->redirect_after = self::$currentIndex.'&id_product='.$product->id.'&id_category='.(!empty($_REQUEST['id_category'])?$_REQUEST['id_category']:'1').'&add'.$this->table.'&action=Customization&token='.($token ? $token : $this->token); $this->redirect_after = self::$currentIndex.'&id_product='.$product->id.'&id_category='.(Tools::getIsset('id_category') ? (int)Tools::getValue('id_category') : '1').'&add'.$this->table.'&action=Customization&token='.($token ? $token : $this->token);
} }
else else
$this->errors[] = Tools::displayError('Product must be created before adding customization possibilities.'); $this->errors[] = Tools::displayError('Product must be created before adding customization possibilities.');
@@ -817,7 +813,7 @@ class AdminProductsControllerCore extends AdminController
public function processProductCustomization($token) public function processProductCustomization($token)
{ {
if (Validate::isLoadedObject($product = new Product((int)(Tools::getValue('id_product'))))) if (Validate::isLoadedObject($product = new Product((int)Tools::getValue('id_product'))))
{ {
foreach ($_POST as $field => $value) foreach ($_POST as $field => $value)
if (strncmp($field, 'label_', 6) == 0 && !Validate::isLabel($value)) if (strncmp($field, 'label_', 6) == 0 && !Validate::isLabel($value))
@@ -825,7 +821,7 @@ class AdminProductsControllerCore extends AdminController
if (!count($this->errors) && !$product->updateLabels()) if (!count($this->errors) && !$product->updateLabels())
$this->errors[] = Tools::displayError('An error occurred while updating customization.'); $this->errors[] = Tools::displayError('An error occurred while updating customization.');
if (!count($this->errors)) if (!count($this->errors))
$this->redirect_after = self::$currentIndex.'&id_product='.$product->id.'&id_category='.(!empty($_REQUEST['id_category'])?$_REQUEST['id_category']:'1').'&action=Customization&add'.$this->table.'&token='.($token ? $token : $this->token); $this->redirect_after = self::$currentIndex.'&id_product='.$product->id.'&id_category='.(Tools::getIsset('id_category') ? (int)Tools::getValue('id_category') : '1').'&action=Customization&add'.$this->table.'&token='.($token ? $token : $this->token);
} }
else else
$this->errors[] = Tools::displayError('Product must be created before adding customization possibilities.'); $this->errors[] = Tools::displayError('Product must be created before adding customization possibilities.');
@@ -845,7 +841,7 @@ class AdminProductsControllerCore extends AdminController
else if (!$object->updatePosition((int)Tools::getValue('way'), (int)Tools::getValue('position'))) else if (!$object->updatePosition((int)Tools::getValue('way'), (int)Tools::getValue('position')))
$this->errors[] = Tools::displayError('Failed to update the position.'); $this->errors[] = Tools::displayError('Failed to update the position.');
else else
$this->redirect_after = self::$currentIndex.'&'.$this->table.'Orderby=position&'.$this->table.'Orderway=asc&action=Customization&conf=5'.(($id_category = (!empty($_REQUEST['id_category'])?$_REQUEST['id_category']:'1')) ? ('&id_category='.$id_category) : '').'&token='.Tools::getAdminTokenLite('AdminProducts'); $this->redirect_after = self::$currentIndex.'&'.$this->table.'Orderby=position&'.$this->table.'Orderway=asc&action=Customization&conf=5'.(($id_category = (Tools::getIsset('id_category') ? (int)Tools::getValue('id_category') : '1')) ? ('&id_category='.$id_category) : '').'&token='.Tools::getAdminTokenLite('AdminProducts');
} }
public function initProcess() public function initProcess()
@@ -858,7 +854,7 @@ class AdminProductsControllerCore extends AdminController
else else
$this->errors[] = Tools::displayError('You do not have permission to delete here.'); $this->errors[] = Tools::displayError('You do not have permission to delete here.');
} }
else if (Tools::getValue('deleteVirtualProductAttribute'))/* Delete a product in the download folder */ elseif (Tools::getValue('deleteVirtualProductAttribute'))/* Delete a product in the download folder */
{ {
if ($this->tabAccess['delete'] === '1') if ($this->tabAccess['delete'] === '1')
$this->action = 'deleteVirtualProductAttribute'; $this->action = 'deleteVirtualProductAttribute';
@@ -866,7 +862,7 @@ class AdminProductsControllerCore extends AdminController
$this->errors[] = Tools::displayError('You do not have permission to delete here.'); $this->errors[] = Tools::displayError('You do not have permission to delete here.');
} }
// Update attachments // Update attachments
else if (Tools::isSubmit('submitAddAttachments')) elseif (Tools::isSubmit('submitAddAttachments'))
{ {
if ($this->tabAccess['add'] === '1') if ($this->tabAccess['add'] === '1')
{ {
@@ -876,7 +872,7 @@ class AdminProductsControllerCore extends AdminController
else else
$this->errors[] = Tools::displayError('You do not have permission to add here.'); $this->errors[] = Tools::displayError('You do not have permission to add here.');
} }
else if (Tools::isSubmit('submitAttachments')) elseif (Tools::isSubmit('submitAttachments'))
{ {
if ($this->tabAccess['edit'] === '1') if ($this->tabAccess['edit'] === '1')
{ {
@@ -887,7 +883,7 @@ class AdminProductsControllerCore extends AdminController
$this->errors[] = Tools::displayError('You do not have permission to edit here.'); $this->errors[] = Tools::displayError('You do not have permission to edit here.');
} }
// Product duplication // Product duplication
else if (isset($_GET['duplicate'.$this->table])) elseif (Tools::getIsset('duplicate'.$this->table))
{ {
if ($this->tabAccess['add'] === '1') if ($this->tabAccess['add'] === '1')
$this->action = 'duplicate'; $this->action = 'duplicate';
@@ -895,7 +891,7 @@ class AdminProductsControllerCore extends AdminController
$this->errors[] = Tools::displayError('You do not have permission to add here.'); $this->errors[] = Tools::displayError('You do not have permission to add here.');
} }
// Product images management // Product images management
else if (Tools::getValue('id_image') && Tools::getValue('ajax')) elseif (Tools::getValue('id_image') && Tools::getValue('ajax'))
{ {
if ($this->tabAccess['edit'] === '1') if ($this->tabAccess['edit'] === '1')
$this->action = 'image'; $this->action = 'image';
@@ -903,7 +899,7 @@ class AdminProductsControllerCore extends AdminController
$this->errors[] = Tools::displayError('You do not have permission to edit here.'); $this->errors[] = Tools::displayError('You do not have permission to edit here.');
} }
// Product attributes management // Product attributes management
else if (Tools::isSubmit('submitProductAttribute')) elseif (Tools::isSubmit('submitProductAttribute'))
{ {
if ($this->tabAccess['edit'] === '1') if ($this->tabAccess['edit'] === '1')
$this->action = 'productAttribute'; $this->action = 'productAttribute';
@@ -911,7 +907,7 @@ class AdminProductsControllerCore extends AdminController
$this->errors[] = Tools::displayError('You do not have permission to edit here.'); $this->errors[] = Tools::displayError('You do not have permission to edit here.');
} }
// Product features management // Product features management
else if (Tools::isSubmit('submitFeatures') || Tools::isSubmit('submitFeaturesAndStay')) elseif (Tools::isSubmit('submitFeatures') || Tools::isSubmit('submitFeaturesAndStay'))
{ {
if ($this->tabAccess['edit'] === '1') if ($this->tabAccess['edit'] === '1')
$this->action = 'features'; $this->action = 'features';
@@ -919,28 +915,28 @@ class AdminProductsControllerCore extends AdminController
$this->errors[] = Tools::displayError('You do not have permission to edit here.'); $this->errors[] = Tools::displayError('You do not have permission to edit here.');
} }
// Product specific prices management // Product specific prices management
else if (Tools::isSubmit('submitPricesModification')) elseif (Tools::isSubmit('submitPricesModification'))
{ {
if ($this->tabAccess['add'] === '1') if ($this->tabAccess['add'] === '1')
$this->action = 'pricesModification'; $this->action = 'pricesModification';
else else
$this->errors[] = Tools::displayError('You do not have permission to add here.'); $this->errors[] = Tools::displayError('You do not have permission to add here.');
} }
else if (Tools::isSubmit('submitPriceAddition')) elseif (Tools::isSubmit('submitPriceAddition'))
{ {
if ($this->tabAccess['add'] === '1') if ($this->tabAccess['add'] === '1')
$this->action = 'priceAddition'; $this->action = 'priceAddition';
else else
$this->errors[] = Tools::displayError('You do not have permission to add here.'); $this->errors[] = Tools::displayError('You do not have permission to add here.');
} }
else if (Tools::isSubmit('deleteSpecificPrice')) elseif (Tools::isSubmit('deleteSpecificPrice'))
{ {
if ($this->tabAccess['delete'] === '1') if ($this->tabAccess['delete'] === '1')
$this->action = 'deleteSpecificPrice'; $this->action = 'deleteSpecificPrice';
else else
$this->errors[] = Tools::displayError('You do not have permission to delete here.'); $this->errors[] = Tools::displayError('You do not have permission to delete here.');
} }
else if (Tools::isSubmit('submitSpecificPricePriorities')) elseif (Tools::isSubmit('submitSpecificPricePriorities'))
{ {
if ($this->tabAccess['edit'] === '1') if ($this->tabAccess['edit'] === '1')
{ {
@@ -951,14 +947,14 @@ class AdminProductsControllerCore extends AdminController
$this->errors[] = Tools::displayError('You do not have permission to edit here.'); $this->errors[] = Tools::displayError('You do not have permission to edit here.');
} }
// Customization management // Customization management
else if (Tools::isSubmit('submitCustomizationConfiguration')) elseif (Tools::isSubmit('submitCustomizationConfiguration'))
{ {
if ($this->tabAccess['edit'] === '1') if ($this->tabAccess['edit'] === '1')
$this->action = 'customizationConfiguration'; $this->action = 'customizationConfiguration';
else else
$this->errors[] = Tools::displayError('You do not have permission to edit here.'); $this->errors[] = Tools::displayError('You do not have permission to edit here.');
} }
else if (Tools::isSubmit('submitProductCustomization')) elseif (Tools::isSubmit('submitProductCustomization'))
{ {
if ($this->tabAccess['edit'] === '1') if ($this->tabAccess['edit'] === '1')
$this->action = 'productCustomization'; $this->action = 'productCustomization';
@@ -969,8 +965,7 @@ class AdminProductsControllerCore extends AdminController
if (!$this->action) if (!$this->action)
parent::initProcess(); parent::initProcess();
if ($this->action == 'save' || $this->action == 'new' if ($this->action == 'save' || $this->action == 'new' && isset($this->available_tabs[Tools::getValue('key_tab')]))
&& isset($this->available_tabs[Tools::getValue('key_tab')]))
$this->tab_display = Tools::getValue('key_tab'); $this->tab_display = Tools::getValue('key_tab');
// Set tab to display if not decided already // Set tab to display if not decided already
@@ -1014,13 +1009,13 @@ class AdminProductsControllerCore extends AdminController
public function ajaxProcessAddImage() public function ajaxProcessAddImage()
{ {
self::$currentIndex = 'index.php?tab=AdminProducts'; self::$currentIndex = 'index.php?tab=AdminProducts';
$allowedExtensions = array('jpeg', 'jpg', 'png', 'jpg'); $allowedExtensions = array('jpeg', 'gif', 'png', 'jpg');
// max file size in bytes // max file size in bytes
$uploader = new FileUploader($allowedExtensions, $this->max_image_size); $uploader = new FileUploader($allowedExtensions, $this->max_image_size);
$result = $uploader->handleUpload(); $result = $uploader->handleUpload();
if (isset($result['success'])) if (isset($result['success']))
{ {
$obj = new Image($result['success']['id_image']); $obj = new Image((int)$result['success']['id_image']);
$json = array( $json = array(
'name' => $result['success']['name'], 'name' => $result['success']['name'],
'status' => 'ok', 'status' => 'ok',
@@ -1048,13 +1043,13 @@ class AdminProductsControllerCore extends AdminController
$id_product_attribute = (int)Tools::getValue('id_product_attribute'); $id_product_attribute = (int)Tools::getValue('id_product_attribute');
if ($id_product && Validate::isUnsignedId($id_product) && Validate::isLoadedObject($product = new Product($id_product))) if ($id_product && Validate::isUnsignedId($id_product) && Validate::isLoadedObject($product = new Product($id_product)))
{ {
$product->deleteAttributeCombination($id_product_attribute); $product->deleteAttributeCombination((int)$id_product_attribute);
$id_product_download = (int)ProductDownload::getIdFromIdAttribute($id_product, $id_product_attribute); $id_product_download = (int)ProductDownload::getIdFromIdAttribute($id_product, $id_product_attribute);
if ($id_product_download) if ($id_product_download)
{ {
$product_download = new ProductDownload($id_product_download); $product_download = new ProductDownload((int)$id_product_download);
$this->deleteDownloadProduct($id_product_download); $this->deleteDownloadProduct((int)$id_product_download);
$product_download->deleteFile(); $product_download->deleteFile();
} }
$product->checkDefaultAttributes(); $product->checkDefaultAttributes();
@@ -1118,7 +1113,7 @@ class AdminProductsControllerCore extends AdminController
{ {
$id_product = (int)Tools::getValue('id_product'); $id_product = (int)Tools::getValue('id_product');
$id_product_attribute = (int)Tools::getValue('id_product_attribute'); $id_product_attribute = (int)Tools::getValue('id_product_attribute');
if ($id_product && Validate::isUnsignedId($id_product) && Validate::isLoadedObject($product = new Product($id_product))) if ($id_product && Validate::isUnsignedId($id_product) && Validate::isLoadedObject($product = new Product((int)$id_product)))
{ {
$combinations = $product->getAttributeCombinationsById($id_product_attribute, $this->context->language->id); $combinations = $product->getAttributeCombinationsById($id_product_attribute, $this->context->language->id);
$product_download = ProductDownload::getAttributeFromIdAttribute($id_product, $id_product_attribute); $product_download = ProductDownload::getAttributeFromIdAttribute($id_product, $id_product_attribute);
@@ -1143,7 +1138,7 @@ class AdminProductsControllerCore extends AdminController
public function ajaxPreProcess() public function ajaxPreProcess()
{ {
if (isset($_GET['update'.$this->table]) && isset($_GET['id_'.$this->table])) if (Tools::getIsset('update'.$this->table) && Tools::getIsset('id_'.$this->table))
{ {
$this->display = 'edit'; $this->display = 'edit';
$this->action = Tools::getValue('action'); $this->action = Tools::getValue('action');
@@ -1153,7 +1148,7 @@ class AdminProductsControllerCore extends AdminController
public function ajaxProcessUpdateProductImageShopAsso() public function ajaxProcessUpdateProductImageShopAsso()
{ {
$this->json = true; $this->json = true;
if (($id_image = $_GET['id_image']) && ($id_shop = (int)$_GET['id_shop'])) if (($id_image = Tools::getValue('id_image')) && ($id_shop = (int)Tools::getValue('id_shop')))
if (Tools::getValue('active') == 'true') if (Tools::getValue('active') == 'true')
$res = Db::getInstance()->execute( $res = Db::getInstance()->execute(
'INSERT INTO '._DB_PREFIX_.'image_shop (`id_image`, `id_shop`) 'INSERT INTO '._DB_PREFIX_.'image_shop (`id_image`, `id_shop`)
@@ -1172,19 +1167,6 @@ class AdminProductsControllerCore extends AdminController
$this->status = 'ok'; $this->status = 'ok';
} }
/**
* Search customers
*/
public function ajaxProcessSearchCustomers()
{
if ($customers = Customer::searchByName(pSQL(Tools::getValue('customer_search'))))
$to_return = array('customers' => $customers, 'found' => true);
else
$to_return = array('found' => false);
$this->content = Tools::jsonEncode($to_return);
}
public function ajaxProcessUpdateImagePosition() public function ajaxProcessUpdateImagePosition()
{ {
$this->json = true; $this->json = true;
@@ -1211,8 +1193,8 @@ class AdminProductsControllerCore extends AdminController
public function ajaxProcessUpdateCover() public function ajaxProcessUpdateCover()
{ {
$this->json = true; $this->json = true;
Image::deleteCover((int)$_GET['id_product']); Image::deleteCover((int)Tools::getValue('id_product'));
$img = new Image((int)$_GET['id_image']); $img = new Image((int)Tools::getValue('id_image'));
$img->cover = 1; $img->cover = 1;
@unlink(_PS_TMP_IMG_DIR_.'product_'.(int)$img->id_product.'.jpg'); @unlink(_PS_TMP_IMG_DIR_.'product_'.(int)$img->id_product.'.jpg');
@@ -1276,7 +1258,7 @@ class AdminProductsControllerCore extends AdminController
protected function checkFeatures($languages, $feature_id) protected function checkFeatures($languages, $feature_id)
{ {
$rules = call_user_func(array('FeatureValue', 'getValidationRules'), 'FeatureValue'); $rules = call_user_func(array('FeatureValue', 'getValidationRules'), 'FeatureValue');
$feature = Feature::getFeature(Configuration::get('PS_LANG_DEFAULT'), $feature_id); $feature = Feature::getFeature((int)Configuration::get('PS_LANG_DEFAULT'), $feature_id);
$val = 0; $val = 0;
foreach ($languages as $language) foreach ($languages as $language)
if ($val = Tools::getValue('custom_'.$feature_id.'_'.$language['id_lang'])) if ($val = Tools::getValue('custom_'.$feature_id.'_'.$language['id_lang']))
@@ -1284,15 +1266,15 @@ class AdminProductsControllerCore extends AdminController
$current_language = new Language($language['id_lang']); $current_language = new Language($language['id_lang']);
if (Tools::strlen($val) > $rules['sizeLang']['value']) if (Tools::strlen($val) > $rules['sizeLang']['value'])
$this->errors[] = Tools::displayError('name for feature').' <b>'.$feature['name'].'</b> '.Tools::displayError('is too long in').' '.$current_language->name; $this->errors[] = Tools::displayError('name for feature').' <b>'.$feature['name'].'</b> '.Tools::displayError('is too long in').' '.$current_language->name;
else if (!call_user_func(array('Validate', $rules['validateLang']['value']), $val)) elseif (!call_user_func(array('Validate', $rules['validateLang']['value']), $val))
$this->errors[] = Tools::displayError('Valid name required for feature.').' <b>'.$feature['name'].'</b> '.Tools::displayError('in').' '.$current_language->name; $this->errors[] = Tools::displayError('Valid name required for feature.').' <b>'.$feature['name'].'</b> '.Tools::displayError('in').' '.$current_language->name;
if (count($this->errors)) if (count($this->errors))
return (0); return 0;
// Getting default language // Getting default language
if ($language['id_lang'] == Configuration::get('PS_LANG_DEFAULT')) if ($language['id_lang'] == Configuration::get('PS_LANG_DEFAULT'))
return ($val); return $val;
} }
return (0); return 0;
} }
/** /**
@@ -1305,7 +1287,7 @@ class AdminProductsControllerCore extends AdminController
/* Updating an existing product image */ /* Updating an existing product image */
if ($id_image = (int)Tools::getValue('id_image')) if ($id_image = (int)Tools::getValue('id_image'))
{ {
$image = new Image($id_image); $image = new Image((int)$id_image);
if (!Validate::isLoadedObject($image)) if (!Validate::isLoadedObject($image))
$this->errors[] = Tools::displayError('An error occurred while loading object image.'); $this->errors[] = Tools::displayError('An error occurred while loading object image.');
else else
@@ -1317,7 +1299,7 @@ class AdminProductsControllerCore extends AdminController
$this->copyFromPost($image, 'image'); $this->copyFromPost($image, 'image');
if (count($this->errors) || !$image->update()) if (count($this->errors) || !$image->update())
$this->errors[] = Tools::displayError('An error occurred while updating image.'); $this->errors[] = Tools::displayError('An error occurred while updating image.');
else if (isset($_FILES['image_product']['tmp_name']) && $_FILES['image_product']['tmp_name'] != null) elseif (isset($_FILES['image_product']['tmp_name']) && $_FILES['image_product']['tmp_name'] != null)
$this->copyImage($product->id, $image->id, $method); $this->copyImage($product->id, $image->id, $method);
} }
} }
@@ -1390,7 +1372,7 @@ class AdminProductsControllerCore extends AdminController
if (!count($this->errors)) if (!count($this->errors))
{ {
$languages = Language::getLanguages(false); $languages = Language::getLanguages(false);
if (!$this->object->updateCategories($_POST['categoryBox'])) if (!$this->object->updateCategories(Tools::getValue('categoryBox')))
$this->errors[] = Tools::displayError('An error occurred while linking object.').' <b>'.$this->table.'</b> '.Tools::displayError('To categories'); $this->errors[] = Tools::displayError('An error occurred while linking object.').' <b>'.$this->table.'</b> '.Tools::displayError('To categories');
else if (!$this->updateTags($languages, $this->object)) else if (!$this->updateTags($languages, $this->object))
$this->errors[] = Tools::displayError('An error occurred while adding tags.'); $this->errors[] = Tools::displayError('An error occurred while adding tags.');
@@ -1427,19 +1409,19 @@ class AdminProductsControllerCore extends AdminController
if (Tools::getValue('resizer') == 'man' && isset($id_image) && is_int($id_image) && $id_image) if (Tools::getValue('resizer') == 'man' && isset($id_image) && is_int($id_image) && $id_image)
$this->redirect_after = self::$currentIndex.'&id_product='.$this->object->id $this->redirect_after = self::$currentIndex.'&id_product='.$this->object->id
.'&id_category='.(!empty($_REQUEST['id_category_default'])?$_REQUEST['id_category_default']:'1') .'&id_category='.(Tools::getIsset('id_category_default') ? (int)Tools::getValue('id_category_default') : '1')
.'&id_image='.$id_image.'&imageresize&toconf=3&submitAddAndStay='.(Tools::isSubmit('submitAdd'.$this->table.'AndStay') ? 'on' : 'off') .'&id_image='.(int)$id_image.'&imageresize&toconf=3&submitAddAndStay='.(Tools::isSubmit('submitAdd'.$this->table.'AndStay') ? 'on' : 'off')
.'&token='.($token ? $token : $this->token); .'&token='.($token ? $token : $this->token);
// Save and stay on same form // Save and stay on same form
if (Tools::isSubmit('submitAdd'.$this->table.'AndStay')) if (Tools::isSubmit('submitAdd'.$this->table.'AndStay'))
$this->redirect_after = self::$currentIndex.'&id_product='.$this->object->id $this->redirect_after = self::$currentIndex.'&id_product='.(int)$this->object->id
.'&id_category='.(!empty($_REQUEST['id_category_default'])?$_REQUEST['id_category_default']:'1') .'&id_category='.(Tools::getIsset('id_category_default') ? (int)Tools::getValue('id_category_default') : '1')
.'&addproduct&conf=3&key_tab='.Tools::getValue('key_tab').'&token='.($token ? $token : $this->token); .'&addproduct&conf=3&key_tab='.Tools::safeOutput(Tools::getValue('key_tab')).'&token='.($token ? $token : $this->token);
else else
// Default behavior (save and back) // Default behavior (save and back)
$this->redirect_after = self::$currentIndex $this->redirect_after = self::$currentIndex
.'&id_category='.(!empty($_REQUEST['id_category_default'])?$_REQUEST['id_category_default']:'1') .'&id_category='.(Tools::getIsset('id_category_default') ? (int)Tools::getValue('id_category_default') : '1')
.'&conf=3&token='.($token ? $token : $this->token); .'&conf=3&token='.($token ? $token : $this->token);
} }
else else
@@ -1459,7 +1441,7 @@ class AdminProductsControllerCore extends AdminController
/* Update an existing product */ /* Update an existing product */
if (isset($id) && !empty($id)) if (isset($id) && !empty($id))
{ {
$object = new $this->className($id); $object = new $this->className((int)$id);
$this->object = $object; $this->object = $object;
if (Validate::isLoadedObject($object)) if (Validate::isLoadedObject($object))
@@ -1470,11 +1452,6 @@ class AdminProductsControllerCore extends AdminController
if ($object->update()) if ($object->update())
{ {
$this->addCarriers(); $this->addCarriers();
if ($id_reason = (int)Tools::getValue('id_mvt_reason') && Tools::getValue('mvt_quantity') > 0 && $id_reason > 0)
{
if (!$object->addStockMvt(Tools::getValue('mvt_quantity'), $id_reason, null, null, $this->context->employee->id))
$this->errors[] = Tools::displayError('An error occurred while updating qty.');
}
$this->updateAccessories($object); $this->updateAccessories($object);
$this->updateDownloadProduct($object, 1); $this->updateDownloadProduct($object, 1);
$this->updateAssoShop((int)$object->id); $this->updateAssoShop((int)$object->id);
@@ -1486,11 +1463,11 @@ class AdminProductsControllerCore extends AdminController
if (!$this->updatePackItems($object)) if (!$this->updatePackItems($object))
$this->errors[] = Tools::displayError('An error occurred while adding products to the pack.'); $this->errors[] = Tools::displayError('An error occurred while adding products to the pack.');
else if (!$object->updateCategories($_POST['categoryBox'], true)) elseif (!$object->updateCategories(Tools::getValue('categoryBox'), true))
$this->errors[] = Tools::displayError('An error occurred while linking object.').' <b>'.$this->table.'</b> '.Tools::displayError('To categories'); $this->errors[] = Tools::displayError('An error occurred while linking object.').' <b>'.$this->table.'</b> '.Tools::displayError('To categories');
else if (!$this->updateTags(Language::getLanguages(false), $object)) elseif (!$this->updateTags(Language::getLanguages(false), $object))
$this->errors[] = Tools::displayError('An error occurred while adding tags.'); $this->errors[] = Tools::displayError('An error occurred while adding tags.');
else if (empty($this->errors)) elseif (empty($this->errors))
{ {
Hook::exec('actionProductUpdate', array('product' => $object)); Hook::exec('actionProductUpdate', array('product' => $object));
Search::indexation(false, $object->id); Search::indexation(false, $object->id);
@@ -1519,7 +1496,7 @@ class AdminProductsControllerCore extends AdminController
$this->confirmations[] = $this->l('Update successful'); $this->confirmations[] = $this->l('Update successful');
else else
// Default behavior (save and back) // Default behavior (save and back)
$this->redirect_after = self::$currentIndex.'&id_category='.(!empty($_REQUEST['id_category'])?$_REQUEST['id_category']:'1').'&conf=4&token='.($token ? $token : $this->token); $this->redirect_after = self::$currentIndex.'&id_category='.(Tools::getIsset('id_category') ? (int)Tools::getValue('id_category') : '1').'&conf=4&token='.($token ? $token : $this->token);
} }
} }
} }
@@ -1537,9 +1514,9 @@ class AdminProductsControllerCore extends AdminController
public function checkProduct() public function checkProduct()
{ {
$className = 'Product'; $className = 'Product';
// @todo : the call_user_func seems to contains only statics values (clasName = "Product") // @todo : the call_user_func seems to contains only statics values (className = 'Product')
$rules = call_user_func(array($this->className, 'getValidationRules'), $this->className); $rules = call_user_func(array($this->className, 'getValidationRules'), $this->className);
$default_language = new Language((int)(Configuration::get('PS_LANG_DEFAULT'))); $default_language = new Language((int)Configuration::get('PS_LANG_DEFAULT'));
$languages = Language::getLanguages(false); $languages = Language::getLanguages(false);
/* Check required fields */ /* Check required fields */
@@ -1561,10 +1538,10 @@ class AdminProductsControllerCore extends AdminController
if ($value = Tools::getValue($field) && Tools::strlen($value) > $maxLength) if ($value = Tools::getValue($field) && Tools::strlen($value) > $maxLength)
$this->errors[] = $this->l('the field').' <b>'.call_user_func(array($className, 'displayFieldName'), $field, $className).'</b> '.$this->l('is too long').' ('.$maxLength.' '.$this->l('chars max').')'; $this->errors[] = $this->l('the field').' <b>'.call_user_func(array($className, 'displayFieldName'), $field, $className).'</b> '.$this->l('is too long').' ('.$maxLength.' '.$this->l('chars max').')';
if (isset($_POST['description_short'])) if (Tools::getIsset('description_short'))
{ {
$saveShort = $_POST['description_short']; $saveShort = Tools::getValue('description_short');
$_POST['description_short'] = strip_tags($_POST['description_short']); $_POST['description_short'] = strip_tags(Tools::getValue('description_short'));
} }
/* Check description short size without html */ /* Check description short size without html */
@@ -1646,7 +1623,7 @@ class AdminProductsControllerCore extends AdminController
$this->errors[] = $this->l('the field').' <b>'.$this->l('display filename attribute').'</b> '.$this->l('is required'); $this->errors[] = $this->l('the field').' <b>'.$this->l('display filename attribute').'</b> '.$this->l('is required');
return false; return false;
} }
else if (!empty($id_product_attribute)) elseif (!empty($id_product_attribute))
{ {
$this->errors[] = $this->l('the field').' <b>'.$this->l('display filename').'</b> '.$this->l('is required'); $this->errors[] = $this->l('the field').' <b>'.$this->l('display filename').'</b> '.$this->l('is required');
return false; return false;
@@ -1666,7 +1643,7 @@ class AdminProductsControllerCore extends AdminController
return false; return false;
} }
} }
else if (!empty($id_product_attribute)) elseif (!empty($id_product_attribute))
{ {
$this->errors[] = $this->l('the field').' <b>'.$this->l('number of days').'</b> '.$this->l('is required'); $this->errors[] = $this->l('the field').' <b>'.$this->l('number of days').'</b> '.$this->l('is required');
return false; return false;
@@ -1696,8 +1673,9 @@ class AdminProductsControllerCore extends AdminController
if ($edit == 1) if ($edit == 1)
{ {
$id_product_download_attribute = ProductDownload::getIdFromIdAttribute((int)$product->id, $id_product_attribute); $id_product_download_attribute = ProductDownload::getIdFromIdAttribute((int)$product->id, $id_product_attribute);
$id_product_download = ($id_product_download_attribute) ? (int)$id_product_download_attribute : (int)Tools::getValue('virtual_product_id'); $id_product_download = $id_product_download_attribute ? (int)$id_product_download_attribute : (int)Tools::getValue('virtual_product_id');
} else }
else
$id_product_download = Tools::getValue('virtual_product_id'); $id_product_download = Tools::getValue('virtual_product_id');
$is_shareable = Tools::getValue('virtual_product_is_shareable'); $is_shareable = Tools::getValue('virtual_product_is_shareable');
@@ -1731,12 +1709,12 @@ class AdminProductsControllerCore extends AdminController
if (!empty($virtual_product_filename_attribute)) if (!empty($virtual_product_filename_attribute))
$filename = $virtual_product_filename_attribute; $filename = $virtual_product_filename_attribute;
else if ($virtual_product_filename) elseif ($virtual_product_filename)
$filename = $virtual_product_filename; $filename = $virtual_product_filename;
else else
$filename = ProductDownload::getNewFilename(); $filename = ProductDownload::getNewFilename();
$download = new ProductDownload($id_product_download); $download = new ProductDownload((int)$id_product_download);
$download->id_product = (int)$product->id; $download->id_product = (int)$product->id;
$download->id_product_attribute = (int)$id_product_attribute; $download->id_product_attribute = (int)$id_product_attribute;
$download->display_filename = $virtual_product_name; $download->display_filename = $virtual_product_name;
@@ -1756,7 +1734,7 @@ class AdminProductsControllerCore extends AdminController
/* unactive download product if checkbox not checked */ /* unactive download product if checkbox not checked */
if ($edit == 1) if ($edit == 1)
{ {
$id_product_download_attribute = ProductDownload::getIdFromIdAttribute((int)$product->id, $id_product_attribute); $id_product_download_attribute = ProductDownload::getIdFromIdAttribute((int)$product->id, (int)$id_product_attribute);
$id_product_download = ($id_product_download_attribute) ? (int)$id_product_download_attribute : (int)Tools::getValue('virtual_product_id'); $id_product_download = ($id_product_download_attribute) ? (int)$id_product_download_attribute : (int)Tools::getValue('virtual_product_id');
} }
else else
@@ -1764,7 +1742,7 @@ class AdminProductsControllerCore extends AdminController
if (!empty($id_product_download)) if (!empty($id_product_download))
{ {
$product_download = new ProductDownload($id_product_download); $product_download = new ProductDownload((int)$id_product_download);
$product_download->date_expiration = date('Y-m-d H:i:s', time() - 1); $product_download->date_expiration = date('Y-m-d H:i:s', time() - 1);
$product_download->active = 0; $product_download->active = 0;
return $product_download->save(); return $product_download->save();
@@ -1777,7 +1755,7 @@ class AdminProductsControllerCore extends AdminController
{ {
if (!empty($id_product_attribute)) if (!empty($id_product_attribute))
{ {
$product_download = new ProductDownload($id_product_attribute); $product_download = new ProductDownload((int)$id_product_attribute);
$product_download->date_expiration = date('Y-m-d H:i:s', time() - 1); $product_download->date_expiration = date('Y-m-d H:i:s', time() - 1);
$product_download->active = 0; $product_download->active = 0;
return $product_download->save(); return $product_download->save();
@@ -1815,9 +1793,7 @@ class AdminProductsControllerCore extends AdminController
{ {
$tagError = true; $tagError = true;
/* Reset all tags for THIS product */ /* Reset all tags for THIS product */
if (!Db::getInstance()->Execute(' if (!Tag::deleteTagsForProduct((int)$product->id))
DELETE FROM `'._DB_PREFIX_.'product_tag`
WHERE `id_product` = '.(int)($product->id)))
return false; return false;
/* Assign tags to this product */ /* Assign tags to this product */
foreach ($languages as $language) foreach ($languages as $language)
@@ -1862,7 +1838,7 @@ class AdminProductsControllerCore extends AdminController
'id' => $product_tab, 'id' => $product_tab,
'selected' => (strtolower($product_tab) == strtolower($this->tab_display)), 'selected' => (strtolower($product_tab) == strtolower($this->tab_display)),
'name' => $this->available_tabs_lang[$product_tab], 'name' => $this->available_tabs_lang[$product_tab],
'href' => $this->context->link->getAdminLink('AdminProducts').'&amp;id_product='.Tools::getValue('id_product').'&amp;action='.$product_tab, 'href' => $this->context->link->getAdminLink('AdminProducts').'&amp;id_product='.(int)Tools::getValue('id_product').'&amp;action='.(int)$product_tab,
); );
} }
@@ -1872,15 +1848,15 @@ class AdminProductsControllerCore extends AdminController
else else
{ {
if ($id_category = (int)Tools::getValue('id_category')) if ($id_category = (int)Tools::getValue('id_category'))
self::$currentIndex .= '&id_category='.$id_category; self::$currentIndex .= '&id_category='.(int)$id_category;
$this->getList($this->context->language->id, !$this->context->cookie->__get($this->table.'Orderby') ? 'position' : null, !$this->context->cookie->__get($this->table.'Orderway') ? 'ASC' : null, 0, null, $this->context->shop->getID(true)); $this->getList($this->context->language->id, !$this->context->cookie->__get($this->table.'Orderby') ? 'position' : null, !$this->context->cookie->__get($this->table.'Orderway') ? 'ASC' : null, 0, null, $this->context->shop->getID(true));
$id_category = Tools::getValue('id_category', 1); $id_category = (int)Tools::getValue('id_category', 1);
$this->tpl_list_vars['is_category_filter'] = Tools::getValue('id_category') ? true : false; $this->tpl_list_vars['is_category_filter'] = Tools::getValue('id_category') ? true : false;
// Generate category selection tree // Generate category selection tree
$helper = new Helper(); $helper = new Helper();
$this->tpl_list_vars['category_tree'] = $helper->renderCategoryTree(null, array($id_category), 'categoryBox', true, false); $this->tpl_list_vars['category_tree'] = $helper->renderCategoryTree(null, array((int)$id_category), 'categoryBox', true, false);
// used to build the new url when changing category // used to build the new url when changing category
$this->tpl_list_vars['base_url'] = preg_replace('#&id_category=[0-9]*#', '', self::$currentIndex).'&token='.$this->token; $this->tpl_list_vars['base_url'] = preg_replace('#&id_category=[0-9]*#', '', self::$currentIndex).'&token='.$this->token;
@@ -1908,7 +1884,7 @@ class AdminProductsControllerCore extends AdminController
$jsonArray = array(); $jsonArray = array();
if ($manufacturers) if ($manufacturers)
foreach ($manufacturers as $manufacturer) foreach ($manufacturers as $manufacturer)
$jsonArray[] = '{"optionValue": "'.$manufacturer['id_manufacturer'].'", "optionDisplay": "'.htmlspecialchars(trim($manufacturer['name'])).'"}'; $jsonArray[] = '{"optionValue": "'.(int)$manufacturer['id_manufacturer'].'", "optionDisplay": "'.htmlspecialchars(trim($manufacturer['name'])).'"}';
die('['.implode(',', $jsonArray).']'); die('['.implode(',', $jsonArray).']');
} }
@@ -1985,7 +1961,7 @@ class AdminProductsControllerCore extends AdminController
if ($this->tabAccess['delete'] && $this->display != 'add') if ($this->tabAccess['delete'] && $this->display != 'add')
$this->toolbar_btn['delete'] = array( $this->toolbar_btn['delete'] = array(
'short' => 'Delete', 'short' => 'Delete',
'href' => $this->context->link->getAdminLink('AdminProducts').'&amp;id_product='.$product->id.'&amp;deleteproduct', 'href' => $this->context->link->getAdminLink('AdminProducts').'&amp;id_product='.(int)$product->id.'&amp;deleteproduct',
'desc' => $this->l('Delete this product'), 'desc' => $this->l('Delete this product'),
'confirm' => 1); 'confirm' => 1);
@@ -1996,7 +1972,7 @@ class AdminProductsControllerCore extends AdminController
//'href' => $this->context->link->getAdminLink('AdminProducts').'&amp;id_product='.$product->id.'&amp;duplicateproduct', //'href' => $this->context->link->getAdminLink('AdminProducts').'&amp;id_product='.$product->id.'&amp;duplicateproduct',
'desc' => $this->l('Duplicate'), 'desc' => $this->l('Duplicate'),
'confirm' => 1, 'confirm' => 1,
'js' => 'if (confirm(\''.$this->l('Also copy images').' ?\')) document.location = \''.$this->context->link->getAdminLink('AdminProducts').'&amp;id_product='.$product->id.'&amp;duplicateproduct\'; else document.location = \''.$this->context->link->getAdminLink('AdminProducts').'&amp;id_product='.$product->id.'&amp;duplicateproduct&amp;noimage=1\';' 'js' => 'if (confirm(\''.$this->l('Also copy images').' ?\')) document.location = \''.$this->context->link->getAdminLink('AdminProducts').'&amp;id_product='.(int)$product->id.'&amp;duplicateproduct\'; else document.location = \''.$this->context->link->getAdminLink('AdminProducts').'&amp;id_product='.(int)$product->id.'&amp;duplicateproduct&amp;noimage=1\';'
); );
// adding button for preview this product // adding button for preview this product
@@ -2013,7 +1989,7 @@ class AdminProductsControllerCore extends AdminController
if (file_exists(_PS_MODULE_DIR_.'statsproduct/statsproduct.php') && $this->display != 'add') if (file_exists(_PS_MODULE_DIR_.'statsproduct/statsproduct.php') && $this->display != 'add')
$this->toolbar_btn['stats'] = array( $this->toolbar_btn['stats'] = array(
'short' => 'Statistics', 'short' => 'Statistics',
'href' => $this->context->link->getAdminLink('AdminStats').'&amp;module=statsproduct&amp;id_product='.$product->id, 'href' => $this->context->link->getAdminLink('AdminStats').'&amp;module=statsproduct&amp;id_product='.(int)$product->id,
'desc' => $this->l('Product sales'), 'desc' => $this->l('Product sales'),
); );
@@ -2074,7 +2050,7 @@ class AdminProductsControllerCore extends AdminController
// getLanguages init this->_languages // getLanguages init this->_languages
$this->getLanguages(); $this->getLanguages();
$languages = $this->_languages; $languages = $this->_languages;
$default_language = (int)(Configuration::get('PS_LANG_DEFAULT')); $default_language = (int)Configuration::get('PS_LANG_DEFAULT');
$this->tpl_form_vars['currentIndex'] = self::$currentIndex; $this->tpl_form_vars['currentIndex'] = self::$currentIndex;
$this->fields_form = array(''); $this->fields_form = array('');
@@ -2089,11 +2065,11 @@ class AdminProductsControllerCore extends AdminController
$this->tpl_form_vars['ad'] = dirname($_SERVER['PHP_SELF']); $this->tpl_form_vars['ad'] = dirname($_SERVER['PHP_SELF']);
if (Validate::isLoadedObject(($this->object))) if (Validate::isLoadedObject(($this->object)))
$id_product = $this->object->id; $id_product = (int)$this->object->id;
else else
$id_product = Tools::getvalue('id_product'); $id_product = (int)Tools::getvalue('id_product');
$this->tpl_form_vars['form_action'] = $this->context->link->getAdminLink('AdminProducts').'&amp;'.($id_product ? 'id_product='.$id_product : 'addproduct'); $this->tpl_form_vars['form_action'] = $this->context->link->getAdminLink('AdminProducts').'&amp;'.($id_product ? 'id_product='.(int)$id_product : 'addproduct');
$this->tpl_form_vars['id_product'] = $id_product; $this->tpl_form_vars['id_product'] = $id_product;
// Transform configuration option 'upload_max_filesize' in octets // Transform configuration option 'upload_max_filesize' in octets
@@ -2155,7 +2131,7 @@ class AdminProductsControllerCore extends AdminController
*/ */
public function processAccounting($token) public function processAccounting($token)
{ {
if (Validate::isLoadedObject($product = new Product((int)(Tools::getValue('id_product'))))) if (Validate::isLoadedObject(($product = new Product((int)Tools::getValue('id_product')))))
{ {
$id_shop = $this->context->shop->getID(); $id_shop = $this->context->shop->getID();
@@ -2169,7 +2145,7 @@ class AdminProductsControllerCore extends AdminController
$tab[] = array( $tab[] = array(
'id_zone' => $zone['id_zone'], 'id_zone' => $zone['id_zone'],
'id_product' => $product->id, 'id_product' => $product->id,
'id_shop' => $id_shop, 'id_shop' => (int)$id_shop,
'num' => $num); 'num' => $num);
// Save to the database the account // Save to the database the account
@@ -2310,7 +2286,7 @@ class AdminProductsControllerCore extends AdminController
} }
} }
else if (Tools::isSubmit('supplier_reference_'.$product->id.'_'.$attribute['id_product_attribute'].'_'.$supplier->id_supplier)) elseif (Tools::isSubmit('supplier_reference_'.$product->id.'_'.$attribute['id_product_attribute'].'_'.$supplier->id_supplier))
{ {
//int attribute with default values if possible //int attribute with default values if possible
if ((int)$attribute['id_product_attribute'] > 0) if ((int)$attribute['id_product_attribute'] > 0)