Merge pull request #64 from topcoder-platform/issues-451

jmgasper · web-flow · commit 626368f001f7 · 2021-03-03T07:27:24.000+11:00
Showing orange handles for 'connect manager'
diff --git a/Topcoder/class.topcoder.plugin.php b/Topcoder/class.topcoder.plugin.php
@@ -1835,7 +1835,7 @@ private static function loadTopcoderUserDetailsByHandle($topcoderHandle) {
                 $roleNames = array_column($topcoderRoles, 'roleName');
                 $lowerRoleNames = array_map('strtolower', $roleNames);
                 $cachedUser['Roles'] = $roleNames;
-                $cachedUser['IsAdmin'] = in_array("admin", $lowerRoleNames) || in_array("administrator", $lowerRoleNames);
+                $cachedUser['IsAdmin'] = count(array_intersect($lowerRoleNames, ["connect manager", "admin", "administrator"])) > 0;
             }
 
             $topcoderRating = self::loadTopcoderRating($topcoderHandle); //loaded by handle

Original file line number	Diff line number	Diff line change
`@@ -1835,7 +1835,7 @@ private static function loadTopcoderUserDetailsByHandle($topcoderHandle) {`
`1835`	`1835`	`$roleNames = array_column($topcoderRoles, 'roleName');`
`1836`	`1836`	`$lowerRoleNames = array_map('strtolower', $roleNames);`
`1837`	`1837`	`$cachedUser['Roles'] = $roleNames;`
`1838`		`- $cachedUser['IsAdmin'] = in_array("admin", $lowerRoleNames) \|\| in_array("administrator", $lowerRoleNames);`
	`1838`	`+ $cachedUser['IsAdmin'] = count(array_intersect($lowerRoleNames, ["connect manager", "admin", "administrator"])) > 0;`
`1839`	`1839`	`}`
`1840`	`1840`
`1841`	`1841`	`$topcoderRating = self::loadTopcoderRating($topcoderHandle); //loaded by handle`