[patch] Vendor

[alien]

На оффе есть шмотки которые продаются у одног вендора за голд а у другого за баджики или еще за какие штуки.
например
http://ru.wowhead.com/item=44128#sold-by
и http://ru.wowhead.com/item=36908#sold-by
В данной реализации таблици сделать такое невозможно
написал маленький патчик.

diff --git a/src/game/Creature.h b/src/game/Creature.h
index 3bb0591..38177b0 100644
--- a/src/game/Creature.h
+++ b/src/game/Creature.h
@@ -276,13 +276,14 @@ enum AttackingTarget
 // Vendors
 struct VendorItem
 {
-    VendorItem(uint32 _item, uint32 _maxcount, uint32 _incrtime, uint32 _ExtendedCost)
-        : item(_item), maxcount(_maxcount), incrtime(_incrtime), ExtendedCost(_ExtendedCost) {}
+    VendorItem(uint32 _item, uint32 _maxcount, uint32 _incrtime, uint8 _UseBuyPrice, uint32 _ExtendedCost)
+        : item(_item), maxcount(_maxcount), incrtime(_incrtime), UseBuyPrice(_UseBuyPrice), ExtendedCost(_ExtendedCost) {}
 
     uint32 item;
     uint32 maxcount;                                        // 0 for infinity item amount
     uint32 incrtime;                                        // time for restore items amount if maxcount != 0
-    uint32 ExtendedCost;
+    uint8  UseBuyPrice;
+	uint32 ExtendedCost;
 };
 typedef std::vector<VendorItem*> VendorItemList;
 
@@ -297,9 +298,9 @@ struct VendorItemData
     }
     bool Empty() const { return m_items.empty(); }
     uint8 GetItemCount() const { return m_items.size(); }
-    void AddItem( uint32 item, uint32 maxcount, uint32 ptime, uint32 ExtendedCost)
+    void AddItem( uint32 item, uint32 maxcount, uint32 ptime, uint8 UseBuyPrice, uint32 ExtendedCost)
     {
-        m_items.push_back(new VendorItem(item, maxcount, ptime, ExtendedCost));
+        m_items.push_back(new VendorItem(item, maxcount, ptime, UseBuyPrice, ExtendedCost));
     }
     bool RemoveItem( uint32 item_id );
     VendorItem const* FindItemCostPair(uint32 item_id, uint32 extendedCost) const;
diff --git a/src/game/ItemHandler.cpp b/src/game/ItemHandler.cpp
index 1d04c14..d3bd5b4 100644
--- a/src/game/ItemHandler.cpp
+++ b/src/game/ItemHandler.cpp
@@ -761,6 +761,7 @@ void WorldSession::SendListInventory(uint64 vendorguid)
 
                 // reputation discount
                 uint32 price = uint32(floor(pProto->BuyPrice * discountMod));
+				if (!crItem->UseBuyPrice) price=0;
 
                 data << uint32(vendorslot +1);              // client size expected counting from 1
                 data << uint32(crItem->item);
diff --git a/src/game/Level2.cpp b/src/game/Level2.cpp
index ad9682f..7264660 100644
--- a/src/game/Level2.cpp
+++ b/src/game/Level2.cpp
@@ -1128,6 +1128,11 @@ bool ChatHandler::HandleNpcAddVendorItemCommand(const char* args)
     if (fincrtime)
         incrtime = atol(fincrtime);
 
+	char* fUseBuyPrice = strtok(NULL, " ");                    //add UseBuyPrice, default: 1
+    uint8 UseBuyPrice = 1;
+    if (fUseBuyPrice)
+        UseBuyPrice = atol(fUseBuyPrice);
+
     char* fextendedcost = strtok(NULL, " ");                //add ExtendedCost, default: 0
     uint32 extendedcost = fextendedcost ? atol(fextendedcost) : 0;
 
@@ -1135,13 +1140,13 @@ bool ChatHandler::HandleNpcAddVendorItemCommand(const char* args)
 
     uint32 vendor_entry = vendor ? vendor->GetEntry() : 0;
 
