w-yuchen / ped Goto Github PK

Notes about command format in the UG is for AB3 and not updated for NUS Module Planner, hence there is no mention about the command constrains for NUS Modules Planner.

Error message provides an example that is:
addp d/Software Engineering Route t/fun t/good prospects

But when this example is actually used as input,

it is not accepted as a valid input.

This is highly likely to confuse the user.

Steps to reproduce:

Assuming plan 1 exists and has semester 2,
first add a module to a plan and semeter:
addm m/cs2100 p/1 s/2
then try to delete it:
deletem m/cs2100 p/1 s/2

Expected:

Successful deletion.

Actual:

This module clearly exists since:

Steps to reproduce:

info 123

Expected:

Error message telling me what are the valid info commands, or just showing all modules as if info 123 is the same as info.

Actual:

No error messages, input not highlighted as red.

Pressing enter does nothing.

It asks for alphanumeric tag names, but rejects anything with spaces, perhaps it can be specified that spaces are allowed?

In the UG, under View module info: info:

In UG:

The titles of other commands all have the actual command that can be used formatted as code (surrounded with ``), as seen here:

The command for setting current semester is current, so it should be current instead of current semester.

The actual app only accepts alphanumeric inputs with no spaces as tag names, but no mention of this in the UG.

In UG:

Actual usage in app:

Since the usage is correctly recorded in the app, it should not pose too much of an issue to most users, but will be an issue for users who want to refer the UG.

Steps to reproduce:

deletem, or any invalid deletem commands.

Expected:

Error message telling me how to use deletem.

Actual:

Error message telling me how to use deletes.

In UG:

Show CAP is not considered valid in the app.

Steps to reproduce:

adds p/1 s/-2 for -2 semester
adds p/1 s/1000 for 1000 semester
adds p/1 s/0 for 0 semester

Expected:

These values are clearly invalid semester numbers and should be rejected as invalid input, just like semester numbers with non-numerical values:

Actual

Semester -2:

Semester 1000:

Semester 0:

Steps to reproduce:

Any invalid addm commands, e.g. addm, or addm m/

Expected:

Error message informing that it is an invalid command format, along with a usage message telling me the correct format of the command.

Actual:

No messages at all, the input is not even highlighted in red as with other invalid inputs.

Steps to reproduce;

Key in current as an input.

Expected:

Invalid command format! 
current s/SEM_NUMBER
Marks the supplied semester as the current one.

Actual:

As seen here, the screenshot still shows the link to AB3's UG.

Steps to reproduce:

addm m/EU4401 p/1 s/2

Expected:

Some warning as I have not added any prerequisites for this module.
This module has prerequisites as stated within the app:

The UG also clearly states the existence of such warnings:

Actual:

No warning, module added successfully.