Assign subgroups/parent groups while creating new group

Moving the subgroup parent selector to the node add/edit form instead of it being on a tab is definitely do-able and would be less annoying. This is already on my TODO list:)

subscribe

subscribing

Would love to see this soon, as it seems like a no-brainer to me. The first place I looked when trying to set up a subgroup was the groups section in the node add/edit form. When I realised the subgroup stuff was a separate tab I didn't think it made much sense.

Incidentally, moving it all to the node add/edit form would also make much more sense when you think about #482156: Generate tokens for subgroup paths because if tokens were available and you could set the parent group at the time of creation, pathauto could generate an appropriate alias based on the group hierarchy. At the moment I'm having to simulate this behaviour using a taxonomy.

roger that!

This patch enables the parent group to be defined on the node add/edit form. Let me know if you see any issues.

Works great for me (tested with hierarchical select enabled).

The only thing I noticed was that the old Subgroups tab remained visible until I cleared the cache, and it generated an error if clicked. I guess that's not really an issue though, since a stable release would already have this patch included.

Thanks for the quick patch!

Edit: Found a bug! If you change the subgroup parent with hierarchical select enabled and then preview the node rather than save it, the change is forgotten. This does not happen with hierarchical select disabled.

good catch on the preview bug. Here is a new patch that fixes the Hierarchical Select menu when a group is being previewed.

This patch completely replaces the previous one posted above. If you've already applied the previous patch, you'll need to undo it then apply this new one.

Yup, that sorted it. Haven't seen any other problems yet so it looks good to me.

This patch has been committed.