Skip to content

Inconsistent case sensitivity usage in commands #2482

@nus-pe-bot

Description

@nus-pe-bot

Description

The find command is case insensitive while the add-to-group command is case sensitive. This makes even less sense when taking into consideration the application uses the Person's name as the unique identifier, so no two Persons will have the same name, so case sensitivity should not be taken into consideration for add-to-group.

Steps to reproduce

  1. Ensure there is an existing person by the name JoJoBiz
  2. Enter a valid add-to-group command: add-to-group n/jojobiz g/1

Expected

The system should accept and add JoJoBiz to the group, as it has the name of an existing Person and is within the input constraints of the add-to-group command

Actual

The system rejects the command and returns "Person not found!" error message

Screenshots

image.png


[original: nus-cs2103-AY2425S2/pe-interim#2479] [original labels: severity.Medium type.FeatureFlaw]

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions