NanoContainer
  1. NanoContainer
  2. NANO-4

StringRegistrationNanoContainerImpl does not support ComponentFactory

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      It should take one in one of it's constructors or something similar.

        Activity

        Jon Tirsen made changes -
        Field Original Value New Value
        Fix Version/s 1.0-beta-1 [ 10147 ]
        Hide
        Aslak Hellesøy added a comment -

        I don't understand. StringNano wraps a pico. It's pico that'
        s responsible for the instantiation (delegating to its component factory).

        Nano shouldn't know about the component factory, that's pico's business.

        pseudo code:

        Pico pico = new Pico( new MyComponentFactory() );
        StringNano nano = new StringNano( pico );

        Show
        Aslak Hellesøy added a comment - I don't understand. StringNano wraps a pico. It's pico that' s responsible for the instantiation (delegating to its component factory). Nano shouldn't know about the component factory, that's pico's business. pseudo code: Pico pico = new Pico( new MyComponentFactory() ); StringNano nano = new StringNano( pico );
        Hide
        Stephan Schmidt added a comment -

        Yes, I had it this way and this does not work.

        PicoContainer pc = new
        picocontainer.defaults.DefaultPicoContainer(myComponentFactory);

        StringRegistrationNanoContainer c =
        new StringRegistrationNanoContainerImpl(pc,
        ... ,
        new StringToObjectConverter());

        Show
        Stephan Schmidt added a comment - Yes, I had it this way and this does not work. PicoContainer pc = new picocontainer.defaults.DefaultPicoContainer(myComponentFactory); StringRegistrationNanoContainer c = new StringRegistrationNanoContainerImpl(pc, ... , new StringToObjectConverter());
        Hide
        Aslak Hellesøy added a comment -

        In what way doesn't it work then? Need more info than just "it doesn't work".

        Show
        Aslak Hellesøy added a comment - In what way doesn't it work then? Need more info than just "it doesn't work".
        Hide
        Aslak Hellesøy added a comment -

        This is obsolete

        Show
        Aslak Hellesøy added a comment - This is obsolete
        Aslak Hellesøy made changes -
        Resolution Won't Fix [ 2 ]
        Fix Version/s 1.0-beta-1 [ 10147 ]
        Status Unassigned [ 1 ] Closed [ 6 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Jon Tirsen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: