order assault units to attack point(not working plz help)

[sunshinex3]

this trigger is supposed to pick all units but it only effects 16 one group of them not all, why not?

• MM MassAttack Rally
  • Events
    • Unit - A unit Begins casting an ability
  • Conditions
    • And - All (Conditions) are true
      • Conditions
        • (Unit-type of (Casting unit)) Equal to Command Center (All Players)
        • (Ability being cast) Equal to MM Rally
  • Actions
    • Set MMRally[(Player number of (Owner of (Casting unit)))] = (Target point of ability being cast)
    • Set RallyUnitGroup[(Player number of (Owner of (Casting unit)))] = (Units owned by (Owner of (Casting unit)) matching ((((Matching unit) is A peon-type unit) Equal to False) and ((((Matching unit) is A structure) Equal to False) and ((((Matching unit) is A Hero) Equal to False) and (((Matching unit) is in ExemptUnitGroup[(Pl
    • Unit Group - Order RallyUnitGroup[(Player number of (Owner of (Casting unit)))] to Attack-Move To MMRally[(Player number of (Owner of (Casting unit)))]
    • Custom script: call DestroyGroup(udg_RallyUnitGroup[GetConvertedPlayerId(GetOwningPlayer(GetSpellAbilityUnit()))])
    • Custom script: call RemoveLocation (udg_MMRally[GetConvertedPlayerId(GetOwningPlayer(GetSpellAbilityUnit()))])

 

[sunshinex3]

well poop, i think i fixed it, their was a little note at the bottom of that unit group action, saying it only effects 12 units

so i made this trigger hope it works....

• MM MassAttack Rally
  • Events
    • Unit - A unit Begins casting an ability
  • Conditions
    • And - All (Conditions) are true
      • Conditions
        • (Unit-type of (Casting unit)) Equal to Command Center (All Players)
        • (Ability being cast) Equal to MM Rally
  • Actions
    • Set MMRally[(Player number of (Owner of (Casting unit)))] = (Target point of ability being cast)
    • Set RallyUnitGroup[(Player number of (Owner of (Casting unit)))] = (Units owned by (Owner of (Casting unit)) matching ((((Matching unit) is A peon-type unit) Equal to False) and ((((Matching unit) is A structure) Equal to False) and ((((Matching unit) is A Hero) Equal to False) and (((Matching unit) is in ExemptUnitGroup[(Pl
    • Unit Group - Pick every unit in RallyUnitGroup[(Player number of (Owner of (Casting unit)))] and do (Actions)
      • Loop - Actions
        • Unit - Order (Picked unit) to Attack-Move To MMRally[(Player number of (Owner of (Casting unit)))]
    • Custom script: call DestroyGroup(udg_RallyUnitGroup[GetConvertedPlayerId(GetOwningPlayer(GetSpellAbilityUnit()))])
    • Custom script: call RemoveLocation (udg_MMRally[GetConvertedPlayerId(GetOwningPlayer(GetSpellAbilityUnit()))])