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

[CS2103-F11-1] ManageEZPZ #62

Open
wants to merge 791 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
791 commits
Select commit Hold shift + click to select a range
cb77411
checkstyle fix
aaron-ljx Apr 6, 2022
1fe6bd8
checkstyle fix
aaron-ljx Apr 6, 2022
36c9393
Update PPP
dannytayjy Apr 6, 2022
7645957
Update PPP
dannytayjy Apr 6, 2022
dc8042f
code edited based on danny's suggestion
aaron-ljx Apr 6, 2022
12bb906
checkstyle fix
aaron-ljx Apr 6, 2022
ce2bc90
edited UG slightly
aaron-ljx Apr 6, 2022
93c37d2
Merge pull request #257 from aaron-ljx/editTaskFix
dannytayjy Apr 6, 2022
7845904
Change new features to enhancement implemented
Denniszedead Apr 6, 2022
925c6a5
Merge branch 'master' of https://github.com/Denniszedead/tp
Denniszedead Apr 6, 2022
5fe6290
Merge branch 'master' into update-ppp
Denniszedead Apr 6, 2022
619670e
Update Denniszedead PPP
Denniszedead Apr 6, 2022
f5ed8fc
Add whitespace in PPP
Denniszedead Apr 6, 2022
c68ba9a
Make minor changes to String messages
dannytayjy Apr 6, 2022
ac67757
Fix String message in TagTaskCommand and UntagTaskCommand
dannytayjy Apr 6, 2022
b562816
Fix bug in DeleteTaskCommand
dannytayjy Apr 6, 2022
ee92495
Update dg on findTask command
Denniszedead Apr 6, 2022
fcce37d
Update details of EditEmployee, DeleteEmployee and DeleteTask in UG
dannytayjy Apr 6, 2022
c205870
Update PPP
dannytayjy Apr 6, 2022
1330adb
Add logger for findTask command
Denniszedead Apr 6, 2022
5e547ad
Edit PPP
alfredkohhh Apr 6, 2022
22cf9b6
Merge pull request #253 from dannytayjy/branch-Update-README-PPP
aaron-ljx Apr 7, 2022
dc55c63
Merge pull request #259 from dannytayjy/branch-FixBugsForV1.4-MinorFixes
aaron-ljx Apr 7, 2022
af175e7
Update Jun Yang's PPP
dannytayjy Apr 7, 2022
14bfbdc
Update ppp.
ChanWeiJie Apr 7, 2022
0f7a386
Change heading of ppp
ChanWeiJie Apr 7, 2022
77c5d6a
Vocab changes.
ChanWeiJie Apr 7, 2022
141a425
Update PPP
dannytayjy Apr 7, 2022
dfe59c5
updated PPP
aaron-ljx Apr 7, 2022
c30af28
Merge pull request #262 from aaron-ljx/updatePPP
ChanWeiJie Apr 7, 2022
2024086
Merge pull request #261 from dannytayjy/branch-UpdatePPP-JunYang
ChanWeiJie Apr 7, 2022
9966137
Merge pull request #243 from ChanWeiJie/branch-PPP
dannytayjy Apr 7, 2022
56c2d49
Merge pull request #260 from alfredkohhh/Branch-UpdatePPP
ChanWeiJie Apr 7, 2022
b75786b
Merge pull request #258 from Denniszedead/update-ppp
dannytayjy Apr 7, 2022
21331a6
Spaced out ppp.
ChanWeiJie Apr 7, 2022
3e1c338
Merge pull request #263 from ChanWeiJie/branch-PPP
alfredkohhh Apr 7, 2022
df89a49
Update PPP to fit a page.
ChanWeiJie Apr 7, 2022
5f1117b
Shorten PPP
Denniszedead Apr 7, 2022
17be2ee
Change base.scss to our product name.
ChanWeiJie Apr 7, 2022
b8177a7
Change product name to big M.
ChanWeiJie Apr 7, 2022
6bb7068
Update AboutUs.md
aaron-ljx Apr 7, 2022
628de7b
English changes.
ChanWeiJie Apr 7, 2022
3d72ec2
Merge pull request #265 from aaron-ljx/editAboutUs
ChanWeiJie Apr 7, 2022
645b4a7
Merge pull request #264 from ChanWeiJie/branch-PPP
alfredkohhh Apr 7, 2022
517966b
Testing page break.
ChanWeiJie Apr 7, 2022
37b1913
Delete extra spaces.
ChanWeiJie Apr 7, 2022
13702e3
Edit PPP
alfredkohhh Apr 7, 2022
6fa95dd
Add fullstops.
ChanWeiJie Apr 7, 2022
75f4e25
Merge branch 'master' of https://github.com/Denniszedead/tp
Denniszedead Apr 7, 2022
2e35e3a
Merge branch 'master' into update-ppp
Denniszedead Apr 7, 2022
159806d
Merge pull request #266 from ChanWeiJie/branch-PPP
alfredkohhh Apr 7, 2022
6fe37cb
Merge pull request #267 from alfredkohhh/Branch-UpdatePPP
ChanWeiJie Apr 7, 2022
1a3b237
Shorten PPP
Denniszedead Apr 7, 2022
5297a07
Shorten PPP
Denniszedead Apr 7, 2022
331c1b6
Edit PPP
alfredkohhh Apr 7, 2022
d038c8b
Shorten ppp
Denniszedead Apr 7, 2022
f33c04b
Refine PPP
Denniszedead Apr 7, 2022
d6eac0b
Merge pull request #268 from alfredkohhh/Branch-UpdatePPP
ChanWeiJie Apr 7, 2022
c34b147
Shortern PPP
Denniszedead Apr 7, 2022
7f2bd79
Add Note to Edit Task
alfredkohhh Apr 7, 2022
9bb269d
Merge pull request #270 from alfredkohhh/Branch-UG-PDF
ChanWeiJie Apr 7, 2022
f754d2d
Add pagebreak
alfredkohhh Apr 7, 2022
5f8cb0e
Merge pull request #272 from alfredkohhh/Branch-UG-PDF
ChanWeiJie Apr 7, 2022
91b2f71
Add page break to see sections
alfredkohhh Apr 7, 2022
7b60554
Merge pull request #273 from alfredkohhh/Branch-UG-PDF
ChanWeiJie Apr 7, 2022
0c17844
Add page breaks for clearer viewing
alfredkohhh Apr 7, 2022
ab30648
Merge pull request #274 from alfredkohhh/Branch-UG-PDF
ChanWeiJie Apr 7, 2022
224f3de
Reedit page breaks
alfredkohhh Apr 7, 2022
a0f932d
Merge pull request #275 from alfredkohhh/Branch-UG-PDF
ChanWeiJie Apr 7, 2022
4b999f5
Reedit page breaks
alfredkohhh Apr 7, 2022
9ad0834
Merge pull request #276 from alfredkohhh/Branch-UG-PDF
ChanWeiJie Apr 7, 2022
9d48d05
Add logger to findEmployee and findTask commands
Denniszedead Apr 7, 2022
0285b4d
Add page break below findTask command
Denniszedead Apr 7, 2022
34c6f45
Add page break at ug contribution section
Denniszedead Apr 7, 2022
028871b
Update Java Docs
ChanWeiJie Apr 7, 2022
b1ac0d2
Merge branch 'master' into branch-fixBugs1.4
ChanWeiJie Apr 7, 2022
c186216
Merge branch 'master' of https://github.com/Denniszedead/tp
Denniszedead Apr 8, 2022
20bc06d
Merge pull request #269 from Denniszedead/update-ppp
alfredkohhh Apr 8, 2022
e1c56e2
Add Test Cases for JsonAdaptedTask, improving code coverage
alfredkohhh Apr 8, 2022
cd34552
Remove unused statements
alfredkohhh Apr 8, 2022
e068373
Merge pull request #279 from alfredkohhh/Branch-StorageTestCases
ChanWeiJie Apr 8, 2022
c522f40
Merge pull request #244 from ChanWeiJie/branch-moreTests
alfredkohhh Apr 8, 2022
1c878bf
Merge branch 'master' of https://github.com/Denniszedead/tp
Denniszedead Apr 8, 2022
4399c2c
Update FindTaskCommandTest.java
ChanWeiJie Apr 8, 2022
d98b0fc
Update FindTaskCommandTest.java
ChanWeiJie Apr 8, 2022
f88a01d
Merge branch 'master' of https://github.com/Denniszedead/tp
Denniszedead Apr 8, 2022
acc192e
updated test cases for edit task
aaron-ljx Apr 8, 2022
ceb706a
Merge pull request #282 from aaron-ljx/updateEditTaskTest
ChanWeiJie Apr 8, 2022
4016014
Update PPP
dannytayjy Apr 8, 2022
6c95157
Merge pull request #277 from Denniszedead/find-logger
ChanWeiJie Apr 8, 2022
f52caf1
Merge pull request #283 from dannytayjy/branch-UpdatePPP-JunYang
ChanWeiJie Apr 8, 2022
a2b7a54
Merge branch 'master' of https://github.com/ChanWeiJie/tp into branch…
ChanWeiJie Apr 8, 2022
aa7ea07
Merge branch 'branch-fixBugs1.4' of https://github.com/ChanWeiJie/tp …
ChanWeiJie Apr 8, 2022
f170e56
Update PPP
dannytayjy Apr 8, 2022
0082b73
Fix testcase for findTaskCommand
Denniszedead Apr 8, 2022
a3a2d90
Fix code to follow coding standard
Denniszedead Apr 8, 2022
80a8d55
Merge branch 'AY2122S2-CS2103-F11-1:master' into fix-testcases
Denniszedead Apr 8, 2022
9443bd3
Merge branch 'master' of https://github.com/Denniszedead/tp
Denniszedead Apr 8, 2022
50cf029
Merge branch 'master' into fix-testcases
Denniszedead Apr 8, 2022
4c91bb5
Merge branch 'fix-testcases' of https://github.com/Denniszedead/tp in…
Denniszedead Apr 8, 2022
5bebcea
Update DG.
ChanWeiJie Apr 8, 2022
4bfa934
Updated model puml image.
ChanWeiJie Apr 8, 2022
2e23094
Merge pull request #284 from dannytayjy/branch-UpdatePPP-JunYang
ChanWeiJie Apr 8, 2022
814b235
Update FindTaskCommandTest.java
Denniszedead Apr 8, 2022
ed81bde
Include methods to avert error.
ChanWeiJie Apr 8, 2022
6132a97
Update javdocs for overriden methods.
ChanWeiJie Apr 8, 2022
11b317f
Update gradle.yml
Denniszedead Apr 8, 2022
bc68034
Update gradle.yml
Denniszedead Apr 8, 2022
69a8b90
Update gradle.yml
Denniszedead Apr 8, 2022
53bb222
Merge branch 'fix-testcases' of https://github.com/Denniszedead/tp in…
Denniszedead Apr 8, 2022
11effd6
Change testcase
Denniszedead Apr 8, 2022
23c431c
Modify test case
Denniszedead Apr 8, 2022
01ec003
Clean up code
Denniszedead Apr 8, 2022
784f46a
Update Reuse Policy.
ChanWeiJie Apr 8, 2022
acc841d
updated UG and DG
aaron-ljx Apr 8, 2022
e22c0cb
Merge branch 'master' of https://github.com/Denniszedead/tp
Denniszedead Apr 8, 2022
85642c1
Merge branch 'master' into update-dg
Denniszedead Apr 8, 2022
e7f89fc
added edit task to command summary
aaron-ljx Apr 8, 2022
d87dfde
added extra info about edit task in DG
aaron-ljx Apr 8, 2022
696a9a2
Update DG to v1.4
Denniszedead Apr 8, 2022
7c9e816
Update Effort in DG.
ChanWeiJie Apr 9, 2022
2914932
Change user stories
Denniszedead Apr 9, 2022
104b32b
Remove extra extension
Denniszedead Apr 9, 2022
2fdf3cf
Add extension
Denniszedead Apr 9, 2022
ef9c5bb
Implement test-cases for AddTodo, TagTask, UntagTask, TagPriority.
alfredkohhh Apr 9, 2022
9369908
Update Use-Cases
alfredkohhh Apr 9, 2022
a32205d
Add "NONE" as a Priority.
alfredkohhh Apr 9, 2022
235188a
Merge pull request #285 from Denniszedead/fix-testcases
alfredkohhh Apr 9, 2022
39e0286
Merge pull request #278 from ChanWeiJie/branch-fixBugs1.4
Denniszedead Apr 9, 2022
7c40aee
Merge branch 'master' of https://github.com/alfredkohhh/tp into Branc…
alfredkohhh Apr 9, 2022
b4ad4bc
Merge pull request #286 from ChanWeiJie/branch-DGUpdates
alfredkohhh Apr 9, 2022
7b3dc6b
Fix bug, refractor EditPersonDescriptor and implement test cases
dannytayjy Apr 9, 2022
62bc991
Merge pull request #289 from Denniszedead/update-dg
ChanWeiJie Apr 9, 2022
1db7460
Merge pull request #288 from aaron-ljx/updatedocs
ChanWeiJie Apr 9, 2022
eb7c142
Merge branch 'master' into Branch-UpdateDG
ChanWeiJie Apr 9, 2022
e622da3
Merge pull request #291 from alfredkohhh/Branch-UpdateDG
ChanWeiJie Apr 9, 2022
a7f3ea9
Update DG details and diagrams
dannytayjy Apr 9, 2022
9196445
Merge branch 'master' into branch-UpdateDG-JunYang
Denniszedead Apr 9, 2022
d9bd026
Merge pull request #292 from dannytayjy/branch-UpdateDG-JunYang
Denniszedead Apr 9, 2022
d7e4a8c
Merge branch 'master' of https://github.com/Denniszedead/tp
Denniszedead Apr 9, 2022
810676b
Update ppp.
ChanWeiJie Apr 9, 2022
2cb2ffb
Add test-cases to raise coverage of all test-cases to 100%
alfredkohhh Apr 9, 2022
a25ef81
Merge pull request #293 from ChanWeiJie/branch-PPP
alfredkohhh Apr 9, 2022
23c88fa
Merge pull request #290 from alfredkohhh/Branch-MoreTestCases
ChanWeiJie Apr 9, 2022
5458c9a
Update png files
alfredkohhh Apr 9, 2022
d37a2df
Merge pull request #294 from alfredkohhh/Branch-UpdatePNG
ChanWeiJie Apr 9, 2022
c0dd25f
Update png files
alfredkohhh Apr 9, 2022
e1967e1
Update png files
alfredkohhh Apr 9, 2022
6a8c6f6
Merge pull request #295 from alfredkohhh/Branch-UpdatePNG
ChanWeiJie Apr 9, 2022
f8f4f6b
Update lightbulb format
alfredkohhh Apr 9, 2022
cee440a
Merge pull request #296 from alfredkohhh/Branch-UpdateUGNote
ChanWeiJie Apr 9, 2022
743965a
Merge branch 'master' of https://github.com/Denniszedead/tp
Denniszedead Apr 9, 2022
94157be
Update lightbulb format
alfredkohhh Apr 9, 2022
d588934
Merge pull request #297 from alfredkohhh/Branch-UpdateUGNote
ChanWeiJie Apr 9, 2022
273f140
Update HTML lightbulb format
alfredkohhh Apr 9, 2022
84e6658
Merge pull request #298 from alfredkohhh/Branch-UpdateUGNote
ChanWeiJie Apr 9, 2022
e7da91b
Added br for take note
alfredkohhh Apr 9, 2022
49926df
Reorder test cases in dg
Denniszedead Apr 9, 2022
7942f84
Merge pull request #299 from alfredkohhh/Branch-UpdateUGNote
ChanWeiJie Apr 9, 2022
49eb90e
Added br for take note
alfredkohhh Apr 9, 2022
e7efd07
Merge pull request #300 from Denniszedead/update-dg
ChanWeiJie Apr 9, 2022
1afc009
added test cases for add event and add deadline + refined test cases …
aaron-ljx Apr 9, 2022
3d4876f
Merge pull request #301 from alfredkohhh/Branch-UpdateUGNote
ChanWeiJie Apr 9, 2022
264512e
Added br for take note
alfredkohhh Apr 9, 2022
8e65143
Merge pull request #303 from alfredkohhh/Branch-UpdateUGNote
ChanWeiJie Apr 9, 2022
6443d1d
fix checkstyle
aaron-ljx Apr 9, 2022
89df322
Add page-break
alfredkohhh Apr 9, 2022
8b2e60b
Merge pull request #304 from alfredkohhh/Branch-UpdateUGNote
ChanWeiJie Apr 9, 2022
84d63d5
Add final page-break
alfredkohhh Apr 9, 2022
c994046
Merge pull request #302 from aaron-ljx/updateaddtest
alfredkohhh Apr 9, 2022
8b900e2
Merge pull request #305 from alfredkohhh/Branch-UpdateUGNote
ChanWeiJie Apr 9, 2022
fff70eb
Code Quality changes for JsonAdaptedTask
alfredkohhh Apr 9, 2022
2310868
Add Javadocs to methods.
alfredkohhh Apr 9, 2022
562e7ab
Fix gradle errors
alfredkohhh Apr 9, 2022
94431fd
done with edit task parser
aaron-ljx Apr 9, 2022
fec3476
added test cases for parsers
aaron-ljx Apr 9, 2022
6cb43e7
Fix test cases and review Javadocs and code quality for DeleteEmployee,
dannytayjy Apr 9, 2022
0d6e7a1
Make minor edits to spacing and code comments in PersonListPanel.java
dannytayjy Apr 9, 2022
22c360d
Improve code quality of TaskCard.java
dannytayjy Apr 9, 2022
8cb272d
Merge branch 'master' into branch-FixBugs-TestCases-CodeQuality
dannytayjy Apr 9, 2022
842cb35
Remove unused imports
dannytayjy Apr 9, 2022
b36738a
added edit task image to DG
aaron-ljx Apr 9, 2022
1d0fecb
Fix prohibited line ending issues
dannytayjy Apr 9, 2022
e90765b
Comment out test case for valid index filtered list
dannytayjy Apr 9, 2022
ac45acd
Update failed test case
dannytayjy Apr 10, 2022
ed22c51
Remove unused imports
dannytayjy Apr 10, 2022
bfaeb0f
Edit test case
dannytayjy Apr 10, 2022
94bff03
Comment out error test cases
dannytayjy Apr 10, 2022
259780b
Remove unused imports
dannytayjy Apr 10, 2022
41e699c
Merge pull request #308 from aaron-ljx/parsertest
ChanWeiJie Apr 10, 2022
8513fc4
Merge pull request #309 from dannytayjy/branch-FixBugs-TestCases-Code…
ChanWeiJie Apr 10, 2022
5bd7334
Merge pull request #307 from alfredkohhh/Branch-Javadocs
ChanWeiJie Apr 10, 2022
7b0050c
Merge pull request #306 from alfredkohhh/Branch-CodeQuality
ChanWeiJie Apr 10, 2022
14c1a0d
Merge pull request #310 from aaron-ljx/refinedocs
ChanWeiJie Apr 10, 2022
a86a7c4
made slight changes to edit task image
aaron-ljx Apr 10, 2022
4098ec9
Merge pull request #311 from aaron-ljx/refinedocs1
ChanWeiJie Apr 10, 2022
521b8e6
Update DG.
ChanWeiJie Apr 10, 2022
be0eb97
Update Index for UG.
ChanWeiJie Apr 10, 2022
35218ad
Merge pull request #313 from ChanWeiJie/branch-UG
alfredkohhh Apr 10, 2022
952a98b
update png
ChanWeiJie Apr 10, 2022
cfc1b23
Merge pull request #312 from ChanWeiJie/branch-DG
alfredkohhh Apr 10, 2022
41841a2
Update DG.
ChanWeiJie Apr 10, 2022
3387132
Update gradle to v1.4
ChanWeiJie Apr 10, 2022
568b958
Merge pull request #314 from ChanWeiJie/branch-DG
alfredkohhh Apr 10, 2022
faa90b7
Merge branch 'master' of https://github.com/Denniszedead/tp
Denniszedead Apr 10, 2022
9f34c3b
Change size of image
Denniszedead Apr 10, 2022
3b36399
Change UML sequence diagram
Denniszedead Apr 10, 2022
c1fefd1
Change UML Sequence diagram
Denniszedead Apr 10, 2022
0e8ff81
Merge pull request #317 from Denniszedead/update-dg
ChanWeiJie Apr 10, 2022
91ff544
Update page break on DG.
ChanWeiJie Apr 10, 2022
77f86fe
Merge branch 'master' of https://github.com/Denniszedead/tp
Denniszedead Apr 10, 2022
05d15d2
Merge pull request #318 from ChanWeiJie/branch-DGSweep
alfredkohhh Apr 10, 2022
2b1e6ff
Update PPP
Denniszedead Apr 10, 2022
ea4f2aa
Update DeveloperGuide.md
ChanWeiJie Apr 10, 2022
1aeeca7
Update DeveloperGuide.md
ChanWeiJie Apr 10, 2022
3c75b9b
Update DeveloperGuide.md
ChanWeiJie Apr 10, 2022
c7ba8cc
Update DeveloperGuide.md
ChanWeiJie Apr 10, 2022
2d64d3d
Update DeveloperGuide.md
ChanWeiJie Apr 10, 2022
e061222
Update DeveloperGuide.md
ChanWeiJie Apr 10, 2022
1fbed18
Update DG Contribution segment
Denniszedead Apr 10, 2022
7e63969
Remove optional section
Denniszedead Apr 10, 2022
b99fc84
Remove unused images
Denniszedead Apr 10, 2022
9ba778c
Merge pull request #319 from Denniszedead/update-ppp
ChanWeiJie Apr 10, 2022
8a9f542
Edit editEmployee notes.
alfredkohhh Apr 10, 2022
fe9eff5
Merge pull request #320 from alfredkohhh/Branch-EditEmployeeEdit
ChanWeiJie Apr 10, 2022
2011b3d
Edit case-sensitive.
alfredkohhh Apr 10, 2022
f6cbe6e
Edit UG wordings and add notes
alfredkohhh Apr 10, 2022
da84784
Bug Fix to storage
alfredkohhh Apr 10, 2022
0735201
Remove souts
alfredkohhh Apr 10, 2022
68e3d65
Merge pull request #321 from alfredkohhh/Branch-EditEmployeeEdit
ChanWeiJie Apr 10, 2022
b9c17c9
Remove unwanted methods.
ChanWeiJie Apr 10, 2022
fe38f36
Merge branch 'master' of https://github.com/ChanWeiJie/tp into Branch…
ChanWeiJie Apr 10, 2022
3d50e1d
Merge pull request #324 from ChanWeiJie/Branch-RemoveUnusedMethods
alfredkohhh Apr 10, 2022
c0de420
Update DeveloperGuide.md
ChanWeiJie Apr 10, 2022
d1b6a9b
Update DeveloperGuide.md
ChanWeiJie Apr 10, 2022
0c7bd77
Update DeveloperGuide.md
ChanWeiJie Apr 10, 2022
369c2fb
Modify test cases for DeleteEmployee and DeleteTask
dannytayjy Apr 10, 2022
57082ba
Make final updates to UI
dannytayjy Apr 10, 2022
0ec6eec
Add Javadoc for ResultDisplay constructor
dannytayjy Apr 10, 2022
bf1d859
Update PPP
dannytayjy Apr 10, 2022
c697e9b
Update PPP
dannytayjy Apr 10, 2022
615b85d
Update PPP
dannytayjy Apr 10, 2022
88a7b9e
Modify test cases for DeleteTask, DeleteEmployee and EditEmployee
dannytayjy Apr 11, 2022
027d04e
Modify test cases for DeleteTask, DeleteEmployee and EditEmployee to
dannytayjy Apr 11, 2022
dcd5506
Merge pull request #327 from dannytayjy/branch-UpdatePPP-JunYang
ChanWeiJie Apr 11, 2022
824049a
Merge pull request #326 from dannytayjy/branch-Final-UpdateUI
ChanWeiJie Apr 11, 2022
5f4e281
Merge pull request #325 from dannytayjy/branch-ModifyTestCases-Delete…
ChanWeiJie Apr 11, 2022
53b868e
Add more sample data
ChanWeiJie Apr 11, 2022
8014798
English
ChanWeiJie Apr 11, 2022
7cb78b6
Merge pull request #331 from ChanWeiJie/branch-moreDummyData
alfredkohhh Apr 11, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
done with edit task parser
  • Loading branch information
