feat: refs #4409 Disable old available triggers

db/.editorconfig

@@ -0,0 +1,13 @@
+# EditorConfig helps developers define and maintain consistent
+# coding styles between different editors and IDEs
+# http://editorconfig.org
+
+root = true
+
+[*]
+indent_style = tab
+indent_size = 4
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true

db/routines/hedera/procedures/order_requestRecalc.sql

@@ -10,6 +10,7 @@ proc: BEGIN
 		LEAVE proc;
 	END IF;
 
-	INSERT INTO orderRecalc SET orderFk = vSelf;
+	-- #4409 Deprecated by MyCDC
+	-- INSERT INTO orderRecalc SET orderFk = vSelf;
 END$$
 DELIMITER ;

db/routines/hedera/triggers/orderRow_afterDelete.sql

@@ -3,7 +3,6 @@ CREATE OR REPLACE DEFINER=`root`@`localhost` TRIGGER `hedera`.`orderRow_afterDel
 	AFTER DELETE ON `orderRow`
 	FOR EACH ROW
 BEGIN
-	CALL stock.log_add('orderRow', NULL, OLD.id);
-    CALL order_requestRecalc(OLD.orderFk);
+	CALL order_requestRecalc(OLD.orderFk);
 END$$
 DELIMITER ;

db/routines/hedera/triggers/orderRow_afterInsert.sql

@@ -3,7 +3,6 @@ CREATE OR REPLACE DEFINER=`root`@`localhost` TRIGGER `hedera`.`orderRow_afterIns
 	AFTER INSERT ON `orderRow`
 	FOR EACH ROW
 BEGIN
-	CALL stock.log_add('orderRow', NEW.id, NULL);
-    CALL order_requestRecalc(NEW.orderFk);
+	CALL order_requestRecalc(NEW.orderFk);
 END$$
 DELIMITER ;

db/routines/hedera/triggers/orderRow_afterUpdate.sql

@@ -3,8 +3,7 @@ CREATE OR REPLACE DEFINER=`root`@`localhost` TRIGGER `hedera`.`orderRow_afterUpd
 	AFTER UPDATE ON `orderRow`
 	FOR EACH ROW
 BEGIN
-	CALL stock.log_add('orderRow', NEW.id, OLD.id);
-    CALL order_requestRecalc(OLD.orderFk);
-    CALL order_requestRecalc(NEW.orderFk);
+	CALL order_requestRecalc(OLD.orderFk);
+	CALL order_requestRecalc(NEW.orderFk);
 END$$
 DELIMITER ;

db/routines/hedera/triggers/order_afterUpdate.sql

@@ -3,11 +3,9 @@ CREATE OR REPLACE DEFINER=`root`@`localhost` TRIGGER `hedera`.`order_afterUpdate
 	AFTER UPDATE ON `order`
 	FOR EACH ROW
 BEGIN
-	CALL stock.log_add('order', NEW.id, OLD.id);
-    
-    IF !(OLD.address_id <=> NEW.address_id)
-    OR !(OLD.company_id <=> NEW.company_id)
-    OR !(OLD.customer_id <=> NEW.customer_id) THEN
+	IF !(OLD.address_id <=> NEW.address_id)
+	OR !(OLD.company_id <=> NEW.company_id)
+	OR !(OLD.customer_id <=> NEW.customer_id) THEN
 		CALL order_requestRecalc(NEW.id);
 	END IF;
 

db/routines/stock/procedures/inbound_addPick.sql

@@ -1,8 +1,8 @@
 DELIMITER $$
 CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `stock`.`inbound_addPick`(
 	vSelf INT,
-    vOutboundFk INT,
-    vQuantity INT
+	vOutboundFk INT,
+	vQuantity INT
 )
 BEGIN
 	INSERT INTO inboundPick

db/routines/stock/procedures/inbound_removePick.sql

@@ -1,9 +1,9 @@
 DELIMITER $$
 CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `stock`.`inbound_removePick`(
 	vSelf INT,
-    vOutboundFk INT,
-    vQuantity INT,
-    vTotalQuantity INT
+	vOutboundFk INT,
+	vQuantity INT,
+	vTotalQuantity INT
 )
 BEGIN
 	IF vQuantity < vTotalQuantity THEN

db/routines/stock/procedures/inbound_requestQuantity.sql

