Cataclysm v1.2 [IMPROVED TO THE MAX]

[Moderator]

12.12
IcemanBo: For long time as NeedsFix. Rejected.

Bribe

Set Int_Cataclysm_Player = (Owner of (Triggering unit)) -> (Triggering player)... and reference this variable instead of calling (Triggering player repeatedly)

(Casting unit) -> (Triggering unit)

(Key (Last created unit)) should be set to a variable as it involves three function calls: GetHandleIdBJ(GetLastCreatedUnit())

When possible, avoid using arrays. Array[2], for example, can be simplified into Variable2.

Maker, v1.2:
You save the point[1] into a hashtable but remove it at the end of the cast trigger. You load it in the looping trigger, but do not use it. Fix that.

 

[88WaRCraFT3]

E p i c.... 5/5, idk what to say
Set Int_Cataclysm_Player = (Owner of (Triggering unit)) change into Triggering Player.
and change all that owner of trig units.
In cataclysm loop you forget to add bj_wanttodestroyground something like that i forget

 

[defskull]

It's this:

• Custom script: set bj_wantDestroyGroup = true

Put this function above the Unit Group - Pick every unit... action in Cataclysm Loop

 

[x3GlikE]

E p i c.... 5/5, idk what to say
Set Int_Cataclysm_Player = (Owner of (Triggering unit)) change into Triggering Player.
and change all that owner of trig units.
In cataclysm loop you forget to add bj_wanttodestroyground something like that i forget

• Custom script: set bj_wantDestroyGroup = true

-Will do thanks,-

 

[Nightmare Moon]

In cataclysm loop you forget to add bj_wanttodestroyground something like that i forget

You mean bj_wantDestroyGroup= true =)
@x3GlikE, from the screenshot it looks like your dummies have hp bars, add locust ability. Wait, But if you add locust it'll make the rock walkable, so i suggest creating a pathing blocker where are rocks and moving blocker with rocks - no hp bars and rocks will be unwalkable...
Coding seems nice and understandable!

 

[defskull]

Btw, it IS set bj.....

 

[SpasMaster]

Give credits to League of Legends & Riot for the idea. =)

 

[Nightmare Moon]

Oh sorry then i wasn't testing
Edit: hold on man, defskull take a better look above the rocks...

 

[x3GlikE]

Give credits to League of Legends & Riot for the idea. =)

LOL riot ptff

 

[SpasMaster]

Here you go:

Jarvan IV heroically leaps to an enemy champion, dealing physical damage and creating a circle of impassable terrain around them for a 3.5 second duration. Jarvan IV can destroy the impassable terrain by activating this again. Champions can still leave or enter the area by using teleport abilities.

Source: Click

 

[x3GlikE]

Sif they copied my spell... I don,t play lol lol.
League of legends "Cataclysm" is LOL check out the tool-tip.

 

[SpasMaster]

Sif they copied my spell... I don,t play lol lol.

They copied the spell which was added like a month ago..
Yours is today. >.>

Anyways, I don't want to judge you or anything. Just say when you are copying a spell

 

[x3GlikE]

Lets not spam the thread now.

 

[Manasurge]

@ spasorc
With so many spells some of them are bound to be similiar.

 

[Marcos DAB]

Seems fine.

 

[xorkatoss]

this spell is epiiiiiiiiicc!! 5/5

offtopic:
you should play league of legends and try to make more of their spells ^.^

 

[Genji Lone]

Hmmm..... Reminds me of that one spell in LoL
OH sorry didn't read comments yeah that spell.

 

[Genji Lone]

You can go out of map with this spell

 

[Magtheridon96]

• -------- ------------------------------ --------
• -------- Do not touch --------
• Set Int_Cataclysm_Temppoint[2] = (Position of (Picked unit))
• Unit - Create 1 Int_Cataclysm_Unit[2] for Int_Cataclysm_Player at Int_Cataclysm_Temppoint[2] facing Default building facing degrees
• Unit - Set level of Cataclysm_Dummy for (Last created unit) to (1 x Int_Cataclysm_SpellLevel)
• Unit - Order (Last created unit) to Undead Banshee - Curse (Picked unit)
• Unit - Order (Last created unit) to Human Sorceress - Slow (Picked unit)
• Unit - Add a 0.50 second Generic expiration timer to (Last created unit)

