Merge pull request #2243 from hongwei1/refactor/removeHasOwnerViewAccess

bugfix/tweaked the position to run populateMigrationOfViewDefinitionP…
OpenBankProject · Jul 14, 2023 · f4fc319 · f4fc319
2 parents 8291d72 + ae994f3
commit f4fc319
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/obp-api/src/main/scala/code/api/util/migration/Migration.scala b/obp-api/src/main/scala/code/api/util/migration/Migration.scala
@@ -63,7 +63,7 @@ object Migration extends MdcLoggable {
       dummyScript()
       addAccountAccessConsumerId()
       populateTableViewDefinition()
-      populateMigrationOfViewDefinitionPermissions()
+      populateMigrationOfViewDefinitionPermissions(startedBeforeSchemifier)
       populateTableAccountAccess()
       generateAndPopulateMissingCustomerUUIDs(startedBeforeSchemifier)
       generateAndPopulateMissingConsumersUUIDs(startedBeforeSchemifier)
@@ -129,10 +129,15 @@ object Migration extends MdcLoggable {
     }  
 
 
-    private def populateMigrationOfViewDefinitionPermissions(): Boolean = {
-      val name = nameOf(populateMigrationOfViewDefinitionPermissions)
-      runOnce(name) {
-        MigrationOfViewDefinitionPermissions.populate(name)
+    private def populateMigrationOfViewDefinitionPermissions(startedBeforeSchemifier: Boolean): Boolean = {
+      if (startedBeforeSchemifier == true) {
+        logger.warn(s"Migration.database.populateMigrationOfViewDefinitionPermissions(true) cannot be run before Schemifier.")
+        true
+      } else {
+        val name = nameOf(populateMigrationOfViewDefinitionPermissions(startedBeforeSchemifier))
+        runOnce(name) {
+          MigrationOfViewDefinitionPermissions.populate(name)
+        }
       }
     }