Skip to content
This repository was archived by the owner on Aug 18, 2024. It is now read-only.

Group roles and permissions UI#243

Closed
zerolab wants to merge 110 commits intoGizra:8.x-1.xfrom
zerolab:196-redux
Closed

Group roles and permissions UI#243
zerolab wants to merge 110 commits intoGizra:8.x-1.xfrom
zerolab:196-redux

Conversation

@zerolab
Copy link
Copy Markdown
Contributor

@zerolab zerolab commented Mar 7, 2017

This PR is a continuation of amitaibu#196. It:

  • Fixes the clashing routes for adding a role
  • Fixes new group roles considered required, rather than standard
  • Adds a role deletion form
  • Adds a group permissions and a group role permissions form

To-Do:

  • clean-up the routing
  • use separate permissions for roles (og_role.add, og_role.update, og_role.delete)
  • allow permissions groups other that "Group" and "Group content"
  • tests

Also needs decision as to when amitaibu#315 comes into force.

@amitaibu

This comment has been minimized.

@zerolab

This comment has been minimized.

@amitaibu

This comment has been minimized.

Comment thread og_ui/src/Form/OgRoleForm.php Outdated
Comment thread og_ui/src/Form/OgRoleForm.php Outdated
@MPParsley

This comment has been minimized.

@jludwig

This comment has been minimized.

@zerolab

This comment has been minimized.

@MPParsley

This comment has been minimized.

@zerolab

This comment has been minimized.

@amitaibu

This comment has been minimized.

@pfrenssen
Copy link
Copy Markdown
Contributor

Created a followup to merge OG and OG UI: #396

@pfrenssen

This comment has been minimized.

@zerolab

This comment has been minimized.

@pfrenssen

This comment has been minimized.

@pfrenssen pfrenssen self-assigned this Jul 4, 2018
CathLind referenced this pull request in CathLind/drupal_og May 7, 2021
CathLind referenced this pull request in CathLind/drupal_og May 7, 2021
CathLind referenced this pull request in CathLind/drupal_og May 7, 2021
CathLind referenced this pull request in CathLind/drupal_og May 7, 2021
CathLind referenced this pull request in CathLind/drupal_og May 7, 2021
@MPParsley MPParsley added this to the 8.x-1.0-beta1 milestone Sep 15, 2021
Comment thread og_ui/src/Form/OgPermissionsForm.php Outdated
@cofihouse
Copy link
Copy Markdown

cofihouse commented Jul 31, 2022

Just want to provide some feedback:
Setting a group to 'private' hides all the field information and functions for the group node from non-members, including things like the 'subscribe' link. I would think an access module should block access to the entities and users of a group, and not necessarily the fields of the group node itself. In my use case, I would like the fields of the actual group node to still be viewable by non-members, so they can decide if they want to join the group or not.
I'm not great at coding modules. I'm going to poke around the code a little, but I probably can't contribute much. In a few weeks I should have some funds to pay for development toward this module.

Comment thread tests/src/Kernel/Access/RolesAndPermissionsUiAccessTest.php
Comment thread tests/src/Kernel/Access/RolesAndPermissionsUiAccessTest.php Outdated
Comment thread src/OgRoleAccessControlHandler.php Outdated
list(...) is forbidden, use [...] instead.
@impara
Copy link
Copy Markdown

impara commented Nov 2, 2022

What is the status of this pull request?

$query = $this->getStorage()->getQuery()
->condition('group_type', $this->groupType, '=')
->condition('group_bundle', $this->groupBundle, '=')
->sort($this->entityType->getKey('weight'));
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add access check to entity query for Drupal 10 compatibility.

Suggested change
->sort($this->entityType->getKey('weight'));
->sort($this->entityType->getKey('weight'))
->accessCheck();

@damienmckenna
Copy link
Copy Markdown

Could someone please update the PR with the latest from the 8.x-1.x branch? It's really out of date and can't be applied anymore. Thank you.

@damienmckenna
Copy link
Copy Markdown

FWIW the current PR does apply cleanly to the current 8.x-1.x branch on d.o.

@damienmckenna
Copy link
Copy Markdown

I created a PR to rebase this PR with the latest from the 8.x-1.x branch: zerolab#11

@damienmckenna
Copy link
Copy Markdown

I also created a PR that adds more accessCheck() calls: zerolab#12

@damienmckenna
Copy link
Copy Markdown

With the latest PR (my fork of it anyway) on 10.2.x when you try to add a new role you get this error:

Drupal\Core\Entity\Exception\UndefinedLinkTemplateException: No link template 'canonical' found for the 'og_role' entity type in Drupal\Core\Entity\EntityBase->toUrl() (line 211 of core/lib/Drupal/Core/Entity/EntityBase.php).

@claudiu-cristea
Copy link
Copy Markdown
Contributor

Since https://www.drupal.org/project/og/issues/3467694 was merged into the 8.x-1.x branch, the development has been moved to https://www.drupal.org/project/og. If this change is still actual, please open an issue at https://www.drupal.org/project/issues/og and provide there a merge request similar with this pull request.

Thank you for contributing to Organic groups module.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3rd party OG permissions don't say which module they are from