[buddypress-trac] [BuddyPress Trac] #7210: Group type frontend integration
buddypress-trac
noreply at wordpress.org
Thu Sep 29 07:34:51 UTC 2016
#7210: Group type frontend integration
-------------------------+--------------------
Reporter: r-a-y | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 2.7
Component: Groups | Version: 2.6.0
Severity: normal | Resolution:
Keywords: has-patch |
-------------------------+--------------------
Comment (by hnla):
Allowing for a value to be set to represent a default checked=checked on
register is a good idea, let the dev or admin decide which 'type'
represents a 'normal' or 'default' group for their install.
Of a lesser note are we concerned that in an existing install of groups if
one then sets a new group 'type' then creates or associates a group to
that type we now have a group list with a column that shows a group type
string(label) for perhaps one group but the rest of the type columns for
groups are empty, likewise create steps shows one group type, the
possibility of creating a non group type exists but isn't clear?
My feeling is that we do:
Types are registered with `bp_groups_register_group_type` this action
prompts BP to assume a type 'Normal' (terminology is wrong but...)
BP sets a type 'normal' for all groups unless create steps or admin
backend has overridden with a new or specific type from the choices (yes
this step is problematic as it means running through all groups adding
values to the type field so perhaps not a great idea).
A new param is provided to allow for a preset checked=checked to exist for
a chosen type.
If this param wasn't set and if BP did set a type 'Normal' the default
selection would be this value 'normal' set by BP automagically.
A new param is provided that allows for a descriptive string to be set on
create steps for the type checkbox items:
`'create_step_types_descrip' => 'Please select the group type from the
available options or leave as the default selection set',`
That last option is really not massively important but could be nice but
how that is generated to screen needs thought so that appropriate markup
would be generated so likely a step too much at this stage.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7210#comment:20>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list