Making a spell in vJASS - Practice Session 1

[Cheezeman]

Anyway, Timers ... do what do you think of ... Mmmmm a spell like Ranger arrow from Dota?
xD

I don't play dota!

 

[Darkness-4ever]

I don't play dota!

Nor do I.

 

[Flame_Phoenix]

I don't play dota!

Nor do I.

Then you both have good taste =P

 

[Batopi]

On the contrary Shanghai, I found the tutorial to be just the right length. I know I'm ressurecting this post but;

Phoenix, I liked how you went into detail into some of the topics, but I would have liked for you to cover the meanings of some of the things inside your Init Function. For instance,

local trigger InstantJusticeTrg = CreateTrigger()        
call TriggerRegisterAnyUnitEventBJ(InstantJusticeTrg, EVENT_PLAYER_UNIT_SPELL_EFFECT ) 
call TriggerAddCondition(InstantJusticeTrg, Condition( function Conditions ) )        
call TriggerAddAction( InstantJusticeTrg, function Actions )

Over all I really liked your tutorial, there were a few grammatical mistakes but I over looked those. As a C++ programmer for two years, the languages look similar so that might have been why I understood most of the meanings.

+Rep, a very good tutorial for vJass.

 

[bigapple90]

This tutorial is not long, finished in like 20 minutes while learning
Thanks for tut + rep

 

[Flame_Phoenix]

Thx for the rep++ guys, it's good to know hard work is appreciated.

local trigger InstantJusticeTrg = CreateTrigger()
call TriggerRegisterAnyUnitEventBJ(InstantJusticeTrg, EVENT_PLAYER_UNIT_SPELL_EFFECT )
call TriggerAddCondition(InstantJusticeTrg, Condition( function Conditions ) )
call TriggerAddAction( InstantJusticeTrg, function Actions )

Ok, you see, a when you click new trigger button, you are NOT really creating a trigger, you are instead creating a document of text, or a project (in C++ if you use Eclipse, or IDE::Blocks). Each text document can contain several triggers inside it. Triggers control what happens when a player does an action. Triggers fire on certain events. In this case, every time a player casts ANY spell, it will fire this trigger, and it will run the code you made.
However we don't want our code to run all the time, this is why we have a function called Conditions. If the conditions are met, then the trigger will move on to actions and it will execute the rest of the code, else the trigger will just stop there and nothing else will happen.
Conditions and Actions are not obligatory inside triggers, but they are a good way of dividing a problem and solving it.

I hope you understand this small explanation among with my other analogies.

 

[Vulcano]

Nice tutorial, it was my first vJass tutorial and i could understand everything.

Btw, why the hate against dota. I often think people are just jealous that they haven't got a map that is so popular. Ok, i must admit some spells suck hard (like Return skill of centaur), but in overall the gameplay is better than wow IMO.

 

[Cheezeman]

Nice tutorial, it was my first vJass tutorial and i could understand everything.

Btw, why the hate against dota. I often think people are just jealous that they haven't got a map that is so popular. Ok, i must admit some spells suck hard (like Return skill of centaur), but in overall the gameplay is better than wow IMO.

Mostly because most map makers finds it boring to play.
Personally I can't stand how the players treat new people ("[All] Oh noes! One player on my team is newbie!" *ragequit*), but that might not be the case for everyone

 

[Flame_Phoenix]

Btw, why the hate against dota.

Because even this simple spell is better then half Dota spells. Morale ? Dota doesn't have enough quality for the reputation it has. Dota is popular because gamers are ignorants that can't distinguish a good quality spell from a rip-off spell.

I now officially say I will start on a new tutorial. I will start it on day 20 July and I intend to cover the basis of structures and timers as well as modularity. Wish me luck people !

 

[Vulcano]

Any way , i don't know any other maps with more than 360 spells.
I usually don't insult new players, except when i tell them what they do wrong and then they keep on doing the same wrong thing :s.

EDIT: gl on tutorial

 

[Cheezeman]

