Invulnerable Field v9

[Moderator]

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

Filter out dead units from being damaged.
Use variables for unit types and for the ability, set them in settings trigger.
Also make the dummy type amounts configurable.
The biggest problem with this spell is that is is just Big Bad Voodoo with instant AoE and some effects. It's not that original. Maybe you could make it to be more interesting.
I just remembered we already have a spell like this-> www.hiveworkshop.com/forums/spells-569/gui-divine-protection-v1-2-a-195702

Old reviews

51 dummies per cast is just too much. Try to use less.
There is no importing instructions.
The dummies use upgrades.

Add importing instructions
Not needed: Unit - Add Invulnerable Field (Unit) to TempUnit
The unit has the ability in object editor.
The damage and AoE arrays are not needed, you can either move them into a separate init trigger or use an equation in the cast trigger.

Dont set TempUnit in the forst loop since you only use last created unit once.
The spell doesn't support levels.
Dummies do not need invulnerability ability, they have locust.
Set dummies to can't raise, does not decay.

23rd Oct 2011
Bribe: If you want to use 39 dummies per cast you need to use some form of dummy recycling or just use fewer dummies (I recommend the latter). This is just too much lag per cast.

 

[Magtheridon96]

I could've sworn I've seen this before..
You should use the 'Update' button instead of re-uploading the resource.

1) Use an integer variable instead of (Integer A) [Integer A is bug prone]
2) Instead of repeating stuff like (Last created unit) and slowing down the trigger, you should store the data in variables.
For example:

• Set Offset = (CasterPosition offset by 700.00 towards (30.00 x (Real((Integer A)))) degrees)
• Unit - Create 1 Laser Effects for TriggerPlayer at Offset facing Default building facing degrees
• Unit - Order (Last created unit) to Human Blood Mage - Siphon Mana Laser_Taker
• Unit - Add a 30.00 second Generic expiration timer to (Last created unit)
• Custom script: call RemoveLocation(udg_Offset)

->

• Set Offset = (CasterPosition offset by 700.00 towards (30.00 x (Real((Integer A)))) degrees)
• Unit - Create 1 Laser Effects for TriggerPlayer at Offset facing Default building facing degrees
• Set TempUnit = (Last created unit)
• Unit - Order TempUnit to Human Blood Mage - Siphon Mana Laser_Taker
• Unit - Add a 30.00 second Generic expiration timer to TempUnit
• Custom script: call RemoveLocation(udg_Offset)

3)

• Set UnitGroup = (Units within 700.00 of CasterPosition matching (((Owner of (Matching unit)) is an enemy of (Owner of (Triggering unit))) Equal to True))
• Unit Group - Pick every unit in UnitGroup and do (Unit - Cause Caster to damage (Picked unit), dealing 300.00 damage of attack type Spells and damage type Normal)

Since you're only setting the group, picking it, manipulating it then destroying it, you can do this instead:

• Custom script: set bj_wantDestroyGroup = true
• Unit Group - Pick every unit in (Units within 700.00 of CasterPosition matching (((Owner of (Matching unit)) is an enemy of (Owner of (Triggering unit))) Equal to True)) and do (Actions)
  • Loop - Actions
    • Unit - Cause Caster to damage (Picked unit), dealing 300.00 damage of attack type Spells and damage type Normal

This way, you don't even need a group variable.

4) I just realized you add that 'set bj_wantDestroyGroup = true' line.
You can remove the line where you set the group variable, and put that '(Units within 700.00 of CasterPosition matching (((Owner of (Matching unit)) is an enemy of (Owner of (Triggering unit))) Equal to True))' inside the 'Pick every unit...' line.
Then, just delete the 'DestroyGroup' line and remove the group variable from the variable list.

5) Hope this helps.

 

[xIceShotx]

I could've sworn I've seen this before..
You should use the 'Update' button instead of re-uploading the resource.

1) Use an integer variable instead of (Integer A) [Integer A is bug prone]
2) Instead of repeating stuff like (Last created unit) and slowing down the trigger, you should store the data in variables.
For example:

• Set Offset = (CasterPosition offset by 700.00 towards (30.00 x (Real((Integer A)))) degrees)
• Unit - Create 1 Laser Effects for TriggerPlayer at Offset facing Default building facing degrees
• Unit - Order (Last created unit) to Human Blood Mage - Siphon Mana Laser_Taker
• Unit - Add a 30.00 second Generic expiration timer to (Last created unit)
• Custom script: call RemoveLocation(udg_Offset)