-    if(!sObjectMgr.IsVendorItemValid(vendor_entry,itemId,maxcount,incrtime,extendedcost,m_session->GetPlayer()))
+    if(!sObjectMgr.IsVendorItemValid(vendor_entry,itemId,maxcount,incrtime,UseBuyPrice,extendedcost,m_session->GetPlayer()))
     {
         SetSentErrorMessage(true);
         return false;
     }
 
-    sObjectMgr.AddVendorItem(vendor_entry,itemId,maxcount,incrtime,extendedcost);
+    sObjectMgr.AddVendorItem(vendor_entry,itemId,maxcount,incrtime,UseBuyPrice,extendedcost);
 
     ItemPrototype const* pProto = ObjectMgr::GetItemPrototype(itemId);
 
diff --git a/src/game/ObjectMgr.cpp b/src/game/ObjectMgr.cpp
index 7beae97..4aa7b96 100644
--- a/src/game/ObjectMgr.cpp
+++ b/src/game/ObjectMgr.cpp
@@ -7988,7 +7988,7 @@ void ObjectMgr::LoadVendors()
 
     std::set<uint32> skip_vendors;
 
-    QueryResult *result = WorldDatabase.Query("SELECT entry, item, maxcount, incrtime, ExtendedCost FROM npc_vendor");
+    QueryResult *result = WorldDatabase.Query("SELECT entry, item, maxcount, incrtime, UseBuyPrice, ExtendedCost FROM npc_vendor");
     if( !result )
     {
         barGoLink bar( 1 );
@@ -8012,14 +8012,15 @@ void ObjectMgr::LoadVendors()
         uint32 item_id      = fields[1].GetUInt32();
         uint32 maxcount     = fields[2].GetUInt32();
         uint32 incrtime     = fields[3].GetUInt32();
-        uint32 ExtendedCost = fields[4].GetUInt32();
-
-        if(!IsVendorItemValid(entry,item_id,maxcount,incrtime,ExtendedCost,NULL,&skip_vendors))
+		uint8  UseBuyPrice 	= fields[4].GetUInt8();
+		uint32 ExtendedCost = fields[5].GetUInt32();
+		
+        if(!IsVendorItemValid(entry,item_id,maxcount,incrtime,UseBuyPrice,ExtendedCost,NULL,&skip_vendors))
             continue;
 
         VendorItemData& vList = m_mCacheVendorItemMap[entry];
 
-        vList.AddItem(item_id,maxcount,incrtime,ExtendedCost);
+        vList.AddItem(item_id,maxcount,incrtime,UseBuyPrice,ExtendedCost);
         ++count;
 
     } while (result->NextRow());
@@ -8290,10 +8291,10 @@ void ObjectMgr::LoadGossipMenuItems()
     sLog.outString(">> Loaded %u gossip_menu_option entries", count);
 }
 
-void ObjectMgr::AddVendorItem( uint32 entry,uint32 item, uint32 maxcount, uint32 incrtime, uint32 extendedcost )
+void ObjectMgr::AddVendorItem( uint32 entry,uint32 item, uint32 maxcount, uint32 incrtime,uint8 UseBuyPrice, uint32 extendedcost )
 {
     VendorItemData& vList = m_mCacheVendorItemMap[entry];
-    vList.AddItem(item,maxcount,incrtime,extendedcost);
+    vList.AddItem(item,maxcount,incrtime,UseBuyPrice,extendedcost);
 
     WorldDatabase.PExecuteLog("INSERT INTO npc_vendor (entry,item,maxcount,incrtime,extendedcost) VALUES('%u','%u','%u','%u','%u')",entry, item, maxcount,incrtime,extendedcost);
 }
@@ -8311,7 +8312,7 @@ bool ObjectMgr::RemoveVendorItem( uint32 entry,uint32 item )
     return true;
 }
 
