It works in multiplayer.
but only for one hero.
but only for one hero.
and isnt multi-unitWhat do you mean ?
UnitAddAbilityBJ could be easily be replaced with UnitAddAbility , only swap the arguments, always use common.j functions instead of blizzard.j functions that just call them.
if not(boolean1) then
return false
endif
if not(boolean2) then
return false
endif
return true
use:
return boolean1 and boolean2
Easier to understand , takes 1 line of text and is much more efficient.Multiboad are easy to navigate when you have a lot of item
This inventory works in multiplayerno it doesn't.
Why 2 versions of each item?
Why if you were using gamecache you used a big array and a loop for the declared items
or was it to make it to remove the whole purpose of gamecache totally destroying the multi instability of the system?
Could you please replace the dummy bj functions with their native equivalents and
Why you have horrible GUI like estructures like this: ?
if ( not ( GetItemTypeId(GetManipulatedItem()) != 'I00M' ) ) then
return false
endif
if ( not ( GetItemTypeId(UnitItemInSlotBJ(GetOrderedUnit(), GetIssuedOrderId()-852001)) == 'I00O' ) ) then
return false
endif
return true