DEMONIC RAGE

[Moderator]

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

21st Sep 2014
TriggerHappy
Spells need to be completely MUI, this only supports one instance.

10:40, 12th Sep 2014
PurgeandFire: Review:
http://www.hiveworkshop.com/forums/2586444-post30.html

 

[NightKnight]

This is my first spell,hope you like it.

 

[GywGod133]

Dont use wait action

You have leaks

should learn more indexing to approved this

 

[Tank-Commander]

• Unit - A unit Begins casting an ability

->

• Unit - A unit starts the effect of an ability

There's no need for 2 triggers, it could be done with 1
Don't use TriggerSleepActions (TSA's or Waits) there are a few ways to avoid them, Artificial waits (Tutorials section) is a recommended method.
There is a total lack of configuration for this ability
Multiple memory leaks (unit group leaks, point leaks)
The spell is unit-specific (Not MUI or MPI)
Your trigger listings should be contained with Trigger Tags ("

• ")
• In summary: I strongly suggest looking at quite a few tutorials on Spell making

 

[NightKnight]

Ok thanks,but as i said this is my first spell.I will try to fix it.

• Unit - A unit Begins casting an ability

->

• Unit - A unit starts the effect of an ability

When i tried that,I found that after the ability ends it gives bonus str(don't know if it does damage or no)

so I cant.

 

[EdgeOfChaos]

First, post your triggers in

• tags for easy reading.
• Now about the code:
• All Event Responses that are used more than once should be saved to a variable, and this variable should be used.
• [TRIGGER]Set CasterUnit = (Triggering unit)

Like that

-----------------------------------

• Unit - Create 1 Dummy (small) for (Owner of (Triggering unit)) at (Position of (Triggering unit)) facing Default building facing degrees

These actions will dynamically create a location, and therefore will leak.

Change them to something like this:

• Set CasterUnit = (Triggering unit)
• Set TempPoint = (Position of CasterUnit)
• Unit - Create 1 Dummy (small) for (Owner of CasterUnit) at TempPoint facing Default building facing degrees
• Unit Group - Add (Last created unit) to Dummygroup
• Unit - Create 1 Dummy (big) for (Owner of CasterUnit) at TempPoint facing Default building facing degrees
• Unit Group - Add (Last created unit) to Dummygroup
• Custom script: call RemoveLocation(udg_TempPoint)

-----------------------------------

The "Wait" action is not good to use because of glitches. You could use a Timer instead, or use one of the pre-existing libraries for adding stats for a time.

-----------------------------------

• Unit Group - Pick every unit in (Units within 850.00 of (Position of (Triggering unit)) matching (((Matching unit) belongs to an enemy of (Owner of (Triggering unit))) Equal to True)) and do (Actions)
  • Loop - Actions
    • Unit - Cause (Triggering unit) to damage (Picked unit), dealing 800.00 damage of attack type Chaos and damage type Universal

This leaks a dynamically created unit-group and a location. How to fix? Like this:

• Set CasterUnit = (Triggering unit)
• Set TempPoint = (Position of CasterUnit)
• Custom script: set bj_wantDestroyGroup=true
• Unit Group - Pick every unit in (Units within 850.00 of TempPoint matching (((Matching unit) belongs to an enemy of (Owner of CasterUnit)) Equal to True)) and do (Actions)
  • Loop - Actions
    • Unit - Cause CasterUnit to damage (Picked unit), dealing 800.00 damage of attack type Chaos and damage type Universal
• Custom script: call RemoveLocation(udg_TempPoint)

 

[Kazeon]

Demonic, all newbies use this word >.>

 

[deathismyfriend]

Demonic, all newbies use this word >.>

That comment was unnecessary and rude. Also I like demonic names and I don't think I'm a noob.

@TO
Anything you use twice or more should be stored into a variable. Example: (Owner of CasterUnit) should be stored into a variable and the player variable should be used.

 

[EdgeOfChaos]

Also I forgot to mention: use timed life to remove the dummies, don't use a Wait then a unit-group remove.

 

[Kazeon]

It was for joke and no rude intention. Thanks for reminding.

 

[NightKnight]

First, post your triggers in

• tags for easy reading.
• Now about the code:
• All Event Responses that are used more than once should be saved to a variable, and this variable should be used.
• [TRIGGER]Set CasterUnit = (Triggering unit)

Like that

-----------------------------------

• Unit - Create 1 Dummy (small) for (Owner of (Triggering unit)) at (Position of (Triggering unit)) facing Default building facing degrees

These actions will dynamically create a location, and therefore will leak.

Change them to something like this:

• Set CasterUnit = (Triggering unit)
• Set TempPoint = (Position of CasterUnit)
• Unit - Create 1 Dummy (small) for (Owner of CasterUnit) at TempPoint facing Default building facing degrees
• Unit Group - Add (Last created unit) to Dummygroup
• Unit - Create 1 Dummy (big) for (Owner of CasterUnit) at TempPoint facing Default building facing degrees
• Unit Group - Add (Last created unit) to Dummygroup
• Custom script: call RemoveLocation(udg_TempPoint)

-----------------------------------

The "Wait" action is not good to use because of glitches. You could use a Timer instead, or use one of the pre-existing libraries for adding stats for a time.

-----------------------------------

• Unit Group - Pick every unit in (Units within 850.00 of (Position of (Triggering unit)) matching (((Matching unit) belongs to an enemy of (Owner of (Triggering unit))) Equal to True)) and do (Actions)
  • Loop - Actions
    • Unit - Cause (Triggering unit) to damage (Picked unit), dealing 800.00 damage of attack type Chaos and damage type Universal

This leaks a dynamically created unit-group and a location. How to fix? Like this:

• Set CasterUnit = (Triggering unit)
• Set TempPoint = (Position of CasterUnit)
• Custom script: set bj_wantDestroyGroup=true
• Unit Group - Pick every unit in (Units within 850.00 of TempPoint matching (((Matching unit) belongs to an enemy of (Owner of CasterUnit)) Equal to True)) and do (Actions)
  • Loop - Actions
    • Unit - Cause CasterUnit to damage (Picked unit), dealing 800.00 damage of attack type Chaos and damage type Universal
• Custom script: call RemoveLocation(udg_TempPoint)

Really thanks,that will help me much to fix it.

Also I forgot to mention: use timed life to remove the dummies, don't use a Wait then a unit-group remove.

I did all what you said in the previous post in 1 trigger,but i don't know how to make timed life to remove a dummy.

 

[EdgeOfChaos]

• Unit - Add a 60.00 second Generic expiration timer to (Last created unit)

That

 

[gorillabull]

Nice this is going to get a director's cut fur sure but just to be safe include the word chaos and death somewhere in there

 

[Tal'Rarity]

Might want to check a few tutorials on indexing or other peoples approved GUI spells.

 

[NightKnight]

• Unit - Add a 60.00 second Generic expiration timer to (Last created unit)

That

thanks,now i will update it.

Nice this is going to get a director's cut fur sure but just to be safe include the word chaos and death somewhere in there

ok good idea,I will add it to the keywords.

 

[EdgeOfChaos]

ok good idea,I will add it to the keywords.

He was trolling.

Now for some more stuff regarding the new triggers:

-------------------------------------------------

1: It needs a configuration trigger to be allowed on the hive spells. A user should never have to manually edit your triggers if they want to do something like change damage from 800 to 750, or the strength gain from 60 to 80.

To fix this, we make a configuration trigger. Example:

• Config
  • Events
    • Map initialization
  • Conditions
  • Actions
    • -------- The range for damaging the units --------
    • Set DemonicRage_Radius = 850.00
    • -------- Damage dealt to enemies --------
    • Set DemonicRage_Damage = 850.00
    • -------- Strength gained by caster --------
    • Set DemonicRage_StrGain = 60

Then alter the other triggers as needed

• Unit Group - Pick every unit in (Units within DemonicRage_Radius of TempPoint matching (((Matching unit) belongs to an enemy of (Owner of CasterUnit)) Equal to True)) and do (Actions)
  • Loop - Actions
    • Unit - Cause CasterUnit to damage (Picked unit), dealing DemonicRage_Damage damage of attack type Chaos and damage type Universal

• Hero - Modify Strength of CasterUnit: Add DemonicRage_StrGain
• Hero - Modify Strength of CasterUnit: Subtract DemonicRage_StrGain

-------------------------------------------

2: Join triggers

Put the damage section of the second trigger into the first section of the first trigger. It should go before the wait. This was the trigger after some minor alterations, and it worked.

• Demonic Rage
  • Events
    • Unit - A unit Begins casting an ability
  • Conditions
    • (Ability being cast) Equal to DEMONIC RAGE
  • Actions
    • Set CasterUnit = (Triggering unit)
    • Set TempPoint = (Position of CasterUnit)
    • Custom script: set bj_wantDestroyGroup=true
    • Unit Group - Pick every unit in (Units within DemonicRage_Radius of TempPoint matching (((Matching unit) belongs to an enemy of (Owner of CasterUnit)) Equal to True)) and do (Actions)
      • Loop - Actions
        • Unit - Cause CasterUnit to damage (Picked unit), dealing DemonicRage_Damage damage of attack type Chaos and damage type Universal
    • Unit - Create 1 Dummy (small) for (Owner of CasterUnit) at TempPoint facing Default building facing degrees
    • Unit - Add a 62.00 second Generic expiration timer to (Last created unit)
    • Unit - Create 1 Dummy (big) for (Owner of CasterUnit) at TempPoint facing Default building facing degrees
    • Unit - Add a 62.00 second Generic expiration timer to (Last created unit)
    • Custom script: call RemoveLocation(udg_TempPoint)
    • Hero - Modify Strength of CasterUnit: Add DemonicRage_StrGain
    • Wait 62.00 seconds
    • Hero - Modify Strength of CasterUnit: Subtract DemonicRage_StrGain

---------------------------------------------

3: Owner of CastingUnit is used more than once; it should be stored in a variable.

• Set CastingPlayer = (Owner of CasterUnit)

-------------------------------------------

4: We need to do something about the Wait. Waits are (generally) unacceptable due to glitches regarding the Wait function

There are ways of adding temporary stats, but they require a unit indexer and a bit of triggering knowledge, including arrays and timers.

If you don't want to go this direction, you could do something really easy: create a dummy unit, and make it cast Inner Fire on you to add extra damage/armor instead of adding extra strength.

-------------------------------------------

5: Documentation. Use the "Comment" function to indicate what you are doing in the blocks of code. This adds readability to your triggers, breaking them up into chunks.

Example:

• Demonic Rage
  • Events
    • Unit - A unit Begins casting an ability
  • Conditions
    • (Ability being cast) Equal to DEMONIC RAGE
  • Actions
    • -------- Demonic Rage: This spell will deal damage to all nearby units and add strength to the caster. --------
    • -------- Declare event-response variables. --------
    • Set CasterUnit = (Triggering unit)
    • Set TempPoint = (Position of CasterUnit)
    • Set CastingPlayer = (Owner of CasterUnit)
    • -------- Deal DemonicRage_Damage damage to units in DemonicRage_Radius --------
    • Custom script: set bj_wantDestroyGroup=true
    • Unit Group - Pick every unit in (Units within DemonicRage_Radius of TempPoint matching (((Matching unit) belongs to an enemy of CastingPlayer) Equal to True)) and do (Actions)
      • Loop - Actions
        • Unit - Cause CasterUnit to damage (Picked unit), dealing DemonicRage_Damage damage of attack type Chaos and damage type Universal
    • -------- Create dummy units to show the DustWave effect. --------
    • Unit - Create 1 Dummy (small) for CastingPlayer at TempPoint facing Default building facing degrees
    • Unit - Add a 62.00 second Generic expiration timer to (Last created unit)
    • Unit - Create 1 Dummy (big) for CastingPlayer at TempPoint facing Default building facing degrees
    • Unit - Add a 62.00 second Generic expiration timer to (Last created unit)
    • Custom script: call RemoveLocation(udg_TempPoint)

 

[NightKnight]

He was trolling.

Now for some more stuff regarding the new triggers:

-------------------------------------------------

Thanks again,I am working on that.

 

[Tal'Rarity]

put your triggers in trigger tags. For example[trigger=Example]tuh da
tuh da
tuh da[/trigger]

its [trigger=]

• don't forget to make the = in front of the ] but after [trigger as well / at the last part right before trigger] but after [

 

[EdgeOfChaos]

I wouldn't just give him a fixed map; that doesn't teach anything about why it's changed, or teach how to change it. Actually doing efficient coding works wonders to improve skill.

 

[Tal'Rarity]

I wouldn't just give him a fixed map; that doesn't teach anything about why it's changed, or teach how to change it. Actually doing efficient coding works wonders to improve skill.

Doesn't look like anything else is helping so maybe he might learn better by comparing his map to my edit.

 

[EdgeOfChaos]

He did improve it from my last post, and said he's working on it again

 

[Tal'Rarity]

He did improve it from my last post, and said he's working on it again

Well he could use it as a sign of what to do next since his still isn't enough to be approved.

 

[Kazeon]

I wouldn't just give him a fixed map; that doesn't teach anything about why it's changed, or teach how to change it. Actually doing efficient coding works wonders to improve skill.

Totally agreed.

Anyway, please, post the trigger properly. Paste them between this tag:
[trigger]>>paste here<<[/trigger]

I wouldn't review until you post it

 

[NightKnight]

Try learning how to do it like this, it is must more efficient and accurate. Hope this helps lead you into the right direction.

Also put your triggers in trigger tags. For example[trigger=Example]tuh da
tuh da
tuh da[/trigger]

its [trigger=]

• don't forget to make the = in front of the ] but after [trigger as well / at the last part right before trigger] but after [[/QUOTE]
• Sorry,but i don't want a fixed map.If you want to help me then give me a tip.
• [quote="Dalvengyr, post: 2585411"]Totally agreed.
• Anyway, please, post the trigger properly. Paste them between this tag:
• [plain][trigger]>>paste here<<

[/plain]

I wouldn't review until you post it

Thanks for that [trigger]>>paste here<<[/trigger]

I have updated the map.Review it.

I deleted the strength bonus (couldn't delete the wait so i delete it + the wait)

 

[Tal'Rarity]

Your using the wrong event, use unit - starts the effect of an ability.

Fix your condition, put the ability into a variable in your config trigger then ability being cast equal to variable.

Your spell is also extremely simple and well I can only rate it 2/5 currently because it sure looks like a unfinished spell to me, like in the testmap I provided for you I suggest doing something else with those 2 dustwaves u create, maybe move them with the caster? It'd make sense then.

Don't triple post either reyo, merge your posts into one. Good job though on mostly fixing it, waits aren't bad just difficult to use.

 

[NightKnight]

Your using the wrong event, use unit - starts the effect of an ability.

Look at page 1 and you will see why I didn't do that.

 

[kakuzu]

Awesome spell , very good work !

 

[Tal'Rarity]

Look at page 1 and you will see why I didn't do that.

Oh that's a neat bug I never saw before, metamorphis double casts. . . Didn't know however since its a morph should of figured. Buggy ability yet again from Blizzard. =)

However even with the event you use it still bugs and double casts when it ends.

So yeah I don't see why you didn't do that.

 

[NightKnight]

Oh that's a neat bug I never saw before, metamorphis double casts. . . Didn't know however since its a morph should of figured. Buggy ability yet again from Blizzard. =)

However even with the event you use it still bugs and double casts when it ends.

So yeah I don't see why you didn't do that.

No I don't think so,try it.

I am working on update for this spell,adding a dummy and damage over time.

 

[Kazeon]

I am working on update for this spell,adding a dummy and damage over time.

Yeah, but you are not allowed to double/triple/more posting. Use edit button instead.

 

[PurgeandFire]

While this is a nice first attempt at a spell, it is too simple to be approved. It is an AoE with a cool effect, but there aren't any other dynamics to the spell. I apologize, but that is our rule.

I am working on update for this spell,adding a dummy and damage over time.

Damage over time is a good idea. That could tip it into being approvable, but in its current state, I'll move it to "Needs Fix".

 

[NightKnight]

Advanced Update,many things changed.I will not say anything just review it (or look in triggers and in the update log (but better to test it yourself)). Hope you like it.For all who downloaded it before the updateELETE YOUR OLD AND DOWNLOAD THIS NEW UPDATE 100% better!

 

[deathismyfriend]

You leak unit groups.

Your triggers are not MUI. Look at this tutorial for help on making it MUI. http://www.hiveworkshop.com/forums/...en-using-triggers-gui-233242/#Section 2 Ch 12

Anything used twice or more should be stored in a variable and the variable should be used.

 

[PoWerJin]

I like the idea of your spell, i see much effort put into this, good job


Maybe you want to make the start effect lasts only for a few seconds. Having a perma effect on the map is not so nice.

4/5

 

[gorillabull]

i suppose this can be used as an effect in some single player naruto map
there is however no point in uploading it here; its too simple and the only thing someone might download it for is the model. go look at a few other spells, check out some tutorials and then try again

 

[kakuzu]

WoW, the new version look really good, i like the effect, very nice work !

 

[NightKnight]

i suppose this can be used as an effect in some single player naruto map
there is however no point in uploading it here; its too simple and the only thing someone might download it for is the model. go look at a few other spells, check out some tutorials and then try again

No,it isn't too simple anymore.I don't care that you still think its too simple.

 

[Tal'Rarity]

No,it isn't too simple anymore.I don't care that you still think its too simple.

You should care because he's right. Fix your leaks and make the spell MUI.

 

[NightKnight]

I like the idea of your spell, i see much effort put into this, good job


Maybe you want to make the start effect lasts only for a few seconds. Having a perma effect on the map is not so nice.

4/5

If you want to use the spell with a timed effect you can just change the timer from 62 seconds to anything you want.

 

[Tank-Commander]

It's unfortunate that after so long some of the original criticisms still hold, regardless here's a few things I picked up when scrying:
- "Unit - A unit Begins casting an ability" -> "Unit - A unit Starts the effect of an ability"
- There is only a need to have a single loop trigger in any spells, also having a time as high as 1 seconds gives a 0.99 second margin of error for the spell time out, 0.03/0.04 are suggested/recommended to give consistent spell durations, this also means that loop config is not needed, if done properly the demonic end trigger would also not be needed
- The spell is still not MUI only permitting a single caster in any given map, this doesn't match the site rules
- The spell has no level support, another required thing for all spells in the section
- You have location leaks (setting a variable to a location doesn't fix the leak if you never use RemoveLocation(loc) to clean it
- Your spell interferes with the game scorescreen, spells should avoid doing this if possible (dummies should be owned by a neutral player)
- Controlling unit speeds needs to be done with care when you're setting a speed rather than applying a modifier that can be timed out, I advise using an aura to apply it (based off slow aura) to avoid issues arising from that
- You have rather a lot of different dummy units, they could be reduced to 1 via the use of one of the various dummy models we have on the sight and attaching the effects to them (though you will need to remember to destroy the effects, the units dying would leave the effect in memory and therefore leak
- Credit the origins of the models you are using in the spell
- Your spells have no target filter aside from enemy, it can damage magic immune units for example which doesn't match ingame mechanics (will also damage corpses)
- Your variables need prefixes to prevent collisions with other spells on the sight, e.g. DR_varName or DemonRage_varName would be suggested "DamageOverTime" is rather too generic for example, they should also be consistently named (give all variables the same prefix)

for the above reasons I'm setting the resource to needs fix as a lot of these are major flaws, I suggest reading on making MUI spells in our tutorials section
As an aside when you update a substandard/too simple resource you should notify moderators so we can take action to put them back into the pending section

 

[NightKnight]

I notified a moderator and he told me the steps to revive it (which is editing it). I don't understand anything from the tutorials so I'll need steps and examples with simple words (I know nothing about programing)
1-Unit - A unit Starts the effect of an ability" causes a bug so I cant do it.
2- The spell is still not MUI only permitting a single caster in any given map, this doesn't match the site rules, what does that mean and how do i do it?
6- "Stats - race = creep" done (will update it when i make important changes).
8- if it wasn't essential thing (will make a problem in game) then I believe it will be better as this (in order to avoid bugs and it looks very complicated.)
9- I copied the models from another map, but if someone knows about the developers then write to me.
10- How do I make a condition? (give an example + how you did it ---> what button).
11- complete (will update it when i make important changes).

Tell me in details about No.2 and how I fix it (rest of the problems after this).

 

[Tank-Commander]

1 - describe the bug in detail that is generated from this, it's not known to cause any errors
2 - MUI means multi-unit instable essentially the ability for lots of different units (with multiple players being able to use it on multiple units at the same time) without causing errors, to do it you need to implement any from a number of methods, this includes Dyanamic indexing, Linked Lists (recommended) & Hashtables (recommended) generally the full details I can't go into within this post as it'd be too long, you can find a starting point here which links to further examples (for dynamic indexing, some automatic systems & hashtables. Linked lists can be found here, for how to do delays properly to time out abilities or effects (such as to avoid the extra triggers there's this tutorial). If you need examples of this not in tutorials (as you've stated you don't follow them) you should have a look at pretty much any GUI approved submission on the site - they all necessarily do it. though I'd recommend recent ones since they will most closely match our existing standards, it would be impossible for me to explain things more concisely than those tutorials sans personal tutelage which I don't have the time for
6 - Stats - race - creep doesn't solve the issue, the unit itself needs to be owned by a neutral player (currently it is given to the player that owns the casting unit, it is here that needs to be changed)
8 - It's not essential, but I'd still suggest reducing the number of them otherwise, the spell doesn't really need all the effects it has. Given everything else is fixed it'd be a nitpick more than anything else
9 - It's generally not our responsibility to do the legwork for you on these things regardless of how or where you obtained them from, but I believe the chronosphere model was created by "PGRU-Unexpected", as for the dust wave model I've no idea, it should also be noted that generally you should only use third-party models if you have the rights to use them (the dust wave could easily be rip from a game we shouldn't rightfully be using) and only use effects essential to your spell generally (though we make exceptions when aesthetics are dependent on it and there's no suitable in game replacement
10 - If (multiple conditions) then -> add condition -> boolean expression -> unit category -> unit type of (picked unit) is magic immune equal to false -> rest of the actions. May want to store (picked unit) in a temporary unit variable to avoid multiple referrals if you want multiple conditions (I'd generally recommend it)

 

[NightKnight]

6-
Unit - Create 1 Dummy2 (lightning) for Neutral Passive at DemonicPoint facing Default building facing degrees
will that fix it?

 

[Tank-Commander]

yeah that's the kind of thing I meant

 

[NightKnight]

1- The ability happens again after it ends.
6/10/11- done?

 

[NightKnight]

Are there leaks in Unit Groups?

 

[Tank-Commander]

1 - this is probably due to how metamorphosis works, really you shouldn't be using it as a base ability (it's known for causing all kinds of weird issues)

Yes the unit groups do leak, you need to set bj_WantDestroyGroup to true every time you create a unit group in that way rather than just the once
You probably also want to move the matching unit part of the unit group selection into the if statement too, and use the temporary variable (also want to exclude structures for example, same way as before)
the biggest issue atm is ofc the lack of MUI, the restructuring necessary to implement it will probably make too substantial of a change to the spell for much to be noted (the method you're using to end the spell for example, does not work in an MUI spell)

Edit: also be sure to keep version numbers up to date on both the map and the thread, it makes it easier to know people have the latest version

 

[NightKnight]

• Demonic Rage
  • Events
    • Unit - A unit Begins casting an ability
  • Conditions
    • (Ability being cast) Equal to DEMONIC RAGE
  • Actions
    • Trigger - Turn on Demonic Loop <gen>
    • Set CasterDR = (Triggering unit)
    • Set Demonic_Caster = CasterDR
    • Set TempPoint = (Position of CasterDR)
    • Custom script: set bj_wantDestroyGroup=true
    • -------- --------------------------------------------------------- --------
    • -------- Damage and AOE (area of effect) --------
    • Set Demonic_UnitGroup = (Units within Demonic_Radius of TempPoint matching (((Matching unit) belongs to an enemy of (Owner of Demonic_Caster)) Equal to True))
    • Unit Group - Pick every unit in Demonic_UnitGroup and do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • ((Picked unit) is Magic Immune) Equal to False
          • Then - Actions
            • Unit - Cause Demonic_Caster to damage (Picked unit), dealing Demonic_Damage damage of attack type Chaos and damage type Universal
          • Else - Actions
    • Custom script: bj_WantDestroyGroup=true
    • -------- --------------------------------------------------------- --------
    • -------- Those first 8 actions is for the effect (Dummies) --------
    • Unit - Create 1 Dummy (small) for Neutral Passive at TempPoint facing Default building facing degrees
    • Unit - Add a Demonic_Effectdummies_timer second Generic expiration timer to (Last created unit)
    • Unit - Create 1 Dummy (big) for Neutral Passive at TempPoint facing Default building facing degrees
    • Unit - Add a Demonic_Effectdummies_timer second Generic expiration timer to (Last created unit)
    • Unit - Create 1 Dummy (crono) for Neutral Passive at TempPoint facing Default building facing degrees
    • Unit - Add a Demonic_Effectdummies_timer second Generic expiration timer to (Last created unit)
    • Unit - Create 1 Dummy2 (raising skull/howl) for Neutral Passive at TempPoint facing Default building facing degrees
    • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
    • -------- Demonic Rage Str gained --------
    • Hero - Modify Strength of Demonic_Caster: Add Demonic_Strength_Gained
    • Hero - Modify Agility of Demonic_Caster: Add Demonic_Agility_Gained
    • Trigger - Turn on Demonic Loop Config <gen>
    • Trigger - Turn on Demonic Loop <gen>
    • Trigger - Turn on Demonic End <gen>
• [trigger/]
• The unit group good as this? or I should revert?

 

[NightKnight]

Also, I can use that same variable with the other groups? or ill have to change it?

• Set Demonic_UnitGroup = (Units within Demonic_Radius of TempPoint matching (((Matching unit) belongs to an enemy of (Owner of Demonic_Caster)) Equal to True))