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 .

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quicksketch’s picture

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

Status: Needs review » Fixed
FileSize
3.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

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.