I now officially say I will start on a new tutorial. I will start it on day 20 July and I intend to cover the basis of structures and timers as well as modularity. Wish me luck people !

Oh my f**king God. I've been waiting for this since... early april?
Only 3 weeks to go for the start of the new tutorial.

 

[Flame_Phoenix]

Any way , i don't know any other maps with more than 360 spells.

Then check my project xD

Thx for the comments btw =P

 

[Anachron]

Really like this tutorial. Problem: It does not introduce to structs, while they are the best to use at making spells. The whole tutorial is very well written, and I will dl it surely.

 

[Flame_Phoenix]

I don't get your point, structs rock for making spells.
As for the rest, it is not a problem. Struct usage is meant to be covered for lesson 2. In lesson 2 I will also introduce the user to timers as well, and I plan to teach him a few tricks with timers.
I may also give a "little smell" about modularity =P

However lesson 2 will take it's time, I will only start making it after day 17 July, the date of (hopefully) my last exam.

Still, thx for your comment =D

 

[onisaiyan808]

I have a question. Is the Init function where the Event is?

 

[Cheezeman]

I have a question. Is the Init function where the Event is?

Yep, kind of. InitTrig_ can be compared to Events in GUI, just that you can do alot more with InitTrig_.

 

[Flame_Phoenix]

Yep, kind of. InitTrig_ can be compared to Events in GUI, just that you can do alot more with InitTrig_.

Good answer. rep++

I am working on my second tutorial BUT ... I can't find a decent program to write it ... does anyone know a better program then notepad or notepad++ that indicates when the page laterals is over?

 

[Vulcano]

i don't know what page laterals are (english is not my main language). But u can try OpenOffice.org (almost same as office but it's free)

 

[Flame_Phoenix]

I know about open office... however it is a main distro for Linux and not Windows =S

 

[Vulcano]

it works fine on my windows vista

 

[FRAN]

nice tuto... i like it... T_T

 

[You Win]

First of all thanks for the tutorial.
Question:
I don't understand how it is MUI when you put it as global for non-constant variables. Can you explain that? thanks

 

[Flame_Phoenix]

MUI -> Multi unit instanceable. It basicaly means that your spell can be cast by more than 1 unit at the same time without blowing your computer.

As for the global, happens that the wc3 engine runs using something called threads. A thread is a very low level mechanism that can execute an action or part of an action. Wc3 engine can not run 2 threads at the same time.
Because we are using the globals groups instantly (meaning we don't use them over time) wc3 engine will run the code 1 thread at a time, thus keeping your spell MUI.
A simple way of understand what happens that might work is this: You can use global variables such as groups to make your spell MUI as long as you do NOT involve those global variables in ANY time depending mechanisms such as waits. I am not using any time mechanism associated to the global variable, so it is MUI. Also, notice that I try to do all my group code very quickly and very close. All code is imperative and I have no useless instructions in the middle, this also helps.

I hope you understand why the spell is MUI.

 

[You Win]

Oh ok... thanks for the explanation, I get what you mean =)

But what if I wanted to use Sleep function? I will have to use the local way which will cause a lot of lags? Is there any alternative way of doing this which will reduce the lag and leak?

 

[Flame_Phoenix]

If you use a Sleep function (that is a wait) and a global variable your spell will not be MUI.The problem with sleeps is that they are inaccurate and cause other problems (i can post a link to a thread in wc3c about that if you need). You can use timers instead of TriggerSleepAction, however they are harder to use for most new coders.

 

[You Win]

As far as I know if we use timer, we will be needing another event in another trigger right?

 

[Cheezeman]

The Magic of Jass: CreateTimer(), StartTimer() and GetExpiredTimer()
With these you'll need only one event which starts a timer, rather than resumes a timer-event.
I think Phoenix can help you further with them if you got questions

 

[Flame_Phoenix]

to use timers... timers are complex, if you want to use them you will need to use an outisde system and make your spell dependant on that system. I recommend the TimerUtils. It has 3 flavors and you can pick the one you find best.

