NanoContainer
  1. NanoContainer
  2. NANO-80

nanocontainer.dynaop enhancements

          Details

          • Type: New Feature New Feature
          • Status: Closed Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: None
          • Component/s: None
          • Labels:
            None
          • Number of attachments :
            1

            Description

            Added the following features to nanocontainer.dynaop:

            1. Added interface DynaPico, and implementation DefaultDynaPico. This class supports configuration of Pico components and Dynaop aspects using one API / class. Supports applying advice to just one component in the container, and container supplied advice.

            2. Added DynaPicoBeanShellContainerBuilder. This class lets you configure your Pico container and apply Dynaop advice to its components in one BeanShell script.

            (I'm working DynaPicoGroovyContainerBuilder. It's not done yet and not included in the attached zip file.)

            3. Added various 'under the hood' classes to support the functionality in items #1 and #2, above.

            4. Changed the package.html documentation to reflect the new features.

            5. Fixed lots of spelling errors in Javadoc.

            6. Made various little stylistic tweaks (adding 'final' modifiers to instance variables where appropriate, etc.)

            7. Added a lot more unit tests.

            The attached zip file is not a patch; it includes everything. If a patch file would be better, let me know, although almost every file has changed.

              Activity

              Stephen Molitor made changes -
              Field Original Value New Value
              Attachment nanocontainer-dynaop.3.zip [ 12200 ]
              Hide
              Permalink
              Paul Hammant added a comment -

              Done. Are jakarta-ORO.jar etc available from the Maven repository? If yes, then perhaps we should chante the project.xml file.

              Show
              Paul Hammant added a comment - Done. Are jakarta-ORO.jar etc available from the Maven repository? If yes, then perhaps we should chante the project.xml file.
              Paul Hammant made changes -
              Assignee Paul Hammant [ paul ]
              Status Open [ 1 ] Closed [ 6 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Stephen Molitor added a comment -

              Paul asks:

              "Are jakarta-ORO.jar etc available from the Maven repository? If yes, then perhaps we should chante the project.xml file."

              Jakarta-oro isn't required at all; that was an oversite on my part. It can be removed from the project.properties file and from the lib/jars directory. It's not mentioned in the project.xml file.

              As best I can tell, dynaop is not in the Maven repository. So unless it gets added we need a copy in lib/jars and a reference to it in project.properties.

              Steve

              Show
              Stephen Molitor added a comment - Paul asks: "Are jakarta-ORO.jar etc available from the Maven repository? If yes, then perhaps we should chante the project.xml file." Jakarta-oro isn't required at all; that was an oversite on my part. It can be removed from the project.properties file and from the lib/jars directory. It's not mentioned in the project.xml file. As best I can tell, dynaop is not in the Maven repository. So unless it gets added we need a copy in lib/jars and a reference to it in project.properties. Steve

                People

                • Assignee:
                  Paul Hammant
                  Reporter:
                  Stephen Molitor
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: