Chemistry
  1. Chemistry
  2. CMIS-120

SessionParameter Set not easy to use

    Details

      Description

      The parameters defined in class SessionParameter should get some improvements:

      minor:

      • interface instead of class

      major:

      • Atom Pub and Web Service binding are implicit determind by various set of URL parameters. If a Atom Pub and a Web Service URL is specified then the binding is ambiguous in runtime. Error handling is difficult becaus of various parameter configurations are illegal.
        Proposal: Introduce a explicite parameter for the binding type to use. Example: *.binding = {AtomPub|SOAP|...}

        . In dependency of a binding different sets of URL parameter need to be set: Atom Pub binding => Atom Pub URL.

      • WebService URL can also be used in various combinations (prefix, non prefix, single service URLs ...). This should be made explicite while one parameter has only one purpose (URL, Prefix URL, Single Service URLs).

        Activity

        Hide
        Florian Müller added a comment -

        On the provider level there is exactly one parameter that determines the binding: BINDING_SPI_CLASS.
        Also, all WEBSERVICES_*_SERVICE parameters must be set for the Web Services binding.

        We certainly can (and should) simplify that on the Client API layer.

        Show
        Florian Müller added a comment - On the provider level there is exactly one parameter that determines the binding: BINDING_SPI_CLASS. Also, all WEBSERVICES_*_SERVICE parameters must be set for the Web Services binding. We certainly can (and should) simplify that on the Client API layer.
        Hide
        Stephan Klevenz added a comment -

        I've seen that and like it. But we have to duplicate the binding class const (CmisProviderFactory.BINDING_SPI*) in the client API. Right?

        Show
        Stephan Klevenz added a comment - I've seen that and like it. But we have to duplicate the binding class const (CmisProviderFactory.BINDING_SPI*) in the client API. Right?
        Hide
        Stephan Klevenz added a comment -

        BindingType at API level introduced.

        Show
        Stephan Klevenz added a comment - BindingType at API level introduced.

          People

          • Assignee:
            Unassigned
            Reporter:
            Stephan Klevenz
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development