You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by dj...@apache.org on 2007/07/11 23:43:36 UTC
svn commit: r555412 -
/openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java
Author: djencks
Date: Wed Jul 11 14:43:36 2007
New Revision: 555412
URL: http://svn.apache.org/viewvc?view=rev&rev=555412
Log:
GERONIMO-3310. Make a mapped-name override a generated (not actually supplied) resource link for message destination refs. This makes messages-destination-refs work in geronimo-openejb, esp. in daytrader
Modified:
openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java
Modified: openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java?view=diff&rev=555412&r1=555411&r2=555412
==============================================================================
--- openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java (original)
+++ openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java Wed Jul 11 14:43:36 2007
@@ -292,7 +292,7 @@
for (MessageDestinationRef ref : bean.getMessageDestinationRef()) {
// skip destination refs with a resource link already assigned
- if (ejbDeployment.getResourceLink(ref.getName()) == null) {
+ if (ref.getMappedName() == null && ejbDeployment.getResourceLink(ref.getName()) == null) {
String destinationId = resolveDestinationId(ref, moduleUri, destinationResolver, destinationTypes);
if (destinationId != null) {
// build the link and add it
Re: svn commit: r555412 - /openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AutoConfig.java
Posted by David Jencks <da...@yahoo.com>.
Oops, the actual issue is GERONIMO-3306.
david jencks
On Jul 11, 2007, at 5:43 PM, djencks@apache.org wrote:
> Author: djencks
> Date: Wed Jul 11 14:43:36 2007
> New Revision: 555412
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=555412
> Log:
> GERONIMO-3310. Make a mapped-name override a generated (not
> actually supplied) resource link for message destination refs.
> This makes messages-destination-refs work in geronimo-openejb, esp.
> in daytrader
>
> Modified:
> openejb/trunk/openejb3/container/openejb-core/src/main/java/org/
> apache/openejb/config/AutoConfig.java
>
> Modified: openejb/trunk/openejb3/container/openejb-core/src/main/
> java/org/apache/openejb/config/AutoConfig.java
> URL: http://svn.apache.org/viewvc/openejb/trunk/openejb3/container/
> openejb-core/src/main/java/org/apache/openejb/config/
> AutoConfig.java?view=diff&rev=555412&r1=555411&r2=555412
> ======================================================================
> ========
> --- openejb/trunk/openejb3/container/openejb-core/src/main/java/org/
> apache/openejb/config/AutoConfig.java (original)
> +++ openejb/trunk/openejb3/container/openejb-core/src/main/java/org/
> apache/openejb/config/AutoConfig.java Wed Jul 11 14:43:36 2007
> @@ -292,7 +292,7 @@
>
> for (MessageDestinationRef ref :
> bean.getMessageDestinationRef()) {
> // skip destination refs with a resource link
> already assigned
> - if (ejbDeployment.getResourceLink(ref.getName
> ()) == null) {
> + if (ref.getMappedName() == null &&
> ejbDeployment.getResourceLink(ref.getName()) == null) {
> String destinationId = resolveDestinationId
> (ref, moduleUri, destinationResolver, destinationTypes);
> if (destinationId != null) {
> // build the link and add it
>
>