Closed (fixed)
Project:
Webform Score
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 Oct 2017 at 20:49 UTC
Updated:
25 Jul 2024 at 06:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
abutze commentedThanks. It would be great to get this comitted to latest dev version.
Comment #3
socialnicheguru commentedIs there a patch
Comment #4
zanoman commentedThanks Cogent,
AFAIU, I tried adding
$webform->save();
in setScoreConfig function in WebformScore.php (unzipped, rezipped package),
uninstalled the module with easyinstall and dumping the core/module/webform_score folder
without luck, still dumping the score settings of the element :(
Could you please elaborate where you added $webform->save()?
Thank you
Comment #5
stefank commentedHi,
Attaching a patch to be able to save the score config data into webform element.
Comment #6
richard.lampitt commentedI am also having this issue.
I've applied the patch, but unfortunately it doesn't seem to be working.
Comment #7
rerab jordhen commentedit's an issue through ThirdpartySetting so cannot be safe . so adding webform plugin can resolved the issue
Comment #8
vishal choudhary@rerab can you please share the screenshot of you facing the issue on this module?
why do we use the Webform Module and you say the issue is resolved?
please provide the steps where you are facing the issue.
Comment #9
nigelcunningham commentedI'm working on this at the moment - I have it mostly working but one bit that's still problematic. I'm hoping to finish it tomorrow.
Comment #10
nigelcunningham commentedI've just pushed my work on this ticket (and related tickets) to the dev branch and would appreciate any test results others can provide.
Comment #11
nicklundy commentedYes! I can confirm dev branch fixes this issue and I'm now able to save configurations.
Comment #12
nigelcunningham commentedComment #13
nigelcunningham commentedComment #14
anne-pierre commentedSame as #11. I have required the dev branch and a quick test was successful.
Thanks Nigel Cunningham.
Comment #15
nigelcunningham commentedThanks for the feedback!