-bool ObjectMgr::IsVendorItemValid( uint32 vendor_entry, uint32 item_id, uint32 maxcount, uint32 incrtime, uint32 ExtendedCost, Player* pl, std::set<uint32>* skip_vendors ) const
+bool ObjectMgr::IsVendorItemValid( uint32 vendor_entry, uint32 item_id, uint32 maxcount, uint32 incrtime, uint8 UseBuyPrice, uint32 ExtendedCost, Player* pl, std::set<uint32>* skip_vendors ) const
 {
     CreatureInfo const* cInfo = GetCreatureTemplate(vendor_entry);
     if(!cInfo)
diff --git a/src/game/ObjectMgr.h b/src/game/ObjectMgr.h
index d1819a7..331a282 100644
--- a/src/game/ObjectMgr.h
+++ b/src/game/ObjectMgr.h
@@ -863,9 +863,9 @@ class ObjectMgr
 
             return &iter->second;
         }
-        void AddVendorItem(uint32 entry,uint32 item, uint32 maxcount, uint32 incrtime, uint32 ExtendedCost);
+        void AddVendorItem(uint32 entry,uint32 item, uint32 maxcount, uint32 incrtime, uint8 UseBuyPrice, uint32 ExtendedCost);
         bool RemoveVendorItem(uint32 entry,uint32 item);
-        bool IsVendorItemValid( uint32 vendor_entry, uint32 item, uint32 maxcount, uint32 ptime, uint32 ExtendedCost, Player* pl = NULL, std::set<uint32>* skip_vendors = NULL ) const;
+        bool IsVendorItemValid( uint32 vendor_entry, uint32 item, uint32 maxcount, uint32 ptime, uint8 UseBuyPrice, uint32 ExtendedCost, Player* pl = NULL, std::set<uint32>* skip_vendors = NULL ) const;
 
         void LoadScriptNames();
         ScriptNameMap &GetScriptNames() { return m_scriptNames; }
diff --git a/src/game/Player.cpp b/src/game/Player.cpp
index 55fee16..4d135bc 100644
--- a/src/game/Player.cpp
+++ b/src/game/Player.cpp
@@ -18469,12 +18469,12 @@ bool Player::BuyItemFromVendorSlot(uint64 vendorguid, uint32 vendorslot, uint32
             return false;
         }
     }
-
+	
     uint32 price  = pProto->BuyPrice * count;
 
     // reputation discount
     price = uint32(floor(price * GetReputationPriceDiscount(pCreature)));
