-
Notifications
You must be signed in to change notification settings - Fork 5
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
Import Export feature #166
Import Export feature #166
Conversation
Also make various changes to PorterManager
Also add DeckImportException
Refactor methods to take in String instead of Path
Now returns a String so user can tell where the file is exported to
Import and Export commands and functions are working
Resolves #57 |
And Formatting
|
||
Deck deckToExport = lastShownList.get(targetIndex.getZeroBased()); | ||
String exportPath = model.exportDeck(deckToExport); | ||
System.out.println(exportPath); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this for debugging?
System.out.println(exportPath); |
@@ -57,6 +57,10 @@ | |||
|
|||
boolean isInsideDeck(); | |||
|
|||
String exportDeck(Deck deck); | |||
|
|||
Deck importDeck (String filepath); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deck importDeck (String filepath); | |
Deck importDeck(String filepath); |
As you add some more methods to existing classes, it would be nice if you can change the test classes to cover those new methods |
Create a PortManager class to handle reading and writing of data to/from files. Uses XmlUtil
Anakin now contains a PortManager
The default location for export/import is the root folder normally called "main". (Same folder as README.adoc)
Includes PortManager, Commands, Parser, Unit Tests.
Todo: Integrated testing