Project

General

Profile

« Previous | Next » 

Revision 7051

If PID is not part of the multipart params, we end up with a NullPointerException. Throw an InvalidRequest in this case rather than ServiceFailure resulting from the NPE.

View differences:

src/edu/ucsb/nceas/metacat/restservice/MNResourceHandler.java
1031 1031
        if (trailingPid == null) {
1032 1032
	        // get the pid string from the body and set the value
1033 1033
	        String pidString = multipartparams.get("pid").get(0);
1034
	        pid.setValue(pidString);
1034
	        if (pidString != null) {
1035
            pid.setValue(pidString);
1036
            
1037
          } else {
1038
              throw new InvalidRequest("1102", "The pid param must be included and contain the identifier.");
1039
              
1040
          }
1035 1041
        } else {
1036 1042
        	// use the pid included in the URL
1037 1043
        	pid.setValue(trailingPid);

Also available in: Unified diff