Right now webform returns false on all uncaught $op values, which overrides the permissions system. It should return NULL. Also, view permissions cannot be used with the drupal node-access system with module like simple_access because the 'view' $op returns the value of the published state of the node, which is not necessary since node_access already does this. This patch removes those unneccesary features.

CommentFileSizeAuthor
webform.patch.txt957 bytescrunchywelch

Comments

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs review » Fixed

'unneccesary features' is a very nice way of putting that. Committed. Thanks again!

John McV’s picture

John McV commented

Hello,
I'm having problems with permissions. Those who have only access to webform results can also delete those results. How do I manually install your patch? I tried manually removing the patch code marked - and adding those with + but it didn't work for me. Also what settings should I use if any with simple access on webforms? For my purposes, it would be ideal if a given result could be diplayed as a node using it's first field as the title. I could then filter them through views and control access that way as well. Anyway, thanks for the great module.
John

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)