CS2103-AY1819S2-W10-1 / main

92%

master: 92%

LAST BUILD ON BRANCH add-archives

branch: add-archives

CHANGE BRANCH

x

Reset

• add-archives
• add-archive-feature
• add-context-swinging
• add-moar-tests
• add-ppp-jon
• add-qj-ppp
• add-thomastanck-ppp
• add-ug-quickstart-image
• autofill-feed
• bump-131ea
• bump-version-13
• delete-all
• exit-cleanly
• feed-tags
• feedbook
• feeds-stuff
• final-final-final-branch
• fix-128
• fix-archive-bug
• master
• move-commands-around
• multiple-entrybooks
• offline-delete
• offline-view
• qjqqyy-add-impl-descr
• release-v1.2
• remove-address
• revert-90-update-dg
• rlrh-patch-1
• rlrh-ppp
• stage-nettest-files
• thomastanck-patch-1
• typos-and-stuff
• update-asciidoc-settings
• update-dg
• update-dg-jonathan
• update-docs
• update-edit-nolink
• update-find-command
• update-ug
• v1.1
• v1.2
• v1.3
• v1.3.1
• v1.4

Build # 431

Build Type

push

travis-ci

Committed by epicfailname

Commit Message

Implement delete command for archives context

DeleteArchiveEntryCommand and DeleteArchiveEntryCommandParser classes created to implement this.
Using the same delete command and parser as the list-context was decided against.
The parser needs to return a specific Command object, so if we were to use the same parser, we need to use the same DeleteCommand class for deleting in both contexts.
This means that we have to rely on ModelManager#context to decide which entrybook to delete the entry from.
Could work, but feels very iffy.

Run Details

1852 of 2022 relevant lines covered (91.59%)

0.92 hits per line