Tame Pet v1.0

[Moderator]

12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.

Maker, Tame Pet v1.0, 3rd Jan

Your display to player group actions leak player groups (forces) Add importing instructions into the map file Pet Dies trigger should not be on all the time Neutral hostile units can't be targeted with the spell Abilities must support leveling up

 

[Magtheridon96]

Tips:
- Use (Triggering unit) instead of (Casting unit) cause (Triggering unit) is faster.

-

• Game - Display to (Player group((Owner of (Casting unit)))) for 10.00 seconds the text: (Game_Time_String + ((Name of (Target unit of ability being cast)) + is now under your control.))

This leaks a Player group (Set that Player group in a variable and destroy it with
the function DestroyForce. Or, since you're only referring to one player, you can
use the global bj_FORCE_PLAYER which is a player group array that has one force
per player. You won't even need to destroy it :>

- (Triggering player) is faster than (Owner of (Triggering unit))

- Cache things so you don't have to repeat the functions over and over again
(Because caching is what makes the world go around)

Thus:

Triggers

• -------- --------------------------------------------- --------
• -------- Sets Variables --------
• -------- --------------------------------------------- --------
• Set Beast_Type[(Player number of (Owner of (Casting unit)))] = (Unit-type of (Target unit of ability being cast))
• Set Beast[(Player number of (Owner of (Casting unit)))] = (Target unit of ability being cast)
• Set Unit_Count = 1
• Unit - Change ownership of (Target unit of ability being cast) to (Owner of (Casting unit)) and Change color
• -------- --------------------------------------------- --------
• -------- Adds Consume Corpse Ability and Release Pet Ability --------
• -------- --------------------------------------------- --------
• Unit - Add Release Pet to (Target unit of ability being cast)
• Unit - Add Consume Corpse to (Target unit of ability being cast)
• -------- --------------------------------------------- --------
• Game - Display to (Player group((Owner of (Casting unit)))) for 10.00 seconds the text: (Game_Time_String + ((Name of (Target unit of ability being cast)) + is now under your control.))
• -------- --------------------------------------------- --------
• Unit - Remove BT_Dummy_1[(Player number of (Owner of (Casting unit)))] from the game
• Unit - Remove BT_Dummy_2[(Player number of (Owner of (Casting unit)))] from the game

->

• Set TempUnit = (Triggering unit)
• Set TempUnit2 = (Target unit of ability being cast)
• Set TempPlayer = (Triggering player)
• Set TempInt = (Player number of TempPlayer)
• -------- --------------------------------------------- --------
• -------- Sets Variables --------
• -------- --------------------------------------------- --------
• Set Beast_Type[TempInt] = (Unit-type of TempUnit2)
• Set Beast[TempInt] = TempUnit2
• Set Unit_Count = 1
• Unit - Change ownership of TempUnit2 to TempPlayer and Change color
• -------- --------------------------------------------- --------
• -------- Adds Consume Corpse Ability and Release Pet Ability --------
• -------- --------------------------------------------- --------
• Unit - Add Release Pet to TempUnit2
• Unit - Add Consume Corpse to TempUnit2
• -------- --------------------------------------------- --------
• Custom script: set udg_TempForce = bj_FORCE_PLAYER[udg_TempInt - 1]
• Game - Display to TempForce for 10.00 seconds the text: (Game_Time_String + ((Name of TempUnit2) + is now under your control.))
• -------- --------------------------------------------- --------
• Unit - Remove BT_Dummy_1[TempInt] from the game
• Unit - Remove BT_Dummy_2[TempInt] from the game

That should be way more efficient :3

edit

Same thing goes for the second trigger:

- (Casting unit) -> (Triggering unit)
- Fix the player group leaks
- Cache all those functions like (Triggering unit), (Player number of (Triggering unit)), etc..
- (Owner of (Casting unit)) -> (Triggering player)

 

[RakEnRoL]

Hello there. I would like to rate this spell of yours. First of all, it is not that unique. It might be a good idea, to have chances in taming the beast. Because you can tame any unit. I could have used the "Charm" spell and increase its casting time to get the same effect. So, for what I see.. It is so simple.

Rating: 2/5
Not that unique
Lacks creativity
Lacks configurables
Easy and Simple spell
Not MUI and Leaks

 

[Midnighters]

I have fixed the leaks and MUI, and in the upcoming updates, I am going to add more to it, such as chances of taming, the levels of monsters you can tame at specific levels, and the like.

 

[RakEnRoL]

I have fixed the leaks and MUI, and in the upcoming updates, I am going to add more to it, such as chances of taming, the levels of monsters you can tame at specific levels, and the like.

Good to see that you are making an effort and giving it time to make the spell better. +Rep

 

[Midnighters]

=D Thanks man, I am also planning to make your pet auto-follow you, and auto-attack threats. There will be a special bit added to this system just for my map, I am not going to spoil it =]

 

[RakEnRoL]

Wow, that would be very interesting and nice at the same time. I would love to see that because pet systems are quite rare. Good luck.

 

[Midnighters]

I'm trying for it.

 

[InfinateAnswers]

Midnighters, one problem, the object editor spell isnt letting you target anything. You should fix that lol.