[Trigger] Explain what I did wrong

[Dragonkillerrus]

• part 1
  • Events
    • Map initialization
  • Conditions
  • Actions
    • Item - Create (Unique)Advanced tactics doctrine at (Center of Region 087 <gen>)
    • Set AdvancedDoctrineItem = (Last created item)

• Tactic Doctrine
  • Events
    • Unit - A unit Acquires an item
  • Conditions
    • (Item-type of (Item being manipulated)) Equal to (Unique)Advanced tactics doctrine
  • Actions
    • Hero - Make OwnerOfAdvancedTcticsDoctr Heroes gain 120.00% experience from future kills
    • Set OwnerOfTcticsDoctr(Player) = (Owner of AdvancedDoctrineItem)

Here is the item btw

Custom Item

• Dragonkillerrus
• Apr 22, 2017

Also tell me if these are necessary

• Player loses Doctrine tacit item
  • Events
    • Unit - A unit Loses an item
  • Conditions
    • (Item-type of (Item being manipulated)) Equal to (Unique)Advanced tactics doctrine
  • Actions
    • Hero - Make OwnerOfAdvancedTcticsDoctr Heroes gain 100.00% experience from future kills

• Player sells Doctrine tackit item
  • Events
    • Unit - A unit Pawns an item (to shop)
  • Conditions
    • (Item-type of (Item being manipulated)) Equal to (Unique)Advanced tactics doctrine
  • Actions
    • Hero - Make OwnerOfAdvancedTacticsDoctr Heroes gain 100.00% experience from future kills

 

[KILLCIDE]

I don't see you storing anything into OwnerofAdvancedTcticsDoctr. What does that variable refer to? Player variables have initial values of "None" if you leave it as default.

 

[SnowYell]

The last 2 would be neccessary in my opinion if I understand your idea correctly. The XP boost should only be available, when the Hero carries the item, right?
For that reason, when he loses it, he should not have the XP boost any longer. Unless you make it a tome.

Why don't you make a trigger and set the unit carrying the item as a variable. Like:

Event:
Hero acquires item
Condition:
Equal to item
Action:
Set unit variable
Set item variable
Set experience for player owner of unit carrying item