https://bugs.koozali.org/show_bug.cgi?id=12154
John Crisp <[email protected]> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|CONFIRMED |RESOLVED
Resolution|--- |FIXED
--- Comment #1 from John Crisp <[email protected]> ---
So two things here.
First, there is probably a clever way to combine save and run IbayLettersSave
and WorkgroupSave/workgroup-update but I am not going to try and do it. So I
reverted part of the previous patch.
Second, it still didn't work. This should work by doing Save and then
restarting when you are ready with your drive assignments.
But there was still and issue with onclick - this now returns a $array of
values:
document.getElementsByName('WorkgroupSave')
So you now need this (which makes the assumption this is the value you want):
document.getElementsByName('WorkgroupSave')[0].value = 1
But we can modify this to be safer by adding an ID here:
print $q->hidden (-name => 'WorkgroupSave', -id => 'WorkgroupSave', -override
=> 1, -default => 0);
And then:
document.getElementById('WorkgroupSave').value = 1
And then it seems to work happily.
* Wed Aug 24 2022 John Crisp <[email protected]>
- Revert button removal and update to the onclick [SME: 12154]
/usr/bin/plague-client build smeserver-tw-logonscript
smeserver-tw-logonscript-1_5-10_el7_sme contribs10
Package smeserver-tw-logonscript enqueued. Job ID: 3920.
--
You are receiving this mail because:
You are the QA Contact for the bug._______________________________________________
Mail for each SME Contribs bug report
To unsubscribe, e-mail [email protected]
Searchable archive at https://lists.contribs.org/mailman/public/contribteam/
