-
Notifications
You must be signed in to change notification settings - Fork 133
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Created a followup to merge OG and OG UI: #396 |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Damien McKenna <damien@mc-kenna.com>
Just want to provide some feedback: |
list(...) is forbidden, use [...] instead.
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')); |
There was a problem hiding this comment.
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.
->sort($this->entityType->getKey('weight')); | |
->sort($this->entityType->getKey('weight')) | |
->accessCheck(); |
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. |
FWIW the current PR does apply cleanly to the current 8.x-1.x branch on d.o. |
I created a PR to rebase this PR with the latest from the 8.x-1.x branch: zerolab#11 |
I also created a PR that adds more accessCheck() calls: zerolab#12 |
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:
|
Since https://www.drupal.org/project/og/issues/3467694 was merged into the Thank you for contributing to Organic groups module. |
This PR is a continuation of amitaibu#196. It:
To-Do:
og_role.add
,og_role.update
,og_role.delete
)Also needs decision as to when amitaibu#315 comes into force.