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.
API page: https://api.drupal.org/comment/reply/12215692
The example given for the substr plugin is invalid YAML. The start and length parameters should be indented the same amount as the source parameter.
Comment | File | Size | Author |
---|---|---|---|
#9 | Fixed_Comment 6.png | 4.22 KB | mahalingam_cs |
#7 | 2938645-docblock-yaml-commection-7.patch | 600 bytes | kmbremner |
#4 | 2938645-docblock-yaml-correction-4.patch | 598 bytes | swarad07 |
Comments
Comment #2
BenStallings CreditAttribution: BenStallings commentedComment #3
TR CreditAttribution: TR commentedYou're correct. How about providing a patch? That documentation is in core/modules/migrate/src/Plugin/migrate/process/Substr.php
Comment #4
swarad07Adding patch for this. Please review.
Comment #5
TR CreditAttribution: TR commentedThe small, simple patch in #4 fixes the documentation issue and doesn't affect any code.
Comment #6
Gábor HojtsyWhy is 6 getting lost from start?
Comment #7
kmbremnercorrect patch based on comment #6
Comment #8
kmbremnerComment #9
mahalingam_cs CreditAttribution: mahalingam_cs at TATA Consultancy Services commentedPatch in comment #7 fixed the issue reported in comment #6
Comment #10
larowlanRemoving review credit for @mahalingam_cs
@mahalingam_cs, please don't add screenshots of patches applied, we have the testbot for that.
Please see what we give credit for
Comment #11
larowlanAdding credit for @Gabor Hojtsy as his review shaped the patch
Comment #14
larowlanCommitted as c88ccd9 and pushed to 8.6.x.
Cherry-picked as 3a8ede6 and pushed to 8.5.x
Thanks