NanoContainer
  1. NanoContainer
  2. NANO-87

nano-struts implementation

          Details

          • Type: New Feature New Feature
          • 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 :
            2

            Description

            Nano component that integrates PicoContainer and Struts. Uses PicoContainer to inject dependencies into Struts actions. See the package.html file in the source for a brief description.

            nano-struts.zip contains the actual nano component. sample-struts.zip is an example Struts application that uses nano-struts.

              Activity

              Stephen Molitor made changes -
              Field Original Value New Value
              Attachment nano-struts.zip [ 12301 ]
              Stephen Molitor made changes -
              Attachment sample-struts.zip [ 12302 ]
              Hide
              Permalink
              Stephen Molitor added a comment -

              Removed the logging / monitor gunk. Please use this version instead of struts.zip

              Show
              Stephen Molitor added a comment - Removed the logging / monitor gunk. Please use this version instead of struts.zip
              Stephen Molitor made changes -
              Attachment struts2.zip [ 12307 ]
              Hide
              Permalink
              Stephen Molitor added a comment -

              Fixed compile bug. I zipped up the wrong stuff in sample-struts.zip. Don't use that; please use this zip file instead.

              Show
              Stephen Molitor added a comment - Fixed compile bug. I zipped up the wrong stuff in sample-struts.zip. Don't use that; please use this zip file instead.
              Stephen Molitor made changes -
              Attachment sample-struts2.zip [ 12308 ]
              Hide
              Permalink
              Stephen Molitor added a comment -

              Removed an unnecessary 'Monitor' related constant from KeyConstants. Doh! Please use struts3.zip; ignore struts2.zip and struts.zip.

              Show
              Stephen Molitor added a comment - Removed an unnecessary 'Monitor' related constant from KeyConstants. Doh! Please use struts3.zip; ignore struts2.zip and struts.zip.
              Stephen Molitor made changes -
              Attachment struts3.zip [ 12309 ]
              Hide
              Permalink
              Stephen Molitor added a comment -

              OK, sorry for the confusion with the attachements. Please use struts3.zip and sample-struts2.zip. Please ignore the other zip files.

              I removed the Monitor / LoggingMonitor stuff that was in struts.zip. Introducing a new interface and a new class was just way overkill for a component that only had 3 simple log statements in it. The idea might be sound, but it felt didn't feel right in this case. It was overdesign. My propeller must have been spinning a bit too fast implementing that stuff.

              So all logging is yanked out.

              Show
              Stephen Molitor added a comment - OK, sorry for the confusion with the attachements. Please use struts3.zip and sample-struts2.zip. Please ignore the other zip files. I removed the Monitor / LoggingMonitor stuff that was in struts.zip. Introducing a new interface and a new class was just way overkill for a component that only had 3 simple log statements in it. The idea might be sound, but it felt didn't feel right in this case. It was overdesign. My propeller must have been spinning a bit too fast implementing that stuff. So all logging is yanked out.
              Paul Hammant made changes -
              Attachment struts2.zip [ 12307 ]
              Paul Hammant made changes -
              Attachment nano-struts.zip [ 12301 ]
              Paul Hammant made changes -
              Attachment sample-struts.zip [ 12302 ]
              Hide
              Permalink
              Paul Hammant added a comment -

              This is great stuff and should go into CVS. I'll do it myself later, if nobody beats me to it.

              Show
              Paul Hammant added a comment - This is great stuff and should go into CVS. I'll do it myself later, if nobody beats me to it.
              Aslak Hellesøy made changes -
              Assignee Paul Hammant [ paul ]
              Hide
              Permalink
              Aslak Hellesøy added a comment -

              Is this completed?

              Show
              Aslak Hellesøy added a comment - Is this completed?
              Hide
              Permalink
              Aslak Hellesøy added a comment -

              re-committed these files "as-is" and removed previous commit (which wasn't picked up by the build)

              Show
              Aslak Hellesøy added a comment - re-committed these files "as-is" and removed previous commit (which wasn't picked up by the build)
              Aslak Hellesøy made changes -
              Fix Version/s 1.0-beta-1 [ 10147 ]
              Resolution Fixed [ 1 ]
              Assignee Paul Hammant [ paul ] Aslak Hellesoy [ rinkrank ]
              Status Open [ 1 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: