Permalink
Browse files

Changed email return for item that can't be equiped anymore. Before t…

…he email was sent with an empty body and the subject was to long to be displayed in the player email. Now the Email is sent with the subject 'Item could not be loaded to inventory.' and the body as the subject message before. (#71)
  • Loading branch information...
armandgabriel authored and H0zen committed Jan 9, 2019
1 parent 368b8e8 commit 5bac361592bd112435abb66e1ff09bcec1c0d3f0
Showing with 3 additions and 3 deletions.
  1. +3 −3 src/game/Object/Player.cpp
@@ -14833,11 +14833,11 @@ void Player::_LoadInventory(QueryResult* result, uint32 timediff)
// send by mail problematic items
while (!problematicItems.empty())
{
std::string subject = GetSession()->GetMangosString(LANG_NOT_EQUIPPED_ITEM);

std::string subject = "Item could not be loaded to inventory.";
std::string content = GetSession()->GetMangosString(LANG_NOT_EQUIPPED_ITEM);
// fill mail
MailDraft draft(subject);

draft.SetSubjectAndBody(subject,content);
for (int i = 0; !problematicItems.empty() && i < MAX_MAIL_ITEMS; ++i)
{
Item* item = problematicItems.front();

2 comments on commit 5bac361

@billy1arm

This comment has been minimized.

Copy link
Member

billy1arm replied Jan 10, 2019

@H0zen @armandgabriel - should this be applied to the other cores ?

@armandgabriel

This comment has been minimized.

Copy link
Contributor

armandgabriel replied Jan 10, 2019

Hello @billy1arm

From what I talked with @H0zen last night this is a small fix regarding just email received by players. It's your choice if you want to integrate it with the other branches/cores as you see is just a change in the subject and body of the email any player receives with any item that can't be equipped anymore.

Kind regards,
Armand Camner ( Lunarspear )

Please sign in to comment.