aaron-ljx committed Apr 9, 2022
commit 94431fd6ba08a73ba3bff1ea4abb7f4734a55395
19 changes: 18 additions & 1 deletion src/main/java/manageezpz/logic/commands/EditTaskCommand.java
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ public class EditTaskCommand extends Command {
* @param desc New description of the Task
* @param date New date of the Task
* @param time New time of the Task
* @param prefixStatusHash To fill up
* @param prefixStatusHash Indicate whether user had declared a prefix
*/
public EditTaskCommand(Index index, String desc, String date, String time,
HashMap<String, Boolean> prefixStatusHash) {
Expand Down Expand Up @@ -255,4 +255,21 @@ public Task updateEvent(Event currentTask, String desc, String date, String time

return updatedEventTask;
}

private boolean arePrefixStatusHashesEqual(HashMap<String, Boolean> other) {
return prefixStatusHash.get("description").equals(other.get("description"))
&& prefixStatusHash.get("date").equals(other.get("date"))
&& prefixStatusHash.get("datetime").equals(other.get("datetime"));
}

@Override
public boolean equals(Object other) {
return other == this // short circuit if same object
|| (other instanceof EditTaskCommand // instanceof handles nulls
&& index.equals(((EditTaskCommand) other).index)
&& desc.equals(((EditTaskCommand) other).desc)
&& date.equals(((EditTaskCommand) other).date)
&& time.equals(((EditTaskCommand) other).time))
&& arePrefixStatusHashesEqual(((EditTaskCommand) other).prefixStatusHash);
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
package manageezpz.logic.parser;

import static manageezpz.commons.core.Messages.MESSAGE_FIELD_NOT_EDITED;
import static manageezpz.commons.core.Messages.MESSAGE_INVALID_COMMAND_FORMAT_BIND;
import static manageezpz.logic.parser.CliSyntax.*;
import static manageezpz.logic.parser.CommandParserTestUtil.assertParseFailure;
import static manageezpz.logic.parser.CommandParserTestUtil.assertParseSuccess;
import static org.junit.jupiter.api.Assertions.assertTrue;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertTrue;

import manageezpz.commons.core.index.Index;
import manageezpz.logic.commands.EditTaskCommand;
import manageezpz.logic.parser.exceptions.ParseException;
import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.Test;

import java.util.HashMap;

public class EditTaskCommandParserTest {

private static final String INVALID_PREAMBLE_CONTAINS_WHITESPACE = "INVALID PREAMBLE";
private static final String VALID_INPUT= "1 desc/ hello date/ 2022-01-01 at/ 1900";
private static final String VALID_INPUT_WITHOUT_PREAMBLE = "desc/ hello date/ 2022-01-01 at/ 1900";

private EditTaskCommandParser parser = new EditTaskCommandParser();

@Test
public void parse_invalidPreamble_failure() {
assertParseFailure(parser, INVALID_PREAMBLE_CONTAINS_WHITESPACE,
String.format(MESSAGE_INVALID_COMMAND_FORMAT_BIND,
EditTaskCommand.MESSAGE_USAGE));

assertParseFailure(parser, "",
String.format(MESSAGE_INVALID_COMMAND_FORMAT_BIND,
EditTaskCommand.MESSAGE_USAGE));
}

@Test
public void parse_onlyPreambleNoPrefixes_failure() {
assertParseFailure(parser, "1",
MESSAGE_FIELD_NOT_EDITED + EditTaskCommand.MESSAGE_USAGE);
}

@Test
public void parse_success() throws ParseException {
ArgumentMultimap argMultimap =
ArgumentTokenizer.tokenize(VALID_INPUT,
PREFIX_DESCRIPTION, PREFIX_AT_DATETIME, PREFIX_DATE);
String desc = argMultimap.getValue(PREFIX_DESCRIPTION).orElse("");
String date = argMultimap.getValue(PREFIX_DATE).orElse("");
String time = argMultimap.getValue(PREFIX_AT_DATETIME).orElse("");
HashMap<String, Boolean> prefixStatusHash = new HashMap<>();
prefixStatusHash.put("description", true);
prefixStatusHash.put("date", true);
prefixStatusHash.put("datetime", true);
EditTaskCommand expectedCommand = parser.parse(VALID_INPUT);
EditTaskCommand actualCommand = new EditTaskCommand(Index.fromZeroBased(0), desc, date, time, prefixStatusHash);
assertEquals(expectedCommand, actualCommand);
}
}