Skip to content

Conversation

jacobembree
Copy link

I'd like to improve the node_access() function. For one thing, I'd like to add the op parameter so that node_access() can be used as an access callback for node/add/%.
@signalpoint do you have any recommendations or suggestions before I start? I'm thinking of using Drupal.user.content_types_user_permissions extensively.
Does node_access() indeed belong in DrupalGap or does it go in jDrupal?

@signalpoint
Copy link
Owner

@jacobembree I think node_access() can stay inside DrupalGap, since I assume folks using just jDrupal are building their own front end and access control layers.

If you'd like to add support for the op parameter, that'd be great. I personally don't use that feature very much on the front end, and have since used just the backend to determine access control, and then on the front end just try to make it so the UX can never land them in a spot where they don't have access to do something.

@jacobembree
Copy link
Author

Almost all of my code came from Drupal's node_access() and node_node_access() functions.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants