Shadowraze v1.0.3

[Moderator]

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

15:37, 10th Jan 2015
IcemanBo: https://www.hiveworkshop.com/threads/shadowraze-v1-0-3-a.260514/post-2638483

 

[ukn0wnD3str0y3r]

Anyone see this? ><

 

[LordDz]

Yes.

 

[ukn0wnD3str0y3r]

Yes.

So how you think?

 

[Lender]

not much to say. its ok

 

[ukn0wnD3str0y3r]

not much to say. its ok

Okay, it is simple i know but will this get approved?

 

[Lender]

if no other spell exist here alike, maybe if not too simple.

EDIT: there is an approved shadowfiend spell pack so I assume it can be approved too

 

[Vinz]

Why do you destroy the effect after the group pick? I suggest to not store it at all and destroy it immediately.

Since it's a bit simple I once again suggest more functionalities. Like silencing shadowrazed people, or deleting their mana for some time. xD

 

[ukn0wnD3str0y3r]

Why do you destroy the effect after the group pick? I suggest to not store it at all and destroy it immediately.

Since it's a bit simple I once again suggest more functionalities. Like silencing shadowrazed people, or deleting their mana for some time. xD

I have an idea, why not add Damage over time @@
EDIT: but sorry, this is based on DotA's one ^^ but i will try making other spell base on dota and custom by myself maybe @@?
EDIT 2: why i cant destroy effect after group pick o.0?

 

[Vinz]

It's not that you can't, but it has no actual difference between a.) variable setting and doing that after / b.) directly destroying Last Created Effect upon creation

 

[ukn0wnD3str0y3r]

Okay...

 

[GywGod133]

• Unit - Add Shadowraze [W] to (Learning Hero)
• Unit - Add Shadowraze [E] to (Learning Hero)
• Unit - Set level of Shadowraze [W] for (Learning Hero) to Sw_Level
• Unit - Set level of Shadowraze [E] for (Learning Hero) to Sw_Level

should be add CONDITION
and if the Learning Hero use twice or more
should be put in a variable

• Unit Group - Pick every unit in (Units within Sw_AoE[Sw_X] of Sw_Point[Sw_X] matching ((((Matching unit) is A structure) Equal to False) and ((((Matching unit) is alive) Equal to True) and (((Matching unit) belongs to an enemy of (Owner of (Triggering unit))) Equal to True)))) and do (Actions)
  • Loop - Actions

ITE please (I know you know that)
and also dont forget to save the picked unit in a variable

 

[ukn0wnD3str0y3r]

• Unit - Add Shadowraze [W] to (Learning Hero)
• Unit - Add Shadowraze [E] to (Learning Hero)
• Unit - Set level of Shadowraze [W] for (Learning Hero) to Sw_Level
• Unit - Set level of Shadowraze [E] for (Learning Hero) to Sw_Level

should be add CONDITION
and if the Learning Hero use twice or more
should be put in a variable

• Unit Group - Pick every unit in (Units within Sw_AoE[Sw_X] of Sw_Point[Sw_X] matching ((((Matching unit) is A structure) Equal to False) and ((((Matching unit) is alive) Equal to True) and (((Matching unit) belongs to an enemy of (Owner of (Triggering unit))) Equal to True)))) and do (Actions)
  • Loop - Actions

ITE please (I know you know that)
and also dont forget to save the picked unit in a variable

Okay, i will change that

EDIT: is the learning trigger should be like this?

• Shadowraze Learn
  • Events
    • Unit - A unit Learns a skill
  • Conditions
    • (Learned Hero Skill) Equal to Shadowraze [Q]
  • Actions
    • -------- Configure the Level --------
    • Set Sw_Level = (Level of Shadowraze [Q] for (Learning Hero))
    • -------- Add two other spells --------
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Level of Shadowraze [W] for (Learning Hero)) Greater than 0
        • (Level of Shadowraze [E] for (Learning Hero)) Greater than 0
      • Then - Actions
        • Unit - Set level of Shadowraze [W] for (Learning Hero) to Sw_Level
        • Unit - Set level of Shadowraze [E] for (Learning Hero) to Sw_Level
      • Else - Actions
        • Unit - Add Shadowraze [W] to (Learning Hero)
        • Unit - Add Shadowraze [E] to (Learning Hero)

 

[GywGod133]

@unknown
like that

 

[ukn0wnD3str0y3r]

So, it is correct right

 

[ukn0wnD3str0y3r]

Updated Verison 1.0.1

 

[Vinz]

I suggest having an effect created on units hit for more bling and usage of the stored variable

 

[ukn0wnD3str0y3r]

Effect change to a dummy unit?

 

[BPower]

