[Defensive] EditCommand

[underthehai]

Update checks for valid tags and index
Update error messages

[codecov-io]

Codecov Report

Merging #178 into master will increase coverage by 0.49%.
The diff coverage is 47.05%.

@@             Coverage Diff              @@
##             master     #178      +/-   ##
============================================
+ Coverage     59.83%   60.33%   +0.49%     
- Complexity      794      803       +9     
============================================
  Files           126      126              
  Lines          2806     2801       -5     
  Branches        353      361       +8     
============================================
+ Hits           1679     1690      +11     
+ Misses          983      961      -22     
- Partials        144      150       +6     

Impacted Files	Coverage Δ	Complexity Δ
...s/logic/command/edit/EditAccommodationCommand.java	79.41% <0.00%> (-4.97%)	12.00 <0.00> (ø)
.../seedu/address/logic/command/edit/EditCommand.java	60.00% <ø> (ø)	1.00 <0.00> (ø)
.../address/logic/command/edit/EditFriendCommand.java	75.86% <0.00%> (-5.62%)	12.00 <0.00> (ø)
...ress/logic/command/edit/EditTravelPlanCommand.java	83.87% <0.00%> (-5.79%)	12.00 <0.00> (ø)
...in/java/seedu/address/logic/parser/ParserUtil.java	42.59% <ø> (ø)	9.00 <0.00> (ø)
.../seedu/address/logic/parser/EditCommandParser.java	50.00% <48.00%> (+12.50%)	5.00 <2.00> (ø)
...ddress/logic/command/edit/EditActivityCommand.java	77.14% <66.66%> (+2.14%)	12.00 <0.00> (ø)
...edu/address/logic/command/edit/EditDescriptor.java	60.86% <66.66%> (+5.05%)	31.00 <4.00> (+6.00)
...rc/main/java/seedu/address/model/ModelManager.java	66.66% <0.00%> (+0.72%)	38.00% <0.00%> (ø%)
... and 1 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a5260b3...a161495. Read the comment docs.

[timjkong]

LGTM

[jeannetoh99]

LGTM