Skip to content

Commit a8e9f51

Browse files
committed
review copilot
1 parent 9ff544b commit a8e9f51

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

dspace-api/src/main/java/org/dspace/app/reportdiff/ReportDiff.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,13 +114,14 @@ public void setup() throws ParseException {
114114
if (commandLine.hasOption('c')) {
115115
specificCheck = parseCheckOption(commandLine.getOptionValue('c'));
116116
if (specificCheck == -1) {
117+
// Error already logged in parseCheckOption
117118
return;
118119
}
119120
}
120121

121122
// `-d`: Dates, show all dates that the report was generated for a specific check type.
122123
if (commandLine.hasOption('d')) {
123-
showDates = commandLine.hasOption('d');
124+
showDates = true;
124125
try {
125126
limit = Long.parseLong(commandLine.getOptionValue("l"));
126127
} catch (NumberFormatException e) {

dspace-api/src/main/java/org/dspace/app/reportdiff/ReportDiffScriptConfiguration.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,8 @@ public Options getOptions() {
4343
"otherwise perform default checks).", HealthReport.getNumberOfChecks() - 1));
4444
options.getOption("c").setType(String.class);
4545

46-
options.addOption("d", "dates", false,options.addOption("l", "limit", true, "Limit the number of report entries to this value. " +
46+
options.addOption("d", "dates", false,
47+
"Limit the number of report entries to this value. " +
4748
"If not specified, all entries are shown.");
4849
options.getOption("l").setType(String.class);
4950

0 commit comments

Comments
 (0)