The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Hi guys,
I was working on a custom Webform handler and some of the configuration was not getting updated when editing the handler config. It worked fine when creating a new one, but when editing some of the settings just didn't get saved.
So I looked around the code a bit and think I found a small issue where the handler instance config doesn't get saved on edit.
Can someone take a look at the patch and maybe we can get some feedback from the devs.
Appreciate it!
| Comment | File | Size | Author |
|---|---|---|---|
| webform_handler.patch | 734 bytes | alecsmrekar |
Comments
Comment #2
jrockowitz commented@alecsmrekar You get the prize for issue of the day.
I just recently saw this issue while working with a handler that has an AJAX callback. I was completely baffled by it and was just starting to figure out this problem.
I am setting this patch to needs review to see if all the tests pass.
Here is the ticket I am working on to understand and resolve my Ajax related issues. #2860434: Create Test Ajax Submission Handler
Comment #3
jrockowitz commentedAll the tests are passing.
@alecsmrekar I am able to edit the 'Contact' form handlers without the patch. I am curious... What in your custom handler is triggering this issue? Is it Ajax?
Comment #4
alecsmrekar commentedExactly, I have 2 ajax fields. The second one is dependent on the first one and there's an ajax update when you change the first one. Other fields work fine.
Comment #5
jrockowitz commentedThe patch works perfectly with #2860434: Create Test Ajax Submission Handler.
@alecsmrekar I know you just wrote 5 line patch but I lost a few hours to this problem because I just could not see the issue properly. So thanks again.
Comment #7
jrockowitz commentedComment #8
alecsmrekar commentedA fresh pair of eyes always helps, I'm glad we sorted it out :)