Commit 222eb5cc authored by Nikolay Lobachev's avatar Nikolay Lobachev
Browse files

Issue #3314382: Remove getRoles method.

parent 05b2e660
Loading
Loading
Loading
Loading
+0 −23
Original line number Diff line number Diff line
@@ -111,29 +111,6 @@ class GroupPermissionsManager implements GroupPermissionsManagerInterface {
    return $this->groupPermissionStorage->getAllActive();
  }

  /**
   * {@inheritdoc}
   */
  public function getGroupRoles(GroupInterface $group) {
    // @todo update this logic
    if (empty($this->groupRoles[$group->id()])) {
      $group_type_id = $group->getGroupType()->id();
      $properties = [
        'group_type' => $group_type_id,
        'permissions_ui' => TRUE,
      ];

      $roles = $this->groupRoleStorage->loadByProperties($properties);

      uasort($roles, '\Drupal\group\Entity\GroupRole::sort');

      $outsider_roles = $this->groupRoleStorage->loadSynchronizedByGroupTypes([$group_type_id]);
      $this->groupRoles[$group->id()] = array_merge($roles, $outsider_roles);
    }

    return $this->groupRoles[$group->id()];
  }

  /**
   * {@inheritdoc}
   */
+0 −11
Original line number Diff line number Diff line
@@ -44,17 +44,6 @@ interface GroupPermissionsManagerInterface {
   */
  public function getAll();

  /**
   * Get group roles.
   *
   * @param \Drupal\group\Entity\GroupInterface $group
   *   Group.
   *
   * @return array
   *   Group roles.
   */
  public function getGroupRoles(GroupInterface $group);

  /**
   * Retrieves Group permission entity for a group.
   *