Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
This issue is to add documentation for the following source plugins:
- ContentEntity.php
- d8/Config.php
- EmbeddedDataSource.php
- EmptySource.php
Proposed resolution
Do it.
Remaining tasks
Review, commit.
Issue fork drupal-3206939
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 3206939-migrate-drupal-plugins-documentation changes, plain diff MR !491
Comments
Comment #3
MatroskeenI have no idea what to add in ContentEntityDeriver.php yet. Please let me know if you have any suggestions.
Comment #4
quietone CreditAttribution: quietone as a volunteer commentedGood question.
Searching shows there are 6 derivers in migrateland and I don't think any of them are documented. At least, D7NodeDeriver and D7TaxonomyTermDeriver are not. But what to do? Well, since there are only 6 and they all do much the same thing it would be efficient to add the documentation for them in one issue. What do you think?
Setting NW for unresolved comments.
Comment #5
quietone CreditAttribution: quietone as a volunteer commented@Matroskeen, what do you think of including the migrate module source plugins in this issue?
Comment #6
MatroskeenIt sounds good to me!
I didn't include DummyQueryTrait.php mentioned in the meta task because it looks good and I'm not sure what can be improved.
Comment #7
quietone CreditAttribution: quietone as a volunteer commentedGlad you agree about doing both migrate and migrate_drupal plugins in one issue! Saves us all a bit of work.
I applied the diff locally and read through all the changes. It all looks good and all issue resolved. I agree about DummyQueryTrait, I too could not improve on what is there. It is sufficient to explain what it does.
The failing test is #3207086: [HEAD BROKEN] Consistent failure in MonthDatePluginTest which has been fixed so this can be RTBC.
Comment #10
catchCommitted/pushed to 9.2.x and cherry-picked to 9.1.x, thanks!