From 4b2d7735b30270ed933ed8ba3d6519183ec87dd6 Mon Sep 17 00:00:00 2001 From: Oliver Schramm Date: Sun, 16 Mar 2014 00:11:12 +0100 Subject: [ticket/7707] Add get_username_string() where possible PHPBB3-7707 --- phpBB/includes/acp/acp_main.php | 6 +----- phpBB/includes/acp/acp_prune.php | 2 +- phpBB/includes/acp/auth.php | 4 ++-- 3 files changed, 4 insertions(+), 8 deletions(-) (limited to 'phpBB/includes/acp') diff --git a/phpBB/includes/acp/acp_main.php b/phpBB/includes/acp/acp_main.php index fd45027b49..4512905539 100644 --- a/phpBB/includes/acp/acp_main.php +++ b/phpBB/includes/acp/acp_main.php @@ -40,11 +40,7 @@ class acp_main $user_row = $db->sql_fetchrow($result); $db->sql_freeresult($result); - $perm_from = '' : '>'); - $perm_from .= ($user_row['user_id'] != ANONYMOUS) ? '' : ''; - $perm_from .= $user_row['username']; - $perm_from .= ($user_row['user_id'] != ANONYMOUS) ? '' : ''; - $perm_from .= ''; + $perm_from = get_username_string('full', $user_row['user_id'], $user_row['username'], $user_row['user_colour']); $template->assign_vars(array( 'S_RESTORE_PERMISSIONS' => true, diff --git a/phpBB/includes/acp/acp_prune.php b/phpBB/includes/acp/acp_prune.php index 0f9ca9bab3..d0e5dfb5f4 100644 --- a/phpBB/includes/acp/acp_prune.php +++ b/phpBB/includes/acp/acp_prune.php @@ -294,7 +294,7 @@ class acp_prune $template->assign_block_vars('users', array( 'USERNAME' => $usernames[$user_id], 'USER_ID' => $user_id, - 'U_PROFILE' => append_sid($phpbb_root_path . 'memberlist.' . $phpEx, 'mode=viewprofile&u=' . $user_id), + 'U_PROFILE' => get_username_string('profile', $user_id, $usernames[$user_id]), 'U_USER_ADMIN' => ($auth->acl_get('a_user')) ? append_sid("{$phpbb_admin_path}index.$phpEx", 'i=users&mode=overview&u=' . $user_id, true, $user->session_id) : '', )); } diff --git a/phpBB/includes/acp/auth.php b/phpBB/includes/acp/auth.php index a023bced0a..06f83b2775 100644 --- a/phpBB/includes/acp/auth.php +++ b/phpBB/includes/acp/auth.php @@ -649,8 +649,8 @@ class auth_admin extends \phpbb\auth\auth { $template->assign_block_vars('role_mask.users', array( 'USER_ID' => $row['user_id'], - 'USERNAME' => $row['username'], - 'U_PROFILE' => append_sid("{$phpbb_root_path}memberlist.$phpEx", "mode=viewprofile&u={$row['user_id']}")) + 'USERNAME' => get_username_string('username', $row['user_id'], $row['username']), + 'U_PROFILE' => get_username_string('profile', $row['user_id'], $row['username'])) ); } $db->sql_freeresult($result); -- cgit v1.2.1 From db6e5a19f2f812012fa15047aa97e74608bff34c Mon Sep 17 00:00:00 2001 From: Oliver Schramm Date: Wed, 26 Mar 2014 17:13:46 +0100 Subject: [ticket/7707] Remove unnecessary variables PHPBB3-7707 --- phpBB/includes/acp/acp_main.php | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'phpBB/includes/acp') diff --git a/phpBB/includes/acp/acp_main.php b/phpBB/includes/acp/acp_main.php index 4512905539..542a8c549b 100644 --- a/phpBB/includes/acp/acp_main.php +++ b/phpBB/includes/acp/acp_main.php @@ -40,12 +40,10 @@ class acp_main $user_row = $db->sql_fetchrow($result); $db->sql_freeresult($result); - $perm_from = get_username_string('full', $user_row['user_id'], $user_row['username'], $user_row['user_colour']); - $template->assign_vars(array( 'S_RESTORE_PERMISSIONS' => true, 'U_RESTORE_PERMISSIONS' => append_sid("{$phpbb_root_path}ucp.$phpEx", 'mode=restore_perm'), - 'PERM_FROM' => $perm_from, + 'PERM_FROM' => get_username_string('full', $user_row['user_id'], $user_row['username'], $user_row['user_colour']), 'L_PERMISSIONS_TRANSFERRED_EXPLAIN' => sprintf($user->lang['PERMISSIONS_TRANSFERRED_EXPLAIN'], $perm_from, append_sid("{$phpbb_root_path}ucp.$phpEx", 'mode=restore_perm')), )); -- cgit v1.2.1 From 908057301c023a7d5efcd4aed0e8b8d4c793f15b Mon Sep 17 00:00:00 2001 From: Oliver Schramm Date: Tue, 1 Apr 2014 22:03:59 +0200 Subject: [ticket/7707] Add comma to last entry of arrays PHPBB3-7707 --- phpBB/includes/acp/auth.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'phpBB/includes/acp') diff --git a/phpBB/includes/acp/auth.php b/phpBB/includes/acp/auth.php index 06f83b2775..5404efded3 100644 --- a/phpBB/includes/acp/auth.php +++ b/phpBB/includes/acp/auth.php @@ -650,8 +650,8 @@ class auth_admin extends \phpbb\auth\auth $template->assign_block_vars('role_mask.users', array( 'USER_ID' => $row['user_id'], 'USERNAME' => get_username_string('username', $row['user_id'], $row['username']), - 'U_PROFILE' => get_username_string('profile', $row['user_id'], $row['username'])) - ); + 'U_PROFILE' => get_username_string('profile', $row['user_id'], $row['username']), + )); } $db->sql_freeresult($result); } -- cgit v1.2.1 From 8f525e1ef5777c1dc77d7eb4d43039c5dd059f88 Mon Sep 17 00:00:00 2001 From: Oliver Schramm Date: Fri, 4 Apr 2014 23:48:14 +0200 Subject: [ticket/7707] Fix undefined variable: perm_from PHPBB3-7707 --- phpBB/includes/acp/acp_main.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'phpBB/includes/acp') diff --git a/phpBB/includes/acp/acp_main.php b/phpBB/includes/acp/acp_main.php index 542a8c549b..4512905539 100644 --- a/phpBB/includes/acp/acp_main.php +++ b/phpBB/includes/acp/acp_main.php @@ -40,10 +40,12 @@ class acp_main $user_row = $db->sql_fetchrow($result); $db->sql_freeresult($result); + $perm_from = get_username_string('full', $user_row['user_id'], $user_row['username'], $user_row['user_colour']); + $template->assign_vars(array( 'S_RESTORE_PERMISSIONS' => true, 'U_RESTORE_PERMISSIONS' => append_sid("{$phpbb_root_path}ucp.$phpEx", 'mode=restore_perm'), - 'PERM_FROM' => get_username_string('full', $user_row['user_id'], $user_row['username'], $user_row['user_colour']), + 'PERM_FROM' => $perm_from, 'L_PERMISSIONS_TRANSFERRED_EXPLAIN' => sprintf($user->lang['PERMISSIONS_TRANSFERRED_EXPLAIN'], $perm_from, append_sid("{$phpbb_root_path}ucp.$phpEx", 'mode=restore_perm')), )); -- cgit v1.2.1