[jboss-jira] [JBoss JIRA] (AS7-6079) EJB 2.1 CMP configuration options missing (sync-on-commit-only, insert-after-ejb-post-create, call-ejb-store-on-clean)

[ https://issues.jboss.org/browse/AS7-6079?page=com.atlassian.jira.plugin.s... ] Tiago Frazão edited comment on AS7-6079 at 1/29/13 7:20 AM: ------------------------------------------------------------- //--- was (Author: tiagoff): Hi Guys, I do still have the issue ... and i had to change a little bit the Jboss code to overcome the problem. When we define the configurations (sync-on-commin-only, insert-after-ejb-post-create, call-ejb-store-on-clean) in "jbosscmp-jdbc.xml" like bellow : <jbosscmp-jdbc> <defaults> <datasource>java:jboss/datasources/XptoDS</datasource> <datasource-mapping>Oracle9i</datasource-mapping> <create-table>false</create-table> <remove-table>false</remove-table> <sync-on-commit-only>true</sync-on-commit-only> <insert-after-ejb-post-create>true</insert-after-ejb-post-create> <call-ejb-store-on-clean>true</call-ejb-store-on-clean> </defaults> <entity> <ejb-name>User</ejb-name> <table-name>USER</table-name> <cmp-field><field-name>name</field-name> <column-name>NAME</column-name></cmp-field> </entity> ... </jbosscmp-jdbc> During the lifecycle of org.jboss.as.cmp.processors.CmpParsingProcessor, the default values previously settled (in <defaults>) are been replaced for the one defined in <entity> scope, but if we do not define anything like above, the CmpParsinProcessor is using the default values of the CmpConfig. (primitive values) I had to change 2 classes to skip this problem and I would like to know if is applicable or make sense for you. public class CmpConfig { private boolean syncOnCommitOnly; private boolean insertAfterEjbPostCreate; private boolean callEjbStoreOnClean = true; private boolean wasSyncOnCommitOnlyChanged = false; private boolean wasInsertAfterEjbPostCreateChanged = false; private boolean wasCallEjbStoreOnCleanChanged = false; public boolean isSyncOnCommitOnly() { return syncOnCommitOnly; } public void setSyncOnCommitOnly(boolean syncOnCommitOnly) { this.syncOnCommitOnly = syncOnCommitOnly; this.wasSyncOnCommitOnlyChanged = true; } public boolean isInsertAfterEjbPostCreate() { return insertAfterEjbPostCreate; } public void setInsertAfterEjbPostCreate(boolean insertAfterEjbPostCreate) { this.insertAfterEjbPostCreate = insertAfterEjbPostCreate; this.wasInsertAfterEjbPostCreateChanged = true; } public boolean isCallEjbStoreOnClean() { return callEjbStoreOnClean; } public void setCallEjbStoreOnClean(boolean callEjbStoreOnClean) { this.callEjbStoreOnClean = callEjbStoreOnClean; this.wasCallEjbStoreOnCleanChanged = true; } public boolean wasSyncOnCommitOnlyChanged() { return wasSyncOnCommitOnlyChanged; } public boolean wasInsertAfterEjbPostCreateChanged() { return wasInsertAfterEjbPostCreateChanged; } public boolean wasCallEjbStoreOnCleanChanged() { return wasCallEjbStoreOnCleanChanged; } } and public JDBCEntityMetaData(JDBCApplicationMetaData jdbcApplication, ParsedEntity parsed, JDBCEntityMetaData defaultValues) { .... .... localHomeClass = defaultValues.getLocalHomeClass(); localClass = defaultValues.getLocalClass(); cmpConfig = defaultValues.getCmpConfig(); // --> added .... .... // cmpConfig = parsed.getCmpConfig(); -- removed // -- added if (parsed.getCmpConfig().wasSyncOnCommitOnlyChanged()) { cmpConfig.setSyncOnCommitOnly(parsed.getCmpConfig().isSyncOnCommitOnly()); } // -- added if (parsed.getCmpConfig().wasInsertAfterEjbPostCreateChanged()) { cmpConfig.setInsertAfterEjbPostCreate(parsed.getCmpConfig().isInsertAfterEjbPostCreate()); } // -- added if (parsed.getCmpConfig().wasCallEjbStoreOnCleanChanged()) { cmpConfig.setCallEjbStoreOnClean(parsed.getCmpConfig().isCallEjbStoreOnClean()); } } Regards -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira