forked from I2P_Developers/i2p.i2p
Console: Prevent editing a client while it's starting
This commit is contained in:
@ -140,13 +140,16 @@ public class ConfigClientsHelper extends HelperBase {
|
|||||||
boolean isConsole = ca.className.equals("net.i2p.router.web.RouterConsoleRunner");
|
boolean isConsole = ca.className.equals("net.i2p.router.web.RouterConsoleRunner");
|
||||||
boolean showStart;
|
boolean showStart;
|
||||||
boolean showStop;
|
boolean showStop;
|
||||||
|
boolean showEdit;
|
||||||
if (isConsole) {
|
if (isConsole) {
|
||||||
showStart = false;
|
showStart = false;
|
||||||
showStop = false;
|
showStop = false;
|
||||||
|
showEdit = true;
|
||||||
} else {
|
} else {
|
||||||
ClientApp clientApp = _context.routerAppManager().getClientApp(ca.className, LoadClientAppsJob.parseArgs(ca.args));
|
ClientApp clientApp = _context.routerAppManager().getClientApp(ca.className, LoadClientAppsJob.parseArgs(ca.args));
|
||||||
showStart = clientApp == null;
|
showStart = clientApp == null;
|
||||||
showStop = clientApp != null && clientApp.getState() == ClientAppState.RUNNING;
|
showStop = clientApp != null && clientApp.getState() == ClientAppState.RUNNING;
|
||||||
|
showEdit = !showStop && (clientApp == null || clientApp.getState() != ClientAppState.STARTING);
|
||||||
}
|
}
|
||||||
String scur = Integer.toString(cur);
|
String scur = Integer.toString(cur);
|
||||||
renderForm(buf, scur, ca.clientName,
|
renderForm(buf, scur, ca.clientName,
|
||||||
@ -161,8 +164,9 @@ public class ConfigClientsHelper extends HelperBase {
|
|||||||
// edit
|
// edit
|
||||||
allowEdit && scur.equals(_edit),
|
allowEdit && scur.equals(_edit),
|
||||||
// show edit button, show update button
|
// show edit button, show update button
|
||||||
// Don't allow edit if it's running, or else we would lose the "handle" to the ClientApp to stop it.
|
// Don't allow edit if it's running or starting, or else we would lose the "handle" to the ClientApp to stop it.
|
||||||
allowEdit && !showStop, false,
|
allowEdit && showEdit,
|
||||||
|
false,
|
||||||
// show stop button
|
// show stop button
|
||||||
showStop,
|
showStop,
|
||||||
// show delete button, show start button
|
// show delete button, show start button
|
||||||
|
Reference in New Issue
Block a user