[PATCH] BEEHIVE-953: EJBAssembler creates invalid deployment descriptor (ejb-local-ref before ejb-ref)

[Chad Schoettger <ch...@gmail.com>]

FWIW, I've attached a patch for this BEEHIVE-953. The only nodes inserted
into the DOM tree by the EJB assembler are the 'ejb-ref' and 'ejb-local-ref'
so I included fixes for both. I verified the fix with Jacob's
ejb-jar.xmland serveral others I had sitting around.

Any code review comments are apprecitated.

- Chad

Re: [PATCH] BEEHIVE-953: EJBAssembler creates invalid deployment descriptor (ejb-local-ref before ejb-ref)

[Eddie O'Neil <ek...@bea.com>]

   Thanks, Chad.  I've applied this patch and tweaked some things to support web.xml and testing. 
Details can be found in the description of SVN change 291314.

   The tests aren't wired into the main DRT run yet; use these commands to run them:

cd system-controls/test/ejb
ant test

and manually verify that the right XML DDs show up in build/.

   At some point, we need to wire-up schema validation to automate this checking.  Any takers?  :)

Eddie



Chad Schoettger wrote:
> FWIW, I've attached a patch for this BEEHIVE-953. The only nodes inserted
> into the DOM tree by the EJB assembler are the 'ejb-ref' and 'ejb-local-ref'
> so I included fixes for both. I verified the fix with Jacob's
> ejb-jar.xmland serveral others I had sitting around.
> 
> Any code review comments are apprecitated.
> 
> - Chad
>