Various improvements and bug fixes
----------------------------------

         Key: EJB-88
         URL: http://jira.andromda.org/browse/EJB-88
     Project: EJB Cartridge
        Type: Bug

    Reporter: Sverker Abrahamsson
 Assigned to: Vance Karimi 
    Priority: Minor


This is a patch that contains some small bug fixes and improvement that has 
been collected over the last months. Sorry for one file containing all bur I'll 
describe the changes here:

There were a couple of places that had statements like this:
            if(BooleanUtils.toBoolean(
                    
(String)this.findTaggedValue(EJB3Profile.TAGGEDVALUE_SEAM_BIJECTION_IN_CREATE, 
true))) 

Well, if the tagged value isn't set this statement doesn't have the intended 
function. Affected files:
cartridges/andromda-ejb3/src/main/java/org/andromda/cartridges/ejb3/metafacades/EJB3SessionAttributeFacadeLogicImpl.java
cartridges/andromda-ejb3/src/main/java/org/andromda/cartridges/ejb3/metafacades/EJB3SessionFacadeLogicImpl.java
cartridges/andromda-ejb3/src/main/java/org/andromda/cartridges/ejb3/metafacades/EJB3SessionOperationFacadeLogicImpl.java


Seam component roles were implemented in back end but not in templates as it 
was a bit complicated. I've made a new implementation in back end that makes 
the template much easier. Note! Two attributes must be removed from UML model, 
those are seamComponentRoleNames and seamComponentRoleScopeTypes and instead 
seamComponentRoles should be added.
Affected files:
cartridges/andromda-ejb3/src/main/java/org/andromda/cartridges/ejb3/metafacades/EJB3SessionFacadeLogicImpl.java
cartridges/andromda-ejb3/src/main/resources/templates/ejb3/SessionBeanBase.vsl


When having association ends to be ordered, the code is generated with List 
implementations (ArrayList). However, the interface used on the getter/setter 
method parameters is java.util.Set so the code won't compile. I've changed the 
default collection interface to be java.util.Collection instead of 
java.util.Set in 
cartridges/andromda-ejb3/src/main/resources/META-INF/andromda/namespace.xml


Compiler warnings are supposed to be an indication that something could be 
wrong with the code even though it compiles. It's good practice to solve those 
warnings. However, now with generics each untyped collection cause a warning 
which might hide real problems (have happened to me a couple of times).  There 
were already @SuppressWarnings annotations at some places in the templates but 
I've walked through the files and added that where it was missing. I've also 
changed some iterations to use foreach statements.
Affected files:
cartridges/andromda-ejb3/src/main/resources/templates/ejb3/DaoBase.vsl
cartridges/andromda-ejb3/src/main/resources/templates/ejb3/DaoLocal.vsl
cartridges/andromda-ejb3/src/main/resources/templates/ejb3/Enumeration.vsl

Sometimes I want to set default values on attributes in uml model. That support 
wasn't implemented in templates so
I added it on both entities and session beans.
Affected files:
cartridges/andromda-ejb3/src/main/resources/templates/ejb3/EntityEmbeddable.vsl
cartridges/andromda-ejb3/src/main/resources/templates/ejb3/SessionBeanBase.vsl


Support for @WebRemote annotation wasn't implemented in template, added it to
cartridges/andromda-ejb3/src/main/resources/templates/ejb3/SessionLocal.vsl


A very minor fix, on these files it wasn't correctly stated in the header which 
template had been used to generate file
cartridges/andromda-ejb3/src/main/resources/templates/ejb3/DaoImpl.vsl
cartridges/andromda-ejb3/src/main/resources/templates/ejb3/DaoLocal.vsl


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.andromda.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

Reply via email to