Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 0735173

Browse files
committedDec 14, 2012
Do not remove current user from list
As mentioned by atrol in issue #10130, removing the current user causes potential loss of functionality when using filters because [myself] (handler_id = -1) is not the same as a specific handler_id. This is a partial revert of 748c4ce.
1 parent 748c4ce commit 0735173

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed
 

‎core/print_api.php

-6
Original file line numberDiff line numberDiff line change
@@ -248,12 +248,6 @@ function print_user_option_list( $p_user_id, $p_project_id = null, $p_access = A
248248
$t_show_realname = ( ON == config_get( 'show_realname' ) );
249249
$t_sort_by_last_name = ( ON == config_get( 'sort_by_last_name' ) );
250250
foreach( $t_users as $t_key => $t_user ) {
251-
# Remove current user from the list (there is a "myself" value)
252-
if( $t_user['id'] == $t_current_user ) {
253-
unset( $t_users[$t_key] );
254-
continue;
255-
}
256-
257251
$t_user_name = string_attribute( $t_user['username'] );
258252
$t_sort_name = utf8_strtolower( $t_user_name );
259253
if( $t_show_realname && ( $t_user['realname'] <> '' ) ) {

0 commit comments

Comments
 (0)
Please sign in to comment.