-
Notifications
You must be signed in to change notification settings - Fork 14
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
Option to add multiple schemas #191
Conversation
final String[] representation, final boolean targetOnly, | ||
final String sipID, final String[] ancestors, final String[] documentation, final String softwareVersion, | ||
final String path, final String submitterAgentName, final String submitterAgentID, final String checksum) | ||
final String[] representation, String[] metadataSchema, final boolean targetOnly, |
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.
Parameter metadataSchema should be final.
final String[] representation, final boolean targetOnly, | ||
final String sipID, final String[] ancestors, final String[] documentation, final String softwareVersion, | ||
final String path, final String submitterAgentName, final String submitterAgentID, final String checksum) | ||
final String[] representation, String[] metadataSchema, final boolean targetOnly, |
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.
Variable 'metadataSchema' should be declared final.
final String sipID, final String[] ancestors, final String[] documentation, final String softwareVersion, | ||
final String path, final String submitterAgentName, final String submitterAgentID, final String checksum) | ||
final String[] representation, String[] metadataSchema, final boolean targetOnly, | ||
final String sipID, final String[] ancestors, final String[] documentation, final String softwareVersion, |
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.
Line is longer than 120 characters (found 141).
final String path, final String submitterAgentName, final String submitterAgentID, final String checksum) | ||
final String[] representation, String[] metadataSchema, final boolean targetOnly, | ||
final String sipID, final String[] ancestors, final String[] documentation, final String softwareVersion, | ||
final String path, final String submitterAgentName, final String submitterAgentID, final String checksum) |
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.
Line is longer than 120 characters (found 141).
@@ -214,6 +209,11 @@ public static Path createEARK2SIP(final String[] metadata, | |||
CLIUtils.printErrors(System.out, "Cannot add metadata to the SIP."); | |||
} | |||
} | |||
if (metadataSchema != null){ |
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 not preceded with whitespace.
@@ -214,6 +209,11 @@ public static Path createEARK2SIP(final String[] metadata, | |||
CLIUtils.printErrors(System.out, "Cannot add metadata to the SIP."); | |||
} | |||
} | |||
if (metadataSchema != null){ | |||
for(String schema : metadataSchema){ |
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.
'for' is not followed by whitespace.
@@ -214,6 +209,11 @@ public static Path createEARK2SIP(final String[] metadata, | |||
CLIUtils.printErrors(System.out, "Cannot add metadata to the SIP."); | |||
} | |||
} | |||
if (metadataSchema != null){ | |||
for(String schema : metadataSchema){ |
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 not preceded with whitespace.
No description provided.