[Solved] Class Item - Trigger Error

[JmHunt]

Apologize in advance for the image. Don't know how to quote Triggers yet.

The Trigger works great except for the "Item - Remove" line. I can't seem to get the "Dragon's Heart" item to be removed upon buying the Ring of Armor. Everything else works perfect.

Any ideas?

Thanks in advance.

 

[Doomlord]

http://www.hiveworkshop.com/forums/miscellaneous-tutorials-456/how-easily-post-triggers-163285/

Nothing helps the helpers more than the code itself.

 

[deathismyfriend]

Is it the right unit that u have in the conditions.

 

[JmHunt]

Alright, thanks for the tip Doomlord. Here is the code:

• Assassin Class Item
  • Events
    • Unit - Tinker 0021 <gen> Sells an item (from shop)
  • Conditions
    • ((Buying unit) is A Hero) Equal to True
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Item-type of (Sold Item)) Equal to Ring of Armor
        • ((Buying unit) has an item of type Dragon's Heart) Equal to True
        • (Unit-type of (Buying unit)) Equal to Illidan (Evil)
      • Then - Actions
        • Item - Remove (Item carried by (Triggering unit) of type Dragon's Heart)
      • Else - Actions
        • Item - Remove (Sold Item)
        • Player - Add 1000 to Player 1 (Red) Current gold

deathismyfriend - It only lets the character in the "If" conditions buy the item. For example, with this code here, it only lets Illidan (Evil) buy the item if he has a Dragon's Heart in his inventory - Which is what I want. But I would like it to remove the Dragon's Heart after buying the item - which it is NOT doing so far.

Thanks for replying!

 

[deathismyfriend]

Ik that I meant is it illidan that is the unit buying it ? If its not illidan( evil) then that's ur problem.
Also change triggering unit to buying unit. That is mostly ur problem.

Edit: u should also store anything u use twice or more into a temp variable and use that for efficiency and a speed increase. Example buying unit

 

[JmHunt]

Yeah, it was Illidan.

"Buying Unit" was exactly what I needed.

Works perfectly now. Thanks so much!

 

[deathismyfriend]

Np lol glad to help also look at my post above urs as I updated it.

 

[JmHunt]

Hey Death,

I ran into a problem with the code. It works, but for some reason, if another hero class tries to buy the item it lets them. Is this because the "Unit Type of Buying Unit" (Illidan) - Is a Hero and so are they? I can't use Illidan as a specific unit because I have a Choose Class area, so I have to him as a Unit Type.

I have read about using variables like you said, but don't have complete knowledge how yet.