From a3f7f1909c61ae2b6609a3909a7069a7d11f1a55 Mon Sep 17 00:00:00 2001
From: lLefevre <loic.lefevre@prestashop.com>
Date: Mon, 5 Dec 2011 14:24:59 +0000
Subject: [PATCH] // adding edit action for combination in
 AdminProductsController

---
 .../products/combinaison/list_footer.tpl      |  3 -
 .../themes/template/products/combinations.tpl |  1 -
 controllers/admin/AdminProductsController.php | 13 +++
 js/admin-products.js                          | 89 +++++++++++++------
 js/attributesBack.js                          |  3 -
 5 files changed, 74 insertions(+), 35 deletions(-)

diff --git a/admin-dev/themes/template/products/combinaison/list_footer.tpl b/admin-dev/themes/template/products/combinaison/list_footer.tpl
index 3691d5e65..b6a245335 100644
--- a/admin-dev/themes/template/products/combinaison/list_footer.tpl
+++ b/admin-dev/themes/template/products/combinaison/list_footer.tpl
@@ -44,9 +44,6 @@
 		var s_impact = $('#span_impact');
 		var s_impact2 = $('#span_weight_impact');
 
-		$('#desc-product-newCombinaison').attr('add', '{l s='Add a new combination'}');
-		$('#desc-product-newCombinaison').attr('cancel', '{l s='Cancel the combination'}');
-
 		init_elems();
 	</script>
 
diff --git a/admin-dev/themes/template/products/combinations.tpl b/admin-dev/themes/template/products/combinations.tpl
index 7b24ab249..af0a0aef8 100644
--- a/admin-dev/themes/template/products/combinations.tpl
+++ b/admin-dev/themes/template/products/combinations.tpl
@@ -75,7 +75,6 @@
 <div class="separation"></div>
 
 <div id="add_new_combination" style="display: none;">
-	<input type="hidden" name="id_product_attribute" id="id_product_attribute" value="0" />
 	<table cellpadding="5" style="width:100%">
 		<tr>
 			<td style="width:150px;vertical-align:top;text-align:right;padding-right:10px;font-weight:bold;" valign="top">
diff --git a/controllers/admin/AdminProductsController.php b/controllers/admin/AdminProductsController.php
index 009102f59..7ab7ad5a5 100644
--- a/controllers/admin/AdminProductsController.php
+++ b/controllers/admin/AdminProductsController.php
@@ -1013,6 +1013,18 @@ class AdminProductsControllerCore extends AdminController
 			if ($id_product && Validate::isUnsignedId($id_product) && Validate::isLoadedObject($product = new Product($id_product)))
 			{
 				$combinaisons = $product->getAttributeCombinaisonsById($id_product_attribute, $this->context->language->id);
+				foreach ($combinaisons as $key => $combinaison)
+					$combinaisons[$key]['attributes'][] = array($combinaison['group_name'], $combinaison['attribute_name'], $combinaison['id_attribute']);
+
+				foreach ($combinaisons as $key => $combinaison)
+				{
+					$jsList = '';
+					asort($combinaison['attributes']);
+					foreach ($combinaison['attributes'] AS $attribute)
+						$jsList .= '\''.addslashes(htmlspecialchars($combinaison['group_name'])).' : '.addslashes(htmlspecialchars($combinaison['attribute_name'])).'\', \''.$combinaison['id_attribute'].'\', ';
+					$combinaisons[$key]['list_attributes'] = rtrim($jsList, ', ');
+				}
+
 				die(Tools::jsonEncode($combinaisons));
 			}
 		}
@@ -2914,6 +2926,7 @@ class AdminProductsControllerCore extends AdminController
 		$data->assign('product', $product);
 		$data->assign('last_content', $content);
 		$data->assign('token', $this->token);
+		$data->assign('currency', $currency);
 		$data->assign($this->tpl_form_vars);
 		$data->assign('link', $this->context->link);
 		$this->tpl_form_vars['product'] = $product;
diff --git a/js/admin-products.js b/js/admin-products.js
index 88d315d8f..c3da50962 100644
--- a/js/admin-products.js
+++ b/js/admin-products.js
@@ -41,7 +41,9 @@ function removeButtonCombinaison(item)
 	$('.process-icon-newCombinaison').removeClass('toolbar-new');
 	$('.process-icon-newCombinaison').addClass('toolbar-cancel');
 	$('#submitProductAttribute').val($('#submitProductAttribute').attr(item));
-	$('#desc-product-newCombinaison div').html($('#desc-product-newCombinaison').attr('cancel'));
+	$('#desc-product-newCombinaison div').html($('#ResetBtn').val());
+	$('id_product_attribute').val(0);
+	init_elems();
 	posC = false;
 }
 
@@ -51,7 +53,7 @@ function addButtonCombinaison(item)
 	$('.process-icon-newCombinaison').removeClass('toolbar-cancel');
 	$('.process-icon-newCombinaison').addClass('toolbar-new');
 	$('#submitProductAttribute').val($('#submitProductAttribute').attr(item));
