player on mount use teleport

[Den]

Есть вот такой маленький недочёт:
Если персонаж на маунте , то он не может использовать портал (городские и маговские порталы) а должен.
Вот оно самое:

if (m_caster->IsMounted() && m_caster->GetTypeId()==TYPEID_PLAYER && !m_IsTriggeredSpell &&
        !IsPassiveSpell(m_spellInfo) && !(m_spellInfo->Attributes & SPELL_ATTR_CASTABLE_WHILE_MOUNTED))
    {
        if (m_caster->IsTaxiFlying())
            return SPELL_FAILED_NOT_ON_TAXI;
        else
            return SPELL_FAILED_NOT_MOUNTED;
    }

Если закоммитить - проблеммы нет. Не могу допереть... что тогда ломается то?
А да, проверка была добавлена в 3089-й ревизии.

[Vladimir]

m_spellInfo->Attributes & SPELL_ATTR_CASTABLE_WHILE_MOUNTED

Куча спелов не должна кастаться на маунте. Клинет это проверяет, но если не будет проверки в ядре будет возможет читинг.

[Den]

А если сделать вот так ? (для примера порталы не все)

// not let players cast spells at mount (and let do it to creatures)
    if (m_caster->IsMounted() && m_caster->GetTypeId()==TYPEID_PLAYER && !m_IsTriggeredSpell &&
        !IsPassiveSpell(m_spellInfo) && !(m_spellInfo->Attributes & SPELL_ATTR_CASTABLE_WHILE_MOUNTED) &&
		// for players on mount use portal 
        !(m_spellInfo->Id == 17609 || m_spellInfo->Id == 17610 || m_spellInfo->Id == 17611 || m_spellInfo->Id == 32270 ||	// Orgrimmar, Thunder Bluff, Undercity, Silvermoon City
        m_spellInfo->Id == 17607 || m_spellInfo->Id == 17608 || m_spellInfo->Id == 17334 || m_spellInfo->Id == 32268))      // Ironforge, Darnassus, Stormwind City, The Exodar 		
    {
        if (m_caster->IsTaxiFlying())
            return SPELL_FAILED_NOT_ON_TAXI;
        else
            return SPELL_FAILED_NOT_MOUNTED;
    }

Или есть более лучший подход к этому делу ?

[Vladimir]

Лучший подход - посмотреть на используемый атрибут и может быть найти более адекватно подходящий под проверку - т.е имеющийся, и для тех случаев, когда надо, как и данный, но неимеющийся для неадеквтных случаев.
Возможно аттрибут подобран неудачно.

Использовать списки спелов - это последнее что может быть рассмотрено как удачное решение. Когда тотально нет альтернатив.