Changeset 3197

Timestamp:

03/04/2010 01:43:09 PM (6 months ago)

Author:

jackpoz

Message:

MODIFIED: Mass replaced NGLog.Error(string error) with sLog.outError(string error). This will save SQL errors to file.log (if enabled in configs).

Location:

trunk/src

Files:

• arcemu-shared/Database/MySQLDatabase.cpp (modified) (1 diff)
• arcemu-world/ArathiBasin.cpp (modified) (1 diff)
• arcemu-world/BattlegroundMgr.cpp (modified) (7 diffs)
• arcemu-world/EyeOfTheStorm.cpp (modified) (1 diff)
• arcemu-world/InstanceCommands.cpp (modified) (1 diff)
• arcemu-world/LocalizationMgr.cpp (modified) (1 diff)
• arcemu-world/LootMgr.cpp (modified) (4 diffs)
• arcemu-world/Master.cpp (modified) (2 diffs)

trunk/src/arcemu-shared/Database/MySQLDatabase.cpp

@@ -159 +159 @@
                 }
                 else
-                        Log.Error("Sql query failed due to [%s], Query: [%s]\n", mysql_error( static_cast<MySQLDatabaseConnection*>(con)->MySql ), Sql);
+                        sLog.outError("Sql query failed due to [%s], Query: [%s]", mysql_error( static_cast<MySQLDatabaseConnection*>(con)->MySql ), Sql);
         }
 

trunk/src/arcemu-world/ArathiBasin.cpp

@@ -685 +685 @@
                 break;
         default:
-                Log.Error("ArathiBasin", "Encountered unhandled areatrigger id %u", id);
+                sLog.outError("ArathiBasin", "Encountered unhandled areatrigger id %u", id);
                 return;
                 break;

trunk/src/arcemu-world/BattlegroundMgr.cpp