->

• Set Offset = (CasterPosition offset by 700.00 towards (30.00 x (Real((Integer A)))) degrees)
• Unit - Create 1 Laser Effects for TriggerPlayer at Offset facing Default building facing degrees
• Set TempUnit = (Last created unit)
• Unit - Order TempUnit to Human Blood Mage - Siphon Mana Laser_Taker
• Unit - Add a 30.00 second Generic expiration timer to TempUnit
• Custom script: call RemoveLocation(udg_Offset)

3)

• Set UnitGroup = (Units within 700.00 of CasterPosition matching (((Owner of (Matching unit)) is an enemy of (Owner of (Triggering unit))) Equal to True))
• Unit Group - Pick every unit in UnitGroup and do (Unit - Cause Caster to damage (Picked unit), dealing 300.00 damage of attack type Spells and damage type Normal)

Since you're only setting the group, picking it, manipulating it then destroying it, you can do this instead:

• Custom script: set bj_wantDestroyGroup = true
• Unit Group - Pick every unit in (Units within 700.00 of CasterPosition matching (((Owner of (Matching unit)) is an enemy of (Owner of (Triggering unit))) Equal to True)) and do (Actions)
  • Loop - Actions
    • Unit - Cause Caster to damage (Picked unit), dealing 300.00 damage of attack type Spells and damage type Normal

This way, you don't even need a group variable.

4) I just realized you add that 'set bj_wantDestroyGroup = true' line.
You can remove the line where you set the group variable, and put that '(Units within 700.00 of CasterPosition matching (((Owner of (Matching unit)) is an enemy of (Owner of (Triggering unit))) Equal to True))' inside the 'Pick every unit...' line.
Then, just delete the 'DestroyGroup' line and remove the group variable from the variable list.

5) Hope this helps.

Thanks, if i get a few more mistakes, ill try to fix the trigger as much as possible, and if it does not happen that quickly, ill make sure to get almost all of that One of them im getting a little confused , please dont think im stupid tho lol
Heres the part

Then, just delete the 'DestroyGroup' line and remove the group variable from the variable list.

Thanks for telling the stuff i should do

 

[xIceShotx]

Allright then, ill try to reduce lag and get some fixes on the spell xD.
Thanks for telling me leaks and fixes i should do!
~xIceShotx

 

[xIceShotx]

Updated to v4 - Read description to see details!
And do not forget to vote on what the spell should look like next version (Look at description to see more details about the pictures)!

Hope you enjoy! ~xIceShotx

 

[Magtheridon96]

• Unit - Add a 30.00 second Generic expiration timer to (Last created unit)
• Unit - Add Invulnerable Field (Unit) to (Last created unit)
• Unit - Order (Last created unit) to Orc Shadow Hunter - Big Bad Voodoo
• Unit - Create 1 SpecialFX for TriggerPlayer at CasterPosition facing Default building facing degrees
• Unit - Add a 30.00 second Generic expiration timer to (Last created unit)

Step 2 in my first post needs to be applied here too.. In fact, it needs to be applied everywhere.

Also, instead of using (Integer A), create a new integer variable and use it. (Integer A) is quite bug-prone.

 

[xIceShotx]

• Unit - Add a 30.00 second Generic expiration timer to (Last created unit)
• Unit - Add Invulnerable Field (Unit) to (Last created unit)
• Unit - Order (Last created unit) to Orc Shadow Hunter - Big Bad Voodoo
• Unit - Create 1 SpecialFX for TriggerPlayer at CasterPosition facing Default building facing degrees
• Unit - Add a 30.00 second Generic expiration timer to (Last created unit)

Step 2 in my first post needs to be applied here too.. In fact, it needs to be applied everywhere.

Also, instead of using (Integer A), create a new integer variable and use it. (Integer A) is quite bug-prone.

Allrightt sirrr thanks for telling me!
---------Edit----------------
Stored the unit data in triggers - Also, if i need store more please tell me, Ill be posting the triggers in next version.
And, I need help. Im kind of misunderstanding what you mean by "Also, instead of using (Integer A), create a new integer variable and use it.". But.... Maby this is what you mean?

