# ✅ \[Android\] Salary Sorter - input of values incorrectly handled - leading to crash **URL:** \ **Category:** Bug Reports\ **Created:** [27 September 2019 16:50 UTC](https://community.monzo.com/t/android-salary-sorter-input-of-values-incorrectly-handled-leading-to-crash/81310 "2019-09-27T16:50:13Z")\ **Posts on this page:** 6\ **Page:** 1
**Author:** ![DeanMarkTaylor](https://dub1.discourse-cdn.com/monzo/user_avatar/community.monzo.com/deanmarktaylor/32/23939_2.png) [@DeanMarkTaylor](https://community.monzo.com/u/DeanMarkTaylor)\ **Post date:** [27 September 2019 16:50 UTC](https://community.monzo.com/t/android-salary-sorter-input-of-values-incorrectly-handled-leading-to-crash/81310/1 "2019-09-27T16:50:13Z")
**Issue:** Crashes during Salary Sorter value input and inputting values is incorrectly handled, incorrectly putting remaining sorting balance in one of the pots. ## Details to reproduce For me and my pots, it can be reproduced each time. Note when entering values they were entered by: 1. Tap to edit number (tapping between `+` and `-`) 2. Keyboard pops up 3. Press delete on keyboard (to delete the `0`) 4. Type new number 5. Tap blue tick button on on-screen keyboard (placed where the “enter” key usually is) ## Steps to reproduce 1. Select payment to sort. 2. Select “Sort Salary” button 3. Enter values in first 4 pots, #1=`25`, #2=`100`, #3=`84`, #4=`25` 4. Skip the 5th pot 5. Enter value `84` for the 6th pot. 6. Note upon entering the digit `8`, the `+` to the right hand side of each of the pots can be seen to be disabled. 7. After confirming the input of `84` via pressing the on-screen keyboard blue tick (enter key) - the keyboard disappears revealing that the remaining “Spending” balance amount has been added to the 7th pot. 8. Tapping to edit 6th pot again (where the input has changed from `84` to `8`) - and tapping `4` to add the `4` back in results in the app crashing. You may need to tap `4` multiple times to cause a crash, my usual action as the app fails to update screen or give user feedback that input is invalid. I’ve also attempted to edit pot values in a different order, always able to cause entire remaining “Spending” amount to unexpectedly appear in one of the pots and input crashing app. ## Details **OS:** Android 10, Security patch level 5th Sept. 2019 **Device:** Pixel 2 XL **App Version:** 3.0.0 (I’m on the beta updates if that helps) ## Screenshots ### Step 3-4 ![image](https://global.discourse-cdn.com/monzo/original/3X/5/2/5224d9e64a3e3e3894c81c5cbd4be999e446b819.jpeg) ### Step 5-6 ![image](https://global.discourse-cdn.com/monzo/original/3X/7/e/7ec5134c3a35b4d59331eca666290b2a6d8066d4.jpeg) ### Step 7 ![image](https://global.discourse-cdn.com/monzo/original/3X/8/1/8111e7b5fb7b61e6b2973afefc5948bba41f04aa.jpeg) ### Step 8 ![image](https://global.discourse-cdn.com/monzo/original/3X/7/5/75175f4c651baa6d604c97a3b559b0a271407911.jpeg) Tagging in @anon59758512 because of involvement in Salary Sorter postings in community. ---
**Author:** ![anon59758512](https://avatars.discourse-cdn.com/v4/letter/a/7ab992/32.png) [@anon59758512](https://community.monzo.com/u/anon59758512)\ **Post date:** [27 September 2019 16:51 UTC](https://community.monzo.com/t/android-salary-sorter-input-of-values-incorrectly-handled-leading-to-crash/81310/2 "2019-09-27T16:51:11Z")
I’ll flag to the team, thanks for the thorough report 🙌 ---
**Author:** ![DeanMarkTaylor](https://dub1.discourse-cdn.com/monzo/user_avatar/community.monzo.com/deanmarktaylor/32/23939_2.png) [@DeanMarkTaylor](https://community.monzo.com/u/DeanMarkTaylor)\ **Post date:** [27 September 2019 16:51 UTC](https://community.monzo.com/t/android-salary-sorter-input-of-values-incorrectly-handled-leading-to-crash/81310/3 "2019-09-27T16:51:55Z")
I have screen capture if need - but obviously reveals personal details. ---
**Author:** ![anon59758512](https://avatars.discourse-cdn.com/v4/letter/a/7ab992/32.png) [@anon59758512](https://community.monzo.com/u/anon59758512)\ **Post date:** [27 September 2019 16:52 UTC](https://community.monzo.com/t/android-salary-sorter-input-of-values-incorrectly-handled-leading-to-crash/81310/4 "2019-09-27T16:52:15Z")
I’m sure this’ll suffice to repro on our end ---
**Author:** ![jamesroutley](https://dub1.discourse-cdn.com/monzo/user_avatar/community.monzo.com/jamesroutley/32/16473_2.png) [@jamesroutley](https://community.monzo.com/u/jamesroutley)\ **Post date:** [27 September 2019 17:03 UTC](https://community.monzo.com/t/android-salary-sorter-input-of-values-incorrectly-handled-leading-to-crash/81310/5 "2019-09-27T17:03:29Z")
This is an amazing bug report - thank you! ---
**Author:** ![DeanMarkTaylor](https://dub1.discourse-cdn.com/monzo/user_avatar/community.monzo.com/deanmarktaylor/32/23939_2.png) [@DeanMarkTaylor](https://community.monzo.com/u/DeanMarkTaylor)\ **Post date:** [23 January 2020 18:34 UTC](https://community.monzo.com/t/android-salary-sorter-input-of-values-incorrectly-handled-leading-to-crash/81310/6 "2020-01-23T18:34:44Z")
I tested this on next salary (~ a month after the next app version was released) … … everything worked as expected (thank you). I can’t see a way to mark it this topic as fixed - so I leave this comment instead. Cheers!