amnykon

Update PR to address Sprunk's comments.

updated to address GoogleFrog's comments, and removed the bool option since guardRankoverrideOption set at 3 is the same as having it off.

Updated default value to 3. As I just returned my new laptop due to too many crashes and boot issues, I am not able to currently test this change; however,...

Updated to remove the extra local var, and fixed a camel case. like I said, I returned my laptop so please test this before merging it. As the funds for...

I got a 1/2 working path using gadget:AllowCommand. This working path is similar to LuaRules/Gadgets/cmd_block_ally_attack.lua and conditionally ignores attack commands. Looks like widget:CommandNotify could also be used instead. Is this...

Agreed with exception of the unit_thower gadget. The disable attack functionality is going to be removed from it, as it will be in a widget.

Requires https://github.com/ZeroK-RTS/Zero-K/pull/4660, (preferably with unwritten part 2) I still need to convert cmd_manual to cmd_attack calls.

> [GoogleFrog](http://zero-k.info/Forum/Thread/34918?page=1&Search=&User=&grorder=&grdesc=False&grpage=1 ): As far as I know, widget:CommandNotify() is sufficient. CommandNotify is not sufficient, ether widgets handle the command returns true, preventing all other widgets from acting, or they...

Currently CMD_DISABLE_ATTACK is Inserted in the Unit Cmd Desc by LuaRules/Gadgets/unit_thrower.lua CMD_DISABLE_ATTACK is entangled in unit_thrower.lua, where it really has nothing to do with that gadget. I am moving it...

Updated to work with changes in PR 4660. That PR is required for this to work.