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

Two users of the same name can be loaded from data #739

Open
nus-pe-bot opened this issue Apr 15, 2020 · 1 comment
Open

Two users of the same name can be loaded from data #739

nus-pe-bot opened this issue Apr 15, 2020 · 1 comment

Comments

@nus-pe-bot
Copy link

If there in data is john_main_schedule.txt and john_schedule.txt then the application loads in two users of the same name. This conflicts with the behavior that you cannot create two users with the same name.


[original: nus-cs2113-AY1920S2/pe-interim#748]
@synCKun
Copy link
Contributor

synCKun commented Apr 15, 2020

Team's Response

This is a good hack that you discovered. Yes you can create main user and another user with the same name but only if you manually add the file into the data folder, which was what i think you did.
The consequence of this is that the app take in an extra user of the same name.

Reason why i am putting NotInScope is because in the UG, we advised for files in the data folder to not be changed because of possibilties like this, and this hack will not be encountered by a typical user that is using the app through CLI.

Duplicate status (if any):

--

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

2 participants