hazza/DSpace
1
0
Fork 0
mirror of https://github.com/DSpace/DSpace.git synced 2025-10-16 14:33:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

Don't call System.exit() in a launcher step; throw something instead.

Browse Source 
At least don't use exit() for a normal completion.  That prevents
streaming multiple commands.
This commit is contained in:
Mark H. Wood
2013-02-06 12:45:33 -05:00
parent 3acfcaba2b
commit dfafcf657f
1 changed files with 4 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
dspace-api/src/main/java/org/dspace/eperson/EPerson.java
View File

@@ -1228,24 +1228,12 @@ public class EPerson extends DSpaceObject
* Tool for manipulating user accounts.
*/
public static void main(String argv[])
throws ParseException, SQLException
{
final PosixParser parser = new PosixParser();
CommandLine command = null;
CommandLine command = parser.parse(globalOptions, argv, true);
try {
command = parser.parse(globalOptions, argv, true);
} catch (ParseException e) {
System.err.println(e.getMessage());
System.exit(1);
}
Context context = null;
try {
context = new Context();
} catch (SQLException ex) {
System.err.println(ex.getMessage());
System.exit(1);
}
Context context = new Context();
// Disable authorization since this only runs from the local commandline.
context.turnOffAuthorisationSystem();
@@ -1277,6 +1265,7 @@ public class EPerson extends DSpaceObject
new HelpFormatter().printHelp("user [options]", globalOptions);
context.abort();
status = 1;
throw new IllegalArgumentException();
}
if (context.isValid())
@@ -1287,8 +1276,6 @@ public class EPerson extends DSpaceObject
System.err.println(ex.getMessage());
}
}
System.exit(status);
}
/** Command to create an EPerson. */