[11877][patch]ZoneOrSort<0

[virusav]

Поле ZoneOrSort с отрицательным значением означает обычную группировку квестов в списке и никаким образом не связано с требованием профессий.
Проверялось на квестах 13311, 12518, 12798 и 12517, по которым сейчас кеш такой же, как и на 335а, но взять можно без профессии начертателя. На 335а было также.

В таблице квестов есть поля RequiredSkill и RequiredSkillValue, которые отвечают за требование профессии, поэтому проверка в ядре лишняя.

diff --git a/src/game/ObjectMgr.cpp b/src/game/ObjectMgr.cpp
index e2d0be3..fcbd610 100644
--- a/src/game/ObjectMgr.cpp
+++ b/src/game/ObjectMgr.cpp
@@ -3782,17 +3782,6 @@ void ObjectMgr::LoadQuests()
                     qinfo->GetQuestId(),qinfo->ZoneOrSort);
                 // no changes, quest not dependent from this value but can have problems at client (note some may be 0, we must allow this so no check)
             }
-
-            //check for proper RequiredSkill value (skill case)
-            if (uint32 skill_id = SkillByQuestSort(-int32(qinfo->ZoneOrSort)))
-            {
-                if (qinfo->RequiredSkill != skill_id)
-                {
-                    sLog.outErrorDb("Quest %u has `ZoneOrSort` = %i but `RequiredSkill` does not have a corresponding value (%u).",
-                        qinfo->GetQuestId(),qinfo->ZoneOrSort,skill_id);
-                    //override, and force proper value here?
-                }
-            }
         }
 
         // RequiredClasses, can be 0/CLASSMASK_ALL_PLAYABLE to allow any class

[Vladimir]

Это проверка непротиворечивости данных. Разве бывают квесты для профессии даваемые тем у кого этой профессии нет. Понятно дело что в клиенте это используется для группировки. Сервер опирающийся на данные клинета вынужден просто использовать их несколько шире клиента.

[virusav]

Группа квестов в списке и требование профессий - это разные вещи, поэтому проверка тут не нужна, проверено на офе.

[virusav]

Патч принят в 11877.