Directory Studio
  1. Directory Studio
  2. DIRSTUDIO-136

Remove the "Apache DS" menu from the Java Perspective

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 1.1.0
    • Labels:
      None

      Description

      Some ideas from Stefan Seelman are:
      (1) Keep it only in the LDAP or Schemas perspective
      (2) Replace the functionality with wizards

      Seemed very wizard like to me. Plus 1 for the wizard idea.

        Activity

        Hide
        Pierre-Arnaud Marcelot added a comment -

        (1) -> Why not. Since we'll be delivering more and more tools related to ADS, maybe we can create (later) an Apache Directory Server perspective from which you can configure everything : Configuration, Dynamic Schema, Triggers/Stored Procedures (and more to come).

        (2) -> A wizard seems cool for a new configuration but what about an existing configuration ? I think we need to keep the configuration editor for exiting configuration and maybe add a wizard for new configuration.

        WDYT ?

        Show
        Pierre-Arnaud Marcelot added a comment - (1) -> Why not. Since we'll be delivering more and more tools related to ADS, maybe we can create (later) an Apache Directory Server perspective from which you can configure everything : Configuration, Dynamic Schema, Triggers/Stored Procedures (and more to come). (2) -> A wizard seems cool for a new configuration but what about an existing configuration ? I think we need to keep the configuration editor for exiting configuration and maybe add a wizard for new configuration. WDYT ?
        Hide
        Stefan Seelmann added a comment -

        You are right, an ADS perspective would be very useful.

        About opening an existing configuration: It would be cool to do this via File->Open. The default detection of a editor in Eclipse is done via the file extension or filename. Unfortunately the filename server.xml is ambiguous, e.g. Tomcat also has a server.xml.

        What we can do is to define a content type with a content describer (interface IContentDescriber) and bind this content type to the configuration editor. The content describer must implement the method
        public int describe(InputStream contents, IContentDescription description) throws IOException;
        What we need is a criteria to determine it the file is an ADS configuration file. I think it is enough to check if the given input stream contains an ADS specific string like "org.apache.directory.server.configuration".

        Some more info about the content type and content describer:
        http://help.eclipse.org/help32/index.jsp?topic=/org.eclipse.platform.doc.isv/guide/runtime_content.htm
        http://www.developer.com/java/data/article.php/3648736 (example using the predifined XMLRootElementContentDescriber)
        http://www.eclipse.org/eclipse/platform-core/documents/content_types.html (seems to be outdated)

        Show
        Stefan Seelmann added a comment - You are right, an ADS perspective would be very useful. About opening an existing configuration: It would be cool to do this via File->Open. The default detection of a editor in Eclipse is done via the file extension or filename. Unfortunately the filename server.xml is ambiguous, e.g. Tomcat also has a server.xml. What we can do is to define a content type with a content describer (interface IContentDescriber) and bind this content type to the configuration editor. The content describer must implement the method public int describe(InputStream contents, IContentDescription description) throws IOException; What we need is a criteria to determine it the file is an ADS configuration file. I think it is enough to check if the given input stream contains an ADS specific string like "org.apache.directory.server.configuration". Some more info about the content type and content describer: http://help.eclipse.org/help32/index.jsp?topic=/org.eclipse.platform.doc.isv/guide/runtime_content.htm http://www.developer.com/java/data/article.php/3648736 (example using the predifined XMLRootElementContentDescriber) http://www.eclipse.org/eclipse/platform-core/documents/content_types.html (seems to be outdated)
        Hide
        Pierre-Arnaud Marcelot added a comment -

        Thanks for those cool links Stefan.
        They'll be a great help.

        +1 for this solution.

        I hope we'll be able to start putting some of the properties of the server.xml file into the DIT very soon, to get rid of this file...

        Show
        Pierre-Arnaud Marcelot added a comment - Thanks for those cool links Stefan. They'll be a great help. +1 for this solution. I hope we'll be able to start putting some of the properties of the server.xml file into the DIT very soon, to get rid of this file...
        Hide
        Pierre-Arnaud Marcelot added a comment -

        Updated Title.

        Show
        Pierre-Arnaud Marcelot added a comment - Updated Title.
        Hide
        Pierre-Arnaud Marcelot added a comment -

        Fixed.

        There's a new item in the "New wizards" menu for creating a new Server Configuration file.
        Opening a Server Configuration file can be achieved from a file inside a project (when the plugin is used inside Eclipse) and from an external file (when the plugin is used inside Eclipse or as an RCP Application).

        Show
        Pierre-Arnaud Marcelot added a comment - Fixed. There's a new item in the "New wizards" menu for creating a new Server Configuration file. Opening a Server Configuration file can be achieved from a file inside a project (when the plugin is used inside Eclipse) and from an external file (when the plugin is used inside Eclipse or as an RCP Application).
        Hide
        Pierre-Arnaud Marcelot added a comment -

        Version 1.1.0 has been released.
        Closing this issue.

        Show
        Pierre-Arnaud Marcelot added a comment - Version 1.1.0 has been released. Closing this issue.

          People

          • Assignee:
            Pierre-Arnaud Marcelot
            Reporter:
            Ole Ersoy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development