Commit b7c301e6 authored by bwaindwain's avatar bwaindwain Committed by Damien McKenna
Browse files

Issue #3189138 by bwaindwain, DamienMcKenna: Form element still visible even... 

Issue #3189138 by bwaindwain, DamienMcKenna: Form element still visible even though permissions turned off.
parent 0c914b8f

CHANGELOG.txt

+2 −0
Original line number Diff line number Diff line
@@ -13,6 +13,8 @@ Metatag 8.x-1.x-dev, xxxx-xx-xx 
  is the route entity.

#3188839 by mortim07, DamienMcKenna, mrshowerman, FiNeX, bwaindwain: Node tokens

  missing.

#3189138 by bwaindwain, DamienMcKenna: Form element still visible even though

  permissions turned off.





Metatag 8.x-1.15, 2020-12-05

metatag_extended_perms/metatag_extended_perms.module

+4 −1
Original line number Diff line number Diff line
@@ -16,6 +16,8 @@ function metatag_extended_perms_field_widget_form_alter(&$element, FormStateInte 
  if ($context['widget'] instanceof MetatagFirehose) {

    $group_manager = \Drupal::getContainer()->get('plugin.manager.metatag.group');



    // Prevent access to the element until at least one permission is granted.

    $element['#access'] = FALSE;

    foreach (Element::children($element) as $group_id) {

      $group = $group_manager->getDefinition($group_id, FALSE);

      if ($group === NULL) {
@@ -33,9 +35,10 @@ function metatag_extended_perms_field_widget_form_alter(&$element, FormStateInte 
        $element[$group_id][$tag_id]['#access'] = \Drupal::currentUser()

          ->hasPermission('access metatag ' . $group_id . '__' . $tag_id);



        // Make the group accessible if user has access to the tag.

        // Make the parent and group accessible if user has access to the tag.

        if ($element[$group_id][$tag_id]['#access']) {

          $element[$group_id]['#access'] = TRUE;

          $element['#access'] = TRUE;

        }

      }

    }