- Issue created by @mathilde_dumond
- 🇨🇭Switzerland mathilde_dumond
Uploading a patch that fixes the problem, I intend to extend it with test coverage.
- 🇨🇭Switzerland mathilde_dumond
Added a check + log in case the module is confused and tries to fetch deleted content.
- last update
almost 2 years ago 5 pass - last update
almost 2 years ago 5 pass - 🇨🇭Switzerland berdir Switzerland
+++ b/src/Plugin/migrate/source/GatherContentMigrateSource.php @@ -343,6 +344,9 @@ class GatherContentMigrateSource extends SourcePluginBase implements ContainerFa } + elseif ($item instanceof ElementSimpleText) { + $value[$key] = ['value' =>$item->getValue()]; + }
missing space after =>
The entity check is strictly speaking not related but fine to have that in here as well.
- Status changed to Needs work
almost 2 years ago 12:20pm 28 April 2023 - Status changed to Needs review
almost 2 years ago 12:31pm 28 April 2023 - last update
almost 2 years ago 5 pass - 🇬🇧United Kingdom JennyDumitrescu
Hi @mathilde_dumond, I've checked the patch in #6 on our dummy site and it works fine pulling in plain text fields.
Is this supposed to also work with plain fields in paragraph fields too? Or will that be separate functionality? We have the ability to map fields into them, but they're not bringing in any content. Thanks! - 🇨🇭Switzerland berdir Switzerland
I assume you mean mapping a repeatable component to paragraphs? that is not yet supported unfortunately, we are looking into it, but it's much more complicated than this. But we are in contact with GatherContent about it and will open a separate issue once we have a plan.
- Status changed to Fixed
almost 2 years ago 8:29am 7 July 2023 -
Berdir →
committed 86e251d7 on 6.x authored by
mathilde_dumond →
Issue #3357051 by mathilde_dumond: Support repeatable text fields
-
Berdir →
committed 86e251d7 on 6.x authored by
mathilde_dumond →
Automatically closed - issue fixed for 2 weeks with no activity.