-
Notifications
You must be signed in to change notification settings - Fork 40
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
Comments
I took a look at the code and see that tests pass. But best if @quicksketch review. |
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 |
Looks like this still needs work, so won't make it into 1.10. Bumping milestone to 1.11. |
I've rebased and resolved a handful of items in backdrop/backdrop#2451, this is ready for another review. |
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. |
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 :) |
backdrop/backdrop#2451 looks great. I've incremented the version number in the |
…y::access() for all Entities. By @moonray, @quicksketch, @jenlampton, and @herbdool.
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. |
Change notice written up at https://api.backdropcms.org/change-records/taxonomy_term_access-and-comment_access-op-edit-parameter-renamed-update. |
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 |
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
The text was updated successfully, but these errors were encountered: