When a site uses a node access module to control access to webforms, it can be confusing to end users to also have the built-in Webform Access Control settings.

This patch adds an option to disable the built-in access control, under the Advanced Options at admin/settings/webform .

CommentFileSizeAuthor
#3 webform_optional_access_control2.patch1.02 KBquicksketch
#2 webform_optional_access_control.patch3.31 KBquicksketch
webform_optional_access_control.patch5.88 KBmatt2000

Comments

quicksketch’s picture

quicksketch
he/him
commented

A better approach here would be to use #access rather than wrapping the fields in a large IF statement. Seems like a good overall option for Webform to me.

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs review » Fixed
StatusFileSize
new webform_optional_access_control.patch3.31 KB

Here's a revised patch that makes less changes and preserves per-webform settings even when the access control feature is disabled. Thanks for the great start on this one! I've committed the patch to both 5 and 6 versions.

quicksketch’s picture

quicksketch
he/him
commented
StatusFileSize
new webform_optional_access_control2.patch1.02 KB

Hmm, your patch took out some messages that I didn't notice before committing. These have been restored.

Status: Fixed » Closed (fixed)

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