Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature presidecms 2812 async db migration priority #1437

Merged
merged 5 commits into from
Jan 9, 2025
+21 −0
Merged

Feature presidecms 2812 async db migration priority #1437

Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
dfe2bef
[twgit] Init feature 'feature-PRESIDECMS-2812_async-db-migration-prio…
Pixl8RusselCole Mar 18, 2024
df7e834
PRESIDECMS-2812 async db migrations with priority
Pixl8RusselCole Mar 18, 2024
9c3a264
PRESIDECMS-2812 handlerExists for asyncPriorityHandler
Pixl8RusselCole Nov 27, 2024
5d62b8e
Merge branch 'stable' into feature-PRESIDECMS-2812_async-db-migration…
Pixl8RusselCole Nov 27, 2024
d1a97f4
PRESIDECMS-2812 missing var
Pixl8RusselCole Nov 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions system/services/updateManager/AppDatabaseMigrationService.cfc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,22 @@ component {
}
}

if ( arguments.async ) {
var migrationsWithPriority = {};

for ( migration in migrations ) {
var priority = $runEvent(
Pixl8RusselCole marked this conversation as resolved.
Show resolved Hide resolved
event = "dbmigrations.#migration#.asyncPriority"
, private = true
, prepostExempt = true
);

migrationsWithPriority[ migration ] = priority ?: 1;
}

migrations = StructSort( migrationsWithPriority, "numeric" );
}

return migrations;
}

Expand Down
Loading