I was going to make a 2nd tutorial about timers ... but unfortunately it was not good enough and I never submitted it.

 

[You Win]

Well at first I was planning on using a loop with a sleep function inside which will make a timer. Can you please show me an example by making a skill using timer also? When you are free that is.

Anyway, I notice that normal WE can't run vJass, and the programs you provide is it the latest version? As in 1.24b?

 

[Cheezeman]

v5d is the latest as far as I know. Wc3c is a respected site and if you wish to extend your jass knowledge I recommend it over hive.

 

[Flame_Phoenix]

To run vJass you need a tool called JassNewGeneration Pack. You can download it over here:
http://www.wc3c.net/showthread.php?t=90999

As for timers, I am a retired coder and so I don't code any longer (I forgot to update my subtitle from "Map maker" to "Retired coder").
If you wish to use timers, you can see my submitted spells, they all use timers over waits and they are not hard to understand.

As for wc3c, Cheezman is right. However don't expect to be well treated, people there are usually very cold. If you have enough patience and will to learn you will make it there, however if you cannot receive criticism i recommend you stay at THW.

 

[You Win]

Hmm.. I have programming basics but not familiar with Jass' syntax. Do you guys think is ok for me to go straight for vJass or should i learn Jass first before i start learning vJass. Cause from what I see here, vJass and Jass not much different just the way you write the syntax is a bit different only.

 

[Cheezeman]

Let me quote Vexorian on that:
JassHelper: Turns your simple code into something that is complicated enough to work.

vJass can't do anything Jass2 can't, but it's so much easier. I'd totally recommend it, even if you're not going to utilize all of its features (I myself only use free global declaration and scopes).

 

[Ice Snake]

Thank Flame_Phoenix, good tutorial.

 

[Anachron]

vJass can't do anything Jass2 can't, but it's so much easier. I'd totally recommend it, even if you're not going to utilize all of its features (I myself only use free global declaration and scopes).

Wrong? vJass supports structs, which is not supported on JASS2. Its basically converted back to JASS2, but its features get lost.

 

[Flame_Phoenix]

Wrong? vJass supports structs, which is not supported on JASS2. Its basically converted back to JASS2, but its features get lost.

Meaning that JASS CAN but in a much harder, insane, and ridiculously not modular way.
We all agree vJass > JASS ;P

 

[Jak-The-Ripper]

//===========================================================================
// I thought it was spelled effect, not effetc?
//A vJASS and JESP spell that will have an effetc depending on the number
//of units in the AOE. If there are more allies than enemies, we heal allies
//if there are more enemies that allies we damage enemies, if the number of
//allies equals the number of enemies we heal the allies and damage the enemies
//
//@author Flame_Phoenix
//
//@credits
//- Blade.dk, for the CopyCroup function
//
//@version 1.0
//===========================================================================
scope InstantJustice initializer Init
//===========================================================================
//=============================SETUP START===================================
//===========================================================================
// Why such long names? SPELL_ID is fine, but why use underscores? DummyID works fine.  I guess I`m a speed freak.
    globals
        private constant integer SPELL_ID = 'ANsi' //the rawcode of the spell
        private constant integer DUMMY_ID = 'h000' //rw of the dummy unit
        private constant string AOE_EFFECT = "Units\\NightElf\\Wisp\\WispExplode.mdl" //effect that will be created when we cast the spell on the AOE
        private constant string HEAL_EFFECT = "Abilities\\Spells\\Human\\HolyBolt\\HolyBoltSpecialArt.mdl" //effect that will be created when we heal units
        private constant string DAMAGE_EFFECT = "Abilities\\Weapons\\Bolt\\BoltImpact.mdl" //effect that will be created when we damage units
        private constant damagetype D_TYPE = DAMAGE_TYPE_NORMAL //the attack type of the spell
        private constant attacktype A_TYPE = ATTACK_TYPE_MAGIC //the damage type of the spell
    endglobals

    private function Range takes integer level returns real
    //returns the range the spell will affect
        return level * 150.
    endfunction

    private function Heal takes integer level returns real
    //returns the heal allies will take
        return level * 75.
    endfunction

    private function Damage takes integer level returns real
    //returns the damage enemies will take
        return level * 50.
    endfunction

    private function Targets takes unit target returns boolean
    //the units the spell will affect
        return (GetWidgetLife(target) > 0.405) and (IsUnitType(target, UNIT_TYPE_STRUCTURE) == false) and (IsUnitType(target, UNIT_TYPE_MAGIC_IMMUNE) == false) and (IsUnitType(target, UNIT_TYPE_MECHANICAL) == false)
    endfunction