Don't repeat (Last Created Unit)
Store it in a variable for use so you dont have to constantly call the function (Last Created Unit).

Same goes for ANY other functions you constantly use in the triggers.

Btw, your dummies have hp bars. Give them the locust ability.


All of these:

• Set JDA_JumpHigh_Distance = 1.00
• Set JDA_DestroyTrees_Dash = True
• Set JDA_Speed = 10.00
• Set JDA_SpecialEffect = Abilities\Spells\Other\Volcano\VolcanoMissile.mdl
• Set JDA_Animation = attack
• Set JDA_AnimationSpeed = 0.60

Along with other constant values should be set at map initialization.

You have to think about efficiency because it's one of the key factors
a spell/system needs to be approved.

 

[EloTheMan]

Jarvan IV Cataclysm

It needs a little more work, the stones got health bars and they move when the path is incorrect due to the units in the landing circle, so maybe you could move them out of the way before the stones are created.

Move them just a little bit out from the ring if they are outside the range and if they are inside you move them a little bit in to the center.

Also you could add a button after the ability has been cast which can be used to destroy the stones if you don't want them to block your way in case the enemy blinks out of the arena. Like the real Cataclysm spell

 

[x3GlikE]

If I add locust units can leave it, its not like a player would waste the spell and cast it on the edge of the map. Anyway I will try my best to improve the spell. Thank you all!

 

[Magtheridon96]

If I add locust units can leave it, its not like a player would waste the spell and cast it on the edge of the map. Anyway I will try my best to improve the spell. Thank you all!

You should either set the selection scale to 0 using WE hacks, or give them locust with 32-64 collision.
It might work

 

[defskull]

WE Hacks, WHAT ?
Just hold SHIFT + Double-click that "Art - Selection Scale" LOL
And put 0 in it ?

 

[x3GlikE]

You can't it goes to 0.10 -.-

 

[defskull]

1. HOLD SHIFT key
2. At the same time (while HOLDING SHIFT key), double-click that field "Art - Selection Scale"
3. Input "0" and press Enter
4. Argue with me more

 

[Adiktuz]

The idea is pretty nice... just do those optimizations that they're telling you... I'm a bit busy to actually read the code...

 

[88WaRCraFT3]

1. HOLD SHIFT key
2. At the same time (while HOLDING SHIFT key), double-click that field "Art - Selection Scale"
3. Input "0" and press Enter
4. Argue with me more

I Agree, and it works 100% but it needs to be -1 or numbers like that cause 0 wouldn't work on how much i know

 

[defskull]

0 works, please, try it :'(

 

[Adiktuz]

please enumerate the changes on the changelog rather than just putting "major improvement"...

 

[x3GlikE]

please enumerate the changes on the changelog rather than just putting "major improvement"...

LOL True I also made it so you can now not see the rocks hp and rocks dont stack ect same as some trigger fixes.

 

[Noobsmoke]

idk what happend, but after some casts i got 3 FPS ?? WFT ??

 

[Adiktuz]

most probably leaks... or just because of model spam...

and oh, please add a changelog which contains every change that you have made...

 

[x3GlikE]

idk what happend, but after some casts i got 3 FPS ?? WFT ??

It's cause you casted it in rows like cast 1 cast 2?

 

[mckill2009]

use Integer as ID...

example:

Set ID = Key (last Creted unit)

then in the loop trigger, reference it as...

Set ID = Key (picked unit)

this;

• Unit Group - Remove (Picked unit) from Int_Cataclysm_Dummy
• Hashtable - Clear all child hashtables of child (Key (ID)) in Int_Cataclysm_Hashtable

the ORDER should be last if you want this spell be used by AI...

• Unit - Add a 0.50 second Generic expiration timer to (Last created unit)
• Unit - Set level of Cataclysm_Dummy_slow for (Last created unit) to (1 x Int_Cataclysm_SpellLevel)
• Unit - Order (Last created unit) to Human Sorceress - Slow (Picked unit)

 

[Maker]

List only the stats of the current level in the tooltip. In the learn tooltip, you can list the stats of all levels.
1 x ?

• Unit - Set level of Cataclysm_Dummy_slow for (Last created unit) to (1 x Int_Cataclysm_SpellLevel)

