Why doesn't this trigger work?

[IllidansHead]

For some reason, this triggger doesn't work.

Events
Player - Player 1 (Red) types a chat message containing -equip basicdarkbook as An exact match

Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    If - Conditions
        equipped_spellbook Equal to none
        (ChosenHero has an item of type Basic Dark Spellbook) Equal to True
    Then - Actions
        Set equipped_spellbook = basicdarkbook
        Item - Remove (Item carried by ChosenHero of type Basic Dark Spellbook)
        Hero - Modify Intelligence of ChosenHero: Add 1
    Else - Actions
        Game - Display to (All players) the text: Cannot equip |cffff0000Basic Dark Spellbook|r. Either you already have a spellbook equipped, or the class you have chosen cannot equip spellbooks.

It's for an equipment system used in my RPG, but it always does the "else", even when the variable IS equal to 'none' and I have a Basic Dark Spellbook.

 

[DDRtists]

For some reason, this triggger doesn't work.

Events
Player - Player 1 (Red) types a chat message containing -equip basicdarkbook as An exact match

Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    If - Conditions
        equipped_spellbook Equal to none
        (ChosenHero has an item of type Basic Dark Spellbook) Equal to True
    Then - Actions
        Set equipped_spellbook = basicdarkbook
        Item - Remove (Item carried by ChosenHero of type Basic Dark Spellbook)
        Hero - Modify Intelligence of ChosenHero: Add 1
    Else - Actions
        Game - Display to (All players) the text: Cannot equip |cffff0000Basic Dark Spellbook|r. Either you already have a spellbook equipped, or the class you have chosen cannot equip spellbooks.

It's for an equipment system used in my RPG, but it always does the "else", even when the variable IS equal to 'none' and I have a Basic Dark Spellbook.

um idk ,but at the cannot equip message, you might want to change that to just the player with the book, otherwise eeryone will think they cant equip

 

[maglore]

Ok first thing that poped into my mind was

equipped_spellbook Equal to none
(ChosenHero has an item of type Basic Dark Spellbook) Equal to True

Not sure if "equipped_spellbook" is a var, Looks to be one thou

but id check the defualt value and then check to find out each place that var can be changed.

Also if "equipped_spellbook" is a Item_Type var then is its "Type" the same as "Basic Dark Spellbook" ... Like did u use the same Item Type in creating the two?

If im way off sorry BUT if im not way off and you dont listen .... :nuts:

 

[Rad]

It should work, you may try wanting to set the variable to none in your init trigger though. Also you should give it the thunder clap ability then detect when they cast it so they dont have to type it all

EDIT: 1150 posts

 

[DDRtists]

It should work, you may try wanting to set the variable to none in your init trigger though. Also you should give it the thunder clap ability then detect when they cast it so they dont have to type it all

EDIT: 1150 posts

Congrats

 

[IllidansHead]

It's a single player RPG, that's why it doesn't matter that it's "All PLayers". Basic Dark Spell book is an item, and there are triggers that set equipped_spellbook to none when you pick a hero that uses spellboooks. The reason equipped spellbook is a string var, is because if you pick say, a swordsman, he can't equip spellbooks, and the default value for equipped_spell is 'unused'. THe thing is, even when equipped spellbook is equal to none, and I have a Basic Dark Spellbook, it does the "Else" section.
EDIT: Here are the tirggers for choosing Dark Mage and Demon Lord, the only two heroes made so far, both use Basic Dark SPellboook.

Select Dark Mage
    Events
        Unit - A unit enters Select Dark Mage <gen>
    Conditions
        (Triggering unit) Equal to Spirit of Vengeance 0003 <gen>
    Actions
        Unit - Replace (Triggering unit) with a Dark Mage using The new unit's default life and mana
        Unit - Move (Last replaced unit) instantly to (Center of (Playable map area))
        Set ChosenHero = (Last created unit)
        Set equipped_spellbook = none
        Hero - Create Basic Dark Spellbook and give it to ChosenHero

Select Demon Lord
    Events
        Unit - A unit enters Select Demon Lord <gen>
    Conditions
        (Triggering unit) Equal to Spirit of Vengeance 0003 <gen>
    Actions
        Unit - Replace (Triggering unit) with a Demon Lord using The new unit's default life and mana
        Unit - Move (Last replaced unit) instantly to (Center of (Playable map area))
        Set ChosenHero = (Last created unit)
        Set equipped_spellbook = none
        Hero - Create Basic Dark Spellbook and give it to ChosenHero

EDIT 2: :banghead: :banghead: DOH! It's so obvious. I used replacement, but CHosen hero is "last created unit". It should work now.

 

[DDRtists]

It's a single player RPG, that's why it doesn't matter that it's "All PLayers".

oh, im sorry

 

[IllidansHead]

oh, im sorry

It's ok. I haven't actually said that anywhere before now. I'm just glad I figured out the problem. Now on to making more heroes. (need 9)\
EDIT: Click here to find out more about my RPG (I'm Nyktos btw).