Pluto
  1. Pluto
  2. PLUTO-356

Pluto Driver not using injected Portal URL Parser

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.1.2, 1.1.4
    • Fix Version/s: 1.1.5
    • Component/s: portal driver
    • Labels:
      None

      Description

      Hi all,

      We are in the process of upgrading to pluto 1.1.2 and have run into a small problem. We have taken advantage of the new IoC to inject the required and optional services into our container services (ContainerServicesImpl). It appears that the PortalURLImpl and RelativePortalURLImpl classes in the driver are not using the injected override service for URL creation (PortalURLParser). The code below is showing the singleton access which always comes from org.apache.pluto.driver.url.impl rather than the container services.

      // The toString() method for both classes listed below

      org.apache.pluto.driver.url.impl.PortalURLImpl
      and
      org.apache.pluto.driver.url.impl.RelativePortalURLImpl

      public String toString() {
      return PortalURLParserImpl.getParser().toString(this);
      }

      Am I just confused as to how this should work, or was this singleton just missed?

      Best,
      Nate

      Nathaniel A. Johnson Voice: 812.855.9905
      Principal Systems Analyst Fax: 812.856.9046
      Systems Integration Team, Indiana University

      1. PLUTO-356.patch
        4 kB
        Elliot Metsger

        Issue Links

          Activity

          Mark Thomas made changes -
          Workflow Default workflow, editable Closed status [ 12565282 ] jira [ 12586007 ]
          Mark Thomas made changes -
          Workflow jira [ 12401786 ] Default workflow, editable Closed status [ 12565282 ]
          Elliot Metsger made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Elliot Metsger made changes -
          Fix Version/s 2.0.0 [ 12312914 ]
          Elliot Metsger made changes -
          Link This issue is cloned as PLUTO-469 [ PLUTO-469 ]
          Elliot Metsger made changes -
          Fix Version/s 1.1.4 [ 12312421 ]
          Elliot Metsger made changes -
          Fix Version/s 1.1.5 [ 12312652 ]
          Affects Version/s 1.1.4 [ 12312421 ]
          Fix Version/s 1.2.0 [ 12312213 ]
          Fix Version/s 2.0.0 [ 12312914 ]
          Elliot Metsger made changes -
          Status Closed [ 6 ] Reopened [ 4 ]
          Resolution Fixed [ 1 ]
          Elliot Metsger made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Elliot Metsger made changes -
          Fix Version/s 1.2.0 [ 12312213 ]
          Elliot Metsger made changes -
          Fix Version/s 1.1.4 [ 12312421 ]
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Elliot Metsger made changes -
          Attachment PLUTO-356.patch [ 12355990 ]
          Elliot Metsger made changes -
          Field Original Value New Value
          Assignee Elliot Metsger [ emetsger ]
          Nate Johnson created issue -

            People

            • Assignee:
              Elliot Metsger
              Reporter:
              Nate Johnson
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development