@@ -1,9 +1,9 @@
 DELIMITER $$
 CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `stock`.`inbound_requestQuantity`(
 	vSelf INT,
-    vRequested INT,
-    vDated DATETIME,
-    OUT vSupplied INT)
+	vRequested INT,
+	vDated DATETIME,
+	OUT vSupplied INT)
 BEGIN
 /**
  * Disassociates inbound picks after the given date until the

db/routines/stock/procedures/inbound_sync.sql

@@ -23,7 +23,7 @@ BEGIN
 		SELECT id, lack, lack < quantity
 			FROM outbound
 			WHERE warehouseFk = vWarehouse
-                AND itemFk = vItem
+				AND itemFk = vItem
 				AND dated >= vDated
 				AND (vExpired IS NULL OR dated < vExpired)
 			ORDER BY dated, created;
@@ -52,7 +52,7 @@ BEGIN
 
 		SET vSupplied = LEAST(vAvailable, vLack);
 
-        IF vSupplied > 0 THEN
+		IF vSupplied > 0 THEN
 			SET vAvailable = vAvailable - vSupplied;
 			UPDATE outbound
 				SET lack = lack - vSupplied
@@ -65,7 +65,7 @@ BEGIN
 			SET vAvailable = vAvailable - vSuppliedFromRequest;
 		END IF;
 
-        IF vSupplied > 0 THEN
+		IF vSupplied > 0 THEN
 			CALL inbound_addPick(vSelf, vOutboundFk, vSupplied);
 		END IF;
 

db/routines/stock/procedures/log_add.sql

@@ -1,21 +0,0 @@
-DELIMITER $$
-CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `stock`.`log_add`(IN `vTableName` VARCHAR(255), IN `vNewId` VARCHAR(255), IN `vOldId` VARCHAR(255))
-proc: BEGIN
-	-- XXX: Disabled while testing
-	LEAVE proc;
-
-	IF vOldId IS NOT NULL AND !(vOldId <=> vNewId) THEN
-		INSERT IGNORE INTO `log` SET
-			tableName = vTableName,
-			tableId = vOldId,
-			operation = 'delete';
-	END IF;
-
-	IF vNewId IS NOT NULL THEN
-		INSERT IGNORE INTO `log` SET
-			tableName = vTableName,
-			tableId = vNewId,
-			operation = 'insert';
-	END IF;
-END$$
-DELIMITER ;

db/routines/stock/procedures/log_refreshBuy.sql

@@ -1,7 +1,7 @@
 DELIMITER $$
 CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `stock`.`log_refreshBuy`(
-    `vTableName` VARCHAR(255),
-    `vTableId` INT)
+	`vTableName` VARCHAR(255),
+	`vTableId` INT)
 BEGIN
 	DROP TEMPORARY TABLE IF EXISTS tValues;
 	CREATE TEMPORARY TABLE tValues
@@ -11,7 +11,7 @@ BEGIN
 				e.id entryFk,
 				t.id travelFk,
 				b.itemFk,
-                e.isRaid,
+				e.isRaid,
 				ADDTIME(t.shipped,
 					IFNULL(t.shipmentHour, '00:00:00')) shipped,
 				t.warehouseOutFk,
@@ -24,7 +24,7 @@ BEGIN
 				ABS(b.quantity) quantity,
 				b.created,
 				b.quantity > 0 isIn,
-                t.shipped < vn.getInventoryDate() lessThanInventory
+				t.shipped < vn.getInventoryDate() lessThanInventory
 			FROM vn.buy b
 				JOIN vn.entry e ON e.id = b.entryFk
 				JOIN vn.travel t ON t.id = e.travelFk
@@ -52,7 +52,7 @@ BEGIN
 				quantity,
 				IF(isIn, isReceived, isDelivered) AND !isRaid
 			FROM tValues
-            WHERE isIn OR !lessThanInventory;
+			WHERE isIn OR !lessThanInventory;
 
 	REPLACE INTO outbound (
 			tableName, tableId, warehouseFk, dated,
@@ -67,7 +67,7 @@ BEGIN
 				quantity,
 				IF(isIn, isDelivered, isReceived) AND !isRaid
 			FROM tValues
-            WHERE !isIn OR !lessThanInventory;
+			WHERE !isIn OR !lessThanInventory;
 
 	DROP TEMPORARY TABLE tValues;
 END$$

db/routines/stock/procedures/log_refreshOrder.sql

@@ -1,13 +1,13 @@
 DELIMITER $$
 CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `stock`.`log_refreshOrder`(
-    `vTableName` VARCHAR(255),
-    `vTableId` INT)
+	`vTableName` VARCHAR(255),
+	`vTableId` INT)
 BEGIN
 	DECLARE vExpireTime INT DEFAULT 20;
 	DECLARE vExpired DATETIME DEFAULT TIMESTAMPADD(MINUTE, -vExpireTime, util.VN_NOW());
 
 	DROP TEMPORARY TABLE IF EXISTS tValues;
-    CREATE TEMPORARY TABLE tValues
+	CREATE TEMPORARY TABLE tValues
 		ENGINE = MEMORY
 		SELECT
 				r.id rowFk,
@@ -23,24 +23,24 @@ BEGIN
 					OR (vTableName = 'order' AND o.id = vTableId)
 					OR (vTableName = 'orderRow' AND r.id = vTableId)
 				)
-                AND !o.confirmed
-                AND r.shipment >= vn.getInventoryDate()
+				AND !o.confirmed
+				AND r.shipment >= vn.getInventoryDate()
 				AND r.created >= vExpired
 				AND r.amount != 0;
 
 	REPLACE INTO outbound (
 			tableName, tableId, warehouseFk, dated,
-            itemFk, created, expired, quantity
+			itemFk, created, expired, quantity
 		)
-        SELECT 'orderRow',
+		SELECT 'orderRow',
 				rowFk,
 				warehouseFk,
 				shipped,
 				itemFk,
 				created,
-                TIMESTAMPADD(MINUTE, vExpireTime, created),
+				TIMESTAMPADD(MINUTE, vExpireTime, created),
 				quantity
-            FROM tValues;
+			FROM tValues;
 
 	DROP TEMPORARY TABLE tValues;
 END$$

db/routines/stock/procedures/log_refreshSale.sql

@@ -1,10 +1,10 @@
 DELIMITER $$
 CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `stock`.`log_refreshSale`(
-    `vTableName` VARCHAR(255),
-    `vTableId` INT)
+	`vTableName` VARCHAR(255),
+	`vTableId` INT)
 BEGIN
 	DROP TEMPORARY TABLE IF EXISTS tValues;
-    CREATE TEMPORARY TABLE tValues
+	CREATE TEMPORARY TABLE tValues
 		ENGINE = MEMORY
 		SELECT
 				m.id saleFk,
@@ -14,7 +14,7 @@ BEGIN
 				t.shipped,
 				ABS(m.quantity) quantity,
 				m.created,
-                TIMESTAMPADD(DAY, tp.life, t.shipped) expired,
+				TIMESTAMPADD(DAY, tp.life, t.shipped) expired,
 				m.quantity < 0 isIn,
 				m.isPicked OR s.alertLevel > 1 isPicked
 			FROM vn.sale m
@@ -32,33 +32,33 @@ BEGIN
 
 	REPLACE INTO inbound (
 			tableName, tableId, warehouseFk, dated,
-            itemFk, expired, quantity, isPicked
+			itemFk, expired, quantity, isPicked
 		)
-        SELECT 'sale',
+		SELECT 'sale',
 				saleFk,
 				warehouseFk,
 				shipped,
 				itemFk,
-                expired,
+				expired,
 				quantity,
-                isPicked
-            FROM tValues
-            WHERE isIn;
+				isPicked
+			FROM tValues
+			WHERE isIn;
 
 	REPLACE INTO outbound (
 			tableName, tableId, warehouseFk, dated,
-            itemFk, created, quantity, isPicked
+			itemFk, created, quantity, isPicked
 		)
-        SELECT 'sale',
+		SELECT 'sale',
 				saleFk,
 				warehouseFk,
 				shipped,
 				itemFk,
 				created,
 				quantity,
-                isPicked
-            FROM tValues
-            WHERE !isIn;
+				isPicked
+			FROM tValues
+			WHERE !isIn;
 
 	DROP TEMPORARY TABLE tValues;
 END$$

db/routines/stock/procedures/outbound_sync.sql

@@ -7,7 +7,7 @@ BEGIN
  * @param vSelf The outbound reference
  */
 	DECLARE vDated DATETIME;
-    DECLARE vItem INT;
+	DECLARE vItem INT;
 	DECLARE vWarehouse INT;
 	DECLARE vLack INT;
 	DECLARE vSupplied INT;
@@ -21,7 +21,7 @@ BEGIN
 		SELECT id, available, available < quantity
 			FROM inbound
 			WHERE warehouseFk = vWarehouse
-                AND itemFk = vItem
+				AND itemFk = vItem
 				AND dated <= vDated
 				AND (expired IS NULL OR expired > vDated)
 			ORDER BY dated;

db/routines/stock/procedures/visible_log.sql

@@ -1,16 +1,16 @@
 DELIMITER $$
 CREATE OR REPLACE DEFINER=`root`@`localhost` PROCEDURE `stock`.`visible_log`(
 	vIsPicked BOOL,
-    vWarehouseFk INT,
-    vItemFk INT,
-    vQuantity INT
+	vWarehouseFk INT,
+	vItemFk INT,
+	vQuantity INT
 )
 proc: BEGIN
 	IF !vIsPicked THEN
 		LEAVE proc;
 	END IF;
 
-    INSERT INTO visible
+	INSERT INTO visible
 		SET itemFk = vItemFk,
 			warehouseFk = vWarehouseFk,
 			quantity = vQuantity

db/routines/stock/triggers/inbound_afterDelete.sql

