TabrizPlv / pe

0 stars 0 forks source link

Duplicates are not case sensitive #4

Open TabrizPlv opened 1 year ago

TabrizPlv commented 1 year ago

Reroll does not detect duplicates when there are same inputs, but different casings. It is reasonable to expect that two Characters with the same Name refer to the same Character even if their names are not in the same casing. For example, make char Bob followed by make char BOB creates two different Characters with Names Bob and BOB respectively

soc-pe-bot commented 1 year ago

Team's Response

Thanks for spotting this!

This issue was actually brought up during the PE-D. While it is true that we did not account for it originally, our team thought about it and felt that this was not something that needs fixing (hence it was not mentioned as a feature flaw in the DG). For example, in the context of the game, one may have an item named iron sword and a separate glitched item named iRoN sWoRd.

Hence, this behaviour is expected in the context of the app.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Considering the context of the app, there can be better ways to show a glitched item other than through its name. Even if the dev team is insistent on using the name, I believe that allowing such duplicates to exist is too huge a sacrifice as it negatively impacts the user's experience. For example, since the names are not case-sensitive, all other commands will also have to be case-sensitive since they have to remember the specific casings of the name of the entity in order to interact with them through the commands.