NanoContainer
  1. NanoContainer
  2. NANO-42

allow using custom ContainerBuilder with NanoContainer

    Details

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

      Description

      it's no longer possible (since extensionsToComposers became private, which actually happens to be good design) to use a custom ContainerBuilder with NanoContainer.

      I think that it would be better if only the constructos that take File arguments used the filename extensions to resolve ScriptedContainerBuilder class. All constructors that currently take an extension argument, should take the builder's class name.

        Activity

        Hide
        Rafal Krzewski added a comment -

        a patch that introduces the described change (i.e. extension -> builderClass in constructor parameters)

        Show
        Rafal Krzewski added a comment - a patch that introduces the described change (i.e. extension -> builderClass in constructor parameters)
        Rafal Krzewski made changes -
        Field Original Value New Value
        Attachment nanoContainerBuilder.diff [ 11480 ]
        Hide
        Aslak Hellesøy added a comment -

        Fixed . thankyouu.

        Show
        Aslak Hellesøy added a comment - Fixed . thankyouu.
        Aslak Hellesøy made changes -
        Resolution Fixed [ 1 ]
        Fix Version/s 1.0-alpha-1 [ 10146 ]
        Assignee Aslak Hellesoy [ rinkrank ]
        Status Open [ 1 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: