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.

Sign up for GitHub

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

Jump to bottom

Add DeletedItem class, RESTORE command #9

Merged
merged 39 commits into from
May 12, 2022
Merged

Add DeletedItem class, RESTORE command #9

merged 39 commits into from
May 12, 2022

Conversation

withaspirit
Copy link
Owner

@withaspirit withaspirit commented May 12, 2022 •
edited
Loading

This satisfies an additional requirement for the Challenge.

Changes

  • (new) DeletedItem - item with comment
  • Command
    • (new) added RESTORE command
    • included comment in regex for DELETE
  • DDL.sql
    • added deleted_items table
  • Inputs.json
    • added cases for Restore item, Delete item with comment
  • DatabaseCLI
    • added restore() method (handles null case)
  • DatabasePresenter
    • added restoreItem() method
    • updated DatabasePresenterTest
  • Database
    • add methods to select a single item by id
    • updated DatabaseTest
  • pom.xml
    • added Apache Common Lang 3 dependency

@withaspirit
DDL.sql: add deleted_items table
8a025ab
@withaspirit
Add DeletedItem class
e7b8f20
@withaspirit
Add DeletedItem class
13851d0
@withaspirit
DeletedItem: methods, javadocs
d0be436
@withaspirit
DeletedItem: shorten method
97348c2
@withaspirit
Added Command: Restore
bf2c5ea
@withaspirit
Database: add methods to select one item
2e3d106
@withaspirit
DDL.sql: fix bug
a8a9722
@withaspirit
DatabaseTest: test insert into deleted_items table
9fc4174
@withaspirit
DatabasePresenter: added restore() method
93aee8d
@withaspirit
DatabasePresenter: deleteItem() inserts into deleted_items after dele…
adfd7a6 
…ting
@withaspirit
DatabasePresenter: database.selectFromTable for single item handles D…
16b5cfa 
…eletedItem correctly
@withaspirit
DatabasePresenterTest: test that restore works successfully
f83c83c
@withaspirit
DatabaseCLI: add restore item command
12e523f
@withaspirit
DatabaseCLITest: simplify file
80eb8a4
@withaspirit
DatabaseCLITest: test restore item
6954f02
@withaspirit
pom.xml: added Apache Commons Lang 3 dependency
89620d7
@withaspirit
DeletedItem: fixed deletedItem printing null value
463eea7
@withaspirit
Database: make compatible with DeletedItems
73e25b5
@withaspirit
Simplify DatabaseTest
75019c7
@withaspirit
Command: added optional comment for DELETE comment, Added deletedItem…
b78e3b3 
…s with comments to inputs.json
@withaspirit
DatabasePresenter: made compatible with DeletedItem
76cae78
@withaspirit
DatabaseCLI: make compatible with DatabasePresenter
b09c223
@withaspirit
DeletedItem: added methods to increase compatibility
e8d91af
@withaspirit
Simplify databaseTest
4155033
@withaspirit
Item: make getAttributeValues method more robust
545108a
@withaspirit
Rename Item.getAttributesValues -> getValuesInSQLFormat
d49cb55
@withaspirit
Remove unnecessary output statements
c242745
@withaspirit
Allow READ from deleted_items
77fb1e0
@withaspirit
Database: Remove testing output statements
f23a7bb
@withaspirit withaspirit changed the title Add DeletedItem class Add DeletedItem class, RESTORE command May 12, 2022
@withaspirit withaspirit merged commit 19b6e44 into master May 12, 2022
@withaspirit withaspirit deleted the DeletedItems branch May 12, 2022 21:34
@withaspirit withaspirit mentioned this pull request May 18, 2022
Closed
27 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@withaspirit