• For each (Integer A) from 1 to 24, do (Actions)
  • Loop - Actions
    • Set Integer = (Integer A)
    • Set Offset = (CasterPosition offset by 700.00 towards (15.00 x (Real(Integer))) degrees)
    • Unit - Create 1 Laser Effects (Hidden) for TriggerPlayer at Offset facing Default building facing degrees
    • Set TempUnit = (Last created unit)
    • Unit - Add a 30.00 second Generic expiration timer to TempUnit
    • Custom script: call RemoveLocation(udg_Offset)

If not, please help!
~xIceShotx

 

[Magtheridon96]

No.
I mean this:

• For each Integer from 1 to 24, do (Actions)
  • Loop - Actions
    • Set Offset = (CasterPosition offset by 700.00 towards (15.00 x (Real(Integer))) degrees)
    • Unit - Create 1 Laser Effects (Hidden) for TriggerPlayer at Offset facing Default building facing degrees
    • Set TempUnit = (Last created unit)
    • Unit - Add a 30.00 second Generic expiration timer to TempUnit
    • Custom script: call RemoveLocation(udg_Offset)

 

[xIceShotx]

No.
I mean this:

• For each Integer from 1 to 24, do (Actions)
  • Loop - Actions
    • Set Offset = (CasterPosition offset by 700.00 towards (15.00 x (Real(Integer))) degrees)
    • Unit - Create 1 Laser Effects (Hidden) for TriggerPlayer at Offset facing Default building facing degrees
    • Set TempUnit = (Last created unit)
    • Unit - Add a 30.00 second Generic expiration timer to TempUnit
    • Custom script: call RemoveLocation(udg_Offset)

Ohhhh! Thanks!
But do i set it like this?

• Set Integer = (Integer A)
• For each (Integer Integer) from 1 to 24, do (Actions)
  • Loop - Actions
    • Set Offset = (CasterPosition offset by 700.00 towards (15.00 x (Real(Integer))) degrees)
    • Unit - Create 1 Laser Effects (Hidden) for TriggerPlayer at Offset facing Default building facing degrees
    • Set TempUnit = (Last created unit)
    • Unit - Add a 30.00 second Generic expiration timer to TempUnit
    • Custom script: call RemoveLocation(udg_Offset)

 

[Magtheridon96]

Close.
This line:

• Set Integer = (Integer A)

is useless.

Also, in that loop, you don't need to set TempUnit to the (Last created unit) because you only used it once.
It's only faster to set the data inside a variable and use it if you're going to use it more than once.
Just use (Last created unit) in that expiration timer line.

 

[xIceShotx]

Close.
This line:

• Set Integer = (Integer A)

is useless.

Also, in that loop, you don't need to set TempUnit to the (Last created unit) because you only used it once.
It's only faster to set the data inside a variable and use it if you're going to use it more than once.
Just use (Last created unit) in that expiration timer line.

Oke Dokey! Will be editing.

 

[xIceShotx]

Updated to Invulnerable Field v5!
Read description to see what has benn changed!

 

[Shartuku]

O_O hey iceshot!

 

[xIceShotx]

O_O hey iceshot!

Woah, lol. Never knew you were a hive member, clan member! xD

 

[Magtheridon96]

• For each (Integer Loop) from 1 to 3, do (Actions)
  • Loop - Actions
    • Unit - Create 1 SpecialFX for TriggerPlayer at CasterPosition facing Default building facing degrees
    • Set TempUnit = (Last created unit)
    • Animation - Change TempUnit flying height to (100.00 x (Real(Loop))) at (100.00 x (Real((Integer A))))
    • Unit - Add a 30.00 second Generic expiration timer to TempUnit

You used (Integer A)


You should make this spell configurable.
Create a new trigger in which you allow users to set variables to modify the spell.
You can create a real array, for example, to modify the spell damage.

Damage[1] => Damage at level 1
Damage[2] => Damage at level 2
Damage[3] => Damage at level 3

You can also create one for the Area of effect.

Range[1] => Range at level 1
Range[2] => Range at level 2
Range[3] => Range at level 3

You'd change this:

