Skip to content

Fix trader crash#2398

Open
vaisest wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
vaisest:fix-trader2
Open

Fix trader crash#2398
vaisest wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
vaisest:fix-trader2

Conversation

@vaisest

@vaisest vaisest commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Fixes #2396

Description of the problem being solved:

Sometime after 0.5 there was a change which made explicit lines an object instead of just a string. Now that was applied to the other mod fields too

The weighed sum field also seems to have been changed to the same format. Fix was copied from pob1

Steps taken to verify a working solution:

  • Search no longer crashes

Link to a build that showcases this PR:

Before screenshot:

After screenshot:

@sardutw

sardutw commented Jul 25, 2026

Copy link
Copy Markdown

For anyone using the Portable version:

Open your PathOfBuildingCommunity-PoE2-Portable folder.
Navigate to src → Classes.
Download the fixed TradeQueryRequests.lua file from:

Copy the downloaded file into the Classes folder.
When Windows asks if you want to replace the existing file, click Yes.

That's it! The Trade Search crash should now be fixed.

Note: This is only a temporary workaround until the fix from PR #2398 is included in an official Path of Building PoE2 release.

@vaisest

vaisest commented Jul 25, 2026

Copy link
Copy Markdown
Contributor Author

Please don't link to mediafire like this. That's sketchy as hell, even if that one is the correct. If you really can't wait and don't know how to launch PoB in dev mode: you can read it directly from the repository: https://raw.githubusercontent.com/vaisest/PathOfBuilding-PoE2/fix-trader2/src/Classes/TradeQueryRequests.lua

@LocalIdentity LocalIdentity added pob1 Should also be merged into PoB1 Repo crash Causes PoB to crash and is High Priority labels Jul 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

crash Causes PoB to crash and is High Priority pob1 Should also be merged into PoB1 Repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failure to Search when using "Trade for these items" feature

3 participants