Fix issues from PR feedback

mantisbt · Dec 30, 2017 · ffcf9a9 · ffcf9a9
1 parent d955ef6
commit ffcf9a9
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 7 deletions.
diff --git a/bug_monitor_add.php b/bug_monitor_add.php
@@ -47,7 +47,7 @@
 $t_data = array( 'issue_id' => $f_bug_id );
 
 if( is_blank( $f_usernames ) ) {
-	$t_data['users'] = array( array( 'id' => $t_logged_in_user_id ) );
+	$t_data['users'] = array( array( 'id' => auth_get_current_user_id() ) );
 } else {
 	$t_usernames = preg_split( '/[,|]/', $f_usernames, -1, PREG_SPLIT_NO_EMPTY );
 	$t_users = array();

diff --git a/core/commands/MonitorAddCommand.php b/core/commands/MonitorAddCommand.php
@@ -31,11 +31,6 @@ class MonitorAddCommand extends Command {
 	 */
 	private $projectId;
 
-	/**
-	 * @var integer
-	 */
-	private $loggedInUserId;
-
 	/**
 	 * @var array array of user ids to add to monitor list of the issue
 	 */

diff --git a/core/constant_inc.php b/core/constant_inc.php
@@ -314,7 +314,7 @@
 define( 'ERROR_USER_CURRENT_PASSWORD_MISMATCH', 812 );
 define( 'ERROR_USER_EMAIL_NOT_UNIQUE', 813 );
 define( 'ERROR_USER_BY_EMAIL_NOT_FOUND', 814 );
-define( 'ERROR_USER_BY_REALNAME_NOT_FOUND', 814 );
+define( 'ERROR_USER_BY_REALNAME_NOT_FOUND', 815 );
 
 # ERROR_AUTH_*
 define( 'ERROR_AUTH_INVALID_COOKIE', 900 );