address feedback

This commit is contained in:
Aaron Kimbre 2025-02-01 01:51:46 -06:00
parent b388b03251
commit 78e52904e5
4 changed files with 19 additions and 20 deletions

View File

@ -70,7 +70,7 @@ namespace Mail {
} }
void SendRequest::Handle() { void SendRequest::Handle() {
SendResponse response(eSendResponse::UnknownError); SendResponse response;
auto* character = player->GetCharacter(); auto* character = player->GetCharacter();
if (character && !(character->HasPermission(ePermissionMap::RestrictedMailAccess) || character->GetParentUser()->GetIsMuted())) { if (character && !(character->HasPermission(ePermissionMap::RestrictedMailAccess) || character->GetParentUser()->GetIsMuted())) {
mailInfo.recipient = std::regex_replace(mailInfo.recipient, std::regex("[^0-9a-zA-Z]+"), ""); mailInfo.recipient = std::regex_replace(mailInfo.recipient, std::regex("[^0-9a-zA-Z]+"), "");
@ -178,7 +178,7 @@ namespace Mail {
} }
void AttachmentCollectRequest::Handle() { void AttachmentCollectRequest::Handle() {
auto response = AttachmentCollectResponse(eAttachmentCollectResponse::UnknownError); AttachmentCollectResponse response;
auto inv = player->GetComponent<InventoryComponent>(); auto inv = player->GetComponent<InventoryComponent>();
if (mailID > 0 && playerID == player->GetObjectID() && inv) { if (mailID > 0 && playerID == player->GetObjectID() && inv) {
@ -211,8 +211,9 @@ namespace Mail {
} }
void DeleteRequest::Handle() { void DeleteRequest::Handle() {
auto response = DeleteResponse(mailID); DeleteResponse response;
response.mailID = mailID;
auto mailData = Database::Get()->GetMail(mailID); auto mailData = Database::Get()->GetMail(mailID);
if (mailData && !(mailData->itemLOT != 0 && mailData->itemCount > 0)) { if (mailData && !(mailData->itemLOT != 0 && mailData->itemCount > 0)) {
Database::Get()->DeleteMail(mailID); Database::Get()->DeleteMail(mailID);
@ -239,7 +240,8 @@ namespace Mail {
} }
void ReadRequest::Handle() { void ReadRequest::Handle() {
auto response = ReadResponse(mailID); ReadResponse response;
response.mailID = mailID;
if (Database::Get()->GetMail(mailID)) { if (Database::Get()->GetMail(mailID)) {
response.status = eReadResponse::Success; response.status = eReadResponse::Success;
@ -255,7 +257,7 @@ namespace Mail {
} }
void NotificationRequest::Handle() { void NotificationRequest::Handle() {
auto response = NotificationResponse(eNotificationResponse::UnknownError); NotificationResponse response;
auto character = player->GetCharacter(); auto character = player->GetCharacter();
if (character) { if (character) {
auto unreadMailCount = Database::Get()->GetUnreadMailCount(character->GetID()); auto unreadMailCount = Database::Get()->GetUnreadMailCount(character->GetID());
@ -352,6 +354,7 @@ void Mail::SendMail(const LWOOBJID sender, const std::string& senderName, LWOOBJ
Database::Get()->InsertNewMail(mailInsert); Database::Get()->InsertNewMail(mailInsert);
if (sysAddr == UNASSIGNED_SYSTEM_ADDRESS) return; // TODO: Echo to chat server if (sysAddr == UNASSIGNED_SYSTEM_ADDRESS) return; // TODO: Echo to chat server
NotificationResponse response;
NotificationResponse(eNotificationResponse::NewMail).Send(sysAddr); response.status = eNotificationResponse::NewMail;
response.Send(sysAddr);
} }

View File

@ -125,8 +125,6 @@ namespace Mail {
struct SendResponse :public MailLUBitStream { struct SendResponse :public MailLUBitStream {
eSendResponse status = eSendResponse::UnknownError; eSendResponse status = eSendResponse::UnknownError;
SendResponse(eSendResponse _status) : MailLUBitStream(eMessageID::SendResponse), status{_status} {};
void Serialize(RakNet::BitStream& bitStream) const override; void Serialize(RakNet::BitStream& bitStream) const override;
}; };
@ -134,8 +132,7 @@ namespace Mail {
eNotificationResponse status = eNotificationResponse::UnknownError; eNotificationResponse status = eNotificationResponse::UnknownError;
LWOOBJID auctionID = LWOOBJID_EMPTY; LWOOBJID auctionID = LWOOBJID_EMPTY;
uint32_t mailCount = 1; uint32_t mailCount = 1;
NotificationResponse(eNotificationResponse _status) : MailLUBitStream(eMessageID::NotificationResponse), status{_status} {}; NotificationResponse() : MailLUBitStream(eMessageID::NotificationResponse) {};
NotificationResponse(eNotificationResponse _status, uint32_t _mailCount) : MailLUBitStream(eMessageID::NotificationResponse), status{_status}, mailCount{_mailCount} {};
void Serialize(RakNet::BitStream& bitStream) const override; void Serialize(RakNet::BitStream& bitStream) const override;
}; };
@ -165,8 +162,7 @@ namespace Mail {
struct AttachmentCollectResponse : public MailLUBitStream { struct AttachmentCollectResponse : public MailLUBitStream {
eAttachmentCollectResponse status = eAttachmentCollectResponse::UnknownError; eAttachmentCollectResponse status = eAttachmentCollectResponse::UnknownError;
uint64_t mailID = 0; uint64_t mailID = 0;
AttachmentCollectResponse(eAttachmentCollectResponse _status) : MailLUBitStream(eMessageID::AttachmentCollectResponse), status{_status} {}; AttachmentCollectResponse() : MailLUBitStream(eMessageID::AttachmentCollectResponse) {};
AttachmentCollectResponse(eAttachmentCollectResponse _status, uint64_t _mailID) : MailLUBitStream(eMessageID::AttachmentCollectResponse), status{_status}, mailID{_mailID} {};
void Serialize(RakNet::BitStream& bitStream) const override; void Serialize(RakNet::BitStream& bitStream) const override;
}; };
@ -174,7 +170,6 @@ namespace Mail {
uint64_t mailID = 0; uint64_t mailID = 0;
LWOOBJID playerID = LWOOBJID_EMPTY; LWOOBJID playerID = LWOOBJID_EMPTY;
DeleteRequest() : MailLUBitStream(eMessageID::DeleteRequest) {}; DeleteRequest() : MailLUBitStream(eMessageID::DeleteRequest) {};
bool Deserialize(RakNet::BitStream& bitStream) override; bool Deserialize(RakNet::BitStream& bitStream) override;
void Handle() override; void Handle() override;
@ -183,8 +178,7 @@ namespace Mail {
struct DeleteResponse : public MailLUBitStream { struct DeleteResponse : public MailLUBitStream {
eDeleteResponse status = eDeleteResponse::UnknownError; eDeleteResponse status = eDeleteResponse::UnknownError;
uint64_t mailID = 0; uint64_t mailID = 0;
DeleteResponse(uint64_t _mailID) : MailLUBitStream(eMessageID::DeleteResponse), mailID{_mailID} {}; DeleteResponse() : MailLUBitStream(eMessageID::DeleteResponse) {};
DeleteResponse(eDeleteResponse _status, uint64_t _mailID) : MailLUBitStream(eMessageID::DeleteResponse), status{_status}, mailID{_mailID} {};
void Serialize(RakNet::BitStream& bitStream) const override; void Serialize(RakNet::BitStream& bitStream) const override;
}; };
@ -201,7 +195,6 @@ namespace Mail {
eReadResponse status = eReadResponse::UnknownError; eReadResponse status = eReadResponse::UnknownError;
ReadResponse() : MailLUBitStream(eMessageID::ReadResponse) {}; ReadResponse() : MailLUBitStream(eMessageID::ReadResponse) {};
ReadResponse(uint64_t _mailID) : MailLUBitStream(eMessageID::ReadResponse), mailID{_mailID} {};
void Serialize(RakNet::BitStream& bitStream) const override; void Serialize(RakNet::BitStream& bitStream) const override;
}; };

View File

@ -217,7 +217,10 @@ namespace GMZeroCommands {
} }
void RequestMailCount(Entity* entity, const SystemAddress& sysAddr, const std::string args) { void RequestMailCount(Entity* entity, const SystemAddress& sysAddr, const std::string args) {
Mail::NotificationResponse(Mail::eNotificationResponse::NewMail, Database::Get()->GetUnreadMailCount(entity->GetCharacter()->GetID())).Send(sysAddr); Mail::NotificationResponse response;
response.status = Mail::eNotificationResponse::NewMail;
response.mailCount = Database::Get()->GetUnreadMailCount(entity->GetCharacter()->GetID());
response.Send(sysAddr);
} }
void InstanceInfo(Entity* entity, const SystemAddress& sysAddr, const std::string args) { void InstanceInfo(Entity* entity, const SystemAddress& sysAddr, const std::string args) {

View File

@ -60,4 +60,4 @@ bool MailInfo::Deserialize(RakNet::BitStream& bitStream) {
DluAssert(bitStream.GetNumberOfUnreadBits() == 0); DluAssert(bitStream.GetNumberOfUnreadBits() == 0);
return true; return true;
} }