LastOrder

[Av3n]

So you reckon you are going ahead with the Table version after much discussion?

-Av3n

[Rising_Dusk]

Updated with a no storage version and the AbortSpell supplementary library.

[wraithseeker]

Can you make it so that it doesn't requires your Unit Indexing Utilties as I uses PUI and do not need that stuff or just make it a standalone system, even better.

[Fledermaus]

You mean like the "No Storage" version in the first post?

[Rising_Dusk]

I still can't believe that people use PUI. It frustrates the heck out of me.

[emjlr3]

personal preference or some other sinister reason?

[Rising_Dusk]

Because UserData exists to be used for indexing. That some people blindly think "Oh noez, UserData, bad!" is a ridiculous notion.

[Blubb-Tec]

Originally Posted by Rising_Dusk

Because UserData exists to be used for indexing. That some people blindly think "Oh noez, UserData, bad!" is a ridiculous notion.

meh, there still exist idiot like me who use UserData directly in their systems, and thus would create a conflict with attach systems that use it too....

funnily, i wanted to request just the same thing: making a private GetUnitId() function will allow users who already use other indexing systems to use your system(which is really useful, btw ~) to implement this.

[Rising_Dusk]

If you have another indexing system, I see no reason why you would want to use this.

[Tyrande_ma3x]

Because it would save a lot of time recreating this thing on your own. It is far more easier to just replace the requirement of this library and edit the functions that return the unit's ID... I agree on the part that this needs some wrappers for different unit indexing systems.

[Rising_Dusk]

Originally Posted by Tyrande_ma3x

Because it would save a lot of time recreating this thing on your own.

You wouldn't have to if you already use another indexing system. UIU is useless if you're using something else already that does the same thing. If you still have a script in your map that uses UserData, I honestly recommend rewriting it to not use it, otherwise just wait for the 1.23b patch and start using handle vars again.

Furthermore, AutoIndex uses the exact same function names as UIU, so replacing it is trivial. We are trying to standardize the syntax of GetUnitId(MyUnit), so this will work with whatever indexing system you happen to use. Just change the requirement at the top of the LastOrder library.

[Tyrande_ma3x]

I was using PUI, that's why I mentioned that.

[Rising_Dusk]

If PUI were updated to use GetUnitId(), then it'd also be interchangeable with UIU. Heck, doing the update yourself is pretty easy.

[Rising_Dusk]

Updated to work with patch 1.24.

[Archmage Owenalacaster]

While this isn't exactly a bug, I have found a strange compatibility issue; AbortSpell doesn't like AutoIndex. When AbortSpell is called, it causes the targeting sequence to be canceled and reordered ad infinitum.