OK, so I might be pusing it here, but doesn't hurt to ask.

You described the AND/OR logic of the multinode_access_patch here: http://drupal.org/node/243731#comment-798303

Doing so will give you:

IF (all OR og) AND (tac)

Instead of:

IF (all OR og OR tac)

I seem to have gotten the first case working with OG and TAC: http://drupal.org/node/243731 Thanks!

I have been looking into writing grants for OG User Roles, so that users can see the content their OGR Roles give them access to outside of the group context. I have a roles table which easily and securely allows me to write the grants and tie them to the correct uids.

Problem is, the AND/OR logic needs to be like this for it to work:

IF ((all OR og) AND (tac)) OR (ogr)

Maybe this will work as well:

IF (all OR og) AND (tac) OR (ogr)

Looks like you've ran into this issue here: http://drupal.org/node/196922

Think it's doable?

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Active » Closed (duplicate)

Yes. Let's pursue that topic over in the main thread -- the core patch. I think it may be too complex to implement using the current system.

This kind of complex logic is possible, but likely needs a UI to establish it, or we have to adjust the $grants array to pass the needed information.

somebodysysop’s picture

somebodysysop commented

I'm on it. You can close this, then.

somebodysysop’s picture

somebodysysop commented

I'm ready to move forward on this. Shall I pursue this here or here?: http://drupal.org/node/196922#comment-816284

The reason I ask is because the moderator appears to want to go in an entirely different direction. This was the logic I was looking for:

if (all OR og) AND (tac) OR (ogr)

and I got it working with my modification of the patch. The patch still works as before, but now can provide this additional logic.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

Um, I think I'm the moderator over there, too.

The problem that I'm having is that people aren't actually testing and reviewing the patch.