-
+	if (!crItem->UseBuyPrice) price=0;
     if (GetMoney() < price)
     {
         SendBuyError( BUY_ERR_NOT_ENOUGHT_MONEY, pCreature, item, 0);

и sql

PHP код:

ALTER TABLE npc_vendor  ADD COLUMN `UseBuyPrice` tinyint(2) UNSIGNED NOT NULL default '1' AFTER incrtime;
UPDATE `command` SET `help`='Syntax: .npc additem #itemId <#maxcount><#incrtime><#usebuyprice><#extendedcost>r\r\n\r\nAdd item #itemid to item list of selected vendor. Also optionally set max count item in vendor item list and time to item count restoring and items ExtendedCost.\r\nIf #usebuyprice=0 then Use only #extendedcost for buy item.' WHERE (`name`='npc additem') 


Теперь если в таблице npc_vendor UseBuyPrice=1 то цена берется как и раньше из item_template.BuyCount а если UseBuyPrice=0 то цена из item_template.BuyCount не учитывается.

ЗЫ может лучше тупо запихнуть цену шмотки в таблицу npc_vendor?

[Energy]

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

ЗЫ может лучше тупо запихнуть цену шмотки в таблицу npc_vendor?

Наверное это наиболее логичный вариант. С другой стороны, из чего считать тогда цену продажи? Кажется это поле есть в вдб кеше.

А что если просто сделать так - при наличии цены из dbc не отображать стоимость в голде?

Случаем нет ли предметов, которые продаются за деньги + нестандартную валюту? Вроде бы пвп вода была... И в таком случае не указывается ли цена в голде в той же строчке в дбц что и в предметах?

[Shadez]

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

В данной реализации таблици сделать такое невозможно

Что значит невозможно? А ExtendedCost нам зачем тогда?

SELECT * FROM `npc_vendor` WHERE item =36908

[Energy]

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

Что значит невозможно? А ExtendedCost нам зачем тогда?

SELECT * FROM `npc_vendor` WHERE item =36908

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

[tempura]

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

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

Ошибаешься. alien сложным патчем продублировал работу колонки extended_cost таблицы npc_vendor.


alien
Большая просьба - использоваться теги [ code ] вместо тегов quote для программного кода и теги [ php ] для sql фрагментов.

[timmit]

Если у вендора принудительно не стоит использовать экстенд - тогда на то золото, что стоит в итем_темплате, если стоит - то только за этот экстенд.
Вики то почитать не судьба была, прежде чем патч писать?

[KiriX]

Шадез и темпура совершенно правы.
Юзаем колонку ExtendedCost для итема у того вендора, у которого она продаётся за обмен, и стандартную колонку цены, у кого она продаётся просто за деньги.
Патч бессмысленный, но за попытку спасибо - вам плюс в опыт работы с кодом пошёл

Теперь если в таблице npc_vendor UseBuyPrice=1 то цена берется как и раньше из item_template.BuyCount а если UseBuyPrice=0 то цена из item_template.BuyCount не учитывается.

item_template.BuyCount - это вовсе не цена - это минимальный объём покупки данной вещи...

[SeT]

Если у вендора принудительно не стоит использовать экстенд - тогда на то золото, что стоит в итем_темплате, если стоит - то только за этот экстенд.
Вики то почитать не судьба была, прежде чем патч писать?

Голд все равно будет браться, даже если стоит ExCost.

[PSZ]

Дак да. Есть такое.

[KiriX]

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

Голд все равно будет браться, даже если стоит ExCost.

Даже если и так - патч всё равно неверный.
А для исправления взятия голда за итем, который можно купить за вещи нужно писать иной патч, что-то типа if (ExCost !=0) BuyPrice=0;
Но тут доп-проверки нужны...

[alien]

item_template.BuyCount - это вовсе не цена - это минимальный объём покупки данной вещи...

ошипятка сори.
так меня кажись многие не поняли.
SeT и PSZ совершенно правы.
Если есть значение ExtendedCos и Item_template.BuyPrice!=0
То тогда у вендора будет отображаться цена за голд и за ExtendedCos.
Тоесть помимо шмотки указанной в ExtendedCos вам нужно будет отдать и голд.
как я уже приводил примеры.
http://ru.wowhead.com/item=44128#sold-by
Тут например мех продается за голд у одного вендора а за Кожу у другого. При этом там где он за кожу он без голды.
вот еще http://ru.wowhead.com/item=36908#sold-by

что-то типа if (ExCost !=0) BayPrice=0;

Так нельзя есть шмотки которые требуют и голд и ексткост.

[YuruY]

Голд серовно будет браться, даже если стоит ExCost.

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

[alien]

http://ru.wowhead.com/item=46750
Вот пример итема которые продается и за екст кост и за голду

[KiriX]

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

http://ru.wowhead.com/item=46750
Вот пример итема которые продается и за екст кост и за голду

А в этом случае цена в деньгах не в екст кост пишется?

[tempura]

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

А в этом случае цена в деньгах не в екст кост пишется?

Вот-вот, мне тоже самое мерещится.

[tempura]

Так, стоп. Сейчас:

1.
item_template.BuyPrice>0
npc_vendor.ExtendedCost=0
- предмет продается за деньги

2.
item_template.BuyPrice>0
npc_vendor.ExtendedCost>0
- предмет продается за деньги и за экстенд

3.
item_template.BuyPrice=0
npc_vendor.ExtendedCost>0
- предмет продается за экстенд, денег не требует.

Разве не так? И что к этому (какой вариант) добавляет предложеный патч?

А, кажется догонять начинаю... Еще вариант:

4.
item_template.BuyPrice=0 (у одного вендора) и
item_template.BuyPrice>0 (у другого вендора)
npc_vendor.ExtendedCost>1
- предмет продается за экстенд у одного, и за деньги+экстенд у другого.

В таком случае... ...а у нас в экстендах по dbc разве нет прайсов с "экстенд+деньги"? (или это у меня по какой-то другой таблице отложилось?)

[alien]

А в этом случае цена в деньгах не в екст кост пишется?

Если это так то все гораздо проще. Только честно не знаю как это посмотреть ))

