Skip to content

Commit

Permalink
Merge pull request #1306 from libris/bugfix/lxl-4262-require-changed-…
Browse files Browse the repository at this point in the history
…by-in-batch-import

Require changedBy to be specified
  • Loading branch information
kwahlin authored Sep 12, 2023
2 parents 74489ae + 77e4f45 commit 1cb9737
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion batchimport/src/main/java/whelk/importer/Parameters.java
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ enum DUPLICATION_TYPE
}

// Check required parameters
if (format == null)
if (format == null || changedBy == null)
{
printUsage();
System.exit(-1);
Expand Down Expand Up @@ -166,6 +166,7 @@ private void printUsage()
System.err.println(" Using this option, a file containing such merge rules can be specified.");
System.err.println();
System.err.println("--changedBy A string to use as descriptionCreator (MARC 040) for imported records.");
System.err.println(" This parameter must be specified.");
System.err.println("--changedIn A string to use for the changedIn column, defaults to \"batch import\".");
System.err.println("--forceUpdate If this option is passed, encoding levels are ignored when deciding if");
System.err.println(" An incoming record is allowed to affect existing records. USE WITH CARE");
Expand Down

0 comments on commit 1cb9737

Please sign in to comment.