- Issue created by @Anybody
- 🇩🇪Germany Anybody Porta Westfalica
One question upfront: Does this require "Allow users to post submissions from a dedicated URL for all webform" option to be enabled at /admin/structure/webform/config?
If yes, maybe this should be added to the documentation or even better use Form States API to ensure it's both enabled?I also enabled this one and tried /form and /webform but both didn't fix the issue.
What am I missing or doing wrong? I'm even quite sure it was working in the past, but sadly only 99% sure I've tested it as anonymous user.
As admin user it works as expected in the dialog! - 🇩🇪Germany Anybody Porta Westfalica
OK, I finally had some time to go into details.
Indeed, it's only possible to use the "Site-wide dialog support", if
So I see some possibilities to solve this, but I think it would make sense to first discuss the way to go, before touching code.
Some resolution ideas:
At webform global settings
/admin/structure/webform/config
and each individual webform configuration (/admin/structure/webform/manage/X/settings
):- ... a warning could be shown, if
Enable site-wide dialog support
checkbox is checked, which informs users that this option also needs theAllow users to post submissions from a dedicated URL for all webform
option to be enabled - ... an error could be triggered on form save, if
Enable site-wide dialog support
checkbox is checked, butAllow users to post submissions from a dedicated URL for all webform
option is disabled. - ... place the dialog option within the
Allow users to post submissions from a dedicated URL
fieldset to show the relation more clearly and make it dependent
Would be great to get maintainer feedback, perhaps there are better alternatives or I'm still missing something.
Now at least with both options enabled, the functionality works as expected! 🎉
But it's super risky to run into this currently, as it's not documented in the admin UI. - ... a warning could be shown, if
- 🇩🇪Germany gogowitsch
Personally, I would go with your solution 2, “an error could be triggered on form save”. It strikes a good balance for site owners to understand what the problem is.
About solution 2, I also like that GET requests are not affected by the small performance penalty of the additional permission check.