is_admin 함수가 로그인한 자신만 검색하나보네요
인자에 id를 받길래 어딘가 현재 로그인한 아이디가 아닌 다른 아이디도 조회할 일이 있어서 저렇게 했나 해서 게시물 처럼 처리했습니다

음 제말을 이해 못하신것같은데요...

그거에 대해서 잘 모르겠다가 아니라요
말씀대로  global $member; 를 추가해서 하는건 함수 설계 의도에 어긋나 보여서 "안한" 겁니다.

만약  global $member;  를 이용해서 함수를 짯다면 애초에 $mb_id 라는 인자가 필요 없지요.

인자를 받을 필요 없이  global $member;  선언후 $member 변수를 이용하면 되지 왜 $mb_id라는 인자(파라미터)를 선언해서 썻을까요

그 때문에 DB에서 조회해서 쓰는 루틴을 추가로 한겁니다.

혹시 제말에서 이해가 안가시는 부분이 있으신지요?

자꾸 핀트가 어긋나서 말씀드리는건데요
그걸 이해 못한다는 이야기 아니구요
제 말은 함수 설계의도에 맞춰서 디비 조회를 했다 라는겁니다.

예를들어서 특정 회원의 관리자 권한이 최고관리자인지 그룹관리자인지 확인해야하는 경우 is_admin 함수를 이용해야하는데 위의 $member 변수를 이용하여 처리하게된다면 해당 회원의 관리자 권한을 확인하는 것이 아닌 현재 로그인한 사용자의 관리자 권한을 확인하는 것이 되겠지요.

제가 드리는 말씀은 해당 함수가 만약 현재 로그인한 정보에 대해서만 조회를 하는 목적으로 만들어졌다면

function is_admin($mb_id) 으로 선언되는 것이 아니라 function is_admin() 으로 선언되어 사용되는 것이 옳지 않았나 하는 겁니다.

$mb_id 라는 인자를 받았기때문에 그에 대한 조회를 한것이구요.

혹시나 관리자님께서 개발할 당시 함수에 대해 최적화나, 메모리 부분에 대해 신경을 써서 $mb_id 라는 인자를 쓰신거면 헐랭이님 말씀대로 $member 전역 변수를 이용해도 아무런 문제가 없겠지요.

하지만 만약에 현재 로그인한 유저가아닌 다른 유저의 권한을 확인하고자 한다면 위의 소스는 문제가 생길수 밖에 없습니다.

저는 $member 전역 변수를 쓰는것이 틀렸다 이해 못한다 라는 소리가 아니라 정말 단순히 그누보드에서 is_admin 함수를 사용하는것이 로그인한 유저의 정보를 확인할때만 쓰는것이 아니라 타 유저의 정보를 확인할 때도 사용이 된다라고 생각해서 남긴 글입니다.

하아... 당췌 제 말을 이해 못하시는군요...

$member 변수를 사용하는 것을 이해 못한다. 그러니 내가 쓴게 맞다 라고 주장하고 있는게 아닙니다.
바로 윗 댓글에 적었다시피 함수 설계 의도가 그렇게 보여서 디비 조회를 했다 라는 겁니다.

타인의 의견을 받아들이시질 못하시는군요...