Re: [Webwork-devel] [PATCH] action.factory package javadoc patch

[Bill B. <bi...@pr...>]

Hello,

Attached is a revised patch for javadoc updates to the action.factory
package.  Since Rickard moved the ActionContext class from within
ActionFactory to a top level class, I updated that as well.

Also removed some redundant or unused imports from the classes I touched.

Thanks,
-Bill

Bill Burton wrote:
> 
> Hello,
> 
> Looks like some of the recent commits cause this patch to have a few
> conflicts when applied.  Also, since I mentioned *Aware interfaces in a
> couple of places, those descriptions should be updated.
> 
> In the meantime while I'm updating the patch so it will apply cleanly,
> could someone do a quick scan to make sure the text looks okay?  Even if
> it's not quite right in places, some of the original comments this patch
> fixes were completely wrong and misleading or nonexistent.  I also took
> great pains to try and make each first sentence a good synopsis of the
> item being documented as this makes the javadoc summaries much more
> meaningful.
> 
> Thanks,
> -Bill
> 
> Bill Burton wrote:
> >
> > Hello,
> >
> > Attached is a patch to significantly improve the javadoc for most classes
> > in the .action.factory package as well is minor changes to
> > .action.CommandDriven javadoc.  Most notably, I documented the action
> > factory proxy delegation chain in DefaultActionFactory and the list of
> > *Aware interfaces supported in ContextActionFactoryProxy.
> >
> > Also, I fixed all misnamed javadoc tags and removed some unused imports.
> >
> > -Bill
>