ЗЫ если оно так как KiriX и tempura вы говорите то попроавить патч под такое дело не проблема.

Только что просмотрел ItemExtendedCost.csv из quice. там нету записей голд + екст
Искал на примере http://ru.wowhead.com/item=44990 теже маунты за них продаются за голд и 5 печатей в таблице есть только запись о печатях

2671;5 Champion's Seal (id=44990)

[KiriX]

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

Если это так то все гораздо проще. Только честно не знаю как это посмотреть ))

ЗЫ если оно так как KiriX и tempura вы говорите то попроавить патч под такое дело не проблема.

Просто проверить купив у вендора и глянув значение BuyPrice в базе для данного итема.

[tempura]

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

Если это так то все гораздо проще. Только честно не знаю как это посмотреть ))

ЗЫ если оно так как KiriX и tempura вы говорите то попроавить патч под такое дело не проблема.

Только что просмотрел ItemExtendedCost.csv из quice. там нету записей голд + екст
Искал на примере http://ru.wowhead.com/item=44990 теже маунты за них продаются за голд и 5 печатей в таблице есть только запись о печатях

Не, не так проверять. Надо найти предмет, который у одного продавца стоит ТОЛЬКО экстенд, а у другого - экстенд+голд. И вот по нему (по номеру из npc_vendor.ExtendedCost - он должен быть одинаковым) смотреть в dbc. А потом - и в игре его же проверить - то бишь купить у обоих продавцов, мониторя состояние денег и экстенда на момент до и после покупки.
Причем - желательно не по абстрактному "ItemExtendedCost.csv из quice", а по совершенно конкретным данным в клиентских dbc последней версии. А то мало ли что бывает... Вот уж поверь - не раз сталкивались с ситуацией, когда человек говорит "ой, а я не там смотрел".
Хотя, конечно, странно...

Вот если и тут будет joppa, вот тогда уже понадобится думать.

[alien]

http://ru.wowhead.com/item=43236
проверил купив. в базе стоит екстракост=2483 а в item_template.BuyPrice=10000
тоесть 1голд и 12 хонора.
на оффе также.


tempura нету таких торговцев у которых одна вещь только за екстенд а у вторых екстенд+годл.
Есть у одних только екстенд у других только голд(это про одну шмотку)
или уже сразу екстенд+голд.(но тогда это шмотку только так и продается)

Так конвертнул DBC из последней версии клиента в cvs.
Посмотрел нету в нем записей о голде.

[tempura]

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

tempura нету таких торговцев у которых одна вещь только за екстенд а у вторых екстенд+годл.
Есть у одних только екстенд у других только голд(это про одну шмотку)
или уже сразу екстенд+голд.(но тогда это шмотку только так и продается)

Это хорошо, что нету. Хотя это и неважно. Я совершенно другое писал.
1. Вещь - ОДНА!!! Одна вещь, один предмет - понимаешь? Предмет - один.
2. Торговца - ДВА. Два разных торговца. Два торговца, и они абсолютно разные - понимаешь? Торговцы разные, и их два.

Одна вещь у двух торговцев имеет разную цену. У первого торговца - экстенд, у второго торговца - экстенд+голд. Одна вещь у двух торговцев с разной ценой.

Понимаешь?

[alien]

Ну я и говорю что нету такого предмета которые продается у одного торговца за экстенд+голд а у другого только за голд.(это я про оффицал)
А у мангоса могу и так сказать как будет.

[tempura]

Собственно - о чем и разговор. Близзы не применяют подобную практику. Перед тем, как что-то делать - просто необходимо убедиться в смысловой нагрузке данного деяния.
На данный момент я считаю, что если подобная работа не реализована, то это недосмотр - потому что впоследствии может потребоваться подобная обработка. Но на текущий момент она просто не требуется. А раз не требуется - то делать ее следует лишь в качестве "интеллектуальной тренировки", то бишь развития мозгов.


