mangos-mirror/server
0
0
Fork 0
mirror of https://github.com/mangosfour/server.git synced 2025-12-13 13:37:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

[11948] Check achievement related dbc data

Browse source 
This will fixed a few crashs for criterias without referred achievements

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
This commit is contained in:
Schmoozerd 2012-04-11 21:55:26 +02:00
parent ea08f7c5d7
commit 31e46daaf8
3 changed files with 27 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

34
src/game/AchievementMgr.cpp
View file

@ -436,8 +436,7 @@ void AchievementMgr::ResetAchievementCriteria(AchievementCriteriaTypes type, uin
AchievementCriteriaEntry const *achievementCriteria = (*i);
AchievementEntry const *achievement = sAchievementStore.LookupEntry(achievementCriteria->referredAchievement);
if (!achievement)
continue;
// Checked in LoadAchievementCriteriaList
// don't update already completed criteria
if (IsCompletedCriteria(achievementCriteria,achievement))
@ -578,6 +577,7 @@ void AchievementMgr::LoadFromDB(QueryResult *achievementResult, QueryResult *cri
progress.timedCriteriaFailed = false;
AchievementEntry const* achievement = sAchievementStore.LookupEntry(criteria->referredAchievement);
// Checked in LoadAchievementCriteriaList
// A failed achievement will be removed on next tick - TODO: Possible that timer 2 is reseted
if (criteria->timeLimit)
@ -717,8 +717,7 @@ void AchievementMgr::StartTimedAchievementCriteria(AchievementCriteriaTypes type
continue;
AchievementEntry const *achievement = sAchievementStore.LookupEntry(achievementCriteria->referredAchievement);
if (!achievement)
continue;
// Checked in LoadAchievementCriteriaList
if ((achievement->factionFlag == ACHIEVEMENT_FACTION_FLAG_HORDE && GetPlayer()->GetTeam() != HORDE) ||
(achievement->factionFlag == ACHIEVEMENT_FACTION_FLAG_ALLIANCE && GetPlayer()->GetTeam() != ALLIANCE))
@ -778,6 +777,7 @@ void AchievementMgr::DoFailedTimedAchievementCriterias()
// Possible failed achievement criteria found
AchievementCriteriaEntry const* criteria = sAchievementCriteriaStore.LookupEntry(iter->first);
AchievementEntry const* achievement = sAchievementStore.LookupEntry(criteria->referredAchievement);
// Checked in LoadAchievementCriteriaList
// Send Fail for failed criterias
if (!IsCompletedCriteria(criteria, achievement))
@ -817,8 +817,7 @@ void AchievementMgr::UpdateAchievementCriteria(AchievementCriteriaTypes type, ui
AchievementCriteriaEntry const *achievementCriteria = *itr;
AchievementEntry const *achievement = sAchievementStore.LookupEntry(achievementCriteria->referredAchievement);
if (!achievement)
continue;
// Checked in LoadAchievementCriteriaList
if ((achievement->factionFlag == ACHIEVEMENT_FACTION_FLAG_HORDE && GetPlayer()->GetTeam() != HORDE) ||
(achievement->factionFlag == ACHIEVEMENT_FACTION_FLAG_ALLIANCE && GetPlayer()->GetTeam() != ALLIANCE))
@ -2269,11 +2268,20 @@ void AchievementGlobalMgr::LoadAchievementCriteriaList()
bar.step();
AchievementCriteriaEntry const* criteria = sAchievementCriteriaStore.LookupEntry(entryId);
if(!criteria)
if (!criteria)
continue;
MANGOS_ASSERT(criteria->requiredType < ACHIEVEMENT_CRITERIA_TYPE_TOTAL && "Not updated ACHIEVEMENT_CRITERIA_TYPE_TOTAL?");
// check if referredAchievement exists!
AchievementEntry const* achiev = sAchievementStore.LookupEntry(criteria->referredAchievement);
if (!achiev)
{
sLog.outDetail("Removed achievement-criteria %u, because referred achievement does not exist", entryId);
sAchievementCriteriaStore.EraseEntry(entryId);
continue;
}
m_AchievementCriteriasByType[criteria->requiredType].push_back(criteria);
m_AchievementCriteriaListByAchievement[criteria->referredAchievement].push_back(criteria);
}
@ -2304,6 +2312,15 @@ void AchievementGlobalMgr::LoadAchievementReferenceList()
if (!achievement || !achievement->refAchievement)
continue;
// Check refAchievement exists
AchievementEntry const* refAchiev = sAchievementStore.LookupEntry(achievement->refAchievement);
if (!refAchiev)
{
sLog.outDetail("Removed achieviement %u, because referred achievement does not exist", entryId);
sAchievementStore.EraseEntry(entryId);
continue;
}
m_AchievementListByReferencedId[achievement->refAchievement].push_back(achievement);
++count;
}
@ -2388,8 +2405,7 @@ void AchievementGlobalMgr::LoadAchievementCriteriaRequirements()
case ACHIEVEMENT_CRITERIA_TYPE_COMPLETE_QUEST:
{
AchievementEntry const* achievement = sAchievementStore.LookupEntry(criteria->referredAchievement);
if(!achievement)
continue;
// Checked in LoadAchievementCriteriaList
// exist many achievements with this criteria, use at this moment hardcoded check to skil simple case
switch(achievement->ID)

2
src/game/World.cpp
View file

@ -1185,8 +1185,8 @@ void World::SetInitialWorldSettings()
sLog.outString( "Loading Skill Fishing base level requirements..." );
sObjectMgr.LoadFishingBaseSkillLevel();
sLog.outString( "Loading Achievements..." );
sLog.outString();
sLog.outString( "Loading Achievements..." );
sAchievementMgr.LoadAchievementReferenceList();
sAchievementMgr.LoadAchievementCriteriaList();
sAchievementMgr.LoadAchievementCriteriaRequirements();

2
src/shared/revision_nr.h
View file

@ -1,4 +1,4 @@
#ifndef __REVISION_NR_H__
#define __REVISION_NR_H__
#define REVISION_NR "11947"
#define REVISION_NR "11948"
#endif // __REVISION_NR_H__