Return-Path: Delivered-To: apmail-directory-dev-archive@www.apache.org Received: (qmail 85342 invoked from network); 10 May 2007 21:48:37 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 10 May 2007 21:48:37 -0000 Received: (qmail 23426 invoked by uid 500); 10 May 2007 21:48:42 -0000 Delivered-To: apmail-directory-dev-archive@directory.apache.org Received: (qmail 23392 invoked by uid 500); 10 May 2007 21:48:42 -0000 Mailing-List: contact dev-help@directory.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Apache Directory Developers List" Delivered-To: mailing list dev@directory.apache.org Received: (qmail 23381 invoked by uid 99); 10 May 2007 21:48:42 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 10 May 2007 14:48:42 -0700 X-ASF-Spam-Status: No, hits=-100.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO brutus.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 10 May 2007 14:48:35 -0700 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id AC4B871403F for ; Thu, 10 May 2007 14:48:15 -0700 (PDT) Message-ID: <30889307.1178833695703.JavaMail.jira@brutus> Date: Thu, 10 May 2007 14:48:15 -0700 (PDT) From: "Stefan Seelmann (JIRA)" To: dev@directory.apache.org Subject: [jira] Commented: (DIRSTUDIO-100) Interface IRootDSE does not sufficiently define return values of methods In-Reply-To: <28562767.1178808195395.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/DIRSTUDIO-100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12494852 ] Stefan Seelmann commented on DIRSTUDIO-100: ------------------------------------------- Stefan, you are totally right. The interface lacks on documentation and the RootDSE class implementation is bad. Thanks for heads up. As you suggest the best way would be to return the OIDs only. BTW, the methods are only used from the RootDSEPropertyPage. > Interface IRootDSE does not sufficiently define return values of methods > ------------------------------------------------------------------------ > > Key: DIRSTUDIO-100 > URL: https://issues.apache.org/jira/browse/DIRSTUDIO-100 > Project: Directory LDAP Studio > Issue Type: Wish > Reporter: Stefan Zoerner > Fix For: 0.8.0 > > > I think the interface org.apache.directory.ldapstudio.browser.core.model.IRootDSE does not sufficiently define the return values for the following methods > * getSupportedExtensions > * getSupportedControls > * getSupportedFeatures > Do they return the OIDs, or a name, or a description (what language)? > The implementation > org.apache.directory.ldapstudio.browser.core.internal.model.RootDSE > for instance replaces OIDs with names from a map file, if they are contained in it. It is therefore not easy to determine whether a given LDAP control etc. is supported or not. I recommend to define that the return values of type String[] always return the OIDs, and a translation into text (if necessary) is due to UI classes only. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.