[GUI]HOLY SPELL v1.2 [MUI]

Moderator · May 25, 2009

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 · May 25, 2009

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! :infl_thumbs_up:

baassee · May 25, 2009

someone rated and no comment? :S

Rmx · May 25, 2009

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.

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***.

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.

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- · May 25, 2009

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 · May 25, 2009

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

\

~seed

baassee · May 25, 2009

@ 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 · May 25, 2009

~seed

What seed .. is it a plant seed

Aspard · May 25, 2009

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 · May 25, 2009

@ rmx

haha I just call him that instead of his real nick

@ asp

thank you, finally someone thought the effect is enough

Sadrax · May 25, 2009

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 · May 25, 2009

@ 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 · May 25, 2009

Rmx said:
What seed .. is it a plant seed

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

Sadrax · May 25, 2009

@ 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 · May 25, 2009

Sadrax said:
Deuterium said:

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

Click to expand...

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

Haha!

Spell Review:

[-]

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 · May 26, 2009

@ 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 · May 26, 2009

baassee said:
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

baassee said:
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

baassee said:
thought the integer A is instant should bug or? ^^

It just bugs sometimes... you never know when

it's a Blizzard bug

baassee · May 26, 2009

@ 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 · May 28, 2009

Hmm spellpack... I see there only one spell :wink:

Anyway nice spell :thumbs_up:

baassee · May 31, 2009

krisserz said:
Hmm spellpack... I see there only one spell Anyway nice spell

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

og-loc · Jun 1, 2009

I like this!

[GUI]HOLY SPELL v1.2 [MUI]

baassee H&L SP Preview v1.2 [MUI] (Map)

Moderator

Moderator

Similar threads