[9544] support CONDITION_NOITEM

[timmit]

копия патча на getmangos.com
http://getmangos.com/community/showt...NDITION_NOITEM

diff --git a/src/game/ObjectMgr.cpp b/src/game/ObjectMgr.cpp
index b887961..f8b041d 100644
--- a/src/game/ObjectMgr.cpp
+++ b/src/game/ObjectMgr.cpp
@@ -7389,6 +7389,8 @@ bool PlayerCondition::Meets(Player const * player) const
             return player->HasAura(value1, SpellEffectIndex(value2));
         case CONDITION_ITEM:
             return player->HasItemCount(value1, value2);
+        case CONDITION_NOITEM:
+            return player->!HasItemCount(value1, 1);
         case CONDITION_ITEM_EQUIPPED:
             return player->HasItemOrGemWithIdEquipped(value1,1);
         case CONDITION_ZONEID:
@@ -7484,6 +7486,16 @@ bool PlayerCondition::IsValid(ConditionType condition, uint32 value1, uint32 val
             }
             break;
         }
+        case CONDITION_NOITEM:
+        {
+            ItemPrototype const *proto = ObjectMgr::GetItemPrototype(value1);
+            if(!proto)
+            {
+                sLog.outErrorDb("Item condition requires to have non existing item (%u), skipped", value1);
+                return false;
+            }
+            break;
+        }
         case CONDITION_ITEM_EQUIPPED:
         {
             ItemPrototype const *proto = ObjectMgr::GetItemPrototype(value1);
diff --git a/src/game/ObjectMgr.h b/src/game/ObjectMgr.h
index 4442f56..d0fb9c5 100644
--- a/src/game/ObjectMgr.h
+++ b/src/game/ObjectMgr.h
@@ -301,6 +301,7 @@ enum ConditionType
     CONDITION_AURA                  = 1,                    // spell_id     effindex
     CONDITION_ITEM                  = 2,                    // item_id      count
     CONDITION_ITEM_EQUIPPED         = 3,                    // item_id      0
+    CONDITION_NOITEM                = 16,                   // item_id      count
     CONDITION_ZONEID                = 4,                    // zone_id      0
     CONDITION_REPUTATION_RANK       = 5,                    // faction_id   min_rank
     CONDITION_TEAM                  = 6,                    // player_team  0,      (469 - Alliance 67 - Horde)
@@ -315,7 +316,7 @@ enum ConditionType
     CONDITION_LEVEL                 = 15,                   // player_level 0, 1 or 2 (0: equal to, 1: equal or higher than, 2: equal or less than)
 };
 
-#define MAX_CONDITION                 16                    // maximum value in ConditionType enum
+#define MAX_CONDITION                 17                    // maximum value in ConditionType enum
 
 struct PlayerCondition
 {

[`win]

эм... возможно покажусь занудой, но не могу не спросить.... а что правит/добавляет патч?
читая последнее предложение вообще не понял о чём речь :\

Сообщение от timmit

Баг функции HasItemCount, что даже при 0 предметах она всегда будет true, поэтому когда мы в условии CONDITION_ITEM ставить 0 предметов, то при наличии оных у нас будет показываться госсип, что в корне не верно 0 это не 1

можно было бы попробовать использовать с !, тобишь

!HasItemCount(pItem, 1, false);

[timmit]

Патч добавляет функционал для gossip системы, когда мы показывается меню только при отсутствии предмета в инвентаре.

Последнее предложение к тому, что если value2 Для HasItemCount=0, как бы наличие 0 предметов, то при обращении к функции с проверкой твоего инвентаря, при наличии даже 1 предмета будет true ,что вообще неправильно вот я и говорю 0 это не 1.. 0 это ноль

[PSZ]

А можно пример, когда это нужно? У какого НПЦ какой госсип должен показываться только когда у персонажа отсутствует определенный предмет?

[Foks]

А !HasItemCount(..., 1, ...) разве не даст нам true когда у игрока отсутствует данный предмет? А если он есть хотя бы один, то даст false.

оно даст false, на сколько я понял, это и надо.
Поэтому выше в посте и предлагал

[timmit]

А если у меня таких 2 предмета? проверка даст труе итого false

[Foks]

Но ведь это и нужно? В итоге условие даст true ТОЛЬКО если нету ни одного такого предмета.

[timmit]

Переписал, так правильно?

Да, всё правильно.

[timmit]

Сообщение от PSZ

А можно пример, когда это нужно? У какого НПЦ какой госсип должен показываться только когда у персонажа отсутствует определенный предмет?

Ой, щас не скажу вам, это нужно в 70% случаев госсипов, когда касается квестов, и когда вам говорят - если вдруг пропадет, я вам дам еще.
Пока вся работа с таким ведется без этого условия (обычно все такие предметы уникальны), но это не совсем правильно для отображения и т.д

[Vladimir]

В [9544] с добавленной поддержкой количества. Спасибо