PicoContainer
  1. PicoContainer
  2. PICO-85

wrap component multicaster functionality into adapter pattern [patch]

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0-beta-4
    • Component/s: PicoContainer (Java)
    • Labels:
      None
    • Number of attachments :
      1

      Description

      the component multicaster functionality was removed form the PicoContainer interface, and classes that depend on it have to link to the DefaultPicoContainer class. To promote interoperability, I've written a ComponentMulticasterPicoAdapter, that is able to add multicaster functionality to any PicoContainer implementation.

      To keep the contract of the exisitng implementation, I had to add a method to the PicoContainer interface: List getUnmanagedInsntaces().
      There were few different ways of approaching the problem of unmanged componetns, and I'm not sure if I've chosen the right one.

        Activity

        Hide
        Rafal Krzewski added a comment -

        a patch that introduces the described functionality

        Show
        Rafal Krzewski added a comment - a patch that introduces the described functionality
        Rafal Krzewski made changes -
        Field Original Value New Value
        Attachment multicasterAdapter.diff [ 11164 ]
        Hide
        Aslak Hellesøy added a comment -

        Excellent patch Rafal! Paul was going to beat me up on this last screw up of mine.

        Besides, moving functionality out to adapter is a very clean and flexible way to do this. Thanks!

        Show
        Aslak Hellesøy added a comment - Excellent patch Rafal! Paul was going to beat me up on this last screw up of mine. Besides, moving functionality out to adapter is a very clean and flexible way to do this. Thanks!
        Aslak Hellesøy made changes -
        Assignee Aslak Hellesoy [ rinkrank ]
        Status Open [ 1 ] Closed [ 6 ]
        Fix Version/s 1.0 [ 10145 ]
        Resolution Fixed [ 1 ]
        Aslak Hellesøy made changes -
        Resolution Fixed [ 1 ]
        Status Closed [ 6 ] Reopened [ 4 ]
        Aslak Hellesøy made changes -
        Fix Version/s 1.0-beta-4 [ 10412 ]
        Status Reopened [ 4 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Fix Version/s 1.0 [ 10145 ]

          People

          • Assignee:
            Aslak Hellesøy
            Reporter:
            Rafal Krzewski
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: