Currently we're hiding access to specific metatags in the form by comparing to a 'context' parameter in hook_metatag_info() when we should probably be using a hook instead (that could fallback to the context).

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Issue tags: +D7 stable release blocker

.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Version: » 7.x-1.x-dev
Assigned: Dave Reid » Unassigned
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

Marked #1497402: Expand permissions range as a duplicate.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Assigned: Unassigned » Dave Reid
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Assigned: Dave Reid » Unassigned
Status: Active » Closed (duplicate)

Lets move this over to #1961354: Optionally control meta tag permissions individually which has some working code that needs review.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Title: Add a hook_metatag_access() for controlling access for viewing and editing metatags. » Add a hook_metatag_access() for controlling access for viewing and editing metatags
Status: Closed (duplicate) » Postponed
Issue tags: -D7 stable release blocker

On second thoughts, lets leave this as a postponed issue and remove it from the stable release blockers, as I don't believe it'll be important once we have extensive permissions available.

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
CreditAttribution: bkosborne at Princeton University commented
Issue summary: View changes

Can this be closed now, since metatag has the ability to specify permissions per metatag now?