Fix use of empty which isn't supported in old php. Remove the code entirely as I think it was just cutnpasted from somewhere else. We don't care if it is a new server or not in the validate code.
Fixes #2540
This commit is contained in:
parent
077c643da0
commit
9a2dd06e1d
|
@ -1,11 +1,11 @@
|
|||
function validateForm( form, newServer ) {
|
||||
function validateForm(form) {
|
||||
var errors = new Array();
|
||||
if ( !form.elements['newServer[Name]'].value ) {
|
||||
errors[errors.length] = "You must supply a name";
|
||||
errors[errors.length] = 'You must supply a name';
|
||||
}
|
||||
if ( errors.length ) {
|
||||
alert( errors.join( "\n" ) );
|
||||
return( false );
|
||||
alert(errors.join("\n"));
|
||||
return false;
|
||||
}
|
||||
return( true );
|
||||
return true;
|
||||
}
|
||||
|
|
|
@ -39,7 +39,7 @@ xhtmlHeaders(__FILE__, translate('Server').' - '.$Server->Name());
|
|||
<h2><?php echo translate('Server').' - '.$Server->Name() ?></h2>
|
||||
</div>
|
||||
<div id="content">
|
||||
<form name="contentForm" method="post" action="<?php echo $_SERVER['PHP_SELF'] ?>" onsubmit="return validateForm(this, <?php echo empty($Server->Name())?'true':'false' ?>)">
|
||||
<form name="contentForm" method="post" action="?" onsubmit="return validateForm(this)">
|
||||
<input type="hidden" name="view" value="<?php echo $view ?>"/>
|
||||
<input type="hidden" name="object" value="server"/>
|
||||
<input type="hidden" name="id" value="<?php echo validHtmlStr($_REQUEST['id']) ?>"/>
|
||||
|
|
Loading…
Reference in New Issue