-
Notifications
You must be signed in to change notification settings - Fork 535
GDCC/8605-add-archival-status-support #8696
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
Changes from all commits
de62791
8c82c61
b354bc3
9c9ac65
221ca0b
8902d9a
a37922b
cefa12c
e1c62af
d2bf93c
ae1c97c
d3a7b04
5295bcd
9223e7d
f5396d8
986f9ff
8750e62
5d617f0
8fcb59c
d2d817e
6a70d42
e498417
8a99685
7362e1c
7410c5b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -1187,4 +1187,12 @@ private DatasetVersion getPreviousVersionWithUnf(DatasetVersion datasetVersion) | |
| return null; | ||
| } | ||
|
|
||
| /** | ||
| * Merges the passed datasetversion to the persistence context. | ||
| * @param ver the DatasetVersion whose new state we want to persist. | ||
| * @return The managed entity representing {@code ver}. | ||
| */ | ||
| public DatasetVersion merge( DatasetVersion ver ) { | ||
| return em.merge(ver); | ||
| } | ||
|
Comment on lines
+1195
to
+1197
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm surprised this merge method doesn't already exist on DatasetVersionServiceBean.java. It is because most changes to versions happen through commands? It is because once a version is published there's no need to go back and change the version (except for deaccessioning, I guess, which is a command)? I don't think it's bad to add this method but I wonder why we're only adding it now.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah - I think everything uses a Command of some sort. I was also surprised that it didn't exist as the dataset service has a merge() and the file service has several methods that don't do much more than a merge. |
||
| } // end class | ||
Uh oh!
There was an error while loading. Please reload this page.