Could we add a feature with a page for adding multiple node permissions to a user, instead of adding multiple user permissions to a node?

Update

This feature has now been committed and is included in the 7.x-1.1 release.

CommentFileSizeAuthor
#3 flexiaccess-autocomplete_bugfix-2113493-3.git_.patch955 bytesrudolfbyker
#1 flexiaccess-useracl1-2113493-1.git_.patch12.89 KBrudolfbyker

Comments

rudolfbyker’s picture

rudolfbyker
Primary language Afrikaans
Location South Africa
commented
Assigned: Unassigned » rudolfbyker
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new flexiaccess-useracl1-2113493-1.git_.patch12.89 KB

I have implemented this feature. I was aiming for the use of a table, which worked very well under some circumstances, and not at all under others. See http://drupal.stackexchange.com/questions/90282/d7-fapi-unexpected-bahvi... for more info. This link is also in the code, until it gets resolved.

Patch attached, please test and let me know if there are bugs!

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Issue summary: View changes

This feature has now been committed and is included in the latest 7.x-1.x-dev release.

Please review. I'll push it to a new official version as soon as I get some feedback from the community that it works and is stable.

rudolfbyker’s picture

rudolfbyker
Primary language Afrikaans
Location South Africa
commented
StatusFileSize
new flexiaccess-autocomplete_bugfix-2113493-3.git_.patch955 bytes

When there are no content types enabled on the Flexi Access config page, then an SQL error is generated in the autocomplete function, because the IN clause gets an empty argument. Here is a patch. Corrected with if (!empty(...))

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented

Patch in #3 committed to the repository.

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Version: 7.x-1.x-dev » 7.x-1.1
Issue summary: View changes
Status: Needs review » Fixed

This feature is in ver. 7.x-1.1.

Status: Fixed » Closed (fixed)

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