A very known and simple spell concept. Actually the shadowfiend was one of my favorite characters in DotA.
There are a few things you should change in your loop trigger:

1. SW_Caster should be a non array variable. Move GetTriggeringUnit function
above the from 1 - 3 loop, the caster is always the same unit

2. SW_Point should be a non array variable.

3. You don't have to store the lastCreatedSpecialEffect into a variable,
simple do DestroyEffect(LastCreatedSpecialEffect).

4. You don't have to store picked unit. It is only used once.

5. Optionally you could add a damage type and attack type variable to the configuration trigger.

 

[GywGod133]

• Unit Group - Pick every unit in (Units within Sw_AoE[Sw_X] of Sw_Point[Sw_X]) and do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • ((Picked unit) is A structure) Equal to False
        • ((Matching unit) is alive) Equal to True
        • ((Picked unit) belongs to an enemy of (Owner of Sw_Caster[Sw_X])) Equal to True
      • Then - Actions
        • Set Sw_Target[Sw_X] = (Picked unit)
        • Unit - Cause Sw_Caster[Sw_X] to damage Sw_Target[Sw_X], dealing Sw_Damage[Sw_Level] damage of attack type Spells and damage type Normal
      • Else - Actions

->>>

• Unit Group - Pick every unit in (Units within Sw_AoE[Sw_X] of Sw_Point[Sw_X]) and do (Actions)
  • Loop - Actions
    • Set Sw_Target[Sw_X] = (Picked unit)
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Sw_Target[Sw_X] is A structure) Equal to False
        • Sw_Target[Sw_X] is alive) Equal to True
        • Sw_Target[Sw_X] belongs to an enemy of (Owner of Sw_Caster[Sw_X])) Equal to True
      • Then - Actions
        • Unit - Cause Sw_Caster[Sw_X] to damage Sw_Target[Sw_X], dealing Sw_Damage[Sw_Level] damage of attack type Spells and damage type Normal
      • Else - Actions

 

[ukn0wnD3str0y3r]

I will change that Thx!

EDIT:

2. SW_Point should be a non array variable

I set it on a variable because some people may like to change the range or AoE

 

[ukn0wnD3str0y3r]

Updated Verison 1.0.2~

 

[BPower]

You have a group leak. Move Effect - Destroy right below the line where you create the effect.

 

[ukn0wnD3str0y3r]

Okay

EDIT: Where is the Group leak can you Show me?

 

[GywGod133]

Okay

EDIT: Where is the Group leak can you Show me?

In the 3nd trigger
You created unit group to dmg the nearby enemies
So u forgot remove it

^_^

 

[Vinz]

Effect change to a dummy unit?

No, create a mass effect on units hit. Just a suggestion though.

 

[ukn0wnD3str0y3r]

In the 3nd trigger
You created unit group to dmg the nearby enemies
So u forgot remove it

^_^

Huh? but i didn't set it in a variable, do it also needed to clear it?
using this ->> "set bj_wantDestroyGroup = true" ??

No, create a mass effect on units hit. Just a suggestion though.

I get it, i will add it soon ^^

 

[BPower]

Huh? but i didn't set it in a variable, do it also needed to clear it?
using this ->> "set bj_wantDestroyGroup = true" ??

Yes. The function pick every unit .... creates a local group which has to be destroyed.

I set it on a variable because some people may like to change the range or AoE

That doesn't make any sense. This is a location variable and not a real variable,
there is nothing you could manipulate. Think about it
It's a normal location (point), so an array is completely unnecessary.

If the triggers in the description are up to date, then you didn't change anything of which I suggested.
Feel free to ask if you didn't understand my suggestions.

 

[ukn0wnD3str0y3r]

So you mean destroy the effect right after when the effect is created?
EDIT: Unit group should set in variable? o.0

 

[BPower]

For the effect, yes.
For the group. You can, but you don't have to. Either use bj_wantDestroyGroup or DestroyGroup and a group variable.

 

[ukn0wnD3str0y3r]

Thanks!

 

[ukn0wnD3str0y3r]

Updated Verison 1.0.3~
[+]Added Mass Effect suggested by Mythic
[+]Now the group is destroyed fix by BPower

 

[ukn0wnD3str0y3r]

May I ask, what else need to fix or change? because some spells post late than mine but it is checked by admin faster ._.

 

[IcemanBo]

Sorry if you had to wait too long.

It is actually just one ability. Not three. So just keep it as one, don't make dublicates for nothing.

Though you maybe should improve the concept of the "one" spell then. Damaging a unit group onCast is very simple.

Needs Fix

 

[ukn0wnD3str0y3r]

So do i need to add something like knockback or DOT for damaged unit and make it more better?

 

[Vinz]

Perhaps make it DoT and apply a slow?