Mana Burst v1.0

[Moderator]

12th Dec 2015
IcemanBo: For long time as NeedsFix. Rejected.

04:23, 24th Jan 2015
IcemanBo: https://www.hiveworkshop.com/threads/mana-burst-v1-0-a.260220/post-2644015

 

[iamdl]

HI!

 

[GywGod133]

good...

but you forgot to save the picked unit in Unit Group
if you done, just null it

• Unit - Create 1 MB_DummyType for (Owner of MB_Caster[MB_Index]) at MB_CasterPoint[MB_Index] facing (Facing of MB_Caster[MB_Index]) degrees

Instead using Owner of Caster, should be use Triggering Player ^^

remove this...

• Set MB_Caster[MB_Integer] = No unit

• Set MB_Dummy[MB_Integer] = No unit

dont null it

 

[iamdl]

good...

but you forgot to save the picked unit in Unit Group
if you done, just null it

• Unit - Create 1 MB_DummyType for (Owner of MB_Caster[MB_Index]) at MB_CasterPoint[MB_Index] facing (Facing of MB_Caster[MB_Index]) degrees

Instead using Owner of Caster, should be use Triggering Player ^^

remove this...

• Set MB_Caster[MB_Integer] = No unit

• Set MB_Dummy[MB_Integer] = No unit

dont null it

Thx ill update ASAP

Edit: Updated!

 

[GywGod133]

long

