mifos-mobile icon indicating copy to clipboard operation
mifos-mobile copied to clipboard

Incorrect logic in LoanApplicationFragment.kt

Open divyank00 opened this issue 4 years ago • 1 comments
trafficstars

Summary: The following code snippets are from LoanApplicationFragment.kt In loanPurposeOptions, we manually add the first item, which is R.string.loan_purpose_not_provided (As shown in Image 1) In onItemSelected we store the id of the selected position in the variable purposeId (As shown in image 2) But we should store the value of position-1 and when position is 0, we should assign -1 to purposeId

Screenshots:

carbon (1)

carbon (2)

divyank00 avatar Jan 06 '21 20:01 divyank00

@miPlodder Can you please review

divyank00 avatar Jan 06 '21 21:01 divyank00