Problems with reboot

[Francesco]

I have a few problems with %reboot%, for some weird reasons

"shutdown.exe -r -f -t" is invoked without the time (0)

I saw this in the core.js file that probably is the cause

	rs = rs.replace(/%reboot%/gi, 'shutdown.exe -r -f -t ');					// force a reboot after X seconds
	rs = rs.replace(/%reboot%/gi, 'shutdown.exe -r -f -t 0');					// force a reboot now

there are those 2 replaces one after another but the reboot time isn't specified for the 1° one causing the reboot to fail (the time parameter is mandatory).

Edited August 24, 2007 by Francesco

[sp00f]

And i thought it was just me....

[Francesco]

I still have the same problem with the latest WPI version 6.3. Any idea?

[Kelsenellenelvian]

Let me try some stuff.

[Francesco]

Let me try some stuff.

If you remove the first of those 2 lines the reboot command ("%reboot%" with no other parameters) works fine. I think all you have to do is add an IF block that checks if there's a number after the %reboot% command and then use the first or second replace accordingly.

Edited January 7, 2008 by Francesco

[Francesco]

Any news about a fix for this bug? I still see it on the latest WPI 6.4

[mritter]

The two checks are very similar, but the frirst one looks for a space after %reboot% while the second does not. So, if you do

%reboot% 5

It will wait 5 seconds before restarting. If you do not specify a number it assumes 0 and shuts down immediately.

[Francesco]

The two checks are very similar, but the frirst one looks for a space after %reboot% while the second does not. So, if you do

%reboot% 5

It will wait 5 seconds before restarting. If you do not specify a number it assumes 0 and shuts down immediately.

I use only %reboot% and it doesn't reboot unless I remove the first of those 2 lines from the core.js or I specify the seconds.

The two lines in the core.js are one immediately next the other so if you don't specify a reboot time the reboot command call will be "shutdown.exe -r -f -t " that won't never work, because -t has no time value. No one of those 2 replaces is looking for a space, they both look for the same identical string.

[mritter]

Weird. Somehow, sometime, the space was removed. There should be a space in the first check:

	rs = rs.replace(/%reboot% /gi, 'shutdown.exe -r -f -t ');					// force a reboot after X seconds

Put a space after the %reboot%.

See the second line where the replace puts a 0 after the -t? In the first one it leaves your specified time there.

Kel: Put this in the latest version.

[Kelsenellenelvian]

KK got ya

Goanna put a refresh version up soon in the next coupla days anyways!