ossappscollective / ossappscollective/OSS-DocumentScanner

[BUG] Missing input validation for Extra Fields with type NUMBER

Open
#647 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug done
Dominant language
C++
Stars
2.5k
Forks
122
Avg merge
1d 58m
Merged PRs (30d)
17

Description

Which app is your issue for

CardWallet

Version

1.13.1

What platform are you using?

Android

OS Version

16

What happened?

For an "Extra Field" with the type "NUMBER" a non-numerical value can be entered and saved and would later be displayed as "NaN".

This should be avoided in the first place by input validation which should allow only numeric values.

Relevant log output

Code of Conduct
  • I agree to follow this project's Code of Conduct

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the Android UI and validation path for Extra Fields with type NUMBER, then reproduce entering a non-numerical value on Android. Done means non-numeric input is rejected before saving and the value cannot later be displayed as "NaN".

Written by the indexing model from the issue text.

Assessment

Tech stack
android, cpp
Domain
mobile-dev
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.