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.
Since most relations probably won't use extra fields (and you have to install to plugin first to add them anyway), turning the checkbox Use 2-step wizard when creating a new %entity_type entity within a %group_type group off is probably a better default for the majority of the use cases.
Comment | File | Size | Author |
---|---|---|---|
#2 | 2867858-2.patch | 479 bytes | seanB |
|
Comments
Comment #2
seanBPatch is attached.
Comment #3
geek-merlin> Since most relations probably won't use extra fields (and you have to install to plugin first to add them anyway)
i don't get what this is. does this still exist?
Comment #4
seanBYes, the relation between a group and a node is stored in a "group content" entity. These "group content" entities are fieldable and the checkbox Use 2-step wizard when creating a new %entity_type entity within a %group_type group allows you to expose the form for this entity.
My argument would be most cases are not adding extra fields for the group content entity, so enabling the checkbox by default does not make sense.
Comment #5
floydm CreditAttribution: floydm at Affinity Bridge commentedAgreed, being able to field and add values on the relationship is extremely powerful but not used in most group content scenarios. It should be an opt-in feature rather than the default behaviour.
I've tested the patch in #2. It works as expected.
Comment #6
kristiaanvandeneyndeThe initial logic was to ensure data integrity: Rather have it on by default and then disable it than risk not collection precious data. But that was back when plugins could be installed without saving the confirmation page. I guess it makes sense to disable this by default now.
Comment #8
kristiaanvandeneynde