Fixed a small logic bug in the name identifier mapping. Bugzilla #311.
authorwassa <wassa@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Thu, 28 Apr 2005 21:30:57 +0000 (21:30 +0000)
committerwassa <wassa@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Thu, 28 Apr 2005 21:30:57 +0000 (21:30 +0000)
git-svn-id: https://subversion.switch.ch/svn/shibboleth/java-idp/trunk@1421 ab3bd59b-922f-494d-bb5f-6f0a3c29deca

src/edu/internet2/middleware/shibboleth/common/provider/BaseNameIdentifierMapping.java

index 2cb77dc..ab8501b 100644 (file)
@@ -65,7 +65,7 @@ public abstract class BaseNameIdentifierMapping implements NameIdentifierMapping
                }
 
                String id = ((Element) config).getAttribute("id");
                }
 
                String id = ((Element) config).getAttribute("id");
-               if (id != null || !id.equals("")) {
+               if (id != null && !id.equals("")) {
                        this.id = id;
                }
 
                        this.id = id;
                }
 
@@ -83,7 +83,7 @@ public abstract class BaseNameIdentifierMapping implements NameIdentifierMapping
 
        public void destroy() {
 
 
        public void destroy() {
 
-       //nothing to do
+       // nothing to do
        }
 
        protected void verifyQualifier(SAMLNameIdentifier nameId, IdentityProvider idProv)
        }
 
        protected void verifyQualifier(SAMLNameIdentifier nameId, IdentityProvider idProv)