И еще - личный момент, который сразу же, с первого момента, вызвал неприятие данного патча - он требует добавления в стандартные таблицы лишнего поля. Я искренне и неподдельно считаю, что любое изменение таблиц - это слишком верьезный шаг, чтобы идти на него от балды "тут вот чуток не совсем как-то так". Вероятно должно быть решение (если онно вдруг понадобится), которое не затрагивает таблицы.

В принципе сейчас полей с уникальными guid-id с автоинкрементом можно навтыкать в каждую таблицу под всякие разные обработки. Да и количество таблиц можно бы утроить-учетверить. Вот только их и так до хрена... Короче - в итоге - думать надо. Сильно думать по любому изменению и добавлению.


PS

Так конвертнул DBC из последней версии клиента в cvs.
Посмотрел нету в нем записей о голде

А десятое поле - разве не деньги? Погоди... или пятнадцатое? Или пятнадцатое поле - ХР?

[alien]

Да я тоже понимаю что редактировать таблицы не есть гуд но по другому невозможно.
Да нету шмоток которые продаются за голд+екстенд в одном месте и только за голд или только за екстенд в другом.
Давайте разберем все по полочкам.
1) Предмет продается только за голд. -- Тут все отлично.
2) Предмет продается только за екстенд. -- Тут все отлично.
3) Предмет у одного торговца продается только за екстенд у второго только за голд. -- Вот это реализовать в текущем состоянии невозможно! -- И такое есть на оффицале.
4) Предмет продается и за екстенд и за голд(тоесть Предмет стоит энное количество голд+екстенд) -- Тоже все нормально.
5) Предмет у одного торговца продается за голд а у второго за екстенд+голд. -- Вот такого не оффе нету и врядли будет.

[tempura]

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

3) Предмет у одного торговца продается только за екстенд у второго только за голд. -- Вот это реализовать в текущем состоянии невозможно! -- И такое есть на оффицале.

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

Но тут уже лично мне хочется наконец увидеть хоть какой-то комментарий от Vladimir. Вот кто точно расставит все точки над "ё" и червяков над "й".

[alien]

PS

А десятое поле - разве не деньги? Погоди... или пятнадцатое? Или пятнадцатое поле - ХР?

Нет 10 это склько http://ru.wowhead.com/item=37829
а 15 это вроде личный рейтинг?

[Vladimir]

Тоесть, подтверждается что нет у вендоров предметов продаваемых одновремеено за money+extanded_cost в одном слоте?

[LordJZ]

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

Тоесть, подтверждается что нет у вендоров предметов продаваемых одновремеено за money+extanded_cost в одном слоте?

Вот эти итемчики продаются за голд+токен: http://www.wowhead.com/items=4?filte...1;crv=0:0;gb=0
Скрин с офа: http://filebeam.com/617de641cae8a3b1...c0ff5e9c65.jpg

[alien]

Vladimir,
Предметы которые продаются за голд+екстенд_кост есть.
Но также есть предметы которые продаются у одного вендора только за голд а у другого только за екстенд кост.

[Neggod]

Пример: Ледо Прославленный на офе и на мангосе

[tempura]

Предмет у одного торговца продается только за екстенд у второго только за голд

Frost Lotus
Frozo the Renowned - 1 frozen orb
Lhara - 4 g

Один предмет. Вполне может быть ошибкой ВоВхеда, либо устаревшими данными.
Один предмет может быть случайностью, он не обеспечивает достаточной частоты выборки.

[alien]

На вовхеде все верно.
Дело в том что за 4г он продается на ярмарке новолунной а за орбы в даларане у ледо.
Также еще я линовал в первом посте на арктический мех. Который также продается на ярмарке за 12г а у чела который торгует рецептами за 10 толстой борейской кожи.
Также у ледо продаются многие штуки за орбы у которых в item_template.BuyPrice стоит цена отличная от 0. Вот тут незнаю насколько ее можно равнять 0.
ЗЫ говорю с увереностью так-как сам на оффе обитаю )

[Vladimir]

Реализовано в [9957]