From f65eae145ca6720fab774600ddda2b04f364dda0 Mon Sep 17 00:00:00 2001 From: Endi Sukma Dewata Date: Thu, 5 Jul 2012 10:22:37 -0400 Subject: 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 --- base/common/src/com/netscape/cms/servlet/request/CertReqParser.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'base/common/src/com/netscape/cms/servlet/request') diff --git a/base/common/src/com/netscape/cms/servlet/request/CertReqParser.java b/base/common/src/com/netscape/cms/servlet/request/CertReqParser.java index 11716115f..fd4589437 100644 --- a/base/common/src/com/netscape/cms/servlet/request/CertReqParser.java +++ b/base/common/src/com/netscape/cms/servlet/request/CertReqParser.java @@ -225,7 +225,7 @@ public class CertReqParser extends ReqParser { IRequest.SERVER_ATTRS + LB + String.valueOf(saCounter++) + RB; if (name.equalsIgnoreCase(IRequest.ISSUED_CERTS) && mDetails && - (req.getRequestStatus().toString().equals(RequestStatus.COMPLETE_STRING) || + (req.getRequestStatus() == RequestStatus.COMPLETE || req.getRequestType().equals(IRequest.GETREVOCATIONINFO_REQUEST))) { X509CertImpl issuedCert[] = req.getExtDataInCertArray(IRequest.ISSUED_CERTS); @@ -468,7 +468,7 @@ public class CertReqParser extends ReqParser { } if (name.equalsIgnoreCase(IRequest.ISSUED_CERTS) && mDetails && - (req.getRequestStatus().toString().equals(RequestStatus.COMPLETE_STRING) || + (req.getRequestStatus() == RequestStatus.COMPLETE || req.getRequestType().equals(IRequest.GETREVOCATIONINFO_REQUEST))) { X509CertImpl issuedCert[] = req.getExtDataInCertArray(IRequest.ISSUED_CERTS); -- cgit