"edit" command allows ending time to be earlier than starting time (and times with invalid format)

[nus-pe-script]

Describe the bug
When I entered the command "edit 1 te/0670" (a nonsense time format which is also earlier than the starting time), the update is successful.

To Reproduce
Steps to reproduce the behavior:

1. Go to input textbox.
2. Type "edit 1 te/0670".
3. Press enter.
4. Click on entry Updated the about us, changed storage and package name #1.
5. See error.

Expected behavior
Program should prevent nonsense updates/entries.

Screenshots

---

Reported by: @ongweekeong
Severity: Low

_{[original: nusCS2113-AY1819S1/pe-1#610]}

[arjo129]

Same as #61 !! @cjunxiang

[cjunxiang]

resolved

[shuanang]

still buggy...

[cjunxiang]

that's cause the PR havent got accepted heh

[shuanang]

ps. PR #119 didn't solve this - so reopened this, to remind myself when I'm testing for bugs in the later release (hopefully tmr night? :P)

[shuanang]

add ds/07112018 ts/0792 de/07112018 te/1000 ss/0900 d/10 pg/A l/Sentosa tz/+8

ts of 0792 is not a valid time!

@cjunxiang bro this is still buggy XD ps. closing issues should be done after the PR is merged to see if the bug is fixed...

[cjunxiang]

add ds/07112018 ts/0792 de/07112018 te/1000 ss/0900 d/10 pg/A l/Sentosa tz/+8

ts of 0792 is not a valid time!

@cjunxiang bro this is still buggy XD ps. closing issues should be done after the PR is merged to see if the bug is fixed...

haha well if it takes 5 days to get a PR reviewed i figure i might as well close it to track my own progress.
anyway i done up a new PR! Thanks for helping to review

[arjo129]

Seems fixed finally!