Problem with Integers

[X-Death]

For some reason this isn't working correctly and I don't understand why. I only want a specific unit to be able to carry one of these items. When I try to pick one up for the first time, it displays both messages and the unit drops the item. What's wrong???

• Events
  • Unit - A unit Acquires an item
• Conditions
  • (Item-type of (Item being manipulated)) Equal to Topaz
  • (Owner of (Triggering unit)) Equal to Player 1 (Red)
• Actions
  • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    • If - Conditions
      • TopazInt[1] Equal to 0
    • Then - Actions
      • Set TopazInt[1] = 1
      • Game - Display to (Player group((Owner of (Triggering unit)))) for 0.70 seconds the text: |cffffcc00Topaz Col...
    • Else - Actions
      • Game - Display to (Player group((Owner of (Triggering unit)))) for 0.70 seconds the text: You've already coll...
      • Hero - Drop (Item being manipulated) from (Triggering unit)

 

[pOke]

You're leaking playergroups, you should just loop thru the triggering units inventory and if it already has one of the item type drop the extra item

 

[deathismyfriend]

You never set the value to 0 again. It is not MUI the way you have it. So if more than one unit picks up the same item type it will drop that item.

Poke's suggestion is the best one to use.