fix problem editing users, a semi-colon screwed up the if statement.
Also change behaviour so that if a user is specified, and doesn't load, generate an error. This also removes a search for empty user id in the dbpull/485/head
parent
70471936a7
commit
2be38d7f32
|
@ -25,11 +25,17 @@ if ( !canEdit( 'System' ) && !$selfEdit )
|
|||
$view = "error";
|
||||
return;
|
||||
}
|
||||
if ( !($newUser = dbFetchOne( 'SELECT * FROM Users WHERE Id = ?', NULL, ARRAY($_REQUEST['uid'])) ) ); {
|
||||
$newUser = array();
|
||||
$newUser['Username'] = $SLANG['NewUser'];
|
||||
$newUser['Enabled'] = 1;
|
||||
$newUser['MonitorIds'] = '';
|
||||
|
||||
if ( $_REQUEST['uid'] ) {
|
||||
if ( !($newUser = dbFetchOne( 'SELECT * FROM Users WHERE Id = ?', NULL, ARRAY($_REQUEST['uid'])) ) ) {
|
||||
$view = "error";
|
||||
return;
|
||||
}
|
||||
} else {
|
||||
$newUser = array();
|
||||
$newUser['Username'] = $SLANG['NewUser'];
|
||||
$newUser['Enabled'] = 1;
|
||||
$newUser['MonitorIds'] = '';
|
||||
}
|
||||
|
||||
$monitorIds = array_flip(explode( ',', $newUser['MonitorIds'] ));
|
||||
|
|
|
@ -25,11 +25,17 @@ if ( !canEdit( 'System' ) && !$selfEdit )
|
|||
$view = "error";
|
||||
return;
|
||||
}
|
||||
if ( !($newUser = dbFetchOne( 'SELECT * FROM Users WHERE Id = ?', NULL, ARRAY($_REQUEST['uid'])) ) ); {
|
||||
$newUser = array();
|
||||
$newUser['Username'] = $SLANG['NewUser'];
|
||||
$newUser['Enabled'] = 1;
|
||||
$newUser['MonitorIds'] = '';
|
||||
|
||||
if ( $_REQUEST['uid'] ) {
|
||||
if ( !($newUser = dbFetchOne( 'SELECT * FROM Users WHERE Id = ?', NULL, ARRAY($_REQUEST['uid'])) ) ) {
|
||||
$view = "error";
|
||||
return;
|
||||
}
|
||||
} else {
|
||||
$newUser = array();
|
||||
$newUser['Username'] = $SLANG['NewUser'];
|
||||
$newUser['Enabled'] = 1;
|
||||
$newUser['MonitorIds'] = '';
|
||||
}
|
||||
|
||||
$monitorIds = array_flip(explode( ',', $newUser['MonitorIds'] ));
|
||||
|
|
Loading…
Reference in New Issue