[Quest API] (Performance) Check event EVENT_COMBINE, EVENT_COMBINE_SUCCESS, EVENT_COMBINE_FAILURE, or EVENT_COMBINE_VALIDATE exist before export and execute (#2896)

* [Quest API] Optionally parse EVENT_COMBINE_SUCCESS and EVENT_COMBINE_FAILURE

- Optionally parse these events instead of always doing so.

* Update tradeskills.cpp

* Update tradeskills.cpp
This commit is contained in:
Alex King 2023-02-13 00:28:27 -05:00 committed by GitHub
parent 9f619859d1
commit 3d6b0e5f74
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -373,12 +373,14 @@ void Object::HandleCombine(Client* user, const NewCombine_Struct* in_combine, Ob
DBTradeskillRecipe_Struct spec; DBTradeskillRecipe_Struct spec;
if (parse->PlayerHasQuestSub(EVENT_COMBINE)) {
if (parse->EventPlayer(EVENT_COMBINE, user, std::to_string(in_combine->container_slot), 0) == 1) { if (parse->EventPlayer(EVENT_COMBINE, user, std::to_string(in_combine->container_slot), 0) == 1) {
auto outapp = new EQApplicationPacket(OP_TradeSkillCombine, 0); auto outapp = new EQApplicationPacket(OP_TradeSkillCombine, 0);
user->QueuePacket(outapp); user->QueuePacket(outapp);
safe_delete(outapp); safe_delete(outapp);
return; return;
} }
}
if (!content_db.GetTradeRecipe(container, c_type, some_id, user->CharacterID(), &spec)) { if (!content_db.GetTradeRecipe(container, c_type, some_id, user->CharacterID(), &spec)) {
@ -446,14 +448,18 @@ void Object::HandleCombine(Client* user, const NewCombine_Struct* in_combine, Ob
} }
// final check for any additional quest requirements .. "check_zone" in this case - exported as variable [validate_type] // final check for any additional quest requirements .. "check_zone" in this case - exported as variable [validate_type]
std::string export_string = fmt::format("check_zone {}", zone->GetZoneID()); if (parse->PlayerHasQuestSub(EVENT_COMBINE_VALIDATE)) {
if (parse->EventPlayer(EVENT_COMBINE_VALIDATE, user, export_string, spec.recipe_id) != 0) { if (parse->EventPlayer(EVENT_COMBINE_VALIDATE, user, fmt::format("check_zone {}", zone->GetZoneID()), spec.recipe_id) != 0) {
user->Message(Chat::Emote, "You cannot make this combine because the location requirement has not been met."); user->Message(
Chat::Emote,
"You cannot make this combine because the location requirement has not been met."
);
auto outapp = new EQApplicationPacket(OP_TradeSkillCombine, 0); auto outapp = new EQApplicationPacket(OP_TradeSkillCombine, 0);
user->QueuePacket(outapp); user->QueuePacket(outapp);
safe_delete(outapp); safe_delete(outapp);
return; return;
} }
}
// Send acknowledgement packets to client // Send acknowledgement packets to client
auto outapp = new EQApplicationPacket(OP_TradeSkillCombine, 0); auto outapp = new EQApplicationPacket(OP_TradeSkillCombine, 0);
@ -514,8 +520,11 @@ void Object::HandleCombine(Client* user, const NewCombine_Struct* in_combine, Ob
RecordPlayerEventLogWithClient(user, PlayerEvent::COMBINE_SUCCESS, e); RecordPlayerEventLogWithClient(user, PlayerEvent::COMBINE_SUCCESS, e);
} }
if (parse->PlayerHasQuestSub(EVENT_COMBINE_SUCCESS)) {
parse->EventPlayer(EVENT_COMBINE_SUCCESS, user, spec.name, spec.recipe_id); parse->EventPlayer(EVENT_COMBINE_SUCCESS, user, spec.name, spec.recipe_id);
} else { }
}
else {
if (player_event_logs.IsEventEnabled(PlayerEvent::COMBINE_FAILURE)) { if (player_event_logs.IsEventEnabled(PlayerEvent::COMBINE_FAILURE)) {
auto e = PlayerEvent::CombineEvent{ auto e = PlayerEvent::CombineEvent{
.recipe_id = spec.recipe_id, .recipe_id = spec.recipe_id,
@ -526,9 +535,11 @@ void Object::HandleCombine(Client* user, const NewCombine_Struct* in_combine, Ob
RecordPlayerEventLogWithClient(user, PlayerEvent::COMBINE_FAILURE, e); RecordPlayerEventLogWithClient(user, PlayerEvent::COMBINE_FAILURE, e);
} }
if (parse->PlayerHasQuestSub(EVENT_COMBINE_FAILURE)) {
parse->EventPlayer(EVENT_COMBINE_FAILURE, user, spec.name, spec.recipe_id); parse->EventPlayer(EVENT_COMBINE_FAILURE, user, spec.name, spec.recipe_id);
} }
} }
}
void Object::HandleAutoCombine(Client* user, const RecipeAutoCombine_Struct* rac) { void Object::HandleAutoCombine(Client* user, const RecipeAutoCombine_Struct* rac) {
@ -701,11 +712,15 @@ void Object::HandleAutoCombine(Client* user, const RecipeAutoCombine_Struct* rac
} }
if (success) { if (success) {
if (parse->PlayerHasQuestSub(EVENT_COMBINE_SUCCESS)) {
parse->EventPlayer(EVENT_COMBINE_SUCCESS, user, spec.name, spec.recipe_id); parse->EventPlayer(EVENT_COMBINE_SUCCESS, user, spec.name, spec.recipe_id);
}
} else { } else {
if (parse->PlayerHasQuestSub(EVENT_COMBINE_FAILURE)) {
parse->EventPlayer(EVENT_COMBINE_FAILURE, user, spec.name, spec.recipe_id); parse->EventPlayer(EVENT_COMBINE_FAILURE, user, spec.name, spec.recipe_id);
} }
} }
}
EQ::skills::SkillType Object::TypeToSkill(uint32 type) EQ::skills::SkillType Object::TypeToSkill(uint32 type)
{ {