Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add function to entity: entity_access #3011

Closed
moonray opened this issue Mar 30, 2018 · 10 comments · Fixed by backdrop/backdrop#2451
Closed

Add function to entity: entity_access #3011

moonray opened this issue Mar 30, 2018 · 10 comments · Fixed by backdrop/backdrop#2451

Comments

@moonray
Copy link

moonray commented Mar 30, 2018

Paragraphs module uses this function, and it's likely other modules converted from Drupal would use it as well. We should add it to core.


PR by @jenlampton: backdrop/backdrop#2451

@moonray moonray changed the title Add functions to entity: entity_access and entity_revision_is_default Add function to entity: entity_access Mar 30, 2018
@herbdool
Copy link

herbdool commented Apr 8, 2018

I took a look at the code and see that tests pass. But best if @quicksketch review.

@quicksketch
Copy link
Member

This looks great! I added a lot of feedback to the PR: backdrop/backdrop#2119 (review)

This PR makes some minor API changes as part of the additions. Notably comment_access() and taxonomy_term_access() previously supported an $op of 'edit', but now they are 'update' (matching nodes). We'll need to make a changelog notice for this change once this is completed.

@jenlampton
Copy link
Member

Looks like this still needs work, so won't make it into 1.10. Bumping milestone to 1.11.

@jenlampton
Copy link
Member

I've rebased and resolved a handful of items in backdrop/backdrop#2451, this is ready for another review.

@quicksketch
Copy link
Member

I left a review on on the code: backdrop/backdrop#2451 (review)

This is still looking really good. I only found a few docblock issues and an apparently unused private property.

@jenlampton
Copy link
Member

I've resolved the feedback on the PR, but since this got bumped to 1.13 it may not get any attention for a little while :)

@quicksketch
Copy link
Member

backdrop/backdrop#2451 looks great. I've incremented the version number in the @since docblocks to 1.13.0. I'll let tests run through again; barring any problems I think this is ready.

@quicksketch
Copy link
Member

I've merged backdrop/backdrop#2451 into 1.x for 1.13.0. Thanks @jenlampton for rerolling! SUPER huge thanks to @moonray for spearheading this and doing a great job on expanding our Entity API and bringing consistency.

@quicksketch
Copy link
Member

Change notice written up at https://api.backdropcms.org/change-records/taxonomy_term_access-and-comment_access-op-edit-parameter-renamed-update.

@jenlampton
Copy link
Member

jenlampton commented May 22, 2019

Reopening this issue since it has some unforeseen complications that are breaking a lot of contrib. See:

I don't really know what to do with the labels/milestones on this one. Should we open a new issue? I'm going to do that so we don't get confused. See #3809

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants