Augyre rotating buff: dropdown shows Elemental Overload but Resolute Technique is applied, and the choice is lost on save
Les mainteneurs répondent en général sous 8 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 75/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Domaine
- backend
Piste de recherche
Commencez par l’option conditionHaveAugyre dans src/Modules/ConfigOptions.lua et suivez la façon dont src/Classes/ConfigTab.lua initialise, affiche, enregistre et recharge les valeurs de liste. Vérifiez si la valeur par défaut choisie fait correspondre le buff Augyre affiché au calcul et le conserve après une nouvelle ouverture ; l’issue fournit les chemins de code concernés et une séquence de reproduction. Le travail est terminé lorsque les deux choix de buff se comportent de manière cohérente après l’enregistrement et le rechargement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Lua
- Étoiles
- 5.5k
- Forks
- 2.4k
- Merge moyen
- 6 j 14 h
- PR mergées (30 j)
- 13
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de PathOfBuildingCommunity/PathOfBuilding
-
Difficulté 2/5 1-3 heures Accessibilité débutants 79/100
PathOfBuildingCommunity/PathOfBuilding#10428 ·
Les mainteneurs répondent en général sous 8 jours
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 60/100
PathOfBuildingCommunity/PathOfBuilding#10420 ·
Les mainteneurs répondent en général sous 8 jours
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
PathOfBuildingCommunity/PathOfBuilding#10395 ·
Les mainteneurs répondent en général sous 8 jours
-
bug: accuracy
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
PathOfBuildingCommunity/PathOfBuilding#9911 ·
Les mainteneurs répondent en général sous 8 jours
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
PathOfBuildingCommunity/PathOfBuilding#7548 · 4 commentaires · 1 réaction ·
Les mainteneurs répondent en général sous 8 jours
Toutes les issues de PathOfBuildingCommunity/PathOfBuilding
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 5 jours
-
enhancement priority::low upstream
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
bug reviewed-by-AI
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
olimorris/codecompanion.nvim#3508 ·
Les mainteneurs répondent en général sous 1 jour
-
The Balanced badge stays on a card after it changes to an enhancement that isn't rebalancedPeut-être pris @Jr-JoshS l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 71/100
Balatro-Multiplayer/BalatroMultiplayerPvP#640 ·
Les mainteneurs répondent en général sous 1 jour
-
Babies count as citizens for housing and the lost-fort check, though a baby uses no bed or seatOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Layr-Labs/arena-fortress#141 ·
Les mainteneurs répondent en général sous 1 jour