Uploaded image for project: 'Pluto'
  1. Pluto
  2. PLUTO-356

Pluto Driver not using injected Portal URL Parser

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Critical
    • Resolution: Fixed
    • 1.1.2, 1.1.4
    • 1.1.5
    • portal driver
    • 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

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            emetsger Elliot Metsger
            natjohns Nate Johnson
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment