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
Added some additional comments to version service. #273
Conversation
app/services/version_service.rb
Outdated
@@ -20,10 +20,15 @@ def initialize(work) | |||
# @option opts [Hash] :vers_md_upd_info If present, used to add to the events datastream and set the desc and significance on the versionMetadata datastream | |||
# @raise [Dor::Exception] if the object hasn't been accessioned, or if a version is already opened | |||
def open(opts = {}) | |||
# Object has been accessioned at least once. |
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.
I'm not sure what this means. Does this mean "this method should only be called on an object that has been accessioned at least once?"
app/services/version_service.rb
Outdated
# During local development, we need a way to open a new version even if the object has not been accessioned. | ||
raise(Dor::Exception, 'Object net yet accessioned') unless | ||
opts[:assume_accessioned] || Dor::Config.workflow.client.lifecycle('dor', work.pid, 'accessioned') | ||
# Raised if for the current version, there are are any incomplete wf steps and there is a versionWF. |
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.
Rephrase "Raised when the current version has any incomplete workflow steps and there is a versionWF"
app/services/version_service.rb
Outdated
raise Dor::VersionAlreadyOpenError, 'Object already opened for versioning' if open? | ||
# Raised if for the current version, there are any incomplete wf steps and there is an accessionWF. | ||
# The submittedWF is part of the accessionWF. |
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.
s/submittedWF/submitted milestone
app/services/version_service.rb
Outdated
raise Dor::VersionAlreadyOpenError, 'Object already opened for versioning' if open? | ||
# Raised if for the current version, there are any incomplete wf steps and there is an accessionWF. |
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.
Can this be rephrased to "Raised when the current version has any incomplete workflow steps and there is an accessionWF"
f4759de
to
831e9b5
Compare
@jcoyne Fixed all. |
No description provided.