• Set MB_TargetGroups[MB_Integer] = (Units within MB_AoE of MB_TargetPoint[MB_Integer] matching ((((Matching unit) is A structure) Equal to MB_Structures) and ((((Matching unit) is alive) Equal to MB_Alive) and (((Matching unit) belongs to an enemy of (Owner of MB_Caster[MB_Integer])) Equal to

9.9 so long..... put it in ITE those conditions...

• Set MB_PickedUnits[MB_Integer] = No unit

->

• Unit Group - Pick every unit in MB_TargetGroups[MB_Integer] and do (Actions)
  • Loop - Actions
    • Set MB_PickedUnits[MB_Integer] = (Picked unit)
      • Unit - Cause MB_Caster[MB_Integer] to damage MB_PickedUnits[MB_Integer], dealing MB_FinalDamage[MB_Integer] damage of attack type Spells and damage type Normal
      • Special Effect - Create a special effect attached to the MB_SpecialEffectsAttachment of (Picked unit) using MB_SpecialEffects
      • Special Effect - Destroy (Last created special effect)
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • MB_FloatingText Equal to True
      • Then - Actions
        • Floating Text - Create floating text that reads ((String(MB_FinalDamage[MB_Integer])) + Damage!) above (Picked unit) with Z offset 0.00, using font size 15.00, color (0.00%, 0.00%, 100.00%), and 0.00% transparency
        • Floating Text - Set the velocity of (Last created floating text) to 64.00 towards 90.00 degrees
        • Floating Text - Change (Last created floating text): Disable permanence
        • Floating Text - Change the lifespan of (Last created floating text) to 5.00 seconds
        • Floating Text - Change the fading age of (Last created floating text) to 4.00 seconds
      • Set MB_PickedUnits[MB_Integer] = No unit

and last...
the TriggeringPlayer must be put it in a variable ^^ sorry i forgot...

 

[iamdl]

• Set MB_TargetGroups[MB_Integer] = (Units within MB_AoE of MB_TargetPoint[MB_Integer] matching ((((Matching unit) is A structure) Equal to MB_Structures) and ((((Matching unit) is alive) Equal to MB_Alive) and (((Matching unit) belongs to an enemy of (Owner of MB_Caster[MB_Integer])) Equal to

9.9 so long..... put it in ITE those conditions...

• Set MB_PickedUnits[MB_Integer] = No unit

->

• Unit Group - Pick every unit in MB_TargetGroups[MB_Integer] and do (Actions)
  • Loop - Actions
    • Set MB_PickedUnits[MB_Integer] = (Picked unit)
      • Unit - Cause MB_Caster[MB_Integer] to damage MB_PickedUnits[MB_Integer], dealing MB_FinalDamage[MB_Integer] damage of attack type Spells and damage type Normal
      • Special Effect - Create a special effect attached to the MB_SpecialEffectsAttachment of (Picked unit) using MB_SpecialEffects
      • Special Effect - Destroy (Last created special effect)
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • MB_FloatingText Equal to True
      • Then - Actions
        • Floating Text - Create floating text that reads ((String(MB_FinalDamage[MB_Integer])) + Damage!) above (Picked unit) with Z offset 0.00, using font size 15.00, color (0.00%, 0.00%, 100.00%), and 0.00% transparency
        • Floating Text - Set the velocity of (Last created floating text) to 64.00 towards 90.00 degrees
        • Floating Text - Change (Last created floating text): Disable permanence
        • Floating Text - Change the lifespan of (Last created floating text) to 5.00 seconds
        • Floating Text - Change the fading age of (Last created floating text) to 4.00 seconds
      • Set MB_PickedUnits[MB_Integer] = No unit

and last...
the TriggeringPlayer must be put it in a variable ^^ sorry i forgot...

Updated!

 

[GywGod133]

you miss this... those Picked unit

• Special Effect - Create a special effect attached to the MB_SpecialEffectsAttachment of (Picked unit) using MB_SpecialEffects

• Floating Text - Create floating text that reads ((String(MB_FinalDamage[MB_Integer])) + Damage!) above (Picked unit) with Z offset 0.00, using font size 15.00, color (0.00%, 0.00%, 100.00%), and 0.00% transparency

 

[iamdl]

you miss this... those Picked unit

• Special Effect - Create a special effect attached to the MB_SpecialEffectsAttachment of (Picked unit) using MB_SpecialEffects

• Floating Text - Create floating text that reads ((String(MB_FinalDamage[MB_Integer])) + Damage!) above (Picked unit) with Z offset 0.00, using font size 15.00, color (0.00%, 0.00%, 100.00%), and 0.00% transparency

Updated again -.-" hope there's nothing wrong with that

 

[Vinz]

A suggestion, how about current mana instead?

Anyways, on to the code; you could choose to not have PickedUnits as array. Also, no need to set it to no unit since it's going to be used over and over again.

 

[BPower]

The locations can be non array variables. You also have two location leaks (TargetPoint and DummyPoint).

Caster2 ---> Owner

 

[ukn0wnD3str0y3r]

A suggestion, how about current mana instead? .

Yeah, i agree. If max mana the damage will be very high ._.
If the unit have 2000 mana, and 125 + 40% is 925 damage
And if you uses as 10/20/30/40% of current mana will be lesser damage. In the same time, you can also set the percentage higher
Don't need to care me, im just continue the suggestions of Mythic :/

 

[Vinz]

Not really about the damage though, it kinda makes sense if you consume mana to deal mana and if you're tired you deal less. :3

 

[ukn0wnD3str0y3r]

I care those damage much ._. xD

 

[IcemanBo]

Please add import instructions.

All your point variables do not need to be array.

(Distance between MB_DummyPoint[MB_Integer] and MB_TargetPoint[MB_Integer]) < 75.00

->

(Distance between MB_DummyPoint[MB_Integer] and MB_TargetPoint[MB_Integer]) < MB_Speed

Set MB_Caster2[]

^That's an inproper name for a player variable.

All points don't need to be indexed, nor deindex.

"PickedUnits" is just a temp unit variable. No array needed. No deindexing needed.

No need to index the temp unit group.

Only use variables with array if really needed!

The final damage only needs to be calculated once, not each iteration. It can be directly be done onCast, as it would make most sense I think.

-------- Set true if the target is alive -------- Set MB_Alive = True

The target is always alive onCast. It more means, that the spell will cancel if target dies.