Порой при написании модулей под Друпал нам кажется, что вот эта функция ну уж точно будет исполняться лишь для текущего пользователя. И потому не нужно аргументом передавать $uid — обойдемся глобальным объектом $user.
На мой взгляд, в большинстве случаев рано или поздно все равно возникнет нужда вызвать функцию для другого — не текущего — пользователя. И придется либо рефакторить, либо изгаляться с подменой и возвращением global $user.
