Issue #2409901: Implement Migration Paths for Flag 7.x
1 open thread
TODO
-
Apply patch to merge request see if it passes. -
Finish making tests. -
Add more tests for various types of data -
Fix until it works
Edited by Matthew Radcliffe
Merge request reports
Activity
added 1 commit
added 1 commit
added 1 commit
added 1 commit
added 1 commit
added 1 commit
added 1 commit
added 1 commit
added 22 commits
-
57743f40...9e12a609 - 12 commits from branch
project:8.x-4.x - ff103b92 - Issue #2409901 by mradcliffe: Adds basic, no-frills test fixture and a couple of kernel tests
- d271c64b - Issue #2409901 by brunodbo, schneidolf, socketwench, dieuwe, marvil07:...
- ca50a9eb - Issue #2409901 by mradcliffe: Adds flag_actions and flag_bookmark as installed in source database
- 17ddb5b5 - Issue #2409901 by mradcliffe: Adds text module to migration tests
- a49ff9cf - Issue #2409901 by mradcliffe: Fixes duplicate flag name in my test fixture
- 3f8e055f - Issue #2409901 by mradcliffe: Adds serialized flag options for node, global...
- 86b7c358 - Issue #2409901 by mradcliffe: Fixes duplicate flagging row
- 6ce6f09a - Issue #2409901 by mradcliffe: Fixes typo in table name
- 86459983 - Issue #2409901 by mradcliffe: Adds missing filter dependency from tests
- 75111790 - Issue #2409901 by mradcliffe: Removes entity, but not entity type migrations...
Toggle commit list-
57743f40...9e12a609 - 12 commits from branch
added 1 commit
- b1c726ab - Add 'd7_user' and 'd7_nodes' as optional dependencies
src/Plugin/migrate/source/d7/Flagging.php 0 → 10064428 'fid', 29 'entity_type', 30 'entity_id', 31 'uid', 32 'sid', 33 'timestamp', 34 ]); 35 36 $query->fields('f', [ 37 'name', 38 'global', 39 ]); 40 41 // If a flag type is provided, only migrate flaggings for that flag. 42 if (isset($this->configuration['flag_type'])) { 43 $query->condition('f.name', $this->configuration['flag_type']); -
Still not 100% sure if there can be multiple flag_type's like suggested in patch #60
$query->condition('f.name', (array) $this->configuration['flag_type'], 'IN'); -
I see, this is used for migrating flags on specific entity types only, so defining multiple flag_types totally make sense.
Edited by Joshua Sedler changed this line in version 14 of the diff
added 1 commit
- c31c335f - Reintroduce possibility to define multiple flag_types
added 39 commits
-
3ad19891...82b1accf - 38 commits from branch
project:8.x-4.x - da1d6e3c - Merge branch flag:8.x-4.x into 2409901-implement-migration-paths
-
3ad19891...82b1accf - 38 commits from branch
Please register or sign in to reply