@@ -404 +404 @@
         if (ar == NULL)
         {
-                Log.Error("BattlegroundMgr", "%s (%u): Couldn't create Arena Instance", __FILE__, __LINE__);
+                sLog.outError("BattlegroundMgr", "%s (%u): Couldn't create Arena Instance", __FILE__, __LINE__);
                 m_queueLock.Release();
                 m_instanceLock.Release();
@@ -583 +583 @@
                                         if ( arena == NULL )
                                         {
-                                                Log.Error("BattlegroundMgr", "%s (%u): Couldn't create Arena Instance", __FILE__, __LINE__);
+                                                sLog.outError("BattlegroundMgr", "%s (%u): Couldn't create Arena Instance", __FILE__, __LINE__);
                                                 m_queueLock.Release();
                                                 m_instanceLock.Release();
@@ -674 +674 @@
                         if(itz == m_queuedGroups[i].end())
                         {
-                                Log.Error("BattlegroundMgr", "Internal error at %s:%u", __FILE__, __LINE__);
+                                sLog.outError("BattlegroundMgr", "Internal error at %s:%u", __FILE__, __LINE__);
                                 m_queueLock.Release();
                                 m_instanceLock.Release();
@@ -721 +721 @@
                                 if(itz == possibleGroups.end())
                                 {
-                                        Log.Error("BattlegroundMgr", "Internal error at %s:%u", __FILE__, __LINE__);
+                                        sLog.outError("BattlegroundMgr", "Internal error at %s:%u", __FILE__, __LINE__);
                                         m_queueLock.Release();
                                         m_instanceLock.Release();
@@ -1227 +1227 @@
                 if(mgr == NULL)
                 {
-                        Log.Error("BattlegroundManager", "Arena CreateInstance() call failed for map %u, type %u, level group %u", mapid, Type, LevelGroup);
+                        sLog.outError("BattlegroundManager", "Arena CreateInstance() call failed for map %u, type %u, level group %u", mapid, Type, LevelGroup);
                         return NULL;      // Shouldn't happen
                 }
@@ -1262 +1262 @@
         if(cfunc == NULL)
         {
-                Log.Error("BattlegroundManager", "Could not find CreateBattlegroundFunc pointer for type %u level group %u", Type, LevelGroup);
+                sLog.outError("BattlegroundManager", "Could not find CreateBattlegroundFunc pointer for type %u level group %u", Type, LevelGroup);
                 return NULL;
         }
@@ -1293 +1293 @@
         if(mgr == NULL)
         {
-                Log.Error("BattlegroundManager", "CreateInstance() call failed for map %u, type %u, level group %u", BGMapIds[Type], Type, LevelGroup);
+                sLog.outError("BattlegroundManager", "CreateInstance() call failed for map %u, type %u, level group %u", BGMapIds[Type], Type, LevelGroup);
                 return NULL;      // Shouldn't happen
         }

trunk/src/arcemu-world/EyeOfTheStorm.cpp

@@ -294 +294 @@
                 break;
         default:
-                Log.Error("EyeOfTheStorm", "Encountered unhandled areatrigger id %u", id);
+                sLog.outError("EyeOfTheStorm", "Encountered unhandled areatrigger id %u", id);
                 return;
                 break;

trunk/src/arcemu-world/InstanceCommands.cpp

@@ -327 +327 @@
         if(mgr == NULL)
         {
-                Log.Error("CreateInstanceGMCommand", "CreateInstance() call failed for map %u", mapid);
+                sLog.outError("CreateInstanceGMCommand", "CreateInstance() call failed for map %u", mapid);
                 return false;      // Shouldn't happen
         }

trunk/src/arcemu-world/LocalizationMgr.cpp

@@ -548 +548 @@
                 if(source_language_id== 0 || dest_language_id == 0)
                 {
-                        Log.Error("LocalizationMgr", "Invalid locale conversion string specified: %u->%u (%s->%s)", source_language_id, dest_language_id, itr->second.c_str(), itr->first.c_str());
+                        sLog.outError("LocalizationMgr", "Invalid locale conversion string specified: %u->%u (%s->%s)", source_language_id, dest_language_id, itr->second.c_str(), itr->first.c_str());
                         continue;
                 }

trunk/src/arcemu-world/LootMgr.cpp

@@ -158 +158 @@
                         if(rp == NULL)
                         {
-                                Log.Error("LoadLootProp", "RandomProp group %u references non-existent randomprop %u.", id, eid);
+                                sLog.outError("LoadLootProp", "RandomProp group %u references non-existent randomprop %u.", id, eid);
                                 continue;
                         }
@@ -190 +190 @@
                         if(rs == NULL)
                         {
-                                Log.Error("LoadLootProp", "RandomSuffix group %u references non-existent randomsuffix %u.", id, eid);
+                                sLog.outError("LoadLootProp", "RandomSuffix group %u references non-existent randomsuffix %u.", id, eid);
                                 continue;
                         }
@@ -247 +247 @@
         if(!result)
         {
-                Log.Error("LootMgr", "Loading loot from table %s failed.", szTableName);
+                sLog.outError("LootMgr", "Loading loot from table %s failed.", szTableName);
                 return;
         }
@@ -263 +263 @@
                 if(entry_id < last_entry)
                 {
-                        Log.Error("LootMgr", "WARNING: Out of order loot table being loaded.\n");
+                        sLog.outError("LootMgr", "WARNING: Out of order loot table being loaded.\n");
                         return;
                 }

trunk/src/arcemu-world/Master.cpp

@@ -254 +254 @@
         else
         {
-                Log.Error( "Config", ">> configs/world.conf" );
+                sLog.outError( "Config", ">> configs/world.conf" );
                 return false;
         }
@@ -262 +262 @@
         else
         {
-                Log.Error("Config", ">> configs/optional.conf");
+                sLog.outError("Config", ">> configs/optional.conf");
                 return false;
         }