[ 
https://issues.apache.org/jira/browse/DIRAPI-147?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13689549#comment-13689549
 ] 

Emmanuel Lecharny commented on DIRAPI-147:
------------------------------------------

Good catch... I will apply the proposed change before releasing (I was just in 
the middle of a release, this is a good thing you filled this JIRA before I 
started the vote !)
                
> LdifEntry equals method does not compare entry attributes when changetype is 
> None
> ---------------------------------------------------------------------------------
>
>                 Key: DIRAPI-147
>                 URL: https://issues.apache.org/jira/browse/DIRAPI-147
>             Project: Directory Client API
>          Issue Type: Bug
>    Affects Versions: 1.0.0-M17
>         Environment: Windows, Linux
>            Reporter: Michael Moorman
>              Labels: Equals(), LdifEntry
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> The equals() method of LdifEntry does not consider the inner entry's 
> attributes when ChangeType is not specified in the LDIF entry. 
> According to the the LDIF spec, when no changetype is specified in an LDIF 
> entry, a changetype "ADD" is to be assumed. 
> An easy fix would be, in the equals() method of LdifEntry.java, add case NONE 
> to the switch at:
>      // Check each different cases
>         switch ( changeType )
>         {
>             case None:          //<-- Add None case fall through
>             case Add:
>                  ......

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to