search

coax / hmailserver-webadmin

hMailServer PHPWebAdmin redesign project
https://www.hmailserver.com
77 stars 19 forks source link

Scripting #51

Open RvdHout opened 5 hours ago

RvdHout commented 5 hours ago

Do the 'reload scripts' and 'check syntax' button do anything?

Sure doesn't look like it... not sure how long this issue existed pre 1.7 Uncaught ReferenceError: formCheck is not defined

RvdHout commented 4 hours ago

correction: check syntax seems to work fine, reload scripts doesn't, should we just remove onsubmit="return formCheck(this);"?

coax commented 4 hours ago

Just checked it, it doesn't output anything (all the work is done in background) so I removed the onsubmit="return formCheck(this);". Result message will be shown on page so there is some sort of response after clicking the button.

coax commented 3 hours ago

Changed in v1.8 (pre-release): https://github.com/coax/hmailserver-webadmin/commit/3c5eb2b43d6cd18e72d5d08f7164c380b5d80ff9

RvdHout commented 1 hour ago

A result message should not have been added (if you ask me) it simple has to post back, it didn't due to Uncaught ReferenceError: formCheck is not defined