Hammer of Thor v1.08

[Moderator]

15:56, 4th Oct 2009
TriggerHappy187:

As I said in my review, this spell is awesome.

Rated Recommended.

 

[Aspard]

Here is the link for the non-passive button of the icon you've chosen:
ReplaceableTextures\CommandButtons\BTNStormHammer.blp

Why the hero has low hit points at the beggining? Also place, please, an ability to the X:0; Y:2 position in the commands panel, for me it's really irritating when I learn skill in one position, but then it appears in another. ;(

• Custom script: call UnitAddAbilityBJ( 'Aloc', udg_Temp_Unit_2 )

Why use BJ if it's easy to call UnitAddAbility? If I'm not mistaken, just the order of the variables will change.

Nevertheless, there are too many triggers for me here >< As far as I still didn't study through Hashtables.

In "Hammer of Thor Lightning Move" you don't remove locations, but just store them into variables. Well, that's all from me... It's really difficult to understand how all this triggers are connected

Otherwise, the spell is decent, I really like it! Concept is creative and the spell seems to be MUI and work properly. I can rate it 3/5.

EDIT: What the heck, why can't I rate spells? *CRY* If somebody has already rated it - I can add my rating, but I can't do it first... T_T

 

[TriggerHappy]

Visually the spell is awesome and fairly unique. I loved it, one of the better GUI spells.

• Replace UnitAddAbilityBJ('Aloc', udg_Temp_Unit_2 ) with UnitAddAbility(udg_Temp_Unit_2, 'Aloc').
• Nice work with hashtables, you are doing everything correctly in your code unless I missed something.

The code is seemingly flawless for GUI spell, I highly reccomend this spell to be approved.

 

[Aspard]

Replace UnitAddAbilityBJ('Aloc', udg_Temp_Unit_2 ) with UnitAddAbility('Aloc', udg_Temp_Unit_2).

No, this will be:

call UnitAddAbility(udg_Temp_Unit_2, 'Aloc')

The order is different

 

[Maker]

Here is the link for the non-passive button of the icon you've chosen:
ReplaceableTextures\CommandButtons\BTNStormHammer.blp

Thanks, updated.

Why the hero has low hit points at the beggining? Also place, please, an ability to the X:0; Y:2 position in the commands panel

Hit points set to 100 % and position changed

• Custom script: call UnitAddAbilityBJ( 'Aloc', udg_Temp_Unit_2 )

Changed that to non-BJ.

In "Hammer of Thor Lightning Move" you don't remove locations

Fixed.

Otherwise, the spell is decent, I really like it! Concept is creative and the spell seems to be MUI and work properly. I can rate it 3/5.

Thanks a lot for your comments and suggestions

Visually the spell is awesome and fairly unique. I loved it, one of the better GUI spells.

Thank you

 

[Aspard]

Well... and now I rated it successfuly... who can explain? Can it be somehow connected to my browser? I use Opera.

Btw: 4/5

 

[BloodSoul]

Nice spell and effects. I like it.
Your revive system, revive dummy also.

 

[Maker]

^Thanks.

Small update on the test map, not the ability. The dummy is no longer revived. They dummy used to be revived for neutral
hostile, and there would be a lot of those attacking you, so you lost health really fast, and the spell attacked those invisible units. All should be good now.

 

[Weep]

Very, very cool spell idea and FX. Nice to see a good use of Hashtables.

One problem I noticed is that the projectile dummies aren't given Locust and their health bars show up.

 

[GLB]

That's a wonderful spell! And the spell's triggering is nice, as welll. Since it is a GUI-spell and most of GUI spells are uncarly triggered, but not this one, I would give 4/5. Have a nice day...

 

[Maker]

One problem I noticed is that the projectile dummies aren't given Locust and their health bars show up.

Thanks for noticing that. Now they have locust.

That's a wonderful spell!

Thanks!

 

[Lambdadelta]

I always like spells that have a triggered birth and death

This. Sandman.

 

[X-OMG-X]

Yea +11 gold! The paladin is getting rich.
Back to topic: Havent tried it yet but it seems quite good. You can configurate alot and that makes it attractive

 

[Maker]

Small update. I changed the lines that turn off the periodic triggers from the end of the triggers to right after units are removed from the unit group. This makes the spell require less CPU time.

 

[NOSAKIAS]

Still has some serious bugs.

Check out my discovery!

 

[NOSAKIAS]

gg

edit: PHAIL IMAGE xD

 

[Maker]

^And how did you manage to do that? Don't tell me you abused the esc thingy, resetting cooldown?

It shouldn't matter anymore anyway.

Update:

v1.05
Changed stun to be added by creating a dummy unit and oredring it to use Storm Bolt on units.
Removed one trigger, added its function to another trigger. This solved a bug that caused the hammer sometimes to not slam to the ground.
Fixed a MUI issue on burn effects.
Removed one wait, it was pointless. Using a timer bound to the hammer now.

 

