IF your going for preventing uninteneded side effects like I assume you were by using %rob.AWAY.$nick Then you should not have gone and changed the code so much.
Side effects I saw...
(1) A likely uninteneded logic hole has been patched (this in fact my be good)
.. Say that if (%rob [ $+ [ $nick ] ] != on) && ($2 ison #) && (%account [ $+ [ $2 ] ] == on) { Failed but global var %Attempt was 1 at the time, then if (%attempt == 1) { should succeed, it no longer does.
(2) Game logic has been altered, originally If you attempt a !rob and you have not robbed in 30 mins, regardless of what happens you can not attempt a !rob again for 30 mins (changed to 5 mins now)
(3) Global var %Attempt is no longer set, this may or may not be used else where to report the success or failure of the last !rob attempt
Personally, If it had been me , I would have just taken a punt that It would have been ok to do and use /set %account$nick away
