Damage detection trigger, NOT working

sunshinex3

You can change this now in User CP.
I need to figure out why this isn't working, it fires off when not using the variables and just using a solid number so the second trigger works but when trying to use the bidedamagetaken variable it doesn't work, so im wondering if it is the custom script that is just below that's the problem?

I have this is in initialization its a custom script i was told would add the event unit takes damage to a trigger
Code:
library DamagedEventAdder initializer init
  private function init takes nothing returns nothing

    //change the name of the trigger to match the name of the trigger you want it added to
    //the format is: gg_trg_ + the name of the trigger with _ instead of spaces (see below)

    //if you change the name of the trigger in the trigger editor (or delete the trigger entirely) you will have to change it here
    //if you have multiple triggers that need this event you can duplicate the below line here for any number of triggers, just use the right name!

    call TriggerRegisterAnyUnitEventBJ(gg_trg_BideActivate, EVENT_PLAYER_UNIT_DAMAGED)
  endfunction
endlibrary
First trigger that i need "unit takes damage"
Trigger:
  • BideActivate
    • Events
    • Conditions
      • ((Damage Target) has buff Biding (NerubianKing)) Equal to True
    • Actions
      • Set VariableSet BideDamageTaken[(Player number of (Owner of (Damage Target)))] = (BideDamageTaken[(Player number of (Owner of (Damage Target)))] + (Damage taken))


this trigger works with a straight number but not with the variable arithmetic that i have here

Trigger:
  • BideDeactivate
    • Events
      • Unit - A unit Is issued an order with no target
    • Conditions
      • And - All (Conditions) are true
        • Conditions
          • ((Ordered unit) has buff Biding (NerubianKing)) Equal to True
          • (Issued order) Equal to (Order(unimmolation))
    • Actions
      • Set VariableSet TempPoint = (Position of (Ordered unit))
      • Unit - Cause (Ordered unit) to damage circular area after 0.00 seconds of radius 400.00 at TempPoint, dealing (BideDamageTaken[(Player number of (Owner of (Ordered unit)))] x (Real((Level of Bide (NerubianKing) for (Ordered unit))))) damage of attack type Spells and damage type Magic
      • Custom script: call RemoveLocation (udg_TempPoint)
[/code]
 
Last edited:

jonas

Well-Known Member
Add some debug messages.
Display the damage taken and the new value of BideDamageTaken in the first trigger.
Display the BideDamageTaken and the Level of Bide in the second trigger.

If your first trigger doesn't fire, remove the condition.

If even without the conditon it doesn't fire, it is the custom script. In that case please repost the custom script with
Code:
 code tags
 

jonas

Well-Known Member
Yep. The code itself looks good to me though.
Can you post a screenshot of the code in your trigger editor?

Maybe it is accidentally turned off, or in the comment section.

Another thing you can try is a new GUI trigger like this:

event : a unit enters the map
condition: type of triggering unit is (Nerubian King)
action: add event to trigger (BideActivate) : (triggering unit) takes damage
 

sunshinex3

You can change this now in User CP.
event : a unit enters the map
condition: type of triggering unit is (Nerubian King)
action: add event to trigger (BideActivate) : (triggering unit) takes damage
"can't find (triggering unit) takes damage"
i think that's why i had to add the custom script library.
 
Last edited:

jonas

Well-Known Member
Unfortunately I don't have Wc3 installed, but I was sure there is a GUI function to make that happen.
If there isn't, you can also try with this line of custom script which should do the same:



Another thing you can do is add the following to the custom script you already have:

Code:
if gg_trg_BideActivate == null then
    call DisplayTextToPlayer(GetLocalPlayer(), 0, 0, "initialization called but trigger is null")
else
    call DisplayTextToPlayer(GetLocalPlayer(), 0, 0, "initialization called and trigger exists")
endif
This should write some text at the beginning of the game. (Let me know if there are any syntax errors... sorry that I can't check that myself)
 

sunshinex3

You can change this now in User CP.
I am going to put this behind me i guess it's not possible with gui this is what i did it turned out fine.

Trigger:
  • BideActivate
    • Events
      • Unit - A unit Is attacked
    • Conditions
      • ((Attacked unit) has buff Biding (NerubianKing)) Equal to True
    • Actions
      • Set VariableSet BideDamageTaken[(Player number of (Owner of (Attacked unit)))] = (BideDamageTaken[(Player number of (Owner of (Attacked unit)))] + 1.00)


Number is increased based on how many attacks are made on the unit instead of damage taken, works well with tanks.


Trigger:
  • BideDeactivate
    • Events
      • Unit - A unit Is issued an order with no target
    • Conditions
      • And - All (Conditions) are true
        • Conditions
          • ((Ordered unit) has buff Biding (NerubianKing)) Equal to True
          • (Issued order) Equal to (Order(unimmolation))
    • Actions
      • Set VariableSet TempPoint = (Position of (Ordered unit))
      • Unit - Cause (Ordered unit) to damage circular area after 0.00 seconds of radius 400.00 at TempPoint, dealing (BideDamageTaken[(Player number of (Owner of (Ordered unit)))] x (Real((Level of Bide (NerubianKing) for (Ordered unit))))) damage of attack type Spells and damage type Magic
      • Set VariableSet BideDamageTaken[(Player number of (Owner of (Ordered unit)))] = 0.00
      • Custom script: call RemoveLocation (udg_TempPoint)



causes damage based on number of attacks made on caster and then resets the damage to 0.
 
General chit-chat
Help Users
  • No one is chatting at the moment.
  • The Helper The Helper:
    wizard and ghan i hope you guys are ready because i am taking you bitches along for the ried
  • The Helper The Helper:
    buckle up
  • Wizard Wizard:
    xD
  • Wizard Wizard:
    Sounds good lol
  • Wizard Wizard:
    I did reply with more info just now about what we did in the past and what we are doing now.
  • The Helper The Helper:
    it is all good
  • The Helper The Helper:
    it is all percolating man
  • The Helper The Helper:
    TheHelper.net #1!
  • Wizard Wizard:
    #1
  • The Helper The Helper:
    LOL I love this place
  • Wizard Wizard:
    ;D
  • Wizard Wizard:
    We all do.
  • O Old Mountain Shadow:
    So what is next for thehelper.net? Is there a new set goals yet?
  • The Helper The Helper:
    yes we are going to be expanding
    +1
  • The Helper The Helper:
    help more people
  • The Helper The Helper:
    thank you for asking Old Mountain Shadow hell ya@
  • The Helper The Helper:
    Look at Mountain Shadow in the chat!
  • The Helper The Helper:
    welcome to thehelper.net
  • O Old Mountain Shadow:
    thanks although I'm really kind of regretting this name lol ... is there a way to change it out or will I need to make a new account?
  • Wizard Wizard:
    One of us can change it for you, I believe. Just send me a PM on what you want.
  • Ghan Ghan:
    Some day we should get the feature where you can change your own name.
  • Wizard Wizard:
    Soon!
  • The Helper The Helper:
    did the mountain shadow get his name changed? personally I think Old Mountain Shadow is a kickass name
  • The Helper The Helper:
    but what do i know lol

    Staff online

    • Ghan
      Administrator - Servers are fun

    Members online

    Affiliates

    Hive Workshop NUON Dome
    Top