Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PE-D][Tester A] Using "|" character in fields will cause issues after re-loading application #86

Open
nus-pe-script opened this issue Nov 3, 2023 · 1 comment
Labels
severity.Low A flaw that is unlikely to affect normal operations of the product type.Bug A bug
Milestone

Comments

@nus-pe-script
Copy link

Consider the following command: expense /amt 20 /date 30-10-2023 /desc this | will override | NO_EXPENSE_CATEGORY /note pasta /cat transportation

The above command indicates creation of expense with desc set to "this | will override | NO_EXPENSE_CATEGORY" and note set to "pasta" and cat set to "transportation"

image.png

After exiting, notice that the data is changed with fields being overridden because of the use of the "|" in the command option values.

image.png


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

@spaceman03 spaceman03 added priority.Low Unlikely to do type.Bug A bug labels Nov 5, 2023
@spaceman03 spaceman03 added this to the v2.1 milestone Nov 5, 2023
@spaceman03 spaceman03 added severity.Low A flaw that is unlikely to affect normal operations of the product and removed priority.Low Unlikely to do labels Nov 5, 2023
@spaceman03
Copy link
Collaborator

Added instructions in UG to avoid such situations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity.Low A flaw that is unlikely to affect normal operations of the product type.Bug A bug
Projects
None yet
Development

No branches or pull requests

2 participants