diff options
author | Abhishek Koneru <akoneru@redhat.com> | 2012-05-23 13:46:25 -0400 |
---|---|---|
committer | Ade Lee <alee@redhat.com> | 2012-05-24 23:09:24 -0400 |
commit | 2408bec41a56378fcf942a68a1ab290464c001d7 (patch) | |
tree | db5b4ba123c2023c854882188851a42fd35348bf /base/common/src/com/netscape/cmscore/ldap | |
parent | be23c86db27b7f66ca16a10ae3ef3f043d1d4b27 (diff) | |
download | pki-2408bec41a56378fcf942a68a1ab290464c001d7.tar.gz pki-2408bec41a56378fcf942a68a1ab290464c001d7.tar.xz pki-2408bec41a56378fcf942a68a1ab290464c001d7.zip |
Patch for fixes for Review Comments
Diffstat (limited to 'base/common/src/com/netscape/cmscore/ldap')
-rw-r--r-- | base/common/src/com/netscape/cmscore/ldap/LdapRule.java | 15 | ||||
-rw-r--r-- | base/common/src/com/netscape/cmscore/ldap/PublisherProcessor.java | 6 |
2 files changed, 10 insertions, 11 deletions
diff --git a/base/common/src/com/netscape/cmscore/ldap/LdapRule.java b/base/common/src/com/netscape/cmscore/ldap/LdapRule.java index eaf5e763a..2fb148492 100644 --- a/base/common/src/com/netscape/cmscore/ldap/LdapRule.java +++ b/base/common/src/com/netscape/cmscore/ldap/LdapRule.java @@ -66,20 +66,20 @@ public class LdapRule implements ILdapRule, IExtendedPluginInfo { mProcessor = processor; Enumeration<String> mappers = mProcessor.getMapperInsts().keys(); Enumeration<String> publishers = mProcessor.getPublisherInsts().keys(); - StringBuffer map=new StringBuffer(); + StringBuffer map = new StringBuffer(); map.append(NOMAPPER); for (; mappers.hasMoreElements();) { String name = mappers.nextElement(); - map.append(",").append(name); + map.append("," + name); } StringBuffer publish = new StringBuffer(); for (; publishers.hasMoreElements();) { String name = publishers.nextElement(); - publish.append(",").append(name); + publish.append("," + name); } epi_params = new String[] { @@ -176,21 +176,20 @@ public class LdapRule implements ILdapRule, IExtendedPluginInfo { * * Enumeration<String> mappers = mProcessor.getMapperInsts().keys(); Enumeration<String> publishers = mProcessor.getPublisherInsts().keys(); - StringBuffer map=new StringBuffer(); + StringBuffer map = new StringBuffer(); map.append(NOMAPPER); for (; mappers.hasMoreElements();) { String name = mappers.nextElement(); - map.append(",").append(name); + map.append("," + name); } - StringBuffer publish=new StringBuffer(); - + StringBuffer publish = new StringBuffer(); for (; publishers.hasMoreElements();) { String name = publishers.nextElement(); - publish.append(",").append(name); + publish.append("," + name); }*/ /* diff --git a/base/common/src/com/netscape/cmscore/ldap/PublisherProcessor.java b/base/common/src/com/netscape/cmscore/ldap/PublisherProcessor.java index e5aef45a5..62e4b8bef 100644 --- a/base/common/src/com/netscape/cmscore/ldap/PublisherProcessor.java +++ b/base/common/src/com/netscape/cmscore/ldap/PublisherProcessor.java @@ -955,7 +955,7 @@ public class PublisherProcessor implements // continue publishing even publisher has errors //log(ILogger.LL_WARN, e.toString()); error = true; - errorRule.append(" ").append(rule.getInstanceName()); + errorRule.append(" " + rule.getInstanceName()); } } @@ -1072,7 +1072,7 @@ public class PublisherProcessor implements // continue publishing even publisher has errors //log(ILogger.LL_WARN, e.toString()); error = true; - errorRule.append(" ").append(rule.getInstanceName()); + errorRule.append(" " + rule.getInstanceName()); } } // set the ldap published flag. @@ -1138,7 +1138,7 @@ public class PublisherProcessor implements // continue publishing even publisher has errors //log(ILogger.LL_WARN, e.toString()); error = true; - errorRule.append(" ").append(rule.getInstanceName()); + errorRule.append(" " + rule.getInstanceName()); } } |