//===========================================================================
//=============================SETUP END=====================================
//===========================================================================
    globals
        private group all
        private group copy
        private boolexpr b
    endglobals
//===========================================================================
//Function made by Blade.dk; Search for [url=http://www.wc3campaigns.com]wc3campaigns.com[/url] for more info
    private function CopyGroup takes group g returns group
        set bj_groupAddGroupDest = CreateGroup()
        call ForGroup(g, function GroupAddGroupEnum)
        return bj_groupAddGroupDest
    endfunction
//===========================================================================
    private function Pick takes nothing returns boolean
        return Targets(GetFilterUnit())
    endfunction
//===========================================================================
    private function Conditions takes nothing returns boolean
        return GetSpellAbilityId() == SPELL_ID
    endfunction
//===========================================================================
// These variables are too long.  You need to shorten them, it kills me.
    private function Actions takes nothing returns nothing
        local location spellLoc = GetSpellTargetLoc()
        local real spellX = GetLocationX(spellLoc)
        local real spellY = GetLocationY(spellLoc)
        local unit caster = GetTriggerUnit()
        local integer level = GetUnitAbilityLevel(caster, SPELL_ID)
        local unit f
        local integer allies = 0
        local integer enemies = 0

        //create the AOE effect
        call DestroyEffect(AddSpecialEffect(AOE_EFFECT, spellX, spellY))

        //counting the units
        call GroupEnumUnitsInRange(all, spellX, spellY, Range(level), b)
        set copy = CopyGroup(all)
        loop
            set f = FirstOfGroup(copy)
            exitwhen(f == null)
            call GroupRemoveUnit(copy, f)
            if IsUnitAlly(f, GetOwningPlayer(caster)) then
                set allies = allies + 1
            else
                set enemies = enemies + 1
            endif
        endloop

        //making the effect of the spell
        if allies > enemies then


            loop
                set f = FirstOfGroup(all)
                exitwhen (f == null)
                call GroupRemoveUnit(all, f)
                //heal allies
                if IsUnitAlly(f, GetOwningPlayer(caster)) then
                    call DestroyEffect(AddSpecialEffectTarget(HEAL_EFFECT, f, "origin")) //the healing effect
                    call SetWidgetLife(f, GetWidgetLife(f) + Heal(level))
                endif
            endloop
        elseif enemies > allies then
            loop
                set f = FirstOfGroup(all)
                exitwhen (f == null)
                call GroupRemoveUnit(all, f)
                //damage enemies
                if IsUnitEnemy(f, GetOwningPlayer(caster)) then
                    call DestroyEffect(AddSpecialEffectTarget(DAMAGE_EFFECT, f, "origin")) //the damaging effect
                    call UnitDamageTarget(caster, f, Damage(level), true, false, A_TYPE, D_TYPE, null)
                endif
            endloop
        elseif allies == enemies then
            loop
                set f = FirstOfGroup(all)
                exitwhen (f == null)
                call GroupRemoveUnit(all, f)
                //heal allies
                if IsUnitAlly(f, GetOwningPlayer(caster)) then
                    call DestroyEffect(AddSpecialEffectTarget(HEAL_EFFECT, f, "origin"))
                    call SetWidgetLife(f, GetWidgetLife(f) + Heal(level))
                //if an unit is not an ally than it is an enemy
                else
                    call DestroyEffect(AddSpecialEffectTarget(DAMAGE_EFFECT, f, "origin"))
                    call UnitDamageTarget(caster, f, Damage(level), true, false, A_TYPE, D_TYPE, null)
                endif
            endloop
        endif

        call RemoveLocation(spellLoc)
        set spellLoc = null
        set caster = null
    endfunction
