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.... :mrgreen: