As reported on Slack, we cannot update unpublished entities because we check for a permission that does not exist. Let's fix this by only making sure we check for pub/unpub permissions when dealing with the view operation.

CommentFileSizeAuthor
#2 group-3163727-2.patch31.77 KBkristiaanvandeneynde
#2 group-3163727-2-test-only.patch30 KBkristiaanvandeneynde

Comments

kristiaanvandeneynde created an issue. See original summary.

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
commented
Status: Active » Needs review
StatusFileSize
new group-3163727-2-test-only.patch30 KB
new group-3163727-2.patch31.77 KB

The last submitted patch, 2: group-3163727-2-test-only.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
commented
Status: Needs review » Fixed
Ikows’s picture

Ikows commented

Hi,

The patch successfully fixed my problem thanks for that :) . But I still have issue with the Clone operation from entity_clone

Any idea ?

Thanks in advance

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Title: Don't deny access to update/delete unpublished entities if we don't support it yrt » Don't deny access to update/delete unpublished entities if we don't support it yet
azinck’s picture

azinck commented

@Ikows: I'm not too clear if you're looking to clone an entire group or, like me, just want group-level permissions for the clone operation, but I've opened a ticket that would allow support for the latter: #3171623: Support other entity operations in group_entity_access()