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

          Nate Johnson created issue -
          Elliot Metsger made changes -
          Field Original Value New Value
          Assignee Elliot Metsger [ emetsger ]
          Hide
          Elliot Metsger added a comment -

          Nate, can you try applying this patch and see if it works for you? Let me know if you have trouble applying it.

          Show
          Elliot Metsger added a comment - Nate, can you try applying this patch and see if it works for you? Let me know if you have trouble applying it.
          Elliot Metsger made changes -
          Attachment PLUTO-356.patch [ 12355990 ]
          Hide
          Nate Johnson added a comment -

          I will try to test this as soon as I can. We are currently using the 1.1.2 binaries. Will this patch work or would I need to upgrade to head?

          Show
          Nate Johnson added a comment - I will try to test this as soon as I can. We are currently using the 1.1.2 binaries. Will this patch work or would I need to upgrade to head?
          Hide
          Elliot Metsger added a comment -

          PortalURLParser is now provided as an argument to the RelativePortalURLImpl constructor. Fix committed to the pluto-1.1.x branch. Nate if you could confirm that this fixes things for you, it would be appreciated. Thanks!

          Show
          Elliot Metsger added a comment - PortalURLParser is now provided as an argument to the RelativePortalURLImpl constructor. Fix committed to the pluto-1.1.x branch. Nate if you could confirm that this fixes things for you, it would be appreciated. Thanks!
          Elliot Metsger made changes -
          Fix Version/s 1.1.4 [ 12312421 ]
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Nate Johnson added a comment -

          I haven't been able to put this into our development code yet, but after talking to Dave at JA-SIG, and finding the correct branch in your version control, I believe this is going to work out just great. Thanks for getting this fix in!

          Show
          Nate Johnson added a comment - I haven't been able to put this into our development code yet, but after talking to Dave at JA-SIG, and finding the correct branch in your version control, I believe this is going to work out just great. Thanks for getting this fix in!
          Hide
          Elliot Metsger added a comment -

          Applied to trunk r554485.

          Show
          Elliot Metsger added a comment - Applied to trunk r554485.
          Elliot Metsger made changes -
          Fix Version/s 1.2.0 [ 12312213 ]
          Elliot Metsger made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Hide
          Elliot Metsger added a comment -

          PLUTO-361 reverted part of this patch. Need to add a unit test for PLUTO-356.

          Show
          Elliot Metsger added a comment - PLUTO-361 reverted part of this patch. Need to add a unit test for PLUTO-356 .
          Elliot Metsger made changes -
          Status Closed [ 6 ] Reopened [ 4 ]
          Resolution Fixed [ 1 ]
          Hide
          Elliot Metsger added a comment -

          Updating fix ver to 1.1.5/2.0.0

          Show
          Elliot Metsger added a comment - Updating fix ver to 1.1.5/2.0.0
          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 ]
          Hide
          Elliot Metsger added a comment -

          Fix committed to 1.1.x branch 609238.

          The 286 merge branch is out of date and this patch can't be easily merged/applied to it.

          Show
          Elliot Metsger added a comment - Fix committed to 1.1.x branch 609238. The 286 merge branch is out of date and this patch can't be easily merged/applied to it.
          Elliot Metsger made changes -
          Fix Version/s 1.1.4 [ 12312421 ]
          Elliot Metsger made changes -
          Link This issue is cloned as PLUTO-469 [ PLUTO-469 ]
          Hide
          Elliot Metsger added a comment -

          PLUTO-469 is this issue opened against 2.0.0.

          Show
          Elliot Metsger added a comment - PLUTO-469 is this issue opened against 2.0.0.
          Elliot Metsger made changes -
          Fix Version/s 2.0.0 [ 12312914 ]
          Elliot Metsger made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Mark Thomas made changes -
          Workflow jira [ 12401786 ] Default workflow, editable Closed status [ 12565282 ]
          Mark Thomas made changes -
          Workflow Default workflow, editable Closed status [ 12565282 ] jira [ 12586007 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          34d 8h 55m 1 Elliot Metsger 18/May/07 03:10
          Resolved Resolved Closed Closed
          82d 11h 31m 1 Elliot Metsger 08/Aug/07 14:42
          Closed Closed Reopened Reopened
          150d 6h 2m 1 Elliot Metsger 05/Jan/08 19:44
          Reopened Reopened Resolved Resolved
          56d 5h 27m 1 Elliot Metsger 02/Mar/08 01:11

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development