@@ -12,11 +12,11 @@ BEGIN
 	DELETE FROM inboundPick
 		WHERE inboundFk = OLD.id;
 
-    CALL visible_log(
+	CALL visible_log(
 		OLD.isPicked,
-        OLD.warehouseFk,
-        OLD.itemFk,
-        -OLD.quantity
+		OLD.warehouseFk,
+		OLD.itemFk,
+		-OLD.quantity
 	);
 END$$
 DELIMITER ;

db/routines/stock/triggers/inbound_beforeInsert.sql

@@ -5,11 +5,11 @@ CREATE OR REPLACE DEFINER=`root`@`localhost` TRIGGER `stock`.`inbound_beforeInse
 BEGIN
 	SET NEW.isPicked = NEW.isPicked OR NEW.dated < util.VN_CURDATE();
 
-    CALL visible_log(
+	CALL visible_log(
 		NEW.isPicked,
-        NEW.warehouseFk,
-        NEW.itemFk,
-        NEW.quantity
+		NEW.warehouseFk,
+		NEW.itemFk,
+		NEW.quantity
 	);
 END$$
 DELIMITER ;

db/routines/stock/triggers/outbound_afterDelete.sql

@@ -12,11 +12,11 @@ BEGIN
 	DELETE FROM inboundPick
 		WHERE outboundFk = OLD.id;
 
-    CALL visible_log(
+	CALL visible_log(
 		OLD.isPicked,
-        OLD.warehouseFk,
-        OLD.itemFk,
-        OLD.quantity
+		OLD.warehouseFk,
+		OLD.itemFk,
+		OLD.quantity
 	);
 END$$
 DELIMITER ;

db/routines/stock/triggers/outbound_beforeInsert.sql

@@ -6,11 +6,11 @@ BEGIN
 	SET NEW.lack = NEW.quantity;
 	SET NEW.isPicked = NEW.isPicked OR NEW.dated < util.VN_CURDATE();
 
-    CALL visible_log(
+	CALL visible_log(
 		NEW.isPicked,
-        NEW.warehouseFk,
-        NEW.itemFk,
-        -NEW.quantity
+		NEW.warehouseFk,
+		NEW.itemFk,
+		-NEW.quantity
 	);
 END$$
 DELIMITER ;

db/routines/vn/procedures/ticket_requestRecalc.sql

@@ -10,6 +10,7 @@ proc: BEGIN
 		LEAVE proc;
 	END IF;
 
-	INSERT INTO ticketRecalc SET ticketFk = vSelf;
+	-- #4409 Deprecated by MyCDC
+	-- INSERT INTO ticketRecalc SET ticketFk = vSelf;
 END$$
 DELIMITER ;

db/routines/vn/triggers/buy_afterDelete.sql

@@ -3,19 +3,14 @@ CREATE OR REPLACE DEFINER=`root`@`localhost` TRIGGER `vn`.`buy_afterDelete`
 	AFTER DELETE ON `buy`
 	FOR EACH ROW
 trig: BEGIN
-	DECLARE vValues VARCHAR(255);
-
 	IF @isModeInventory OR @isTriggerDisabled THEN
 		LEAVE trig;
 	END IF;
 
-	CALL stock.log_add('buy', NULL, OLD.id);
-
 	INSERT INTO entryLog
 		SET `action` = 'delete',
 			`changedModel` = 'Buy',
 			`changedModelId` = OLD.id,
 			`userFk` = account.myUser_getId();
-
 END$$
 DELIMITER ;

db/routines/vn/triggers/buy_afterInsert.sql

@@ -7,8 +7,6 @@ trig: BEGIN
 		LEAVE trig;
 	END IF;
 
-	CALL stock.log_add('buy', NEW.id, NULL);
-
 	CALL buy_afterUpsert(NEW.id);
 END$$
 DELIMITER ;

db/routines/vn/triggers/buy_afterUpdate.sql

@@ -12,14 +12,6 @@ trig: BEGIN
 		LEAVE trig;
 	END IF;
 
-	IF !(NEW.id <=> OLD.id)
-	OR !(NEW.entryFk <=> OLD.entryFk)
-	OR !(NEW.itemFk <=> OLD.itemFk)
-	OR !(NEW.quantity <=> OLD.quantity)
-	OR !(NEW.created <=> OLD.created) THEN
-		CALL stock.log_add('buy', NEW.id, OLD.id);
-	END IF;
-
 	CALL buy_afterUpsert(NEW.id);
 
 	SELECT w.isBuyerToBeEmailed, t.landed

db/routines/vn/triggers/entry_afterUpdate.sql

@@ -3,18 +3,11 @@ CREATE OR REPLACE DEFINER=`root`@`localhost` TRIGGER `vn`.`entry_afterUpdate`
 	AFTER UPDATE ON `entry`
 	FOR EACH ROW
 BEGIN
-	IF NOT(NEW.id <=> OLD.id)
-	OR NOT(NEW.travelFk <=> OLD.travelFk)
-	OR NOT(NEW.isRaid <=> OLD.isRaid) THEN
-		CALL stock.log_add('entry', NEW.id, OLD.id);
-	END IF;
-
 	IF NOT (NEW.travelFk <=> OLD.travelFk) THEN
 		CALL travel_requestRecalc(OLD.travelFk);
 		CALL travel_requestRecalc(NEW.travelFk);
 	END IF;
 
-	
 	IF NOT (NEW.travelFk <=> OLD.travelFk) THEN
 		CREATE OR REPLACE TEMPORARY TABLE tmp.buysToCheck
 			SELECT b.id

db/routines/vn/triggers/sale_afterDelete.sql

@@ -12,7 +12,6 @@ BEGIN
 			`changedModelId` = OLD.id,
 			`userFk` = account.myUser_getId();
 
-	CALL stock.log_add('sale', NULL, OLD.id);
 	CALL ticket_requestRecalc(OLD.ticketFk);
 
 	SELECT account.myUser_getName() INTO vUserRole;

db/routines/vn/triggers/sale_afterInsert.sql

@@ -7,7 +7,6 @@ BEGIN
 		CALL util.throw('Cannot insert a service item into a ticket');
 	END IF;
 
-	CALL stock.log_add('sale', NEW.id, NULL);
 	CALL ticket_requestRecalc(NEW.ticketFk);
 
 	IF NEW.quantity > 0 THEN

db/routines/vn/triggers/sale_afterUpdate.sql

@@ -6,15 +6,6 @@ BEGIN
 	DECLARE vIsToSendMail BOOL;
 	DECLARE vUserRole VARCHAR(255);
 
-	IF !(NEW.id <=> OLD.id)
-	OR !(NEW.ticketFk <=> OLD.ticketFk)
-	OR !(NEW.itemFk <=> OLD.itemFk)
-	OR !(NEW.quantity <=> OLD.quantity)
-	OR !(NEW.created <=> OLD.created)
-	OR !(NEW.isPicked <=> OLD.isPicked) THEN
-		CALL stock.log_add('sale', NEW.id, OLD.id);
-	END IF;
-
 	IF !(NEW.price <=> OLD.price)
 	OR !(NEW.ticketFk <=> OLD.ticketFk)
 	OR !(NEW.itemFk <=> OLD.itemFk)

db/routines/vn/triggers/ticket_afterUpdate.sql

@@ -3,13 +3,6 @@ CREATE OR REPLACE DEFINER=`root`@`localhost` TRIGGER `vn`.`ticket_afterUpdate`
 	AFTER UPDATE ON `ticket`
 	FOR EACH ROW
 BEGIN
-
-	IF !(NEW.id <=> OLD.id)
-		OR !(NEW.warehouseFk <=> OLD.warehouseFk)
-		OR !(NEW.shipped <=> OLD.shipped) THEN
-			CALL stock.log_add('ticket', NEW.id, OLD.id);
-	END IF;
-
 	IF !(NEW.clientFk <=> OLD.clientFk)
 		OR !(NEW.addressFk <=> OLD.addressFk)
 		OR !(NEW.companyFk <=> OLD.companyFk) THEN

db/routines/vn/triggers/travel_afterUpdate.sql

@@ -3,8 +3,6 @@ CREATE OR REPLACE DEFINER=`root`@`localhost` TRIGGER `vn`.`travel_afterUpdate`
 	AFTER UPDATE ON `travel`
 	FOR EACH ROW
 BEGIN
-	CALL stock.log_add('travel', NEW.id, OLD.id);
-
 	IF NOT(NEW.shipped <=> OLD.shipped) THEN
 		UPDATE entry
 			SET commission = entry_getCommission(travelFk, currencyFk,supplierFk)