nullptr fix

src/PlayerbotAI.cpp

@@ -4214,19 +4214,6 @@ bool PlayerbotAI::AllowActive(ActivityType activityType)
         }
     }
 
-    // only keep updating till initializing time has completed,
-    // which prevents unneeded expensive GameTime calls.
-    if (_isBotInitializing)
-    {
-        _isBotInitializing = GameTime::GetUptime().count() < sPlayerbotAIConfig->maxRandomBots * 0.11;
-
-        // no activity allowed during bot initialization
-        if (_isBotInitializing)
-        {
-            return false;
-        }
-    }
-
     // General exceptions
     if (activityType == PACKET_ACTIVITY)
     {

src/PlayerbotAI.h

@@ -611,7 +611,6 @@ private:
     Item* FindItemInInventory(std::function<bool(ItemTemplate const*)> checkItem) const;
     void HandleCommands();
     void HandleCommand(uint32 type, const std::string& text, Player& fromPlayer, const uint32 lang = LANG_UNIVERSAL);
-    bool _isBotInitializing = false;
 
 protected:
     Player* bot;

src/Playerbots.cpp

@@ -108,7 +108,7 @@ public:
                     "|cffcccccchttps://github.com/liyunfan1223/mod-playerbots|r");
             }
 
-            if (sPlayerbotAIConfig->enabled || sPlayerbotAIConfig->randomBotAutologin)
+            /*if (sPlayerbotAIConfig->enabled || sPlayerbotAIConfig->randomBotAutologin)
             {
                 std::string roundedTime =
                     std::to_string(std::ceil((sPlayerbotAIConfig->maxRandomBots * 0.11 / 60) * 10) / 10.0);
@@ -117,7 +117,7 @@ public:
                 ChatHandler(player->GetSession()).SendSysMessage(
                     "|cff00ff00Playerbots:|r bot initialization at server startup takes about '" 
                     + roundedTime + "' minutes.");
-            }
+            }*/
         }
     }
 

src/factory/PlayerbotFactory.cpp

@@ -166,27 +166,18 @@ void PlayerbotFactory::Init()
         }
         
         ItemTemplate const* proto = sObjectMgr->GetItemTemplate(gemId);
-        if (!proto)
+        if (proto) {
-        {
+            if (proto->ItemLevel < 60)
-            continue;
+                continue;
-        }
 
-        if (proto->ItemLevel < 60)
+            if (proto->Flags & ITEM_FLAG_UNIQUE_EQUIPPABLE)
-        {
+               continue;
-            continue;
-        }
-        
-        if (proto->Flags & ITEM_FLAG_UNIQUE_EQUIPPABLE)
-        {
-            continue;
         }
         
         if (sRandomItemMgr->IsTestItem(gemId))
-        {
+            continue;
-           continue;
-        }
 
-        if (!sGemPropertiesStore.LookupEntry(proto->GemProperties))
+        if (!proto || !sGemPropertiesStore.LookupEntry(proto->GemProperties))
         {
             continue;
         }
@@ -194,6 +185,7 @@ void PlayerbotFactory::Init()
         // LOG_INFO("playerbots", "Add {} to enchantment gems", gemId);
         enchantGemIdCache.push_back(gemId);
     }
+    
     LOG_INFO("playerbots", "Loading {} enchantment gems", enchantGemIdCache.size());
 }
 
@@ -1028,14 +1020,16 @@ void PlayerbotFactory::ClearSkills()
     }
     bot->SetUInt32Value(PLAYER_SKILL_INDEX(0), 0);
     bot->SetUInt32Value(PLAYER_SKILL_INDEX(1), 0);
+
     // unlearn default race/class skills
-    PlayerInfo const* info = sObjectMgr->GetPlayerInfo(bot->getRace(), bot->getClass());
+    if (PlayerInfo const* info = sObjectMgr->GetPlayerInfo(bot->getRace(), bot->getClass())) {
-    for (PlayerCreateInfoSkills::const_iterator itr = info->skills.begin(); itr != info->skills.end(); ++itr)
+        for (PlayerCreateInfoSkills::const_iterator itr = info->skills.begin(); itr != info->skills.end(); ++itr)
-    {
+        {
-        uint32 skillId = itr->SkillId;
+            uint32 skillId = itr->SkillId;
-        if (!bot->HasSkill(skillId))
+            if (!bot->HasSkill(skillId))
-            continue;
+                continue;
-        bot->SetSkill(skillId, 0, 0, 0);
+            bot->SetSkill(skillId, 0, 0, 0);
+        }
     }
 }