The main fields defined in og_og_fields_info() have description strings for each field that describe what the field *does* once it exists on the entity, eg:
'description' => t('Determine if this should be a group.'),
'description' => t('Determine if group should use default roles and permissions.'),
However, those in og_access_og_fields_info() are phrased as an action from the point of view of adding that field to the entity type:
'description' => t('Add Group access field to group types.'),
'description' => t('Add Group access field to group-content types, that may override the group settings.'),
This is confusing. These should be changed so they read in the same way, so something like:
'description' => t('Determine access to the group.'),
'description' => t('Determine access to the group content, which may override the group settings.'),
Comment | File | Size | Author |
---|---|---|---|
#1 | 1865824.og_.og_access_og_fields_info-descritions.patch | 1010 bytes | joachim |
Comments
Comment #1
joachim CreditAttribution: joachim commentedHere's a patch with that change.
Comment #2
amitaibuCommitted, thanks.
Comment #3.0
(not verified) CreditAttribution: commentedclearer explanation