Re: Explanation of three patches checked into patches_3.1.0-beta1 branch

Simone Bordet <[email protected]> Mon, 23 Mar 2009 17:08:06 +0100
Newsgroups gmane.comp.java.mx4j.devel
Message-ID <[email protected]>
2009/3/23 Allen Servedio <[email protected]>:
> Hi,
>
> I have checked in three patches to the patches_3.1.0-beta1 branch
> (https://mx4j.svn.sourceforge.net/svnroot/mx4j/branches/patches_3.1.0-beta1).
> They basically do the following to the HttpAdapter:
>
> 1) (Revision 2274) If an operation's parameter is of type Object, it treats
> it as type String. The company I work for has a few MBeans with API's that
> have Object parameters for operations that accept Strings but also a few
> other types. This was the simplest way I saw to open up these API's for use
> (without having to add String typed methods just to allow their use that is
> :-) ).
>
> 2) (Revision 2275) The second feature I added was a check box above each
> operation's submit button that - when checked - told the system to return
> the raw output of a method (unescaped and with none of the MX4J navigation
> or other UI chrome). This can be especially handy if the operation returns
> XML that you just want to script pulling and saving (which is what we do).
>
> 3) (Revision 2276) The last feature I added allows people to specify an XSL
> document to apply to the raw output of an operation (along with instructions
> on how to use this and the second feature above).
>
> I broke each of these into their own revision to make it easier to see what
> I did (esp. if the feature actually is unnecessary because of some existing
> feature that I overlooked :-) ).
>
> I will next look at the existing tests in the project and see how tests can
> be added for each of these features and do so (I apologize ahead of time for
> not having tests in place already, but that is another reason I put these
> changes in a branch instead of trunk). I also wanted to try to get these
> features up for review quickly so that people could see what I was trying to
> do and the changes could be made to have them fit the standards of this
> project.

Allen, great job and please apply to trunk, do not waste time working
on a branch (that has later to be reported to trunk).

Thanks !

Simon
-- 
http://bordet.blogspot.com
---
Finally, no matter how good the architecture and design are,
to deliver bug-free software with optimal performance and reliability,
the implementation technique must be flawless.   Victoria Livschitz

------------------------------------------------------------------------------
Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are
powering Web 2.0 with engaging, cross-platform capabilities. Quickly and
easily build your RIAs with Flex Builder, the Eclipse(TM)based development
software that enables intelligent coding and step-through debugging.
Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com