diff --git a/src/NPCManager.cpp b/src/NPCManager.cpp index 1d80a93..bd72fb0 100644 --- a/src/NPCManager.cpp +++ b/src/NPCManager.cpp @@ -94,20 +94,49 @@ void NPCManager::sendToViewable(Entity *npc, void *buf, uint32_t type, size_t si static void npcBarkHandler(CNSocket* sock, CNPacketData* data) { sP_CL2FE_REQ_BARKER* req = (sP_CL2FE_REQ_BARKER*)data->buf; - // get bark IDs from task data - TaskData* td = Missions::Tasks[req->iMissionTaskID]; - std::vector barks; - for (int i = 0; i < 4; i++) { - if (td->task["m_iHBarkerTextID"][i] != 0) // non-zeroes only - barks.push_back(td->task["m_iHBarkerTextID"][i]); + int taskID = req->iMissionTaskID; + // ignore req->iNPC_ID as it is often fixated on a single npc in the region + + if (Missions::Tasks.find(taskID) == Missions::Tasks.end()) { + std::cout << "mission task not found: " << taskID << std::endl; + return; } - if (barks.empty()) - return; // no barks + TaskData* td = Missions::Tasks[taskID]; + auto& barks = td->task["m_iHBarkerTextID"]; + + Player* plr = PlayerManager::getPlayer(sock); + std::vector> npcLines; + + for (Chunk* chunk : plr->viewableChunks) { + for (auto ent = chunk->entities.begin(); ent != chunk->entities.end(); ent++) { + if (ent->kind != EntityKind::SIMPLE_NPC) + continue; + + BaseNPC* npc = (BaseNPC*)ent->getEntity(); + if (npc->type < 0 || npc->type >= NPCData.size()) + continue; // npc unknown ?! + + int barkType = NPCData[npc->type]["m_iBarkerType"]; + if (barkType < 1 || barkType > 4) + continue; // no barks + + int barkID = barks[barkType - 1]; + if (barkID == 0) + continue; // no barks + + npcLines.push_back(std::make_pair(npc->id, barkID)); + } + } + + if (npcLines.size() == 0) + return; // totally no barks + + auto& [npcID, missionStringID] = npcLines[Rand::rand(npcLines.size())]; INITSTRUCT(sP_FE2CL_REP_BARKER, resp); - resp.iNPC_ID = req->iNPC_ID; - resp.iMissionStringID = barks[Rand::rand(barks.size())]; + resp.iNPC_ID = npcID; + resp.iMissionStringID = missionStringID; sock->sendPacket(resp, P_FE2CL_REP_BARKER); }