(tomee) branch xbean-4.30-SNAPSHOT updated: Revert explicit declaration of constructor arg types

[rzo1]

This is an automated email from the ASF dual-hosted git repository.

rzo1 pushed a commit to branch xbean-4.30-SNAPSHOT
in repository https://gitbox.apache.org/repos/asf/tomee.git


The following commit(s) were added to refs/heads/xbean-4.30-SNAPSHOT by this 
push:
     new 1f3277af8b Revert explicit declaration of constructor arg types
1f3277af8b is described below

commit 1f3277af8be628847e341c9c7194e4d6fcfd6c94
Author: Richard Zowalla <13417392+r...@users.noreply.github.com>
AuthorDate: Sun Jan 4 17:22:57 2026 +0100

    Revert explicit declaration of constructor arg types
---
 .../org/apache/openejb/assembler/classic/Assembler.java  | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git 
a/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
 
b/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
index 014d20bc6b..70244de315 100644
--- 
a/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
+++ 
b/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/Assembler.java
@@ -3155,18 +3155,18 @@ public class Assembler extends AssemblerTool implements 
org.apache.openejb.spi.A
             }
             serviceRecipe.setProperty("Definition", 
PropertiesHelper.propertiesToString(props));
             //explicitly set the argument types to avoid ambiguity in factory 
resolution.
-            serviceRecipe.setConstructorArgTypes(
-                    new Class[]{
-                            String.class, boolean.class, Class.class, 
String.class,
-                            Duration.class, Duration.class, Duration.class,
-                            boolean.class}
+//            serviceRecipe.setConstructorArgTypes(
+//                    new Class[]{
+//                            String.class, boolean.class, Class.class, 
String.class,
+//                            Duration.class, Duration.class, Duration.class,
+//                            boolean.class}
             );
         } // else: any other kind of resource relying on it? shouldnt be
 
         //explicitly set the constructor types to avoid ambiguity in ctor 
resolution.
-        if (serviceInfo.types.contains("Queue") || 
serviceInfo.types.contains(Queue.class.getName())) {
-            serviceRecipe.setConstructorArgTypes(new Class[]{String.class});
-        }
+//        if (serviceInfo.types.contains("Queue") || 
serviceInfo.types.contains(Queue.class.getName())) {
+//            serviceRecipe.setConstructorArgTypes(new Class[]{String.class});
+//        }
 
         replaceResourceAdapterProperty(serviceRecipe);