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/