refs #5890

db/changes/234002/01-itemShelvingSale.sql

@@ -1,13 +1,20 @@
+ALTER TABLE `vn`.`itemShelvingSale` DROP COLUMN IF EXISTS isPicked;
 
-ALTER TABLE `vn`.`itemShelvingSale` ADD isPicked TINYINT(1) DEFAULT FALSE NOT NULL;
+ALTER TABLE`vn`.`itemShelvingSale` 
+    ADD isPicked TINYINT(1) DEFAULT FALSE NOT NULL;
+
+ALTER TABLE `vn`.`productionConfig` DROP COLUMN IF EXISTS orderMode;
+
+ALTER TABLE `vn`.`productionConfig`
+    ADD orderMode ENUM('Location', 'Age') NOT NULL DEFAULT 'Location';
 
 DELIMITER $$
-CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `vn`.`itemShelvingSale_addByCollection`(
+CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `vn`.`itemShelvingSale_reserveByCollection`(
 	vCollectionFk INT(11)
 )
 BEGIN
 /**
- * Guarda la ubicación para el contenido de una colección
+ * Reserva cantidades con ubicaciones para el contenido de una colección
  *
  * @param vCollectionFk Identificador de collection
  */
@@ -27,7 +34,7 @@ BEGIN
 				AND st.saleFk IS NULL
 				AND NOT s.isPicked;
 
-	CALL itemShelvingSale_addNew();
+	CALL itemShelvingSale_reserve();
 END$$
 DELIMITER ;
 
@@ -87,9 +94,6 @@ BEGIN
 		SET visible = IF(vIsItemShelvingSaleEmpty, 0, GREATEST(0,visible - vQuantity))
 		WHERE id = vItemShelvingFk;
 
-	/*Se podría preguntar  por la cantidad correcta de la ubicació o insertar en una tabla 
-		para que lo tenga en cuenta la gente que está haciendo inventario*/
-	
 	IF vIsItemShelvingSaleEmpty THEN
 		OPEN vSales;
 l:		LOOP
@@ -104,7 +108,7 @@ l:		LOOP
 			ENGINE = MEMORY
 			SELECT vCursorSaleFk, vUserFk;
 
-			CALL itemShelvingSale_addNewWhitUser();
+			CALL itemShelvingSale_reserveWhitUser();
 			DROP TEMPORARY TABLE tmp.sale;
 	
 		END LOOP;
@@ -124,7 +128,7 @@ l:		LOOP
 				AND NOT isPicked;
 
 	IF vRemainingQuantity THEN 
-		CALL itemShelvingSale_addBySale (vSaleFk, vRemainingQuantity, NULL);
+		CALL itemShelvingSale_reserveBySale (vSaleFk, vRemainingQuantity, NULL);
 	
 		SELECT SUM(quantity) INTO vRemainingQuantity
 			FROM itemShelvingSale
@@ -156,10 +160,10 @@ DELIMITER ;
 
 
 DELIMITER $$
-CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `vn`.`itemShelvingSale_addNew`()
+CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `vn`.`itemShelvingSale_reserve`()
 BEGIN
 /**
- * Guarda la ubicacion para un conjunto de sales del mismo wareHouse
+ * Reserva cantidades con ubicaciones para un conjunto de sales del mismo wareHouse
  *
  * @table tmp.sale(saleFk, userFk)
  */
@@ -227,12 +231,10 @@ BEGIN
 								ish.visible >= s.quantity DESC,
 								s.quantity MOD ish.grouping = 0 DESC,
 								ish.grouping DESC,
-								IF(pc.orderMode = 'Location', p.pickingOrder, ish.created)
-								LIMIT 100000000000000000
+								IF(pc.orderMode = 'Location', p.pickingOrder, ish.created)	
 					)sub
 				)sub2 
-		WHERE quantityToReserve > 0
-		;
+		WHERE quantityToReserve > 0;
 
 	INSERT INTO itemShelvingSale(
 			itemShelvingFk,
@@ -252,14 +254,14 @@ DELIMITER ;
 
 
 DELIMITER $$
-CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `vn`.`itemShelvingSale_addBySale`(
+CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `vn`.`itemShelvingSale_reserveBySale`(
 	vSelf INT , 
 	vQuantity INT,
 	vUserFk INT
 )
 BEGIN
 /**
- * Reserva la ubicación para una saleFk
+ * Reserva cantida y ubicación para una saleFk
  *
  * @param vSelf Identificador de la venta
  * @param vQuantity Cantidad a reservar
@@ -269,7 +271,7 @@ BEGIN
 		ENGINE = MEMORY
 		SELECT vSelf saleFk, vUserFk userFk;
 			
-	CALL itemShelvingSale_addNew();
+	CALL itemShelvingSale_reserve();
 END$$
 DELIMITER ;
 
@@ -287,13 +289,3 @@ BEGIN
 
 END$$
 DELIMITER ;
-
-ALTER TABLE `vn`.`itemShelvingSale` DROP COLUMN IF EXISTS isPicked;
-
-ALTER TABLE`vn`.`itemShelvingSale` 
-    ADD isPicked TINYINT(1) DEFAULT FALSE NOT NULL;
-
-ALTER TABLE `vn`.`productionConfig` DROP COLUMN IF EXISTS orderMode;
-
-ALTER TABLE `vn`.`productionConfig`
-    ADD orderMode ENUM('Location', 'Age') NOT NULL DEFAULT 'Location';