The tp from ay2223s1-cs2113-f11-2

appointment view seems to list appointments with the wrong index number, starting from 2, unless they have some other meaning, which is not specified in the UG. This is causing severe failure in other features.

I believe the failure of indexing occurs when add appointment using appointment add

Change indexing logic to ensure they always start from 1

Labels: severity.High type.FeatureFlaw
original: Brominne/ped#1

Remove appointments

cancel a customer’s appointment.

Allocate appointment to employee

[PE-D][Tester C] Pet add: Pet name can be empty

When I add a pet with no name bue a space, your app still recognizes it and add to the pet list. I think name should not be empty

Labels: type.FunctionalityBug severity.Low
original: bdthanh/ped#4

[PE-D][Tester A] Jumping index when removing and adding services

Labels: type.FunctionalityBug severity.Medium
original: Brandon-OS/ped#3

[PE-D][Tester A] Expected output of pet health status does not match guide

Labels: type.DocumentationBug severity.Low
original: Brandon-OS/ped#1

[PE-D][Tester B] Failure to in adding task associated with an appointment

Add task failure, when following the example walkthrough of UG. I guess it can be due to being unable to find appointment #1 due to the wrong indexing of the appointment.

Rectifying the indexing issue in issue #1 could resolve this problem

Labels: severity.High type.FunctionalityBug
original: Brominne/ped#3

[PE-D][Tester A] Accepts false dates as parameter

Labels: type.FunctionalityBug severity.Medium
original: Brandon-OS/ped#4

Parse the input

Add some probability to the pats status

Add some probability to the pats status like after each appointment and each task. Make it relative to species.

Delete employee

Exception handling: Parser

[PE-D][Tester D] Index for subsequent employee is invalid after the first employee is removed from employees list

Issue

After I attempted to remove the first employee from the employee list, when I add a new employee I am expected to see the new employee be given an index number of 1. However, the program gave an index number 2 to the new employee instead.

Suggestion

Maybe generate the index number dynamically when listing.

Output

Labels: severity.Medium type.FunctionalityBug
original: chydarren/ped#6

[PE-D][Tester D] Adding a pet with only health without other tags will result in the program to exit unexpectedly

Issue

As seen below, when I tried to add a pet with only its health, the program does not catch the error in tags given and cause me to exit unexpectedly.

Suggestion

Suggestion is to check for all present tags.

Output

Labels: severity.High type.FunctionalityBug
original: chydarren/ped#2

Add the durability of the service

Add the durability of the service and make it decrease after each task.(can make it 100% first then no need to change the interface with other classes)

[PE-D][Tester A] Skipping index in view after removing a pet and adding more pets

Labels: type.FunctionalityBug severity.Medium
original: Brandon-OS/ped#2

[PE-D][Tester C] Empty health value causes the app to crash

When I add a pet with no health value, the error is not handled properly causing the app to be crashed. Moreover, you do not have storage feature, then user will lose all of his/her data

Labels: type.FunctionalityBug severity.High
original: bdthanh/ped#3

Remove a pet

in case the owner wants to drop the membership

Remove a service from the list of services the clinic can provide

Register a new pet

keep track of new pets

Add to the list of services this clinic can provide

select corresponding service when entering an appointment

Exception handling: Petparser

Change the status of pets

know whether the pets are recovered

Remove tasks

View all the upcoming appointments

keep track of the clinic’s schedule.

Set the appointment status

[PE-D][Tester D] Removing a pet without specifying its index result in the program to exit unexpectedly

Issue

As shown below, attempt to remove the pet without specifying its index will cause me to exit unexpectedly.

Suggestion

Suggest to check for the availability of the parameters after checking for the presence of tag, and disallow all empty parameters to proceed.

Output

Labels: severity.High type.FunctionalityBug
original: chydarren/ped#3

[PE-D][Tester D] Invalid employee name with special symbols could be added into the employees list

Issue

I am unsure if this was the intended behavior for the program that the team would like to keep, but I suggest to check for just alphabets in the name, as its quite rare that names have special symbols and numbers alone. That said, I cannot rule out this possibility entirely.

Suggestion

Check for special symbols and numbers before adding the employee name. Restrict to certain characters rather than allowing all kinds of input.

Output

Labels: severity.VeryLow type.FeatureFlaw
original: chydarren/ped#4

Find the DG uml tool

[PE-D][Tester D] No message is shown with an invalid health integer for pet and pet was not added into the list

Issue

As seen below, when you attempt to add the second pet with health that is invalid, the program does not inform that the health is invalid. The pet is not added into the list but the user is not informed.

Suggestion

Return an error message if any of the tags or parameters are invalid, and inform the user that the desired action was not performed, for the avoidance of doubt.

Output

Labels: type.FunctionalityBug severity.Low
original: chydarren/ped#1

Add appointments

I can schedule a customer’s appointment.

Add tasks

Exception handling: EmployeeParser, serviceparser

[PE-D][Tester D] Adding an employee with duplicate tags result in duplicated tag and parameter to be entered into the list

Issue

As depicted below, adding an employee using duplicated name tags will result in the duplicated tag and parameter to be entered into the list as part of the first employee's name.

Suggestion

Suggest to check for any presence of duplicated tags.

Output

Labels: severity.VeryLow type.FeatureFlaw
original: chydarren/ped#5

View tasks

check all of the tasks assigned to a staff

Add new employees

newly hired staff can find their place.

[PE-D][Tester A] Skipping index in employee view

Labels: severity.Medium type.FeatureFlaw
original: Brandon-OS/ped#6

[PE-D][Tester E] When pet add is a negative health, does not show up any Ui display

i.e. when the pet add n/Eliza s/cat h/-1 is keyed, no feedback display is shown to the user or acceptance or error. However, under pet view, it is not recorded so not a big error. Can consider adding an error to notify the user that negative numbers are not allowed

Similar to the previous small bug for string length more than 1, because health should be input as 0 and 1 only as seen in the user guide. Would be good to create that restriction to prevent unknown feedback display.

Labels: type.FeatureFlaw severity.Low
original: ivanthengwr/ped#1