-	$('#desc-product-newCombinaison div').html($('#desc-product-newCombinaison').attr('add'));
+	$('#desc-product-newCombinaison div').html($('#submitProductAttribute').val());
 	posC = true;
 }
 
@@ -135,37 +137,68 @@ function editProductAttribute(ids, token)
 		context: this,
 		async: false,
 		success: function(data) {
-			console.log(data[0]);
-			console.log(data[1]);
 			$('#add_new_combination').show();
+			$('#attribute_quantity').show();
 			$('#product_att_list').html('');
+			removeButtonCombinaison('update');
+			$.scrollTo('#add_new_combination', 1200, { offset: -100 });
+
+			var wholesale_price = Math.abs(data[0]['wholesale_price']);
+			var price = Math.abs(data[0]['price']);
+			var weight = Math.abs(data[0]['weight']);
+			var unit_impact = Math.abs(data[0]['unit_price_impact']);
+			var reference = data[0]['reference'];
+			var ean = data[0]['ean13'];
+			var quantity = data[0]['quantity'];
+			var image = false;
+			var old_attr = Array(data[0]['list_attributes']);
+
+			var product_att_list = '';
 			for(i=0;i<data.length;i++)
 			{
-				// update values of fields
-				$('#id_product_attribute').val(data[i]['id_product_attribute']);
-				$('#product_att_list').append('<option value='+data[i]['id_attribute']+' groupid='+data[i]['id_attribute_group']+'>'+data[i]['group_name']+' : '+data[i]['attribute_name']+'</option>');
-				$('#attribute_reference').val(data[i]['reference']);
-				$('#attribute_ean13').val(data[i]['ean13']);
-				$('#attribute_upc').val(data[i]['upc']);
-				$('#attribute_wholesale_price').val(Math.abs(data[i]['wholesale_price']));
-				$('#attribute_price').val(Math.abs(data[i]['price']));
-				$('#attribute_priceTI').val(data[i]['attribute_reference']);
-				$('#attribute_weight').val(Math.abs(data[i]['weight']));
-				$('#attribute_unity').val(Math.abs(data[i]['unit_price_impact']));
-				if ($('#attribute_ecotax').length != 0)
-					$('#attribute_ecotax').val(data[i]['ecotax']);
-				$('#minimal_quantity').val(data[i]['minimal_quantity']);
-				$('#attribute_quantity').html(data[i]['quantity']);
-				$('#attribute_minimal_quantity').val(data[i]['minimal_quantity']);
-				$('#available_date').val(data[i]['available_date']);
-
-				if (data[i]['default_on'] == 1)
-					$('#attribute_default').checked = true;
-				else
-					$('#attribute_default').checked = false;
-				removeButtonCombinaison('update');
-				$.scrollTo('#add_new_combination', 1200, { offset: -100 });
+				product_att_list += data[i]['group_name']+' : ';
+				product_att_list += data[i]['attribute_name']+', ';
+				product_att_list += data[i]['id_attribute']+', ';
 			}
+
+			var old_attr = Array(product_att_list.substr(0, (product_att_list.length-2)));
+			var id_product_attribute = data[0]['id_product_attribute'];
+			var default_attribute = data[0]['default_on'];
+			var eco_tax = data[0]['ecotax'];
+			var upc = data[0]['upc'];
+			var minimal_quantity = data[0]['minimal_quantity'];
+			var available_date = data[0]['available_date'];
+			var virtual_product_name_attribute = null;
+			var virtual_product_filename_attribute = null;
+			var virtual_product_nb_downloable = null;
+			var virtual_product_expiration_date_attribute = null;
+			var virtual_product_nb_days = null;
+			var is_shareable = null;
+
+			fillCombinaison(
+				wholesale_price,
+				price,
+				weight,
+				unit_impact,
+				reference,
+				ean,
+				quantity,
+				image,
+				old_attr,
+				id_product_attribute,
+				default_attribute,
+				eco_tax,
+				upc,
+				minimal_quantity,
+				available_date,
+				virtual_product_name_attribute,
+				virtual_product_filename_attribute,
+				virtual_product_nb_downloable,
+				virtual_product_expiration_date_attribute,
+				virtual_product_nb_days,
+				is_shareable
+			);
+			calcImpactPriceTI();
 		}
 	});
 }
diff --git a/js/attributesBack.js b/js/attributesBack.js
index 977d77e1c..4d4433c3b 100644
--- a/js/attributesBack.js
+++ b/js/attributesBack.js
@@ -220,9 +220,6 @@ function check_unit_impact()
 
 function init_elems()
 {
-	/*$('#stock_mvt_attribute').hide();
-	$('#initial_stock_attribute').show();
-	$('#attr_qty_stock').hide();*/
 	var elem = getE('product_att_list');
 
 	if (elem.length)