As mentioned in the post at #453346: Access checks and performance, complex node access checks kill the benefits of the module. However, basic role based authorization using user_access() still seems to be very performant and would be a great feature for this project.

Thanks,
Chris

CommentFileSizeAuthor
#1 js-1154300-1.patch1021 bytescpliakas

Comments

cpliakas’s picture

cpliakas commented
StatusFileSize
new js-1154300-1.patch1021 bytes

The attached patch adds an "access arguments" key, which if passed loads the user module and passes the arguments to user access. This allows for basic role based authorization without having to explicitly check for them in the JS callback.

~Chris

cpliakas’s picture

cpliakas commented
Title: Allow for basic role based authentication » Allow for basic role based authorization

Changing title...

michielnugter’s picture

michielnugter commented
Status: Active » Patch (to be ported)

This is implemented in the 7.x version already. The 6.x version is now only supported as a patch release. If someone ports the 7.x solution back to 6.x I'll be happy to include it in the release.

michielnugter’s picture

michielnugter commented
Version: 6.x-2.x-dev » 7.x-1.x-dev
Status: Patch (to be ported) » Closed (fixed)