Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The edit command allows names that are all numbers #6

Open
autumn-sonata opened this issue Oct 28, 2022 · 0 comments
Open

The edit command allows names that are all numbers #6

autumn-sonata opened this issue Oct 28, 2022 · 0 comments

Comments

@autumn-sonata
Copy link
Owner

autumn-sonata commented Oct 28, 2022

The application allows the name of clients to only be decimal numbers:

image.png

It is unlikely that a name will have numerical values in them, given they are not nicknames, and given the context of this application being a financial advisor planner.

Further internal checking can be done to prevent such behaviour. Taken from person/Name.java:

image.png

The VALIDATION_REGEX used is alphanumeric, and therefore the application accepts names that are only numbers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant