Skip to content
Permalink
Browse files

Fix: `if` not working for content migration steps and friends

  • Loading branch information...
gggeek committed Feb 14, 2019
1 parent a8d641a commit 7a37ca02f631e01ecff8dd7287ec00c836ac072a
Showing with 33 additions and 1 deletion.
  1. +30 −1 Resources/config/services.yml
  2. +3 −0 WHATSNEW.md
@@ -318,8 +318,9 @@ services:
calls:
- ['setRepository', ['@ezpublish.api.repository']]
- ['setReferenceResolver', ['@ez_migration_bundle.reference_resolver.customreference']]
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
- ['setConfigResolver', ['@ezpublish.config.resolver']]
# SF bug? this method is not called for child srevices as it belongs to a trait instead of teh base class...
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]

ez_migration_bundle.executor.content_manager:
parent: ez_migration_bundle.executor.repository
@@ -333,6 +334,8 @@ services:
- '@ez_migration_bundle.complex_field_manager'
- '@ez_migration_bundle.executor.location_manager'
- '@ez_migration_bundle.helper.sort_converter'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -349,6 +352,8 @@ services:
- '@ez_migration_bundle.executor.location_manager'
- '@ez_migration_bundle.helper.sort_converter'
- '@ez_migration_bundle.content_version_matcher'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -361,6 +366,8 @@ services:
- '@ez_migration_bundle.reference_resolver'
- '@ez_migration_bundle.complex_field_manager'
- '@ez_migration_bundle.helper.sort_converter'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -369,12 +376,16 @@ services:
class: '%ez_migration_bundle.executor.content_type_group_manager.class%'
arguments:
- '@ez_migration_bundle.content_type_group_matcher'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

ez_migration_bundle.executor.language_manager:
class: '%ez_migration_bundle.executor.language_manager.class%'
parent: ez_migration_bundle.executor.repository
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -385,6 +396,8 @@ services:
- '@ez_migration_bundle.content_matcher'
- '@ez_migration_bundle.location_matcher'
- '@ez_migration_bundle.helper.sort_converter'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -394,6 +407,8 @@ services:
arguments:
- '@ez_migration_bundle.object_state_matcher'
- '@ez_migration_bundle.object_state_group_matcher'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -402,6 +417,8 @@ services:
parent: ez_migration_bundle.executor.repository
arguments:
- '@ez_migration_bundle.object_state_group_matcher'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -411,6 +428,8 @@ services:
arguments:
- '@ez_migration_bundle.role_matcher'
- '@ez_migration_bundle.helper.limitation_converter'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -419,6 +438,8 @@ services:
parent: ez_migration_bundle.executor.repository
arguments:
- '@ez_migration_bundle.section_matcher'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -428,6 +449,8 @@ services:
arguments:
- '@ez_migration_bundle.tag_matcher'
- '@?ezpublish.api.service.tags'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -437,6 +460,8 @@ services:
arguments:
- '@ez_migration_bundle.trash_matcher'
- '@ez_migration_bundle.helper.sort_converter'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -446,6 +471,8 @@ services:
arguments:
- '@ez_migration_bundle.user_matcher'
- '@ez_migration_bundle.user_group_matcher'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -456,6 +483,8 @@ services:
- '@ez_migration_bundle.user_group_matcher'
- '@ez_migration_bundle.role_matcher'
- '@ez_migration_bundle.section_matcher'
calls:
- ['setReferenceMatcher', ['@ez_migration_bundle.reference_matcher']]
tags:
- { name: ez_migration_bundle.executor }

@@ -8,6 +8,9 @@ Version 5.8.0 (unreleased)
* Fix: usage of the `-a` flag when running `kaliop:migration:mass_migrate` and when running `kaliop:migration:migrate -p`
was not propagated to subprocesses

* Fix: the `if` element was not giving a afatl error for all migration steps affecting repository elements (Content,
Location, etc...), at least for Symfony version 2.7.10

* New: the `kaliop:migration:migrate` and `kaliop:migration:mass_migrate` now accept a `--force` flag that will execute
migrations that were previously executed or skipped or failed.
*NB* this flag is useful when testing migrations, but should be used sparingly in production context, as replaying

0 comments on commit 7a37ca0

Please sign in to comment.
You can’t perform that action at this time.