PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Sorting uniques by phys or chaos taken falls back to name sorting

Open
#6,543 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
Lua
Stars
5.4k
Forks
2.4k
Avg merge
1d 12h
Merged PRs (30d)
26

Description

Check version
  • I'm running the latest version of Path of Building and I've verified this by checking the changelog
Check for duplicates
  • I've checked for duplicate issues by using the search function of the issue tracker
How is Path of Building expected to behave?

The highest item of the list is the one that gives the most value of the specified criteria - e.g. max phys taken. Subsequent items are sorted following the same criteria.

How does Path of Building behave?

The highest item on the list is Abberath's Hooves, then Abberath's Horn, etc.

How to reproduce the issue
  1. Open PoB on a new (empty) build.
  2. Sort uniques by phys taken.
Character build code

No response

Screenshots

image

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the issue in Path of Building with a new empty build, then inspect the unique-item sorting entry point; no file or test is named in the report. Compare sorting by phys taken with the expected descending criteria order, and consider the issue done when subsequent items follow that criterion instead of falling back to name sorting.

Written by the indexing model from the issue text.

Assessment

Tech stack
lua
Domain
game-dev
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.