Al'akir's Rage 1.2

[Moderator]

15th Feb 2015
IcemanBo: map is updateable again.

Al'akir's Rage 1.2 | Reviewed by Maker | 16th Jun 2013

NEEDS FIX

Using GetUnitDefaultPropWindow like that sets the prop window to a wrong value. Store the value when casting, and load the value from a variable afterwards Flying units get bugged

[tr]

Old reviews Reviewed by Maker Required changes If I follow your importing instructions, the ability might not work. The ability condition and unit type created can be wrong Use GetUnitDefaultPropWindow(udg_KUS_targer) instead of 1 when you reset propulsion window Flush child hashtable Suggested changes Use a variable for the ability and dummy unit type. Set them up in setup trigger and use them in cast trigger Reduce the dummy's sight range The tooltp could list the hotkey You could filter out dead units from being knocked up The knock up would be more impressive if it used some equation instead of going up or down at a constant speed Al'akir's Rage - Reviewed by Maker - 11.02.2013 Required changes are coloured red TRIGGERING call PrepareKnockUp() can cause a compile error, there is no guarantee that it will be compiled before the trigger that calls it Filter out structures from being knocked up You might remove Crow Form from unit that had it before the knock up OBJECT EDITOR Set the dummy's death type to Can't raise, does not decay Locust makes the dummy invulnerable, remove inlvunerability ability from it Set the dummy's cast point to 0, vision to lower value and remove upgrades used GENERAL Lacking importing instructions, the ability in the condition could be wrong after importing

 

[Wrda]

Well I think you didn't give credits to the owner of this model: war3mapImported\LightningWrath.mdx
About the spell:
It is simple ye, but it is still approvable and cool (didn't test)
Maybe you should put the radius variable with arrays, so it increases the radius per level (if the user wants).
And put a descripition about the spell?

 

[Tank-Commander]

Radius should be configurable to a per-level, you refer to (picked unit) after you already cached it, just use the variable again, also "last created unit" should be cached and this spell is absurdly simple

• Unit - Create 1 Footman for Neutral Passive at tloc facing Default building facing degrees

I'm going to assume this "Footman" is a modified dummy? since if not, it should be

 

[Wrda]

• Set KUS_target = (Picked unit)

Should be before

• Unit - Order (Last created unit) to Orc Far Seer - Chain Lightning (Picked unit)

 

[Chaosy]

@weda post 1
1. will credit him when I find the creator again
2. maybe, it's kinda simple to add if would want that.
3. Will add that soon

@tank-guy
oh yes will fix that.
also I COULD make it more advanced by triggering the knockup but since I allready made a system for that which makes it kinda pointless

@wrda post 2
will fix

 

[Chaosy]

updated.
sorry for double posting but I want a own post for it.

1. spell desp added
2. changed the dummy name
3. minor trigger change
4. changed variable names

 

[Wrda]

• Unit - Cause AR_unit to damage (Picked unit), dealing AR_damage[(Level of Al'akir's Rage for AR_unit)] damage of attack type Spells and damage type Normal

(Picked Unit) -> KUS_target you missed that one
Maybe you could make configurable the attack type and damage type

 

[Almia]

huh?

 

[Losam]

Fix

Overall, it has simple functions, leaks and no description of each function (action, condition or event) of each Trigger.
I know a little GUI / Triggers and know that each action is possible, but it is necessary to explain to those unfamiliar.
Furthermore ...
The property's Dummy is incomplete / wrong;
Lack of descriptions ENOUGH

 

[Tank-Commander]

I don't believe there are any leaks, but you do have a point on those trigger names, should have the prefix to match the spell

 

[Chaosy]

updated.
1. the crow form remove bug is fixed I used a old version of my knockup system but that is updated now.
2. removed the invurnable ability from the dummy
3. changed death thingy to can't rise doesn't decay for the dummy
4. as said knockup system updated so I don't use a custom function it got is own tigger now.
5. created a note in the readme which tell that you might consider checking the condition in the cast trigger in case it bugs.

 

[Almia]

NOTE:
Using a system w/c haven't been approved also makes your spell not approvable

 

[Chaosy]

I knew that, its just to wait.

 

[Chaosy]

updated.
-the spell + dummy is now stored in variables @init trigger
-the spell wont hit dead units
-dummy sight range reduced

 

[Chaosy]

there, we go. most likel the last update on this spell. updated the knockup system used.

 

[Toxic 75]

Like Maker say, need to update the knock out system, 3/5 from me.

 

[Chaosy]

Decided it was time to update the knockup system after 2 months of afking

-updated

 

[msongyboi]

it's simple but it's pretty cool

4/5 from me