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.

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

migration_diff on none default Datasource uses dump in wrong folder #809

Closed
maidzen opened this issue Jan 28, 2025 · 2 comments · Fixed by #819
Closed

migration_diff on none default Datasource uses dump in wrong folder #809

maidzen opened this issue Jan 28, 2025 · 2 comments · Fixed by #819
Labels

Comments

@maidzen
Copy link
Contributor

maidzen commented Jan 28, 2025

I have multiple Datasources.

The migrations are split in a Folders per datasource using the -connection and -source parameter

  • config/Migrations (default datasource)
  • config/MigrationsLogs (second Datasource "logs" )

....

If i use migration_diff on "logs"

bin/cake bake migration_diff -s MigrationsLogs -c logs

the system looks for the Dump File in "config/Migrations" instead of "config/MigrationsLogs"

atm. we copy the dump file before executing the migration but this can easy lead to errors by human mistake.

@markstory markstory added this to the 4.x (CakePHP 5) milestone Jan 28, 2025
@markstory markstory added the bug label Jan 28, 2025
@dereuromark
Copy link
Member

Do you have an idea for a fix? Feel free to PR it.

@maidzen
Copy link
Contributor Author

maidzen commented Mar 7, 2025

sure thing, i hope it can be merged soon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants