Submitted by Kruecke on Fri, 04/25/2014 - 07:53
In the greylisting module (Virtualmin -> Email Messages -> Email Greylisting) the "select all" / "invert selection" buttons do not work. They are just dead.
Status:
Closed (fixed)
Comments
Submitted by andreychek on Fri, 04/25/2014 - 09:30 Comment #1
I'm seeing this behavior too Jamie. I looked into it a bit, and I may have stumbled onto the cause.
The JavaScript error I receive is:
Uncaught TypeError: Cannot set property 'checked' of undefined
Looking at the source, the code in question is this here:
onClick='f = document.forms[0]; ff = f.d; ff.checked = true; r = document.getElementById("row_"+ff.id); if (r) { r.className = "mainsel" }; for(i=0; i<f.d.length; i++) { ff = f.d[i]; if (!ff.disabled) { ff.checked = true; r = document.getElementById("row_"+ff.id); if (r) { r.className = "mainsel" } } } return false'
The problem may be that there is another form in the HTML that actually appears before the "Whitelisted clients" form containing the select all / invert links (the first form is the one for enabling/disabling Postgrey).
I suspect the above code is accidentally referencing that first form in the first part of that line:
f = document.forms[0]
Submitted by JamieCameron on Fri, 04/25/2014 - 16:09 Comment #2
Thanks, I see the cause of this - it will be fixed in the next Virtualmin release.
Submitted by Issues on Fri, 05/09/2014 - 16:11 Comment #3
Automatically closed -- issue fixed for 2 weeks with no activity.