rename rebuild_mat_view + optimize it when the object_id is known

Author: derlin

other/mysql/bbdata2-structure.sql

@@ -1,5 +1,5 @@
 -- MySQL Script generated by MySQL Workbench
--- Tue Sep 15 19:58:14 2020
+-- Tue Sep 15 20:27:02 2020
 -- Model: New Model    Version: 1.0
 -- MySQL Workbench Forward Engineering
 
@@ -363,15 +363,15 @@ END$$
 DELIMITER ;
 
 -- -----------------------------------------------------
--- procedure rebuild_mat_view
+-- procedure rebuild_userperms
 -- -----------------------------------------------------
 
 USE `bbdata2`;
-DROP procedure IF EXISTS `rebuild_mat_view`;
+DROP procedure IF EXISTS `rebuild_userperms`;
 
 DELIMITER $$
 USE `bbdata2`$$
-CREATE PROCEDURE `rebuild_mat_view`()
+CREATE PROCEDURE `rebuild_userperms`()
 BEGIN
     DELETE FROM userperms; -- don't use truncate: aises an error when called from a trigger
     INSERT IGNORE INTO userperms 
@@ -418,6 +418,39 @@ END$$
 
 DELIMITER ;
 
+-- -----------------------------------------------------
+-- procedure rebuild_userperms_oid
+-- -----------------------------------------------------
+
+USE `bbdata2`;
+DROP procedure IF EXISTS `rebuild_userperms_oid`;
+
+DELIMITER $$
+USE `bbdata2`$$
+CREATE PROCEDURE `rebuild_userperms_oid`(IN oid INT)
+BEGIN
+    -- same as rebuild_userperms, but delete/insert only lines with the given object
+    -- this is an optimisation: objects are often created/altered so it is worth it !
+    DELETE FROM userperms WHERE object_id = oid; 
+    INSERT IGNORE INTO userperms 
+    SELECT DISTINCT * FROM (
+			-- all admin members of the ugrp owner of the object have write permission
+        SELECT ug.user_id, o.id, ug.is_admin 
+        FROM users_ugrps ug INNER JOIN objects o ON o.ugrp_id = ug.ugrp_id
+      	WHERE o.id = oid
+        UNION
+        -- all members of an ugrp added to an object group having the object have read permission
+        SELECT ug.user_id, og.object_id, 0
+        FROM users_ugrps ug
+        INNER JOIN rights r ON ug.ugrp_id = r.ugrp_id 
+        INNER JOIN objects_ogrps og ON r.ogrp_id = og.ogrp_id
+        INNER JOIN objects o ON og.object_id = o.id AND o.id = oid
+        WHERE ug.ugrp_id <> o.ugrp_id
+    ) tmp;
+END$$
+
+DELIMITER ;
+
 -- -----------------------------------------------------
 -- View `ogrps_read`
 -- -----------------------------------------------------
@@ -476,7 +509,7 @@ USE `bbdata2`$$
 CREATE TRIGGER `objects_AINS` AFTER INSERT ON `objects` FOR EACH ROW
 	-- DAFUCK ???!!!! 
     -- INSERT IGNORE INTO objects_ogrps(ogrp_id, object_id) VALUES (NEW.ugrp_id, NEW.id);
-	CALL rebuild_mat_view();$$
+	CALL rebuild_userperms_oid(NEW.id);$$
 
 
 USE `bbdata2`$$
@@ -494,49 +527,49 @@ USE `bbdata2`$$
 DROP TRIGGER IF EXISTS `objects_ADEL` $$
 USE `bbdata2`$$
 CREATE TRIGGER `objects_ADEL` AFTER DELETE ON `objects` FOR EACH ROW
-	CALL rebuild_mat_view();$$
+	CALL rebuild_userperms_oid(OLD.id);$$
 
 
 USE `bbdata2`$$
 DROP TRIGGER IF EXISTS `objects_ogrps_AINS` $$
 USE `bbdata2`$$
 CREATE TRIGGER `objects_ogrps_AINS` AFTER INSERT ON `objects_ogrps` FOR EACH ROW
-	CALL rebuild_mat_view();$$
+	CALL rebuild_userperms_oid(NEW.object_id);$$
 
 
 USE `bbdata2`$$
 DROP TRIGGER IF EXISTS `objects_ogrps_ADEL` $$
 USE `bbdata2`$$
 CREATE TRIGGER `objects_ogrps_ADEL` AFTER DELETE ON `objects_ogrps` FOR EACH ROW
-	CALL rebuild_mat_view();$$
+	CALL rebuild_userperms_oid(OLD.object_id);$$
 
 
 USE `bbdata2`$$
 DROP TRIGGER IF EXISTS `rights_AINS` $$
 USE `bbdata2`$$
 CREATE TRIGGER `rights_AINS` AFTER INSERT ON `rights` FOR EACH ROW
-	CALL rebuild_mat_view();$$
+	CALL rebuild_userperms();$$
 
 
 USE `bbdata2`$$
 DROP TRIGGER IF EXISTS `rights_ADEL` $$
 USE `bbdata2`$$
 CREATE TRIGGER `rights_ADEL` AFTER DELETE ON `rights` FOR EACH ROW
-	CALL rebuild_mat_view();$$
+	CALL rebuild_userperms();$$
 
 
 USE `bbdata2`$$
 DROP TRIGGER IF EXISTS `users_ugrps_AINS` $$
 USE `bbdata2`$$
 CREATE TRIGGER `users_ugrps_AINS` AFTER INSERT ON `users_ugrps` FOR EACH ROW
-	CALL rebuild_mat_view()$$
+	CALL rebuild_userperms();$$
 
 
 USE `bbdata2`$$
 DROP TRIGGER IF EXISTS `users_ugrps_ADEL` $$
 USE `bbdata2`$$
 CREATE TRIGGER `users_ugrps_ADEL` AFTER DELETE ON `users_ugrps` FOR EACH ROW
-	CALL rebuild_mat_view();$$
+	CALL rebuild_userperms();$$
 
 
 DELIMITER ;