• Custom script: set bj_wantDestroyGroup = true
• Unit Group - Pick every unit in (Units within 700.00 of CasterPosition matching (((Owner of (Matching unit)) is an enemy of TriggerPlayer) Equal to True)) and do (Actions)
  • Loop - Actions
    • Unit - Cause Caster to damage (Picked unit), dealing 300.00 damage of attack type Spells and damage type Normal

to something like this:

• Set IF_level = (Level of Invulnerable Field for Caster)
• Custom script: set bj_wantDestroyGroup = true
• Unit Group - Pick every unit in (Units within IF_spellAoE[IF_level] of CasterPosition matching (((Owner of (Matching unit)) is an enemy of TriggerPlayer) Equal to True)) and do (Actions)
  • Loop - Actions
    • Unit - Cause Caster to damage (Picked unit), dealing Damage[IF_level] damage of attack type Spells and damage type Normal

edit
Also, as Bribe said, you're creating way too many dummies.
You should be recycling dummies in this case.
-Actually, it would be way better if you just use less dummies.

 

[xIceShotx]

• For each (Integer Loop) from 1 to 3, do (Actions)
  • Loop - Actions
    • Unit - Create 1 SpecialFX for TriggerPlayer at CasterPosition facing Default building facing degrees
    • Set TempUnit = (Last created unit)
    • Animation - Change TempUnit flying height to (100.00 x (Real(Loop))) at (100.00 x (Real((Integer A))))
    • Unit - Add a 30.00 second Generic expiration timer to TempUnit

You used (Integer A)


You should make this spell configurable.
Create a new trigger in which you allow users to set variables to modify the spell.
You can create a real array, for example, to modify the spell damage.

Damage[1] => Damage at level 1
Damage[2] => Damage at level 2
Damage[3] => Damage at level 3

You can also create one for the Area of effect.

Range[1] => Range at level 1
Range[2] => Range at level 2
Range[3] => Range at level 3

You'd change this:

• Custom script: set bj_wantDestroyGroup = true
• Unit Group - Pick every unit in (Units within 700.00 of CasterPosition matching (((Owner of (Matching unit)) is an enemy of TriggerPlayer) Equal to True)) and do (Actions)
  • Loop - Actions
    • Unit - Cause Caster to damage (Picked unit), dealing 300.00 damage of attack type Spells and damage type Normal

to something like this:

• Set IF_level = (Level of Invulnerable Field for Caster)
• Custom script: set bj_wantDestroyGroup = true
• Unit Group - Pick every unit in (Units within IF_spellAoE[IF_level] of CasterPosition matching (((Owner of (Matching unit)) is an enemy of TriggerPlayer) Equal to True)) and do (Actions)
  • Loop - Actions
    • Unit - Cause Caster to damage (Picked unit), dealing Damage[IF_level] damage of attack type Spells and damage type Normal

edit
Also, as Bribe said, you're creating way too many dummies.
You should be recycling dummies in this case.
-Actually, it would be way better if you just use less dummies.

lol, no problem, ill add more levels but i kind of considered this as a final ability like Mountain King - Avatar (Only has one level) but, i guess ill create some more levels for it
Thanks for your opinion!
~xIceShotx

 

[Magtheridon96]

And could you stop saying '~xIceShotx' at the end of every post? =P
We know the posts are legit and we know they're coming from you.

~Magtheridon96
xD

 

[xIceShotx]

And could you stop saying '~xIceShotx' at the end of every post? =P
We know the posts are legit and we know they're coming from you.

~Magtheridon96
xD

Lol sureee
I need help tho on "how to recycle a dummy?" and i cannot reduce units because if i reduce beams to 12 it looks SUPER skinny lol.

 

[Magtheridon96]

Well, 39 is just too much dude.
For recycling dummies, you can nag Bribe about it cause he's the first one to propose the idea ;D

 

[WizardYu]

Epiccc!

 

[maddeem]

Damage, AoE, and other things MUST be configurable to a specific caster, so like the damage can be set to 2x int +100.
Also, dont use dummies to cast lightning, trigger it. It will reduce lag and units required to C-N-P.
(Suggestion) For the casting Medhivs or however you spell that, those should be mirror images of the caster.
If a spell isn't done over time (periodically or through timers) then I think it is too simple, therefore I rate this 2/5

~maddeem

rofl

 

[SpectreD]

very nice

 

[jdo]

you have seen this in candy war (halloween Edition)