Page 1 of 1
2x Factory Reset button to web interface ?
Posted: 03 Mar 2018, 06:40
by harm501
Hello,
Question,
Why two times "Factory Reset" button to web interface ?
I noticed this is in GIT version: mega-20180302 and mega-20180303
Is there a reason for, or what is the difference between the two ?
Harald
Re: 2x Factory Reset button to web interface ?
Posted: 03 Mar 2018, 11:41
by vader
It's a bug in the source code. See here....
TXBuffer += F("<TR><TD HEIGHT=\"30\">");
addButton(TXBuffer.buf, F("/?cmd=reset"), F("Factory Reset"));
TXBuffer += F("<TD>");
TXBuffer += F("Erase all settings files");
TXBuffer += F("<TR><TD HEIGHT=\"30\">");
addButton(TXBuffer.buf, F("/?cmd=reset"), F("Factory Reset"));
TXBuffer += F("<TD>");
TXBuffer += F("Erase all settings files");
Re: 2x Factory Reset button to web interface ?
Posted: 03 Mar 2018, 12:47
by harm501
OK, I see the same code twice, simple bug fix
Thank you "vader"
Re: 2x Factory Reset button to web interface ?
Posted: 04 Mar 2018, 07:06
by harm501
In GIT version: mega-20180304 same bug !!
Re: 2x Factory Reset button to web interface ?
Posted: 04 Mar 2018, 16:53
by TD-er
And already fixed + merged. In new builds it will be gone.
Re: 2x Factory Reset button to web interface ?
Posted: 04 Mar 2018, 17:00
by vader
I'm compiling my own ESPeasy version and so didn't had that problem....
