Trigger not working help please

[Legendary]

hi guys i want to ask why does this trigger dosen't restict anyone? from getting a certain spell hmmmm i have set the condition to warrior only but i tried a mage to learn the mortal strike and after a sec a noticed that the mortal strike was in the mages skills i duno why can you guys help me?

• Mortal Strike Learned
  • Events
    • Unit - A unit Acquires an item
  • Conditions
    • (Unit-type of (Hero manipulating item)) Equal to Heroes_1st_Job[1]
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • And - All (Conditions) are true
          • Conditions
            • (Item-type of (Item being manipulated)) Equal to [|cffffd700Q|r] - |cff90ee90Mortal Strike|r
      • Then - Actions
        • Set Units = (Triggering unit)
        • Set Ability[(Player number of (Owner of Units))] = Mortal Strike
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • And - All (Conditions) are true
              • Conditions
                • (Level of Ability[(Player number of (Owner of Units))] for Units) Less than 1
          • Then - Actions
            • Unit - Add Ability[(Player number of (Owner of Units))] to Units
            • Game - Display to (Player group((Owner of Units))) for 10.00 seconds the text: Mortal Strike Learn...
          • Else - Actions
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (Level of Ability[(Player number of (Owner of Units))] for Units) Less than 3
              • Then - Actions
                • Unit - Increase level of Ability[(Player number of (Owner of Units))] for Units
                • Game - Display to (Player group((Owner of Units))) for 5.00 seconds the text: (Mortal Strike's Level + ( is + ( + (String((Level of Mortal Strike for Units))))))
              • Else - Actions
                • Game - Display to (Player group((Owner of Units))) for 5.00 seconds the text: This spell is alrea...
                • Player - Add 1 to (Owner of Units) Current lumber
      • Else - Actions
        • Game - Display to (All players) for 5.00 seconds the text: |cff6d6d6dThis spel...

 

[deathismyfriend]

Add the condition unit type if triggering unit equal to warrior

 

[Legendary]

i did that's why i use the heroes 1st job[1] is set to my warrior

• (Unit-type of (Hero manipulating item)) Equal to Heroes_1st_Job[1]

 

[deathismyfriend]

ur getting the unit type of an item that wont work u need to change it to unit type of unit. then u need item type of item equal to that item

 

[Legendary]

• (Unit-type of (Hero manipulating item)) Equal to Heroes_1st_Job[1]

• (Item-type of (Item being manipulated)) Equal to [|cffffd700Q|r] - |cff90ee90Mortal Strike|r

is this correct? if not can you make me a example trigger?

 

[deathismyfriend]

• (Unit-type of (Hero manipulating item)) Equal to Heroes_1st_Job[1]

• (Item-type of (Item being manipulated)) Equal to [|cffffd700Q|r] - |cff90ee90Mortal Strike|r

is this correct? if not can you make me a example trigger?

• (Unit-type of (triggering unit)) Equal to Heroes_1st_Job[1]

• (Item-type of (Item being manipulated)) Equal to [|cffffd700Q|r] - |cff90ee90Mortal Strike|r

this is the right way. look at the difference unit type of triggering unit. you cant get a unit type of an item.

 

[Legendary]

• (Unit-type of (triggering unit)) Equal to Heroes_1st_Job[1]

• (Item-type of (Item being manipulated)) Equal to [|cffffd700Q|r] - |cff90ee90Mortal Strike|r

this is the right way. look at the difference unit type of triggering unit. you cant get a unit type of an item.

Hmmm i get it let me try

• Mortal Strike Learned
  • Events
    • Unit - A unit Acquires an item
  • Conditions
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • And - All (Conditions) are true
          • Conditions
            • (Unit-type of (Triggering unit)) Equal to Defender
            • (Item-type of (Item being manipulated)) Equal to [|cffffd700Q|r] - |cff90ee90Mortal Strike|r
      • Then - Actions
        • Set Units = (Triggering unit)
        • Set Ability[(Player number of (Owner of Units))] = Mortal Strike
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • And - All (Conditions) are true
              • Conditions
                • (Level of Ability[(Player number of (Owner of Units))] for Units) Less than 1
          • Then - Actions
            • Unit - Add Ability[(Player number of (Owner of Units))] to Units
            • Game - Display to (Player group((Owner of Units))) for 10.00 seconds the text: Mortal Strike Learn...
          • Else - Actions
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (Level of Ability[(Player number of (Owner of Units))] for Units) Less than 3
              • Then - Actions
                • Unit - Increase level of Ability[(Player number of (Owner of Units))] for Units
                • Game - Display to (Player group((Owner of Units))) for 5.00 seconds the text: (Mortal Strike's Level + ( is + ( + (String((Level of Mortal Strike for Units))))))
              • Else - Actions
                • Game - Display to (Player group((Owner of Units))) for 5.00 seconds the text: This spell is alrea...
                • Player - Add 1 to (Owner of Units) Current lumber
      • Else - Actions
        • Game - Display to (All players) for 5.00 seconds the text: |cff6d6d6dThis spel...

 

[deathismyfriend]

you dont need the and condition. put them in the conditions block. get rid of the ITE.
plz dont double post.

 

[Legendary]

still not working

• Mortal Strike Learned
  • Events
    • Unit - A unit Acquires an item
  • Conditions
    • And - All (Conditions) are true
      • Conditions
        • (Unit-type of (Triggering unit)) Equal to Defender
        • (Item-type of (Item being manipulated)) Equal to [|cffffd700Q|r] - |cff90ee90Mortal Strike|r
  • Actions
    • Set Units = (Triggering unit)
    • Set Ability[(Player number of (Owner of Units))] = Mortal Strike
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • And - All (Conditions) are true
          • Conditions
            • (Level of Ability[(Player number of (Owner of Units))] for Units) Less than 1
      • Then - Actions
        • Unit - Add Ability[(Player number of (Owner of Units))] to Units
        • Game - Display to (Player group((Owner of Units))) for 10.00 seconds the text: Mortal Strike Learn...
      • Else - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Level of Ability[(Player number of (Owner of Units))] for Units) Less than 3
          • Then - Actions
            • Unit - Increase level of Ability[(Player number of (Owner of Units))] for Units
            • Game - Display to (Player group((Owner of Units))) for 5.00 seconds the text: (Mortal Strike's Level + ( is + ( + (String((Level of Mortal Strike for Units))))))
          • Else - Actions
            • Game - Display to (Player group((Owner of Units))) for 5.00 seconds the text: This spell is alrea...
            • Player - Add 1 to (Owner of Units) Current lumber

 

[deathismyfriend]

take out the and condition u almost never need to use and condition. if its not working then try debuging it to find what works and what doesnt.
also u should store player number of owner of units into a variable.

 

[Legendary]

take out the and condition u almost never need to use and condition. if its not working then try debuging it to find what works and what doesnt.

i can't i just used that system so i can't and i duno how to fix it again

 

[deathismyfriend]

put a message in between lines of code. then remember the last one and the action after that is bad.

 

[Legendary]

i don't get it hm

 

[deathismyfriend]

really....

message 1
action1
message 2
action2
so on so on....

last message displayed is 2 then action2 is the bad action