[jbpm-issues] [JBoss JIRA] Updated: (JBPM-2946) class org.jbpm.ejb.impl.CommandListenerBean.java the lookup has the JNDI name harcoded

class org.jbpm.ejb.impl.CommandListenerBean.java the lookup has the JNDI name harcoded -------------------------------------------------------------------------------------- Key: JBPM-2946 URL: https://jira.jboss.org/browse/JBPM-2946 Project: jBPM Issue Type: Patch Security Level: Public(Everyone can see) Components: Runtime Engine Affects Versions: jBPM 3.2.x Reporter: eric schabell Assignee: Alejandro Guizar Priority: Minor Working with jBPM 3.2.8 but I have found it still is the same in jBPM 3.x TRUNK. This issue was raised in issue 00063190, but is resolved in JBoss environments only with their answer. To be more flexible the fix below would be nice. Class org.jbpm.ejb.impl.CommandListenerBean.java the lookup has the JNDI name harcoded: LocalCommandServiceHome commandServiceHome = (LocalCommandServiceHome) initial.lookup("java:comp/env/ejb/LocalCommandServiceBean"); So this can't be configured through jbpm-cfg.xml. This causes problems when the EJB JNDI name is different from ejb/LocalCommandServiceBean. I wonder if it's possible to fix it so it works like the org.jbpm.scheduler.ejbtimer.EntitySchedulerServiceFactory.java, which has this: String timerEntityHomeJndiName = "java:comp/env/ejb/TimerEntityBean"; [...] timerEntityHome = (LocalTimerEntityHome) lookup(timerEntityHomeJndiName); This way it can be configured through jbpm-cfg.xml: <service name="scheduler"> <factory> <bean class="org.jbpm.scheduler.ejbtimer.EntitySchedulerServiceFactory"> <field name="timerEntityHomeJndiName"> <string value="TimerEntityBean"/> </field> </bean> </factory> </service> This is sort of like the issue resolved here https://jira.jboss.org/browse/JBPM-960 Patch would be something like this in CommandListenerBean.java starting at line 179 (entire method provided here): private Session createSession() throws JMSException { if (jmsConnection == null) { // lookup factory and create jms connection try { log.debug("trying to create session with CommanListenerBean"); Context initial = new InitialContext(); // Changed the following line to change the hardcoded comp:env/ejb JNDI // ConnectionFactory jmsConnectionFactory = (ConnectionFactory) initial.lookup("java:comp/env/jms/JbpmConnectionFactory"); ConnectionFactory jmsConnectionFactory = (ConnectionFactory) initial.lookup("jbpmJobExecConFact"); jmsConnection = jmsConnectionFactory.createConnection(); log.debug("done creating session with CommanListenerBean"); } catch (NamingException e) { throw new EJBException("error retrieving jms connection factory", e); } } /* * if the connection supports xa, the session will be transacted, else the session will auto * acknowledge; in either case no explicit transaction control must be performed - see ejb 2.1 - * 17.3.5 */ return jmsConnection.createSession(false, Session.AUTO_ACKNOWLEDGE); }