JoanneJo / pe

0 stars 0 forks source link

Vendor created on another account is brought in to another account #11

Open JoanneJo opened 7 months ago

JoanneJo commented 7 months ago

Steps:

  1. Login to an account
  2. add vendorsupply
  3. Login to another account
  4. item already exists

Expected: data from different accounts should be stored differently

image.png

nus-pe-bot commented 7 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Any user will be able to access the same data, hence what is the purpose of a user system?

image.png


[original: nus-cs2113-AY2324S1/pe-interim#788] [original labels: severity.Medium type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Unable to tell which feature this bug is tagged to, furthermore a user system enables security to some level and being able to access different data could be a development of the feature in future iterations

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.IssueUnclear`] - [x] I disagree **Reason for disagreement:** Usually user systems are set up to enable access control which is to grant or restrict access to different parts of the program based on the user roles. If all users are sharing the same data file like in the program, then it seems like there is no need to have a login system to verify the user's identity. It may even cause more problems by storing usernames and passwords, due to the need to protect these systems and data. Since the feature is not implemented in a way that it should work, this would be a feature flaw. Additionally, having a shared file storage mechanism is not allowed. ![image.png](https://raw.githubusercontent.com/JoanneJo/pe/main/files/60b60ce7-6bb6-4b88-86d1-689f1a3f8584.png)