//===========================================================================
    private function Init takes nothing returns nothing
        local trigger InstantJusticeTrg = CreateTrigger()
        call TriggerRegisterAnyUnitEventBJ(InstantJusticeTrg, EVENT_PLAYER_UNIT_SPELL_EFFECT )
        call TriggerAddCondition(InstantJusticeTrg, Condition( function Conditions ) )
        call TriggerAddAction( InstantJusticeTrg, function Actions )

        //setting globals
        set all = CreateGroup()
        set copy = CreateGroup()
        set b = Condition(function Pick)

        //preloading effects
        call Preload(AOE_EFFECT)
        call Preload(HEAL_EFFECT)
        call Preload(DAMAGE_EFFECT)

        //preloading the ability
// BJ...? I didn`t look that up, but I think it has a native replacement.
        set bj_lastCreatedUnit = CreateUnit(Player(PLAYER_NEUTRAL_PASSIVE), DUMMY_ID, 0, 0, 0)
        call UnitAddAbility(bj_lastCreatedUnit, SPELL_ID)
        call KillUnit(bj_lastCreatedUnit)

    endfunction
endscope

You also rely on a system in a tutorial. Why rely on a system when you are teaching? This isn`t a very good spell to teach, tbh.

 

[Arisato]

nah, this is the BEST OVER THE BEST BEST BEST ULTIMATIVELY EXTREMELY

GOOD TUTORIAL!!!!!!!!!!!!

 

[Flame_Phoenix]

You also rely on a system in a tutorial. Why rely on a system when you are teaching? This isn`t a very good spell to teach, tbh.

Because teaching modularity is also a good thing to do, and a good thing for you to learn.The great problem in THW is because of people like you that refuse to be modular and to follow community standards. At the time this tutorial was written, it compiled to all known standards of the coding community.

@Uzumaki.Minato: Thx

 

[Venusblood]

Nice tutorial

I have just figured out Jass recently and now I find making skills in vjass is more interesting.I have a knowledge in c and a little in c++.but I just found to check out all the funtion names in programming jass is terrible.However,expecting your tutorial on timer(i did not find one good tutorial on this one in hive)

 

[Vulcano]

Flame Phoenix isn't writing that tutorial about timers, he said in 2009 he would write it during summer. But he still didn't write it, so he'll probably never write it.

In case you want even more c syntax in JASS you can also use cJASS (google it)
example:

JASS:
function funcName takes type1 varName returns type2
    statements
endfunction

//would be in cJASS:
type2 funcName(type1 varName)
{
    statements
}

 

[Vulcano]

Flame Phoenix isn't writing that tutorial about timers, he said in 2009 he would write it during summer. But he still didn't write it, so he'll probably never write it.

In case you want even more c syntax in JASS you can also use cJASS (google it)
example:

JASS:
function funcName takes type1 varName returns type2
    statements
    return bla
endfunction

//would be in cJASS:
type2 funcName(type1 varName)
{
    statements
    return bla
}

 

[Flame_Phoenix]

Yes, I was supposed to write another tutorial, however I didn't had as much time as I expected neither the support I needed. I started making the timer tutorial but it simply wasn't good enough and so I decided to not post it.

Vulcano, perhaps you are talking about ZINC ?

 

[Jak-The-Ripper]

cJASS has a Turin Syntax, like JASS does.

ZINC has a C/++/# syntax.

 

[Flame_Phoenix]

I don't understand how a value oriented paradigm is better than an object oriented paradigm, which is why I value vJass and ZINC over any other extension people may have created. OOP is the future and that's what I tried to teach people.

Thx for your comments though, they are always appreciated.

 

[Miss_Foxy]

But how do you apply this trigger to a specific unit?

PS: Erm... I'm really sorry for asking this but, what does vJASS mean? Like, what does the 'v' stand for and why is it there? And, what is the difference between vJASS and JASS, I'm really sorry...

 

[Flame_Phoenix]

The answer to your question is right after the introduction:

The "v" is for very and not for "Vexorian". Vexorian wanted to call it JASS++ however Zoxc suggested vJASS for veryJASS and Vexorian liked the idea. Thanks to Vexorian for explaining me the origin of the name.

 

[Cweener]

For some reason it doesn't work,

I changed all the constants in the setup area, and copied the dummy unit, and the spell to get it to work, but it just won't.

If it matters, trigger name = InstantJustice (same as test map)

Anyways, The error is that it won't let me run the map in wc3. (indicates syntax)

//===========================================================================
//A vJASS and JESP spell that will have an effetc depending on the number
//of units in the AOE. If there are more allies than enemies, we heal allies
//if there are more enemies that allies we damage enemies, if the number of 
//allies equals the number of enemies we heal the allies and damage the enemies
//
//@author Flame_Phoenix 
//
//@credits
//- Blade.dk, for the CopyCroup function 
//
//@version 1.0
//===========================================================================
scope InstantJustice initializer Init
//===========================================================================
//=============================SETUP START===================================
//===========================================================================
    globals
        private constant integer SPELL_ID = 'A00W'  //the rawcode of the spell
        private constant integer DUMMY_ID = 'h002'  //rw of the dummy unit
        private constant string AOE_EFFECT = "Units\\NightElf\\Wisp\\WispExplode.mdl"  //effect that will be created when we cast the spell on the AOE
        private constant string HEAL_EFFECT = "Abilities\\Spells\\Human\\HolyBolt\\HolyBoltSpecialArt.mdl"  //effect that will be created when we heal units
        private constant string DAMAGE_EFFECT = "Abilities\\Weapons\\Bolt\\BoltImpact.mdl"  //effect that will be created when we damage units
        private constant damagetype D_TYPE = DAMAGE_TYPE_NORMAL //the attack type of the spell
        private constant attacktype A_TYPE = ATTACK_TYPE_MAGIC  //the damage type of the spell
    endglobals
    
    private function Range takes integer level returns real
    //returns the range the spell will affect
        return level * 300.
    endfunction
    
    private function Heal takes integer level returns real 
    //returns the heal allies will take
        return level * 500.
    endfunction
    
    private function Damage takes integer level returns real
    //returns the damage enemies will take
        return level * 500.
    endfunction
    
    private function Targets takes unit target returns boolean
    //the units the spell will affect
        return (GetWidgetLife(target) > 0.405) and (IsUnitType(target, UNIT_TYPE_STRUCTURE) == false) and (IsUnitType(target, UNIT_TYPE_MAGIC_IMMUNE) == false) and (IsUnitType(target, UNIT_TYPE_MECHANICAL) == false)
    endfunction
//===========================================================================
//=============================SETUP END=====================================
//=========================================================================== 
    globals
        private group all
        private group copy
        private boolexpr b
    endglobals
//===========================================================================  
//Function made by Blade.dk; Search for [url]www.wc3campaigns.com[/url] for more info   
    private function CopyGroup takes group g returns group
        set bj_groupAddGroupDest = CreateGroup()
        call ForGroup(g, function GroupAddGroupEnum)
        return bj_groupAddGroupDest
    endfunction
//===========================================================================  
    private function Pick takes nothing returns boolean
        return Targets(GetFilterUnit())
    endfunction
//===========================================================================    
    private function Conditions takes nothing returns boolean
        return GetSpellAbilityId() == SPELL_ID
    endfunction
