Making Invisible

[Rad]

You used (Triggering Player) Whereas the player is not triggering, it is the unit. Change that to Owner of (Triggering unit) And i suggest you start to get the hang on the (Casting unit) etc, i dont know why this site doesnt believe in anything except triggering unit but later on you will be SORRY!

 

[Kitt]

Hah, woops ^^; Also, i'm switching to your syntax, it looks a little cleaner.

as for triggering unit, i usally just forget there are other ways, lol. But yes, i agree, casting unit will be much better.

I'll fix things up and report back ina bit.

Meow.

 

[Sargon]

i dont know why this site doesnt believe in anything except triggering unit

We don't? I, for one, believe that Triggering whatever should only be used if there's no other event response thing that fits.

 

[Rad]

More power to you every time i suggest they change triggering unit someone always complains "It doesn't matter" Which it does for example if you use circle of power hero selection, entering units, last created units, etc.. Triggering unit could mess you up. I think its better to teach them ASAP

 

[Kitt]

Haha, no i do a lot of programming and i know that the more specific you are, the less chance of an unexpected error. It's well worth the time to spend a few extra seconds looking for a closer match, considering you could spend hours trying to fix an unseen error.

Alright, for anyone interested in this skill, simply add the following things:

A) I used Fan of knifes as a the base ability and took out the damage and stuff, then changed the effect to frost nova<effect>

B) add the following in your triggers:

MakeInvisible
    Events
        Unit - A unit Begins casting an ability
    Conditions
        (Ability being cast) Equal to Ghost Blanket 
    Actions
        Unit Group - Pick every unit in (Units within 500.00 of (Position of (Casting unit))) and do (Actions)
            Loop - Actions
                If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                    If - Conditions
                        (Picked unit) Not equal to (Triggering unit)
                        (Owner of (Picked unit)) Equal to (Owner of (Casting unit))
                    Then - Actions
                        Unit - Add Ghost to (Picked unit)
                        Set TotalUnitsAffected = (TotalUnitsAffected + 1)
                        Set UnitsAffected[TotalUnitsAffected] = (Picked unit)
                    Else - Actions
                        Do nothing
        Wait 20.00 seconds
        For each (Integer UnitsAffectedLoop) from 1 to TotalUnitsAffected, do (Actions)
            Loop - Actions
                Unit - Remove Ghost from UnitsAffected[UnitsAffectedLoop]

you can change the wait to whatever you want, the range of effect can be changed also. if you've got any questions, just ask^^

 

[Sargon]

Waits are annoying. I believe there's a better solution than waits. Can't remember what it is though.

 

[Rad]

So you finally got it working? Thats good to here! Some problems with the wait trigger... if more than 1 unit has this ability it will reset the unit groups then the rest would be stuck with ghost (which would be fun) Also if u do any replace unit actions then the new unit will either be stuck with it or automatically lose it...

 

[Sanitarium]

Better solution than waits...JASS!!
And seeing as your a programmer, I'd suggest learning it. It's well worth it and not too much different from most other languages.

 

[Kitt]

Hmm... those are issues. I should also reset the TotalUnitsAffected variable, that could cause problem XD

Any idea how to address those?

 

[Rad]

Yep... there it goes... my brain has stopped

 

[Kitt]

lol, alright^^ well it should work for now, i'll just keep those errors in mind. If you get anymore idea's feel free to post back^^

Mreow!

 

[Rad]

Ok i gave the hamster in my head a pellet so its running again... lol

Heres a few ways you can fix those:
A: Make it so only 1 hero has that ability and only 1 of that hero allowed on the map (Or convert all those actions to arrays per player if you wish to have 1 per player)
B: On each Unit - Replace with unit you should add an if/then/else checking if any unit in the unit group is equal to the to-be-replaced unit, if it is then replace unit THEN add last replaced to the unit group again

 

[Heptameron]

MakeInvisible
    Events
        Unit - A unit Begins casting an ability
    Conditions
        (Ability being cast) Equal to Ghost Blanket 
    Actions
        Unit Group - Pick every unit in (Units within 500.00 of (Position of (Casting unit))) and do (Actions)
            Loop - Actions
                If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                    If - Conditions
                        (Picked unit) Not equal to (Triggering unit)
                        (Owner of (Picked unit)) Equal to (Owner of (Casting unit))
                    Then - Actions
                        Unit - Add Ghost to (Picked unit)
                        Set TotalUnitsAffected = (TotalUnitsAffected + 1)
                        Set UnitsAffected[TotalUnitsAffected] = (Picked unit)
                    Else - Actions
                        Do nothing
        Wait 20.00 seconds
        For each (Integer UnitsAffectedLoop) from 1 to TotalUnitsAffected, do (Actions)
            Loop - Actions
                Unit - Remove Ghost from UnitsAffected[UnitsAffectedLoop]

I suggest you use a unit group instead of an array of units in this case.

Also, you do know you won't be able to cast this spell twice in 20 seconds, right?

 

[Kitt]

yeah, the spell delay is larger than 20 seconds, it's about 40, so a quick person would fight half the battle invisible.

 

[Kitt]

Axtually i'm having another error with this ability and another. for some reason, if the hero has both abilities, if you try to sue either one, it uses both. it casts them both, and triggers them both. here's the other bilities code, any diea's?

Decoy
    Events
        Unit - A unit Starts the effect of an ability
    Conditions
        (Ability being cast) Equal to Decoy 
    Actions
        Unit - Create 1 (Unit-type of (Casting unit)) for (Owner of (Casting unit)) at (Position of (Casting unit)) facing Default building facing degrees
        Unit - Change color of (Last created unit) to (Color of (Triggering player))
        Unit - Turn collision for (Casting unit) Off
        Unit - Order (Casting unit) to Move To (Random point in (Current camera bounds))
        Unit - Add a 17.00 second Generic expiration timer to (Last created unit)
        Unit - Add Ghost to (Casting unit)
        Wait 3.00 seconds
        Unit - Turn collision for (Casting unit) On
        Wait 14.00 seconds
        Unit - Order (Casting unit) to Stop
        Unit - Remove Ghost from (Casting unit)

 

[MaltTaquila]

if u want invis u can just base it off permanent invisiblility and set the fade time to 0.00. no trigs needed happy to help ya!

 

[Heptameron]

if u want invis u can just base it off permanent invisiblility and set the fade time to 0.00. no trigs needed happy to help ya!

...he did that.

But, if you read the thread, um, you'll see he didn't want it invisible permanently. Therefore he needed triggers.

The problem you've having now might be the fact that you have two abilites based off the same one on one unit, which is a no-no.

 

[Xoator]

ya u lost me to

 

[Vexorian]

Cloaking like invisibility? Needs 2 abilities: Permanent Invisibility + Ghost (visible)

Unbelievers might want to take a look at the cloaking field ability: http://vexorian.wc3campaigns.com/spells/moretemplates2.w3x