Updates to PropelMigrationTask to better handle multiple datasources. #343

Merged
merged 1 commit into from Apr 24, 2012

Projects

None yet

2 participants

@dotEvan
Contributor
dotEvan commented Apr 24, 2012
  • Moves updating of the timestamp out of the main migration loop per datasource. Timestamps will now only get updated when all SQL on all datasources pass and it doesn't exit out prior with false.
  • Fixes issue where migrations that only specify change to one datasource don't update the timestamp in all other datasources. This caused issues when running new migrations since it will always start with the lowest migration of all the datasources.
@dotEvan dotEvan Updates to PropelMigrationTask to better handle multiple datasources.
- Moves updating of the timestamp out of the main migration loop per datasource. Timestamps will now only get updated when all SQL on all datasources pass and it doesn't exit out prior with false.
- Fixes issue where migrations that only specify change to one datasource don't update the timestamp in all other datasources. This caused issues when running new migrations since it will always start with the lowest migration of all the datasources.
2097c08
@willdurand
Member

Seems ok, thank you.

Could you port this patch on Propel2 as well? Thanks again

@willdurand willdurand merged commit e769175 into propelorm:master Apr 24, 2012
@willdurand willdurand referenced this pull request in propelorm/Propel2 Jul 27, 2012
Closed

Port PR #343 from Propel 1.6 #192

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment