#409 Client::ConsumeFoodDrink item to be deleted from DB before removing item

Geschlossen
vor 2 Jahren geöffnet von image · 2 Kommentare
image kommentierte vor 2 Jahren

``` else {

        GetPlayer()->GetEquipmentList()->RemoveItem(slot, true);
        database.DeleteItem(GetPlayer()->GetCharacterID(), item, "EQUIPPED");
    }```

wrong order, database.DeleteItem should be first since we have RemoveItem boolean true which deletes the item

``` else { GetPlayer()->GetEquipmentList()->RemoveItem(slot, true); database.DeleteItem(GetPlayer()->GetCharacterID(), item, "EQUIPPED"); }``` wrong order, database.DeleteItem should be first since we have RemoveItem boolean true which deletes the item
devn00b kommentierte vor 2 Jahren
Besitzer

Fixed pending git update.

client.cpp line 10671 -> 10675

    else {

database.DeleteItem(GetPlayer()->GetCharacterID(), item,EQUIPPED"); GetPlayer()->GetEquipmentList()->RemoveItem(slot, true);

    }
Fixed pending git update. client.cpp line 10671 -> 10675 else { database.DeleteItem(GetPlayer()->GetCharacterID(), item,EQUIPPED"); GetPlayer()->GetEquipmentList()->RemoveItem(slot, true); }
devn00b kommentierte vor 2 Jahren
Besitzer

54a61e0c39

I've seen, nor heard any further issues from this patch, closing related issues. I Will re-open if some problem pops back up.

https://git.eq2emu.com/devn00b/EQ2EMu/commit/54a61e0c391817e2e01cb5f93bc9fd8c13526afa I've seen, nor heard any further issues from this patch, closing related issues. I Will re-open if some problem pops back up.
devn00b hat vor 2 Jahren geschlossen
Anmelden, um an der Diskussion teilzunehmen.
Laden…
Abbrechen
Speichern
Hier gibt es bis jetzt noch keinen Inhalt.