// Some bugs fix

admin-dev/themes/template/products/suppliers.tpl

@@ -49,7 +49,7 @@
 
 				<thead>
 					<tr>
-						<th>{l s='product name'}</th>
+						<th>{l s='Product name'}</th>
 						<th width="150">{l s='Supplier reference'}</th>
 						<th width="150">{l s='Unit price tax excluded'}</th>
 						<th width="150">{l s='Unit price currency'}</th>

classes/ProductSupplier.php

@@ -166,7 +166,9 @@ class ProductSupplierCore extends ObjectModel
 	{
 		$suppliers = new Collection('ProductSupplier');
 		$suppliers->where('a.id_product = '.(int)$id_product);
-		$suppliers->groupBy('a.id_supplier');
+
+		if ($group_by_supplier)
+			$suppliers->groupBy('a.id_supplier');
 
 		return $suppliers;
 	}

classes/stock/Stock.php

@@ -90,6 +90,7 @@ class StockCore extends ObjectModel
 	public function getFields()
 	{
 		$this->validateFields();
+
 		$fields['id_warehouse'] = (int)$this->id_warehouse;
 		$fields['id_product'] = (int)$this->id_product;
 		$fields['id_product_attribute'] = (int)$this->id_product_attribute;
@@ -99,6 +100,7 @@ class StockCore extends ObjectModel
 		$fields['physical_quantity'] = (int)$this->physical_quantity;
 		$fields['usable_quantity'] = (int)$this->usable_quantity;
 		$fields['price_te'] = (float)round($this->price_te, 6);
+
 		return $fields;
 	}
 

classes/stock/StockManager.php

@@ -210,7 +210,8 @@ class StockManagerCore implements StockManagerInterface
 			$products_pack = Pack::getItems($id_product, (int)Configuration::get('PS_LANG_DEFAULT'));
 			foreach ($products_pack as $product_pack)
 			{
-				$pack_id_product_attribute = Product::getDefaultAttribute($id_product_attribute, 1); //@TODO is there a better way to retrieve the product attribute assciated to the pack ?
+				//@TODO is there a better way to retrieve the product attribute assciated to the pack ?
+				$pack_id_product_attribute = Product::getDefaultAttribute($id_product_attribute, 1);
 				$this->removeProduct($product_pack->id, $pack_id_product_attribute, $product_pack->pack_quantity * $quantity, $warehouse, $id_order);
 			}
 		}

controllers/admin/AdminProductsController.php

@@ -1994,7 +1994,7 @@ public function getList($id_lang, $orderBy = null, $orderWay = null, $start = 0,
 					$suppliers_to_associate[] = $supplier['id_supplier'];
 
 			// Delete already associated suppliers if needed
-			foreach ($associated_suppliers as $key => &$associated_supplier)
+			foreach ($associated_suppliers as $key => $associated_supplier)
 				if (!in_array($associated_supplier->id_supplier, $suppliers_to_associate))
 				{
 					$associated_supplier->delete();
@@ -3266,7 +3266,7 @@ public function getList($id_lang, $orderBy = null, $orderWay = null, $start = 0,
 				$supplier['is_selected'] = false;
 				$supplier['is_default'] = false;
 
-				foreach ($associated_suppliers as &$associated_supplier)
+				foreach ($associated_suppliers as $associated_supplier)
 					if ($associated_supplier->id_supplier == $supplier['id_supplier'])
 					{
 						$associated_supplier->name = $supplier['name'];