Vengeful Hunt Ver0.2

[Moderator]

20:18, 8th Dec 2009
TriggerHappy:

Review for Vengeful Hunt

This spell is far to simple, and the stun effect looks very choppy. I'm sorry but this can never be approved due to it's simplicity.

Status

Rejected

 

[unk]

hmm why my img does not work ?

 

[Billy the Cat]

Doesn't seem to work properly yet
What was the file extension?

I don't know if size matters, but maybe that's the problem.

By the way you got to add the description here so people can see if they want to download the map
Nobody is going to download all maps to see the spells in it.

 

[-Berz-]

1. "Noobspeel Ver0.1" This is not a spell name, and it's wrong anyway it should be spell not "speel"

2. You don't have a screenshot(I think you should change a file name to .jpg or something)

3. Lacks description "Just take it ya got description in there !... " now I'm so attracted by this that I have to download //sarcasm

Please, fix it.

 

[Thanathos]

Sorry but i don´t think this get approved, even if you fix the things Berzerker told you. The spell itself is just tooooo simple, you could probably do that with just using the object editor. Additionally the coding sucks, this has nearly more leaks then lines. If you want to improve yourself then come up with a better idea and ask around in the triggers and scripts forum.

~TNT

 

[naitsirk]

I tested it... and Well Leaks, and It kinda Sucked... Too easy to be Approved for Hive

 

[unk]

man what the **** i suppose to make =)) i'm noob sorry my next spell ill make MORE DIFICULT EVEN FOR YA PRO GUYS ...just w8

 

[StaberFire]

Being new to spellmaking is no excuse for leaks, considering the vast amount of tutorials and coding threads here.

 

[Guishiu]

You'll get it
If you want someone to take your frustration out on , take it out on me
Im never on

 

[unk]

staberfire ya know ...hmm that men kind is learning from theirs mistake's .... and give me some time i'l upgrade this spell and i'll make it whitout any leak's....

 

[-Berz-]

"leak's", you mean "leaks" right?

Yea, fix the things mentioned, it shouldn't be that hard.

 

[unk]

done ^^

{EDIT}
if there is smth wrong again i'l update it soon !

 

[Maker]

1) Don't double post.

2)

• Set TargetLoc = (Target point of ability being cast)
• Set TargetLoc = (Target point of ability being cast)

It makes no sense to set the same location to two different variables. Use only one variable.

3)

• Special Effect - Create a special effect at eFTect1 using Abilities\Spells\NightElf\Taunt\TauntCaster.mdl

^Add "Destroy last created special effect" after that.

4) Vengeful Spirit and Sentry share the same event and condition. Combine the triggers.

5) Move sentry runs all the time, even if there are no instances of the spell active. Turn it on/off when needed.

6) You could use a static group for the dummies, instead of creating/destroying it again and again.

7) Spell has only one level.

8) The spell isn't easily configurable.

9) Bad grammar in tooltip:

"Create a special Sentry for 15 second's and Trow a Sentry that make's 100 dmg."

second's -> seconds
Trow a Sentry = WTF?
make's = makes (although make is a bad verb to use here I think)

 

[unk]

Maker i done it now ^.^ ya can test it ....

 

[atheistguy]

I award thee *****iest description award. congrats uve earned it

 

[Maker]

You should turn off "Move sentry" when there are no units of type "dummy unit" in the game. So remove that "turn off" from "remove sentry" and add a check <TempGroup> is empty int Move Sentry. If it is empy, turn off the trigger.