CONTENTBOX-1500 #resolve

Defensive Coding Needed for 2023_12_12_164002_EditorsCanClearCaches.cfc Migration
Ortus-Solutions · Dec 20, 2023 · 21da1bb · 21da1bb
1 parent 60e0f99
commit 21da1bb
Showing 1 changed file with 49 additions and 19 deletions.
diff --git a/modules/contentbox/migrations/2023_12_12_164002_EditorsCanClearCaches.cfc b/modules/contentbox/migrations/2023_12_12_164002_EditorsCanClearCaches.cfc
@@ -16,17 +16,31 @@ component {
 	function up( schema, qb ){
 		var today  = now();
 		var permID = createUUID();
-		qb.newQuery()
-			.from( "cb_permission" )
-			.insert( {
-				"permissionID" : permID,
-				"createdDate"  : today,
-				"modifiedDate" : today,
-				"isDeleted"    : 0,
-				"permission"   : "RELOAD_CACHES",
-				"description"  : "Ability to reload caches"
-			} );
-		systemOutput( "√ - Reload Caches permission created", true );
+		if (
+			!qb.newQuery()
+				.from( "cb_permission" )
+				.where( "permission", "RELOAD_CACHES" )
+				.count()
+		) {
+			qb.newQuery()
+				.from( "cb_permission" )
+				.insert( {
+					"permissionID" : permID,
+					"createdDate"  : today,
+					"modifiedDate" : today,
+					"isDeleted"    : 0,
+					"permission"   : "RELOAD_CACHES",
+					"description"  : "Ability to reload caches"
+				} );
+			systemOutput( "√ - Reload Caches permission created", true );
+		} else {
+			var permId = qb
+				.newQuery()
+				.from( "cb_permission" )
+				.where( "permission", "RELOAD_CACHES" )
+				.first()
+				.permissionID;
+		}
 
 		// Assign to Editors and Administrators
 		var admin = qb
@@ -36,10 +50,18 @@ component {
 			.where( "role", "Administrator" )
 			.first();
 
-		qb.newQuery()
-			.from( "cb_rolePermissions" )
-			.insert( { "FK_roleID" : admin.roleID, "FK_permissionID" : permID } );
-		systemOutput( "√ - Admin role updated with new permissions", true );
+		if (
+			!qb.newQuery()
+				.from( "cb_rolePermissions" )
+				.where( "FK_permissionID", permID )
+				.where( "FK_roleID", admin.roleID )
+				.count()
+		) {
+			qb.newQuery()
+				.from( "cb_rolePermissions" )
+				.insert( { "FK_roleID" : admin.roleID, "FK_permissionID" : permID } );
+			systemOutput( "√ - Admin role updated with new permissions", true );
+		}
 
 		var editor = qb
 			.newQuery()
@@ -48,10 +70,18 @@ component {
 			.where( "role", "Editor" )
 			.first();
 
-		qb.newQuery()
-			.from( "cb_rolePermissions" )
-			.insert( { "FK_roleID" : editor.roleID, "FK_permissionID" : permID } );
-		systemOutput( "√ - Editor role updated with new permissions", true );
+		if (
+			!qb.newQuery()
+				.from( "cb_rolePermissions" )
+				.where( "FK_permissionID", permID )
+				.where( "FK_roleID", editor.roleID )
+				.count()
+		) {
+			qb.newQuery()
+				.from( "cb_rolePermissions" )
+				.insert( { "FK_roleID" : editor.roleID, "FK_permissionID" : permID } );
+			systemOutput( "√ - Editor role updated with new permissions", true );
+		}
 	}
 
 	function down( schema, qb ){