Core: Fixed a few warnings and silly typos in scripts

src/server/collision/Maps/TileAssembler.cpp

@@ -500,10 +500,10 @@ namespace VMAP
         {
             model.setGroupModels(groupsArray);
 
-            std::string filename(iSrcDir);
+            std::string worldModelFileName(iSrcDir);
-            filename.push_back('/');
+            worldModelFileName.push_back('/');
-            filename.append(pModelFilename).append(".vmo");
+            worldModelFileName.append(pModelFilename).append(".vmo");
-            success = model.writeFile(filename);
+            success = model.writeFile(worldModelFileName);
         }
 
         //std::cout << "readRawFile2: '" << pModelFilename << "' tris: " << nElements << " nodes: " << nNodes << std::endl;

src/server/game/Tools/PlayerDump.cpp

@@ -443,9 +443,10 @@ DumpReturn PlayerDumpReader::LoadDump(const std::string& file, uint32 account, s
     typedef PetIds::value_type PetIdsPair;
     PetIds petids;
 
-    uint8 gender;
+    uint8 gender = GENDER_NONE;
-    uint8 race;
+    uint8 race = RACE_NONE;
-    uint8 playerClass;
+    uint8 playerClass = 0;
+
     SQLTransaction trans = CharacterDatabase.BeginTransaction();
     while (!feof(fin))
     {

src/server/scripts/EasternKingdoms/MoltenCore/boss_ragnaros.cpp

@@ -110,7 +110,7 @@ class boss_ragnaros : public CreatureScript
 
             void KilledUnit(Unit* /*victim*/)
             {
-                if (urand(0, 99 < 25))
+                if (urand(0, 99) < 25)
                     DoScriptText(SAY_KILL, me);
             }
 

src/server/scripts/EasternKingdoms/ScarletMonastery/boss_mograine_and_whitemane.cpp

@@ -102,7 +102,7 @@ public:
         {
             if (m_instance)
             {
-                if (m_instance->GetData(TYPE_MOGRAINE_AND_WHITE_EVENT != NOT_STARTED))
+                if (m_instance->GetData(TYPE_MOGRAINE_AND_WHITE_EVENT) != NOT_STARTED)
                     m_instance->SetData(TYPE_MOGRAINE_AND_WHITE_EVENT, FAIL);
             }
         }