From 7b308ded1b122183fce001df09ee937ed55e3f6d Mon Sep 17 00:00:00 2001 From: Tom Hicks Date: Fri, 25 Jun 2021 23:26:59 -0700 Subject: [PATCH] Fixes selection bug in AbilityScorePicker. --- .../monstercards/ui/components/AbilityScorePicker.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/app/src/main/java/com/majinnaibu/monstercards/ui/components/AbilityScorePicker.java b/app/src/main/java/com/majinnaibu/monstercards/ui/components/AbilityScorePicker.java index a6268e7..0c276ae 100644 --- a/app/src/main/java/com/majinnaibu/monstercards/ui/components/AbilityScorePicker.java +++ b/app/src/main/java/com/majinnaibu/monstercards/ui/components/AbilityScorePicker.java @@ -69,12 +69,12 @@ public class AbilityScorePicker extends LinearLayout { mHolder.spinner.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() { @Override public void onItemSelected(AdapterView parent, View view, int position, long id) { - mSelectedValue = (AbilityScore) parent.getItemAtPosition(position); + setValue((AbilityScore) parent.getItemAtPosition(position)); } @Override public void onNothingSelected(AdapterView parent) { - mSelectedValue = AbilityScore.STRENGTH; + setValue(mSelectedValue = AbilityScore.STRENGTH); } }); mHolder.spinner.setSelection(ArrayHelper.indexOf(AbilityScore.values(), mSelectedValue)); @@ -93,7 +93,8 @@ public class AbilityScorePicker extends LinearLayout { public void setValue(AbilityScore value) { if (value != mSelectedValue) { - mHolder.spinner.setSelection(ArrayHelper.indexOf(AbilityScore.values(), mSelectedValue)); + mSelectedValue = value; + mHolder.spinner.setSelection(ArrayHelper.indexOf(AbilityScore.values(), value)); if (mOnValueChangedListener != null) { mOnValueChangedListener.onValueChanged(value); }