trigger problem?

[Sui-cookie]

hey can you guys spot anything wrong with this trigger? its suppose to allow only one powerup item, but when you dont have one at all it makes you drop it, then the second time you try to pick it up it works, but then wont allow other class items ...or powerup items....

• Untitled Trigger 001 Copy
  • Events
    • Unit - A unit Acquires an item
  • Conditions
  • Actions
    • For each (Integer A) from 1 to Maximum number of inventory slots (6), do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Item-class of (Item carried by (Triggering unit) in slot (Integer A))) Equal to Powerup
          • Then - Actions
            • Set tempinteger = (tempinteger + 1)
          • Else - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • tempinteger Greater than or equal to 2
      • Then - Actions
        • Set tempinteger = 0
        • Hero - Drop (Item being manipulated) from (Triggering unit)
        • Game - Display to (All players matching ((Matching player) Equal to (Owner of (Triggering unit)))) the text: You are already wei...
      • Else - Actions

 

[Pharaoh_]

Add the second if/then/else right after the Set tempinteger = ((tempinteger) + 1) action and the tempinteger = ((tempinteger) + 1) action in the "Else" part of the second if then else:

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
  • (Item-class of (Item carried by (Triggering unit) in slot (Integer A))) Equal to Powerup
  • Then - Actions
  • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • tempinteger Greater than or equal to 2
  • Then - Actions
    • Set tempinteger = 0
    • Hero - Drop (Item being manipulated) from (Triggering unit)
    • Game - Display to (All players matching ((Matching player) Equal to (Owner of (Triggering unit)))) the text: You are already wei...
  • Else - Actions
    • Set tempinteger = (tempinteger + 1)
  • Else - Actions

 

[Sui-cookie]

no go :[ it still takes 2+ clicks for it to actually get into your inventory

new trigger:

• Untitled Trigger 001 Copy
  • Events
    • Unit - A unit Acquires an item
  • Conditions
  • Actions
    • For each (Integer A) from 1 to 6, do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Item-class of (Item carried by (Triggering unit) in slot (Integer A))) Equal to (Item-class of (Item being manipulated))
          • Then - Actions
            • Set tempinteger = (tempinteger + 1)
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • tempinteger Greater than 1
              • Then - Actions
                • Set tempinteger = 0
                • Hero - Drop (Item being manipulated) from (Triggering unit)
                • Game - Display to (All players matching ((Matching player) Equal to (Owner of (Triggering unit)))) the text: nope
              • Else - Actions
                • Set tempinteger = (tempinteger + 1)
          • Else - Actions

 

[cleeezzz]

(Item-class of (Item carried by (Triggering unit) in slot (Integer A))) Equal to (Item-class of (Item being manipulated))

change to powerup.

(Item-class of (Item carried by (Triggering unit) in slot (Integer A))) Equal to (Item-class powerup)

(something like that)

# Else - Actions

    * Set tempinteger = (tempinteger + 1)

wth is that else action ? remove it

 

[Sui-cookie]

thx, after that it still didnt work...but then i went to get something to drink....and it hit me..... i use tempinteger on some of the initial things..like a multiboards and other random stuff....so.... it may be "contaminated" so i just added set tempinteger = 0 before it all and it worked perfectly

thx guys :]
ug sry pharoah i have to wait another 24 hours before i can give you rep :/

Spoiler

...

• Untitled Trigger 001 Copy
  • Events
    • Unit - A unit Acquires an item
  • Conditions
  • Actions
    • Set tempinteger = 0
    • For each (Integer A) from 1 to 6, do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Item-class of (Item carried by (Triggering unit) in slot (Integer A))) Equal to Powerup
          • Then - Actions
            • Set tempinteger = (tempinteger + 1)
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • tempinteger Greater than 1
              • Then - Actions
                • Set tempinteger = 0
                • Hero - Drop (Item being manipulated) from (Triggering unit)
                • Game - Display to (All players matching ((Matching player) Equal to (Owner of (Triggering unit)))) the text: nope
              • Else - Actions
          • Else - Actions

...