[GUI]HOLY SPELL v1.2 [MUI]

[Moderator]

20:07, 4th Jun 2009
hvo-busterkomo:
Overall a good spell. It's leak free, MUI, and configurable. My only complaint is that you don't store the caster in a variable.

 

[thereallywhitekid]

Seems pretty fine. I personally liked the healing thing. I would probably give you a 4.87/5 in documentation and 4/5 in the spell. Good Job!

 

[baassee]

someone rated and no comment? :S

 

[Rmx]

REVIEW RULES

Rating System :

0 - 15 = Shame on U
16 - 25 = Unacceptable
26 - 35 = Lacking
36 - 45 = Useful
46 - 55 = Recommended - Highly Recommended
56 - 60 = Directors Cut

Recommended to read the How to improve on every review.

Review

Idea and Creativity - 6/10.
Spell Description and In game Balance - 3/10.
Triggering and Scripting - 9.5/10.
Special effects *Eye Candy* - 5/10.
User Friendly *Documentation* - 10/10.
Efficiency - 8/10.

Is it Multi Instanceable - [OFC it is instant].

Final Rating - [41.5].

Status - ***UseFull***.

How to improve

The spell is very common and lack of special effect.... also the unit moving in their angles even if they have no collision they will still go around the tree or ext.. so make them flying unit that be better, also the spcial effects are very nooby and simple make it better didn't see any leaks and the documentation was really good anyway one thing could be IMPROVED the bj_Wanttodestroygroup ...... Best to destroy group is manually call destroygroup(udg_?) anyway make it nicer have fun.

My View of the Spell

I myself like simple spells but not that simple any trigger could of done that that is shame Bassee to you i am sad u promised me a good spellpack still it's not a spellpack waiting for it !!! i hope i won't regret it .. anyway +REP have fun ~~RMX~~.

 

[-Berz-]

Mmmm... ;D

It's simple! I like it =)

It could use some better special effects, but its ok only the part with holy light(dunno maybe it's just me :S)

Very well documented, more trigger comments than trigger actions

And you have a nice spell here =)

4/5 from me.

 

[Deuterium]

Well baassee I'll be checking the spell and giving you a nice review later tonight \

~seed

 

[baassee]

@ rmx

the thing is that the dummies are alrdy flying units / flying movement type :S

also if you noticed the trigger header, this was one of the most easiest once, got one even easier (worse! ) haha

this is the easy level stage thought might change it to very easy.

also I just had the bj destroy thing to test something new and also get rid of one simple variable but yeah I know and I like destroygroup more but thought newbies need to see some new lines too

@ berzerk

yeah I know about the special effects but I thought that would be enough as I made it a little better then a normal, spawn holy light in 3 different circles or 16 impale lines

@ seed

haha wont get so good rate on thsi one thought so don't expect anything to goody

EDIT:

Updated!

Forgot to remove the target point! My bad

 

[Rmx]

~seed

What seed .. is it a plant seed

 

[Aspard]

I like it, even the effects, dunno why everybody aren't satisfied with it Good job, waiting for additional spells. Won't rate now. =)

 

[baassee]

@ rmx

haha I just call him that instead of his real nick

@ asp

thank you, finally someone thought the effect is enough

 

[Sadrax]

well the spell is... GREAT
healing and damage dealing. good for rpg maps
what can i say... heal and dmg dealing at same time is good idea. you could make it to deal not 0.75 but 2.5 of amount hp healed to undead or at least 1.5 (pwning for ghouls ^^) but even without it the spell is great. you could also decrease dmg to 0.4 or 0.35 and add stun instead. this are just some ideas how you can change this
4.5/5

 

[baassee]

@ sad

as the damage is fully customizeable I get your point.

But that's pretty easy to make just add another condition and another if/then/else

add condition if unit is undead = not true in the regualar damage one

and a one with it as true

but I get your points and I'm satisfied how it is now, it's just a preview right?

~baassee

 

[Deuterium]

What seed .. is it a plant seed

Haha well apparently yesteryda baassee decided to give me a new nickname

 

[Sadrax]

@ baassee

it's just a preview right?

well. you can call it that way i just wrote some ideas that can be used


@ Deuterium

Haha well apparently yesteryda baassee decided to give me a new nickname

you will need to change your signature ^_^
instead of "msaeed28 = Deuterium"
you will have "msaeed28 = Deuterium = seed"

 

[Deuterium]

Haha well apparently yesteryda baassee decided to give me a new nickname

you will need to change your signature ^_^
instead of "msaeed28 = Deuterium"
you will have "msaeed28 = Deuterium = seed"

Haha!



Spell Review:

Holy Light

[-]

1. Although you mention that you didn't do it of laziness, that doesn't mean I won't point it out.
You destroyed the group using a BJ:

• Custom script: set bj_wantDestroyGroup = true

Destroy it manually instead:

• Custom script: call DestroyGroup(udg_<whichgroup>)

2. Special effects bug (as demonstrated in the attached picture). I think setting Movement - Type to Fly would fix that issue...


3. Uses Integer A as the loop integer. Avoid using that as it might cause the spell to bug and use an integer variable instead.


[+]

4. Works efficiently and does what it's supposed to do.


5. MUI


6. Leakless


7. Detailed documentation and explanation!


8. Special effects are simple and nice.


9. User-friendly and easily adjustable.

Well baassee, I think this simple triggering became a simple task to you. Good job!
I like the fact that you're explaining what each little action does in order to teach people new to spell-making.
Anyways, just fix the two negative things mentioned in the review - specially the special effects.

 

[baassee]

@ seed

well seems to that I have to destroy it with the normal one after all

schiesse the units didnt ahve fly movement :S

thought the integer A is instant should bug or? ^^

ty for the review

 

[Deuterium]

well seems to that I have to destroy it with the normal one after all

I mean why use a BJ when you know how to avoid it

schiesse the units didnt ahve fly movement :S

Use a unit that does instead... and remove that import by setting the model as none.mdl

thought the integer A is instant should bug or? ^^

It just bugs sometimes... you never know when it's a Blizzard bug

 

[baassee]

@ seed

well wanted to keep it so few variables as possible

the thing is with the movement that I didn't set it as flying, it's foot, changing now

gaahh really don't want another variable but here we go

 

[krisserz]

Hmm spellpack... I see there only one spell Anyway nice spell

 

[baassee]

Hmm spellpack... I see there only one spell Anyway nice spell

it's a preview of my upcomming spellpack ...

 

[og-loc]

I like this!