Hammer of Protection

[deepstrasz]

Please remove the assets as they will be downloaded with the map separately.

 

[Noctiss_Season2]

Please remove the assets as they will be downloaded with the map separately.

thanks, never uploaded nothing so i dont even know how all that works :3

 

[Daffa]

An interesting spell that combines both defensive and offensive capabilities. Definitely not something that commonly came up into the mind
I personally find this spell pleasing and has potentials for a powerful Paladin class spell.

Here are some feedback to allow beginners a better time importing this spell:

• Put the following abilities into Ability Code variables:
  • Channel [Hammer of Protection] (the main ability)
  • Hardened Skin (the custom Hardened Skin ability)
  • Inner Fire
  • Channel [Hammer of Wrath] (the throw ability)
• Same goes to the Peasant dummy unit created for casting the Inner Fire
• Provide a step by step import manual from the test map to the user map, you can check other resources on how they provide this instruction. It can be inside the map OR on the description. This includes the ability object needs to be copied, which triggers, what to set up, and so on.
• At the current state, remember to inform the user this is currently can only be used by one hero at one time. This is important, since by default user expects a MUI spell (a spell that can be used by multiple units from multiple players)

Now, on to the spell itself:

• The first ability, on second cast onward, does not get hidden
• A configurable trigger is missing, this is a trigger designed to allow users to modify values without looking through all the triggers. You can check approved spells on examples of configurables.
• • Unit - Create 1 . Peasant for (Owner of HammerCaster) at (HammerCasterTempPos offset by 125.00 towards 90.00 degrees.) facing Default building facing degrees
  This particular action leaks a point since Point with Polar Offset creates a new point, so you need to create the point with another variable before using it on this action.
• Lastly, I hope you consider looking into one of the following to get this spell into MUI:
  • Visualize: Dynamic Indexing
  • GUI Unit Indexer 1.4.0.0
  • GUI Spell System v1.8.0.0
  • Hashtables and MUI

Of course there are more, but I think these are good starting points to cover

 

[Noctiss_Season2]

An interesting spell that combines both defensive and offensive capabilities. Definitely not something that commonly came up into the mind
I personally find this spell pleasing and has potentials for a powerful Paladin class spell.

Here are some feedback to allow beginners a better time importing this spell:

• Put the following abilities into Ability Code variables:
  • Channel [Hammer of Protection] (the main ability)
  • Hardened Skin (the custom Hardened Skin ability)
  • Inner Fire
  • Channel [Hammer of Wrath] (the throw ability)
• Same goes to the Peasant dummy unit created for casting the Inner Fire
• Provide a step by step import manual from the test map to the user map, you can check other resources on how they provide this instruction. It can be inside the map OR on the description. This includes the ability object needs to be copied, which triggers, what to set up, and so on.
• At the current state, remember to inform the user this is currently can only be used by one hero at one time. This is important, since by default user expects a MUI spell (a spell that can be used by multiple units from multiple players)

Now, on to the spell itself:

• The first ability, on second cast onward, does not get hidden
• A configurable trigger is missing, this is a trigger designed to allow users to modify values without looking through all the triggers. You can check approved spells on examples of configurables.
• • Unit - Create 1 . Peasant for (Owner of HammerCaster) at (HammerCasterTempPos offset by 125.00 towards 90.00 degrees.) facing Default building facing degrees
  This particular action leaks a point since Point with Polar Offset creates a new point, so you need to create the point with another variable before using it on this action.
• Lastly, I hope you consider looking into one of the following to get this spell into MUI:
  • Visualize: Dynamic Indexing
  • GUI Unit Indexer 1.4.0.0
  • GUI Spell System v1.8.0.0
  • Hashtables and MUI

Of course there are more, but I think these are good starting points to cover

Thank you very much for the reply/feedback. It made me happy to see you find the spell interesting.

First of all, sorry about the leak, i thought i got all of them, seems i forgor one. So that is fixed. About the spell being usable only by 1 single unit, yes. I made the spell for a dungeoner map, something like sunken city, where the spell is unique to that 1 unit. It was meant to be a unique unit cast, but that is mostly because of the reason i stated later in this reply.
I added some more writing about how to import this ability to a comment trigger.
About the first ability not being hidden on second cast and onward, i genuinely don't know why that happens. I tried it on a test map, and a couple of times it works and gets hidden each time, and then some it doesn't. I tried to observe a pattern or something, but it seems to be purely random. If that is some kind of a hardcoded "unhidden abilities cannot be rehidden" kind of issue, then i guess its just gonna have to be a "throw spell appears during duration" on button position 1:1 or something. Bottom line i have no clue how to fix that

Now for the biggest roadblock of making this ability pass with flying colors: my triggering skills. As i said previously, i made this spell for my map with triggers within my knowledge and capability, and a friend said hey, cool spell, upload that to hive, at worst it gets deleted or a simple rating. And i honestly didnt expect much else. All of the things you suggested, unit indexer and on, are things I'm aware of existing and i understand that they could improve this spell to be bug free and also usable my many units at the same time. The only reason i didn't implement anything like that is simply... because i didnt know how. My triggering skills hit a big roadblock there. I have some knowledge about triggers, i made some buttons, some UI stuff, and i can make a buncha interesting stuff, but its usually scuffed or kind of inefficient, on behalf of my lack of knowledge. So for that reason, at this time, i cannot upgrade this spell from a unique cast to a indexed cast. Perhaps after my triggering skills grow more, or if i find someone on the hive discord with whomstve help we can take a crack at it, but for now that is in the future. As of this moment, my triggering skills hit a roadblock and im unnable to convert this spell from unique to index. Im honestly kinda happy to even see it work in the first place

 

[Antares]

I like the comments you put throughout your code, but explaining why you have to store locations is really not necessary.

This is a quite nice-looking spell, but like Daffa already pointed out, there are some problems with its implementation. The activation ability does not get hidden on the second cast and every cast after that. Our submission rules specify that a spell must be MUI or MPI to be accepted. I'm sure the people in the World Editor Help Zone can help you with getting your spell to be MUI.

Awaiting Update