//===========================================================================
    private function Actions takes nothing returns nothing
        local location spellLoc = GetSpellTargetLoc()
        local real spellX = GetLocationX(spellLoc)
        local real spellY = GetLocationY(spellLoc)
        local unit caster = GetTriggerUnit()
        local integer level = GetUnitAbilityLevel(caster, SPELL_ID)
        local unit f
        local integer allies = 0
        local integer enemies = 0
        
        //create the AOE effect
        call DestroyEffect(AddSpecialEffect(AOE_EFFECT, spellX, spellY))
        
        //counting the units
        call GroupEnumUnitsInRange(all, spellX, spellY, Range(level), b)
        set copy = CopyGroup(all)
        loop
            set f = FirstOfGroup(copy)
            exitwhen(f == null)
            call GroupRemoveUnit(copy, f)
            if IsUnitAlly(f, GetOwningPlayer(caster)) then
                set allies = allies + 1
            else
                set enemies = enemies + 1
            endif
        endloop
        
        //making the effect of the spell
        if allies > enemies then
            loop
                set f = FirstOfGroup(all)
                exitwhen (f == null)
                call GroupRemoveUnit(all, f)
                //heal allies
                if IsUnitAlly(f, GetOwningPlayer(caster)) then
                    call DestroyEffect(AddSpecialEffectTarget(HEAL_EFFECT, f, "origin")) //the healing effect
                    call SetWidgetLife(f, GetWidgetLife(f) + Heal(level))
                endif
            endloop
        elseif enemies > allies then
            loop
                set f = FirstOfGroup(all)
                exitwhen (f == null)
                call GroupRemoveUnit(all, f)
                //damage enemies
                if IsUnitEnemy(f, GetOwningPlayer(caster)) then
                    call DestroyEffect(AddSpecialEffectTarget(DAMAGE_EFFECT, f, "origin"))  //the damaging effect
                    call UnitDamageTarget(caster, f, Damage(level), true, false, A_TYPE, D_TYPE, null)
                endif
            endloop
        elseif allies == enemies then
            loop
                set f = FirstOfGroup(all)
                exitwhen (f == null)
                call GroupRemoveUnit(all, f)
                //heal allies 
                if IsUnitAlly(f,  GetOwningPlayer(caster)) then
                    call DestroyEffect(AddSpecialEffectTarget(HEAL_EFFECT, f, "origin"))
                    call SetWidgetLife(f, GetWidgetLife(f) + Heal(level))
                //if an unit is not an ally than it is an enemy
                else
                    call DestroyEffect(AddSpecialEffectTarget(DAMAGE_EFFECT, f, "origin"))
                    call UnitDamageTarget(caster, f, Damage(level), true, false, A_TYPE, D_TYPE, null)
                endif
            endloop
        endif
        
        call RemoveLocation(spellLoc)
        set spellLoc = null
        set caster = null
    endfunction    
//===========================================================================
    private function Init takes nothing returns nothing
        local trigger InstantJusticeTrg = CreateTrigger()
        call TriggerRegisterAnyUnitEventBJ(InstantJusticeTrg, EVENT_PLAYER_UNIT_SPELL_EFFECT )
        call TriggerAddCondition(InstantJusticeTrg, Condition( function Conditions ) )
        call TriggerAddAction( InstantJusticeTrg, function Actions )
        
        //setting globals
        set all = CreateGroup()
        set copy = CreateGroup()
        set b = Condition(function Pick)
        
        //preloading effects
        call Preload(AOE_EFFECT)
        call Preload(HEAL_EFFECT)
        call Preload(DAMAGE_EFFECT)
        
        //preloading the ability
        set bj_lastCreatedUnit = CreateUnit(Player(PLAYER_NEUTRAL_PASSIVE), DUMMY_ID, 0, 0, 0)
        call UnitAddAbility(bj_lastCreatedUnit, SPELL_ID)
        call KillUnit(bj_lastCreatedUnit)
        
    endfunction
endscope

I bet I just fucked something up >.<, anyways, let me know please :/

 

[Flame_Phoenix]

You do realize you need JNGP to run the script right?

 

[Cweener]

Of course. I'm using JNPG.