The Phoenix Strike

[Moderator]

12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.

Required changes

• Add importing instructions into the map file

Suggested changes

• Set dummies to can't raise, does not decay
• You only need two dummy unit types, get rid of Dummy (PS, 2)
• Store the ability level at the time of casting, and set the ability level based on that
  The caster could level up the ability while the Phoenix is flying
• Only use english comments in the triggers
• Improve grammar/spelling in your tooltip
  List the level and hotkey like BLizzard's abilities do
  Also check how level related values should be displayed
• You could use an initialization trigger where you set ability, unit type, effects...

 

[zv27]

post trigger...

 

[Jay the Editor]

Here's an efficient way to destroy trees: [General] Destroy only trees

 

[Losam]

All I ask to apologize that I did not give credit in the next version (1.01) I I will.

 

[Losam]

Hey Moderator, If I use the Custom Script:

• Custom script: set bj_wantDestroyGroup = true

I Need Destroy Group Unit?

 

[Jay the Editor]

You don't need to destroy the unit group again if you use that. Just simply pick every units... and so on with the custom scripts above the picking.
By the way, don't make double post. The 'EDIT' button at the bottom right is there for reason.

 

[Losam]

Ok!,Tanks for Help.

Are You A Moderator?

My Spell is MUI?

 

[Jay the Editor]

I'm not a moderator. Sorry.
Your spell should be MUI. I didn't check it thoroughly. My eyes are too tired.
Some parts you should change:

[1] The way you destroy trees is not efficient. Try this: [General] Destroy only trees (It's on my 2nd comment)

[2] You don't need this script

• Skip remaining actions

[3] Some variables are constant and don't need to be indexed. For example, the special effects

[4] You don't need the custom scripts of "set bj_wantDestoryGroup = true" if you're going to set it into variable and destroy it. Use either one of them.

Show

• Custom script: set bj_wantDestroyGroup=true
• Set PS_UtGp[PS_TempInt] = (Units within PS_AoE[PS_TempInt] of TempLoc2 matching (((Matching unit) belongs to an enemy of (Owner of PS_Caster[PS_TempInt])) Equal to True))
• Unit Group - Pick every unit in PS_UtGp[PS_TempInt] and do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (PS_Caster[PS_TempInt] belongs to an enemy of (Owner of (Picked unit))) Equal to True
        • ((Picked unit) is alive) Equal to True
      • Then - Actions
        • -------- --------
        • Special Effect - Create a special effect attached to the chest of (Picked unit) using PS_SE_2[PS_TempInt]
        • Special Effect - Destroy (Last created special effect)
        • -------- --------
        • Unit - Cause PS_Caster[PS_TempInt] to damage (Picked unit), dealing PS_Damage[PS_TempInt] damage of attack type PS_AkTe[PS_TempInt] and damage type PS_DeTe[PS_TempInt]
        • -------- --------
      • Else - Actions
• Custom script: call DestroyGroup(udg_PS_UtGp[udg_PS_TempInt])

[5] You're picking up dead units too.

Last but not least, you shouldn't make triple post. I've mentioned it in my previous comment. Use the "EDIT" button if you forgot to mention anything.

EDIT: You should read the moderator's review. It's clearly listed of everything what you should fix.

 

[Adiktuz]

what does the spell does?

 

[Losam]

Sorry for not having spoken, Just Spell This makes the Following:

You Select A Ground / Spot, The Hero Phoenix that will create a will to the point selected, during the Walk of the Phoenix, Burn It Any Enemy of the Hero, and when it arrives at the Point, she will die of the damage causing an explosion that point Damages Leave and Staggered Any Opponent's Organic Hero.