- First commit to issue fork.
- 🇷🇴Romania stefan.butura
Patch based on #4. Fixed an error appearing when there are no paragraphs of a specific type and removed comments.
- 🇧🇪Belgium fernly
I can confirm patch #11 is not working when you use a modal (paragraphs stable widget) to add a paragraph.
I'm looking in to this. I think it sucks when an editor puts in all the effort to build up a page to see tons of errors when saving. - Status changed to Needs review
9 months ago 10:46am 20 March 2024 - 🇧🇪Belgium fernly
I fixed the patch to work with the Paragraphs (stable) widget and Drupal 10. The latter can be tricky because this module supports multiple Drupal versions. However, the used hook
hook_field_widget_complete_WIDGET_TYPE_form_alter
is a new one for Drupal 10 only and replaceshook_field_widget_multivalue_WIDGET_TYPE_form_alter()
.So if older Drupal versions need to be supported, the hook
hook_field_widget_multivalue_WIDGET_TYPE_form_alter()
needs to be added again as well, containing the same code.Note that this patch was not written for the legacy paragraphs widget. I haven't tested that.
- Status changed to Needs work
6 months ago 6:02am 6 July 2024 - 🇨🇦Canada pavlosdan
Looks like this hook exists for Drupal 9 as well so we should be good in that regard: https://api.drupal.org/api/drupal/core%21modules%21field%21field.api.php... (at least it exists for 9.5).
I've removed support for D8 given its EOL and paragraphs dropping it as well.
I've tested the patch with all the edit modes that the "Paragraphs (stable)" widget has and it looks like it works as advertised. Thanks a lot for driving this forward!
Setting this back to needs work to see if we can get this working for the legacy widget as well.
- Merge request !7Issue #3088250: Disallow adding more than the allowed amount of paragraph items. → (Merged) created by pavlosdan
- Status changed to Needs review
6 months ago 7:10am 6 July 2024 - 🇨🇦Canada pavlosdan
Looks like the same code works for the legacy widget as well and we just needed the extra hook for that one. Setting this back to needs review.
-
pavlosdan →
committed 1c002c9a on 8.x-1.x
Issue #3088250 by pavlosdan, Fernly, jlblanco, stefan.butura, demonde:...
-
pavlosdan →
committed 1c002c9a on 8.x-1.x
- Status changed to Fixed
4 months ago 12:07am 16 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.