Augyre rotating buff: dropdown shows Elemental Overload but Resolute Technique is applied, and the choice is lost on save
I maintainer di solito rispondono entro 8 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 75/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Ambito
- backend
Direzione di ricerca
Inizia dall’opzione conditionHaveAugyre in src/Modules/ConfigOptions.lua e segui come src/Classes/ConfigTab.lua inizializza, visualizza, salva e ricarica i valori delle liste. Verifica se il valore predefinito scelto fa sì che il buff Augyre visualizzato corrisponda al calcolo e persista dopo la riapertura; l’issue indica i percorsi del codice pertinenti e una sequenza per riprodurre il problema. Il lavoro è completo quando entrambe le scelte del buff si comportano in modo coerente dopo il salvataggio e il ricaricamento.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
What happens
The Config tab option "Augyre rotating buff" shows "Elemental Overload" by default, but the calculation actually uses Resolute Technique until you pick a value yourself. If you do pick "Elemental Overload", it is lost on save, and the reopened build is back to Resolute Technique while the dropdown still says "Elemental Overload".
Steps (derived from the code):
- New build, equip Augyre (Void Sceptre). Config tab shows "Augyre rotating buff: Elemental Overload".
- Check Calcs: Resolute Technique is active (hits cannot crit, 100% hit chance), not Elemental Overload.
- Change the dropdown to "Resolute Technique", then back to "Elemental Overload". Elemental Overload is now applied.
- Save and reopen.
Expected: what the dropdown shows is what gets applied, and it is the same after reopening.
Actual: steps 1-2 show one option and apply the other; after step 4 the build is back to Resolute Technique with "Elemental Overload" displayed.
Cause
The option has no defaultIndex:
So a new or loaded config set starts with input.conditionHaveAugyre = nil (NewConfigSet only fills list values that have a defaultIndex):
But the dropdown displays input or GetDefaultState(var), and the default state of a list is its first entry:
- https://github.com/PathOfBuildingCommunity/PathOfBuilding/blob/a69391d6daaaea1aaf8054668a094aad88dd7eaa/src/Classes/ConfigTab.lua#L1018
- https://github.com/PathOfBuildingCommunity/PathOfBuilding/blob/a69391d6daaaea1aaf8054668a094aad88dd7eaa/src/Classes/ConfigTab.lua#L686-L687
A list's apply only runs for a non-nil input (ConfigTab.lua#L1150-L1152), so with nil neither branch adds Elemental Overload. The item's own "You have Resolute Technique while you do not have Elemental Overload" then applies.
On save, any input equal to GetDefaultState is left out (ConfigTab.lua#L971), so an explicit "EleOverload" is not written. On load it comes back as nil.
Among the list options without defaultIndex, this appears to be the only one whose first entry changes the calculation. The others I checked (calc modes, "None" entries) treat nil the same as their first entry. More generally, Save skips values equal to GetDefaultState, but Load/NewConfigSet only restore defaultState/defaultIndex, so any option where those two differ loses an explicit value on reload. The numeric "0" case is #10143 / #10354.
Possible fix
Add defaultIndex = 1 to conditionHaveAugyre, so the stored input, the dropdown and the save default agree. Alternatively, have Save compare against the value NewConfigSet would restore rather than GetDefaultState.
- Lingua principale
- Lua
- Stelle
- 5.5k
- Fork
- 2.4k
- Merge medio
- 6g 14h
- PR unite (30g)
- 13
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di PathOfBuildingCommunity/PathOfBuilding
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
PathOfBuildingCommunity/PathOfBuilding#10428 ·
I maintainer di solito rispondono entro 8 giorni
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 60/100
PathOfBuildingCommunity/PathOfBuilding#10420 ·
I maintainer di solito rispondono entro 8 giorni
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
PathOfBuildingCommunity/PathOfBuilding#10395 ·
I maintainer di solito rispondono entro 8 giorni
-
bug: accuracy
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
PathOfBuildingCommunity/PathOfBuilding#9911 ·
I maintainer di solito rispondono entro 8 giorni
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
PathOfBuildingCommunity/PathOfBuilding#7548 · 4 commenti · 1 reazione ·
I maintainer di solito rispondono entro 8 giorni
Tutte le issue di PathOfBuildingCommunity/PathOfBuilding
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 5 giorni
-
enhancement priority::low upstream
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
bug reviewed-by-AI
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
olimorris/codecompanion.nvim#3508 ·
I maintainer di solito rispondono entro 1 giorno
-
The Balanced badge stays on a card after it changes to an enhancement that isn't rebalancedForse già presa @Jr-JoshS l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 71/100
Balatro-Multiplayer/BalatroMultiplayerPvP#640 ·
I maintainer di solito rispondono entro 1 giorno
-
Babies count as citizens for housing and the lost-fort check, though a baby uses no bed or seatAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Layr-Labs/arena-fortress#141 ·
I maintainer di solito rispondono entro 1 giorno