Фильтр отображения предметов у торговца.

[alien]

Вобщем это некое обсуждение перед реализацией в патче.
Возьмем к примеру этого NPC
http://ru.wowhead.com/npc=32216
На оффе он не показывает те маунты которые вы не можете купить будучи другой фракцией.
Вопрос вот в чем какие фильтры у торговцев на оффе есть?

[tempura]

В смысле - по расе, по классу, по фракции?
По уровню или скилам ставить фильтр точно смысла нет.

[Fedia22]

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

В смысле - по расе, по классу, по фракции?
По уровню или скилам ставить фильтр точно смысла нет.

у человека на оффе не отображается "полёты в непогоду" хотя он ещё 74 и он их не выучил, предположение что появиться на 77, проверим!

[Lightunit]

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

у человека на оффе не отображается "полёты в непогоду" хотя он ещё 74 и он их не выучил, предположение что появиться на 77

На мангосе, книгу "полёты в непогоду", можно купить на 77 лвле. До достижения 77 уровня, НИП даже не торгует.

[Fedia22]

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

На мангосе, книгу "полёты в непогоду", можно купить на 77 лвле. До достижения 77 уровня, НИП даже не торгует.

я не про книгу! я про обучение!

[timmit]

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

я не про книгу! я про обучение!

а причем тут обучение и эта тема? Или мы уже читать разучились?

[YuruY]

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

Вобщем это некое обсуждение перед реализацией в патче.
Возьмем к примеру этого NPC
http://ru.wowhead.com/npc=32216
На оффе он не показывает те маунты которые вы не можете купить будучи другой фракцией.
Вопрос вот в чем какие фильтры у торговцев на оффе есть?

М.б: "item_template" => Faction=1/2.

[NeatElves]

Задача сложнее.)

1. Берем вендора с турнира, например http://ru.wowhead.com/npc=33657, у него есть 2 маунта одинаковых http://ru.wowhead.com/item=46745 и http://ru.wowhead.com/item=46756, далал еще на 332 скрины - первый видно только своим, второй всем остальным...

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

[alien]

Вобщем я еще вот к чему.
Берем этого нпс http://ru.wowhead.com/npc=32216
Он продает к примеру http://ru.wowhead.com/item=44689 и http://ru.wowhead.com/item=44690
В "item_template" Faction стоит верно.
Но в самом сервере нету ограничение на отображение этих предметов в доступных для покупки предметов. То есть сервер не фильтрует их.
На стороне сервера есть только вот такой фильтр http://github.com/mangos/mangos/blob...temHandler.cpp

if((pProto->AllowableClass & _player->getClassMask()) == 0 && pProto->Bonding == BIND_WHEN_PICKED_UP && !_player->isGameMaster())
                    continue;

Собственно надо просто добавить фильтр по фракции. Щас добавил проверю как это выглядит на сервере.
Но тут еще 1 маленькая проблемка с вендорами нарисовалась.

Вот патч на фракцию

diff --git a/src/game/ItemHandler.cpp b/src/game/ItemHandler.cpp
index 41796a7..3d9057e 100644
--- a/src/game/ItemHandler.cpp
+++ b/src/game/ItemHandler.cpp
@@ -756,7 +756,8 @@ void WorldSession::SendListInventory(uint64 vendorguid)
             {
                 if((pProto->AllowableClass & _player->getClassMask()) == 0 && pProto->Bonding == BIND_WHEN_PICKED_UP && !_player->isGameMaster())
                     continue;
-
+		if((pProto->AllowableRace & _player->getRaceMask()) == 0 && pProto->Bonding == BIND_WHEN_PICKED_UP && !_player->isGameMaster())
+			continue;
+
                 ++count;
 
                 // reputation discount

Так и вот еще 1 патчик ))
Если после применения фильтров у вендора не будет ни одного предмета который он может предложить на продажу то тогда вендор не откроет окно )

diff --git a/src/game/ItemHandler.cpp b/src/game/ItemHandler.cpp
index 41796a7..ce94160 100644
--- a/src/game/ItemHandler.cpp
+++ b/src/game/ItemHandler.cpp
@@ -774,9 +775,11 @@ void WorldSession::SendListInventory(uint64 vendorguid)
         }
     }
 
-    if ( count == 0 || data.size() != 8 + 1 + size_t(count) * 8 * 4 )
-        return;
-
+	if(data.size() != 8 + 1 + size_t(count) * 8 * 4)
+		return;
+    if ( count == 0 )
+        data << uint8(0);
+	   
     data.put<uint8>(count_pos, count);
     SendPacket( &data );
 }

+ if ( count == 0 )
+ data << uint8(0);
тут дело вот в чем отснифил пакет с оффа получил пакет длинной 10 байт
а у нас если вендор не предлогает ни одной шмотки на продажу то 9 ))
Вот собственно байтик дописываем )

[TOM_RUS]

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

тут дело вот в чем отснифил пакет с оффа получил пакет длинной 10 байт
а у нас если вендор не предлогает ни одной шмотки на продажу то 9 ))
Вот собственно байтик дописываем )

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

И вообще проверка

    if ( count == 0 || data.size() != 8 + 1 + size_t(count) * 8 * 4 )
        return;

имхо лишняя...

[alien]

Да я видел что в текущем состоянии пакет не отправляется.
Но если вообще убрать
if ( count == 0 || data.size() != 8 + 1 + size_t(count) * 8 * 4 )
return;
то клиент небудет реагировать на пакет. Нужен дописать тот байтик.

А проверку оставил так-как хрен его знает какая логика была у того человека который ее писал, и не мне решать убирать или оставлять ее.

[alien]

Вопрос к администрации.
Можно как-то 7 пост перенести в раздел Патчи на рассмотрении
или просто от Vladimir какой-нибудь коммент услышать

[tempura]

он увидит... И если нужно - перенесет.