[mjllonir]

Thank you for making a spell about me . No but really this is amazing! Great job!

 

[NOSAKIAS]

^And how did you manage to do that? Don't tell me you abused the esc thingy, resetting cooldown?

It shouldn't matter anymore anyway.

Update:

v1.05
Changed stun to be added by creating a dummy unit and oredring it to use Storm Bolt on units.
Removed one trigger, added its function to another trigger. This solved a bug that caused the hammer sometimes to not slam to the ground.
Fixed a MUI issue on burn effects.
Removed one wait, it was pointless. Using a timer bound to the hammer now.

In the first image yeah, I did. In the second no.

 

[Mainy]

very nice 4/5

 

[aMazenSkillz]

looks awesome. keep up the good work

 

[Maker]

Updated to v1.06, now that I figured out a way to make the mana shield effect to disappear instantly.

The effect now disappears from sight as the hammer slams the ground, since the hammer releases enegies stored in it dealing damage to units withing range.

 

[Trototo]

Not bad ^^

 

[Lautanen]

It looks cool. I found one mistake in spell:

"After a while the hammer slams the ground, stunning and setting enemies on fire for three seconds."

Where´s the fire? You should edit the buff...

 

[Maker]

Thanks for reporting. It was a very simple bug.

v1.07
Better tooltip.
Units are now set on fire like they should be.

 

[xorkatoss]

wow dude
it's been a while since i saw such an awesome spell
but isn't the hammer of thor supposed to control lightning?
it would have been better i think to make it attack units with lightning
anyway this spell is awesome !! 5/5

 

[Kam]

Casting a spell based on Shockwave on top of these causes a fatal error every time, no matter how many are present.

 

[Maker]

Can you provide a test map and describe the situation with as many details as you can.

EDIT: Should be fixed now:

v1.08
All dummies now have Locust.
Fixed a bug that caused the dummies to not get cleared if their target was dead.

 

[Doom_AngeaL]

5/5; I couldn't find a bug or even a flaw. Nice work!

 

[Tomek320]

Wow ur spells are realy awesome but how to i fix leaks?

 

[Maker]

Wow ur spells are realy awesome but how to i fix leaks?

That really doesn't belong here. You should ask that in another thread

 

[Paillan]

what power do i use for this man?!

 

[Maker]

what power do i use for this man?!

What language would one use to interpret this question

Could you rephrase the question?

 

[Paillan]

ok ok them singh... how do i use this power in a map?
do you understoon now? ¬¬

 

[Maker]

ok ok them singh... how do i use this power in a map?
do you understoon now? ¬¬

http://www.hiveworkshop.com/forums/...8/importing-spell-problem-186177/#post1803404

 

[Paillan]

thanks but... where do i make a variable, i forgot it
9.9/10 good work, exelent, but i never gibe perfect notes

 

[Paillan]

and something,if i want to upload a map, it mast bee in english?

 

[xIceShotx]

and something,if i want to upload a map, it mast bee in english?

i dont think it has to but it would be recommended for people to understand better, but are you trying to translate that message (not the triggers in the future you will make)(The quote i just put)? or are you just saying it? just curious

 

[Mediumunwell]

Another beautiful spell. Hard to implement, though. I keep overlooking a variable or something I think. I'll figure it out, though, I think.

Edit: Oh, your instructions should do the trick . Thank you. Step 11 jumped out at me, and I definetly forgot it.

 

[Greenwhy]

Maker Why do you add the Locust ability to the HoT_dummy_unit when the unit already has this ability by default as per the object editor?

• Actions
  • -------- -------------------- --------
  • Unit - Create 1 Hammer_of_Thor_Dummy for Neutral Passive at Temp_Loc_1 facing (Random angle) degrees
  • Set Temp_Unit_2 = (Last created unit)
  • Custom script: call UnitAddAbility(udg_Temp_Unit_2, 'Aloc')
  • -------- -------------------- --------

 

[Maker]

It is a mistake. The action can be removed if the dummy has the ability in object editor.

 

[Greenwhy]

Ah ok. No probs, Just wasn't sure if I was missing something.

 

[Auildere]

http://prntscr.com/16jjwe

Is it i need to copy the dummy?

 

[Maker]

Yeah, copy the dummy.

You need to create a variable called Temp_Lightning

 

[Auildere]

oh thx

 

[Auildere]

did you seen the prtscreen link cuz that's my prob now.

 

[Himperion]

Hello, i'm trying to import this spell in my map, but i found a big problem in the "Lightning Move" Detonator.

http://imageshack.com/i/f1lgz6p

http://imageshack.com/i/0508p7p

 

[Maker]

Create a lightning variable called Temp_Lightning

 

[Himperion]

Ok i made the spell work. In my case was Temp_Real_3 the nonexistent variable. I copy and rename Temp_Real_2 xD.