You store the point...

• Hashtable - Save Handle OfInt_Cataclysm_Temppoint[1] as (Key point1) of (Key (Last created unit)) in Int_Cataclysm_Hashtable

...but remove it in the same trigger

• Custom script: call RemoveLocation(udg_Int_Cataclysm_Temppoint[1])

Then you load it in the loop trigger:

• Set Int_Cataclysm_Temppoint[1] = (Load (Key point1) of (Key (Picked unit)) in Int_Cataclysm_Hashtable)

That would return null since you removed it. But you're not using it for anything.
Change the speed to 0.

• Animation - Change JD_Unit[JD_Integers[3]] flying height to JD_JumpHigh[JD_Integers[3]] at 1000000000.00

The settings block in the cast trigger should be at the initialization trigger.

 

[Magtheridon96]

You have to set constant values at Map Initialization.
Please listen to me!

EDIT:

I'm going to make this easier for you.
See these variables:

• Set JDA_JumpHigh_Distance = 1.00
• Set JDA_DestroyTrees_Dash = True
• Set JDA_Speed = 10.00
• Set JDA_SpecialEffect = Abilities\Spells\Other\Volcano\VolcanoMissile.mdl
• Set Int_Cataclysm_Unit[1] = Rock
• Set Int_Cataclysm_Unit[2] = Cataclysm_Dummy2
• Set Int_Cataclysm_Duration = 10.00
• Set Int_Cataclysm_UnitsN = 15

Also, don't use integer A
Use Integer Variable since Integer A can cause some wierd bugs.

• For each (Integer A) from 1 to Int_Cataclysm_UnitsN, do (Actions)
  • Loop - Actions
    • Set Int_Cataclysm_Temppoint[2] = (Int_Cataclysm_Temppoint[1] offset by 350.00 towards ((360.00 / (Real(Int_Cataclysm_UnitsN))) x (Real((Integer A)))) degrees)
    • Unit - Create 1 Int_Cataclysm_Unit[1] for Int_Cataclysm_Player at Int_Cataclysm_Temppoint[2] facing Default building facing degrees
    • Unit - Add a Int_Cataclysm_Duration second Generic expiration timer to (Last created unit)
    • Special Effect - Create a special effect at Int_Cataclysm_Temppoint[2] using Abilities\Spells\Other\Volcano\VolcanoDeath.mdl
    • Special Effect - Destroy (Last created special effect)
    • -------- ------------------------------ --------
    • -------- Remove Leaks --------
    • Custom script: call RemoveLocation(udg_Int_Cataclysm_Temppoint[2])
    • -------- ------------------------------ --------
    • -------- ------------------------------ --------

One last comment, use custom script to save the Handle Id of the Last
Created Unit in an integer so you don't have to constantly call Key(Last Created Unit)

EDIT:

Another comment, don't use an array for the dummy units in the settings.
Just create 2 seperate variables.
Arrays can increase the size of a variable greatly (relatively), so they're best
used when storing a lot of data (maybe 5 instances is good)

 

[rechy89]

When i spam more than once, the rocks stay at the point and does not
die why? It mention duration 10s

 

[Furbolg]

very nice spell!!! but units mess up where the rocks form

 

[x3GlikE]

Hmm, I will check it out.

 

[Manasurge]

I don't find this issue in the triggers but when you cast multiple times in the map the spell bugs out and leaves the rocks

 

[x3GlikE]

I wonder why it happens..?

 

[RaijinSpark]

It also lags the second time I cast it.

And if the caster dies while those rocks. It won't be removed, it becomes permanent.

 

[Manasurge]

Cataclysm v1.2 BUG REPORT

Screenshot Provided of bug, perhaps someone more used to reviewing triggers may find the issue.

 

[xorkatoss]

Screenshot Provided of bug, perhaps someone more used to reviewing triggers may find the issue.

it's not a bug man, after 50 casts comes ultimate cataclysm which spawns rocks on the entire map!! xD

I don't find this issue in the triggers but when you cast multiple times in the map the spell bugs out and leaves the rocks

how to solve your problem:
set spell cooldown above 10 sec
it's not like you are going to set the cooldown of this ability to 0 in your map lol