PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Sorting uniques yields incorrectly sorted items when main skill is minion based
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Lua
- Sterne
- 5.4k
- Forks
- 2.4k
- Ø Merge
- 1 T. 12 Std.
- Gemergte PRs (30 T.)
- 26
Beschreibung
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
What is the expected behaviour/value?
I understand the sorting might be tricky because some sorting options need to apply to minions, such as DPS or life when a minion skill is selected.
Except some sorting options seem to player-only (strength, dexterity, intelligence, effective hit pool, and so on). So for these, I would expect to have items being sorted based on the character itself rather than the skill selected.
What is the actual behaviour/value?
When sorting unique items, many of the sorting options yields the wrong results.
I identified two cases where sorting results are either alphabetically sorted (case 1) or seemingly random (case 2)
The root cause of the issue seems to be linked with having a Main Skill configured on a minion skill (in my case Absolution).
Selecting a non-minion skill causes the sorting option to yield an adequately sorted list.
Case 1 - Sorting options yielding alphabetically sorted list
Examples (with 1st result highlighted)
-
Sort by Mana:

-
Sort by Strength:

-
Sort by Dexterity:

Case 2 - Sorting options yielding list in seemingly random order
-
Sort by Effective Hit Pool:- 1st item:

- 2nd item:

- 10th item:

- 1st item:
How to reproduce the issue
- Add any minion skill as main skill, for example:
Absolution - Sort unique items by metrics only affecting the player, for example:
Strength,Dexterity,IntelligenceorEffective Health Pool - Then switch the main skill to a non-minion skill => results are different from
step 2.
Character build code
https://pobb.in/2pE4Szyp5TuB
Screenshots
No response
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
In der Meldung wird keine Quelldatei und kein Test angegeben. Lade den verlinkten Charakter-Build, wähle eine Minion-Fertigkeit wie Absolution aus und reproduziere das Sortieren einzigartiger Gegenstände nach Strength, Dexterity, Mana oder Effective Hit Pool. Vergleiche anschließend mit einer Nicht-Minions-Hauptfertigkeit. Als erledigt gilt, dass nur auf den Spieler bezogene Metriken unabhängig von der ausgewählten Fertigkeit konsistent sortiert werden und eine Regressionstestabdeckung für die gemeldeten Fälle vorhanden ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- lua
- Bereich
- desktop
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100