yes redundancy is a good thing, try this instead
waituntil {!isnull (finddisplay 46)};
sleep 60;
if ((getPlayerUID player) in ["######","######"]) then {
player addaction [("<t color=""#0074E8"">" + ("Tools Menu") +"</t>"),"admintools\Eexcute.sqf","",5,false,true,"",""];
};