Uploaded image for project: 'Directory Client API'
  1. Directory Client API
  2. DIRAPI-262

misleading message "The matched Dn should not be set when the result code is one of..."?

        Details

        • Type: Bug
        • Status: Resolved
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: 1.0.0-M32
        • Fix Version/s: 1.0.0-M33
        • Labels:
          None

          Description

          Hi guys,

          isnt https://github.com/apache/directory-shared/blob/trunk/ldap/codec/core/src/main/java/org/apache/directory/api/ldap/codec/actions/ldapResult/StoreMatchedDN.java#L121 the opposite of what it means? Seems to me the previous cases exactly do what this message says it shouldnt do. Can be just a phrasing issue but maybe this message should be more expressive, wdyt?

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            elecharny Emmanuel Lecharny added a comment - - edited

            You are right ! The sentence should be : "The matched Dn should not be set when the result code is not one of..."

            I have fixed it with http://svn.apache.org/viewvc?rev=1713590&view=rev

            (FTR, I will move this JIRA to the DIRAPI project).

            Thanks !

            Show
            elecharny Emmanuel Lecharny added a comment - - edited You are right ! The sentence should be : "The matched Dn should not be set when the result code is not one of..." I have fixed it with http://svn.apache.org/viewvc?rev=1713590&view=rev (FTR, I will move this JIRA to the DIRAPI project). Thanks !
            Hide
            Permalink
            elecharny Emmanuel Lecharny added a comment -

            Fixed with http://svn.apache.org/viewvc?rev=1713590&view=rev

            Show
            elecharny Emmanuel Lecharny added a comment - Fixed with http://svn.apache.org/viewvc?rev=1713590&view=rev

              People

              • Assignee:
                Unassigned
                Reporter:
                romain.manni-bucau Romain Manni-Bucau
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development