diff options
| author | Endi Sukma Dewata <edewata@redhat.com> | 2012-07-05 10:22:37 -0400 |
|---|---|---|
| committer | Endi Sukma Dewata <edewata@redhat.com> | 2012-07-10 14:23:18 -0500 |
| commit | f65eae145ca6720fab774600ddda2b04f364dda0 (patch) | |
| tree | 05b90f929c4a1b8193a64a3c79cbda09e1b85492 /base/common/src/com/netscape/cms/servlet/cert/scep | |
| parent | a3c83b5f63c9ced5c527cb3b9306d390c3d192ec (diff) | |
| download | pki-f65eae145ca6720fab774600ddda2b04f364dda0.tar.gz pki-f65eae145ca6720fab774600ddda2b04f364dda0.tar.xz pki-f65eae145ca6720fab774600ddda2b04f364dda0.zip | |
Refactored RequestStatus.
The RequestStatus has been modified to use a map to convert string
into RequestStatus instance. The string constants in RequestStatus
are no longer needed because instances can be compared using
equal sign directly or equals().
Ticket #161
Diffstat (limited to 'base/common/src/com/netscape/cms/servlet/cert/scep')
| -rw-r--r-- | base/common/src/com/netscape/cms/servlet/cert/scep/CRSEnrollment.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/base/common/src/com/netscape/cms/servlet/cert/scep/CRSEnrollment.java b/base/common/src/com/netscape/cms/servlet/cert/scep/CRSEnrollment.java index dc0b0c474..c3d36561d 100644 --- a/base/common/src/com/netscape/cms/servlet/cert/scep/CRSEnrollment.java +++ b/base/common/src/com/netscape/cms/servlet/cert/scep/CRSEnrollment.java @@ -1770,8 +1770,8 @@ public class CRSEnrollment extends HttpServlet { crsResp.setPKIStatus(CRSPKIMessage.mStatus_FAILURE); crsResp.setFailInfo(CRSPKIMessage.mFailInfo_badAlg); } - } else if (status.equals(RequestStatus.REJECTED_STRING) || - status.equals(RequestStatus.CANCELED_STRING)) { + } else if (status == RequestStatus.REJECTED || + status == RequestStatus.CANCELED) { crsResp.setPKIStatus(CRSPKIMessage.mStatus_FAILURE); crsResp.setFailInfo(CRSPKIMessage.mFailInfo_badRequest); } else { // not complete |