Skip to content

Core/Items: Add durability sell penalty when selling damaged items to…#31841

Open
kik0desu wants to merge 3 commits into
TrinityCore:3.3.5from
kik0desu:fix-sell-durability-penalty
Open

Core/Items: Add durability sell penalty when selling damaged items to…#31841
kik0desu wants to merge 3 commits into
TrinityCore:3.3.5from
kik0desu:fix-sell-durability-penalty

Conversation

@kik0desu
Copy link
Copy Markdown

@kik0desu kik0desu commented May 9, 2026

… vendors

Changes proposed:

  • Selling a damaged item to a vendor now properly reduces the gold received based on lost durability, matching what the client already displays.

Issues addressed:

none

Tests performed:

builds, tested in-game

before:

sellcost.durability.fix.before.mp4

after:

sellcost.durability.fix.after.mp4

@imstuckbro
Copy link
Copy Markdown

I just can't remember if this is blizzlike but I do remember on some servers I've played on another scenario where vendor doesn't want your item till you repair it.

@Shauren
Copy link
Copy Markdown
Member

Shauren commented May 11, 2026

What is the point of this function you added? It's exactly a copypaste of CalculateDurabilityRepairCost minus config rate

@kik0desu
Copy link
Copy Markdown
Author

kik0desu commented May 11, 2026

What is the point of this function you added? It's exactly a copypaste of CalculateDurabilityRepairCost minus config rate

I just didn't want to modify the existing function to add a new parameter, afaik the rate config should not affect the sell cost at all.

@kik0desu
Copy link
Copy Markdown
Author

did an update, lmk if it's better like this

Comment thread src/server/shared/DataStores/DBCStructure.h Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants