XCOMMONS-2777: Add support for jakarta.inject annotations

XCOMMONS-2797: Allow accessing a javax.inject.Provider as a jakarta.inject.Provider and the opposite XCOMMONS-2963: Upgrade to Servlet 5.0 XCOMMONS-2962: Provide a javax/jakarta bridge for Servlet APIs XCOMMONS-2994: Upgrade to Websocket 2.1.1 XCOMMONS-2108: Upgrade to Bean Validation 3.0.2 XCOMMONS-2109: Upgrade to Hibernate Validator 8.0.1 XCOMMONS-2475: Use Expressly instead of Apache EL
xwiki · Jan 10, 2025 · 9879228 · 9879228
1 parent bff04ec
commit 9879228
Show file tree

Hide file tree

Showing 148 changed files with 13,536 additions and 505 deletions.
diff --git a/pom.xml b/pom.xml
@@ -255,11 +255,12 @@
     <antlr4.version>4.13.2</antlr4.version>
 
     <!-- Servlet specifications -->
-    <servlet.version>3.1.0</servlet.version>
+    <javax.servlet.version>3.1.0</javax.servlet.version>
+    <jakarta.servlet.version>5.0.0</jakarta.servlet.version>
 
-    <!-- EL implementation -->
-    <!-- Use the same version as the one that comes with the custom XWiki Jetty application server -->
-    <apache-el.version>9.0.90</apache-el.version>
+    <!-- WebSocket specifications -->
+    <javax.websocket.version>1.1</javax.websocket.version>
+    <jakarta.websocket.version>2.1.1</jakarta.websocket.version>
 
     <webdrivermanager.version>5.9.2</webdrivermanager.version>
 
@@ -757,13 +758,6 @@
         <groupId>com.fasterxml.jackson.module</groupId>
         <artifactId>jackson-module-jsonSchema</artifactId>
         <version>${jackson.version}</version>
-        <exclusions>
-          <!-- We use jakarta.validation:jakarta.validation-api -->
-          <exclusion>
-            <groupId>javax.validation</groupId>
-            <artifactId>validation-api</artifactId>
-          </exclusion>
-        </exclusions>
       </dependency>
       <dependency>
         <groupId>com.fasterxml.jackson.datatype</groupId>
@@ -958,6 +952,13 @@
         <version>3.30.2-GA</version>
       </dependency>
 
+      <!-- Injection -->
+      <dependency>
+        <groupId>jakarta.inject</groupId>
+        <artifactId>jakarta.inject-api</artifactId>
+        <version>2.0.1</version>
+      </dependency>
+
       <!-- Triggered by several Maven related projects (Doxia, Maven, Aether) as transitive dependency. We need
            these explicit versions to help Maven decide which version to use. -->
       <dependency>
@@ -1259,15 +1260,31 @@
       <dependency>
         <groupId>javax.servlet</groupId>
         <artifactId>javax.servlet-api</artifactId>
-        <version>${servlet.version}</version>
+        <version>${javax.servlet.version}</version>
+      </dependency>
+      <dependency>
+        <groupId>jakarta.servlet</groupId>
+        <artifactId>jakarta.servlet-api</artifactId>
+        <version>${jakarta.servlet.version}</version>
         <!-- It should always be provided as the API jar is provided by the Servlet Container in which XWiki is
              deployed -->
         <scope>provided</scope>
       </dependency>
       <dependency>
         <groupId>javax.websocket</groupId>
         <artifactId>javax.websocket-api</artifactId>
-        <version>1.1</version>
+        <version>${javax.websocket.version}</version>
+      </dependency>
+      <dependency>
+        <groupId>jakarta.websocket</groupId>
+        <artifactId>jakarta.websocket-api</artifactId>
+        <version>${jakarta.websocket.version}</version>
+        <scope>provided</scope>
+      </dependency>
+      <dependency>
+        <groupId>jakarta.websocket</groupId>
+        <artifactId>jakarta.websocket-client-api</artifactId>
+        <version>${jakarta.websocket.version}</version>
         <scope>provided</scope>
       </dependency>
       <!-- Needed for Legacy modules. The version needs to be in sync with the one used by the AspectJ plugin -->
@@ -1282,24 +1299,30 @@
         <artifactId>cssparser</artifactId>
         <version>0.9.30</version>
       </dependency>
+
       <!-- Bean validation -->
+      <dependency>
+        <groupId>javax.validation</groupId>
+        <artifactId>validation-api</artifactId>
+        <version>2.0.1.Final</version>
+      </dependency>
       <dependency>
         <groupId>jakarta.validation</groupId>
         <artifactId>jakarta.validation-api</artifactId>
-        <version>2.0.2</version>
+        <version>3.0.2</version>
       </dependency>
       <dependency>
         <groupId>org.hibernate.validator</groupId>
         <artifactId>hibernate-validator</artifactId>
-        <version>6.2.5.Final</version>
+        <version>8.0.1.Final</version>
         <scope>runtime</scope>
       </dependency>
       <dependency>
-        <groupId>org.mortbay.jasper</groupId>
-        <artifactId>apache-el</artifactId>
-        <version>${apache-el.version}</version>
-        <!-- The EL implementation is supposed to come with the application server, but we need one for tests -->
-        <scope>test</scope>
+        <groupId>org.glassfish.expressly</groupId>
+        <artifactId>expressly</artifactId>
+        <version>5.0.0</version>
+        <!-- The implementation of Expression Language is supposed to come with the application server -->
+        <scope>provided</scope>
       </dependency>
 
       <!-- Mail -->
@@ -2049,25 +2072,6 @@
               </rules>
             </configuration>
           </execution>
-          <!-- Check that we use the right version of the Validation API -->
-          <execution>
-            <id>enforce-jakarta.validation-api</id>
-            <goals>
-              <goal>enforce</goal>
-            </goals>
-            <configuration>
-              <skip>${xwiki.enforcer.enforce-jakarta.validation-api.skip}</skip>
-              <rules>
-                <bannedDependencies>
-                  <searchTransitive>true</searchTransitive>
-                  <message>Use jakarta.validation:jakarta.validation-api instead</message>
-                  <excludes>
-                    <exclude>javax.validation:validation-api</exclude>
-                  </excludes>
-                </bannedDependencies>
-              </rules>
-            </configuration>
-          </execution>
           <!-- Check that we use the right version of the Hibernate Validator -->
           <execution>
             <id>enforce-hibernate-validator</id>

diff --git a/xwiki-commons-core/pom.xml b/xwiki-commons-core/pom.xml
@@ -64,6 +64,7 @@
     <module>xwiki-commons-websocket</module>
     <module>xwiki-commons-xml</module>
     <module>xwiki-commons-xstream</module>
+    <module>xwiki-commons-jakartabridge</module>
   </modules>
   <build>
     <pluginManagement>
@@ -238,6 +239,68 @@
                 </item>
               </differences>
             </revapi.differences>
+
+            <!-- Jakarta -->
+            <revapi.differences>
+              <justification>revapi false positives</justification>
+              <criticality>allowed</criticality>
+              <differences>
+                <item>
+                  <ignore>true</ignore>
+                  <code>java.class.nonFinalClassInheritsFromNewClass</code>
+                  <old>class org.xwiki.component.embed.GenericProvider&lt;T&gt;</old>
+                  <new>class org.xwiki.component.embed.GenericProvider&lt;T&gt;</new>
+                  <superClass>org.xwiki.component.internal.AbstractGenericProvider&lt;T&gt;</superClass>
+                </item>
+                <item>
+                  <ignore>true</ignore>
+                  <code>java.class.nonFinalClassInheritsFromNewClass</code>
+                  <old>class org.xwiki.component.embed.GenericProvider&lt;T&gt;</old>
+                  <new>class org.xwiki.component.embed.GenericProvider&lt;T&gt;</new>
+                  <superClass>org.xwiki.component.internal.JavaXGenericProvider&lt;T&gt;</superClass>
+                </item>
+              </differences>
+            </revapi.differences>
+            <revapi.differences>
+              <justification>Moved to WebSocket 2.1.1</justification>
+              <criticality>highlight</criticality>
+              <differences>
+                <item>
+                  <ignore>true</ignore>
+                  <code>java.class.noLongerImplementsInterface</code>
+                  <old>class org.xwiki.websocket.AbstractPartialMessageHandler&lt;T&gt;</old>
+                  <new>class org.xwiki.websocket.AbstractPartialMessageHandler&lt;T&gt;</new>
+                  <interface>javax.websocket.MessageHandler.Partial&lt;T&gt;</interface>
+                </item>
+                <item>
+                  <ignore>true</ignore>
+                  <code>java.class.noLongerImplementsInterface</code>
+                  <old>class org.xwiki.websocket.AbstractPartialMessageHandler&lt;T&gt;</old>
+                  <new>class org.xwiki.websocket.AbstractPartialMessageHandler&lt;T&gt;</new>
+                  <interface>javax.websocket.MessageHandler</interface>
+                </item>
+                <item>
+                  <ignore>true</ignore>
+                  <code>java.class.noLongerImplementsInterface</code>
+                  <old>class org.xwiki.websocket.AbstractPartialStringMessageHandler</old>
+                  <new>class org.xwiki.websocket.AbstractPartialStringMessageHandler</new>
+                  <interface>javax.websocket.MessageHandler.Partial&lt;java.lang.String&gt;</interface>
+                </item>
+                <item>
+                  <ignore>true</ignore>
+                  <code>java.class.noLongerImplementsInterface</code>
+                  <old>class org.xwiki.websocket.AbstractPartialStringMessageHandler</old>
+                  <new>class org.xwiki.websocket.AbstractPartialStringMessageHandler</new>
+                  <interface>javax.websocket.MessageHandler</interface>
+                </item>
+                <item>
+                  <ignore>true</ignore>
+                  <regex>true</regex>
+                  <code>java.method.parameterTypeChanged</code>
+                  <old>parameter .* org.xwiki.websocket.WebSocketContext::.*</old>
+                </item>
+              </differences>
+            </revapi.differences>
           </analysisConfiguration>
         </configuration>
       </plugin>

diff --git a/xwiki-commons-core/xwiki-commons-component/xwiki-commons-component-api/pom.xml b/xwiki-commons-core/xwiki-commons-component/xwiki-commons-component-api/pom.xml
@@ -55,13 +55,17 @@
       <groupId>org.slf4j</groupId>
       <artifactId>slf4j-api</artifactId>
     </dependency>
-    <!-- We add this dependency here so that users of the Component API just need to depend on this artifact and
-         don't have to explicitly add a dependency on javax.inject:java.inject. -->
+    <dependency>
+      <groupId>jakarta.inject</groupId>
+      <artifactId>jakarta.inject-api</artifactId>
+    </dependency>
+    <!-- Keep supporting both the Jakarta and Javax injection APIs -->
     <dependency>
       <groupId>javax.inject</groupId>
       <artifactId>javax.inject</artifactId>
       <version>1</version>
     </dependency>
+
     <!-- Testing dependencies -->
     <dependency>
       <groupId>org.xwiki.commons</groupId>

diff --git a/...omponent-api/src/main/java/org/xwiki/component/descriptor/DefaultComponentDependency.java b/...omponent-api/src/main/java/org/xwiki/component/descriptor/DefaultComponentDependency.java
@@ -26,8 +26,9 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
+import java.util.Set;
 
-import javax.inject.Provider;
+import jakarta.inject.Provider;
 
 import org.apache.commons.lang3.builder.HashCodeBuilder;
 import org.xwiki.component.util.DefaultParameterizedType;
@@ -42,6 +43,9 @@
  */
 public class DefaultComponentDependency<T> extends DefaultComponentRole<T> implements ComponentDependency<T>
 {
+    private static final Set<Class<?>> SPECIAL_ROLES =
+        Set.of(List.class, Collection.class, Map.class, javax.inject.Provider.class, Provider.class);
+
     /**
      * @see #getName()
      */
@@ -115,7 +119,8 @@ public boolean equals(Object object)
     {
         boolean result;
 
-        // See http://www.technofundo.com/tech/java/equalhash.html for the detail of this algorithm.
+        // See http://www.technofundo.com/tech/java/equalhash.html for the detail of
+        // this algorithm.
         if (this == object) {
             result = true;
         } else {
@@ -192,7 +197,7 @@ public Class<T> getRole()
     {
         Class mapping = getMappingType();
 
-        if (mapping == List.class || mapping == Collection.class || mapping == Map.class || mapping == Provider.class) {
+        if (SPECIAL_ROLES.contains(mapping)) {
             return ReflectionUtils.getTypeClass(ReflectionUtils.getLastTypeGenericArgument(getRoleType()));
         } else {
             return mapping;
@@ -204,7 +209,7 @@ public void setRole(Class<T> role)
     {
         Class mapping = getMappingType();
 
-        if (mapping == List.class || mapping == Collection.class || mapping == Map.class || mapping == Provider.class) {
+        if (SPECIAL_ROLES.contains(mapping)) {
             Type ownerType;
             Class<?> rawType;
             if (getRoleType() instanceof ParameterizedType) {

diff --git a/...api/src/main/java/org/xwiki/component/internal/multi/AbstractGenericComponentManager.java b/...api/src/main/java/org/xwiki/component/internal/multi/AbstractGenericComponentManager.java
@@ -21,7 +21,7 @@
 
 import java.lang.reflect.Type;
 
-import javax.inject.Inject;
+import jakarta.inject.Inject;
 
 import org.xwiki.component.descriptor.ComponentDescriptor;
 import org.xwiki.component.manager.ComponentManager;

diff --git a/...-api/src/main/java/org/xwiki/component/internal/multi/DefaultComponentManagerManager.java b/...-api/src/main/java/org/xwiki/component/internal/multi/DefaultComponentManagerManager.java
@@ -22,8 +22,8 @@
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
 
-import javax.inject.Inject;
-import javax.inject.Singleton;
+import jakarta.inject.Inject;
+import jakarta.inject.Singleton;
 
 import org.xwiki.component.annotation.Component;
 import org.xwiki.component.manager.ComponentLookupException;

diff --git a/...t-api/src/main/java/org/xwiki/component/internal/namespace/DefaultNamespaceValidator.java b/...t-api/src/main/java/org/xwiki/component/internal/namespace/DefaultNamespaceValidator.java
@@ -25,10 +25,10 @@
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
-import javax.inject.Inject;
-import javax.inject.Named;
-import javax.inject.Provider;
-import javax.inject.Singleton;
+import jakarta.inject.Inject;
+import jakarta.inject.Named;
+import jakarta.inject.Provider;
+import jakarta.inject.Singleton;
 
 import org.apache.commons.lang3.StringUtils;
 import org.xwiki.component.annotation.Component;

diff --git a/...-component-api/src/test/java/org/xwiki/component/descriptor/DefaultComponentRoleTest.java b/...-component-api/src/test/java/org/xwiki/component/descriptor/DefaultComponentRoleTest.java
@@ -30,7 +30,7 @@
  * @version $Id$
  * @since 3.3M1
  */
-public class DefaultComponentRoleTest
+class DefaultComponentRoleTest
 {
     private interface Role
     {

diff --git a/...i/src/test/java/org/xwiki/component/internal/namespace/DefaultNamespaceValidatorTest.java b/...i/src/test/java/org/xwiki/component/internal/namespace/DefaultNamespaceValidatorTest.java
@@ -34,7 +34,7 @@
  *
  * @version $Id$
  */
-public class DefaultNamespaceValidatorTest
+class DefaultNamespaceValidatorTest
 {
     private DefaultNamespaceValidator validator = new DefaultNamespaceValidator();
 

diff --git a/...ons-component-api/src/test/java/org/xwiki/component/internal/namespace/NamespaceTest.java b/...ons-component-api/src/test/java/org/xwiki/component/internal/namespace/NamespaceTest.java
@@ -31,7 +31,7 @@
  * 
  * @version $Id$
  */
-public class NamespaceTest
+class NamespaceTest
 {
     @Test
     void equalsAndHashCode()

diff --git a/...omponent-api/src/test/java/org/xwiki/component/internal/namespace/NamespaceUtilsTest.java b/...omponent-api/src/test/java/org/xwiki/component/internal/namespace/NamespaceUtilsTest.java
@@ -31,7 +31,7 @@
  *
  * @version $Id$
  */
-public class NamespaceUtilsTest
+class NamespaceUtilsTest
 {
     @Test
     void getPrefix()

diff --git a/...ns-component-api/src/test/java/org/xwiki/component/util/DefaultParameterizedTypeTest.java b/...ns-component-api/src/test/java/org/xwiki/component/util/DefaultParameterizedTypeTest.java
@@ -32,7 +32,7 @@
  *
  * @version $Id$
  */
-public class DefaultParameterizedTypeTest
+class DefaultParameterizedTypeTest
 {
     @Test
     void toStringTest()

diff --git a/...mmons-component-api/src/test/java/org/xwiki/component/util/ReflectionMethodUtilsTest.java b/...mmons-component-api/src/test/java/org/xwiki/component/util/ReflectionMethodUtilsTest.java
@@ -30,7 +30,7 @@
  *
  * @version $Id$
  */
-public class ReflectionMethodUtilsTest
+class ReflectionMethodUtilsTest
 {
     @Test
     void testGetMethodParameterAnnotations() throws SecurityException, NoSuchMethodException

diff --git a/...iki-commons-component-api/src/test/java/org/xwiki/component/util/ReflectionUtilsTest.java b/...iki-commons-component-api/src/test/java/org/xwiki/component/util/ReflectionUtilsTest.java
@@ -46,7 +46,7 @@
  *
  * @version $Id$
  */
-public class ReflectionUtilsTest
+class ReflectionUtilsTest
 {
     private static interface TestInterfaceSimple
     {

diff --git a/...type/src/main/resources/archetype-resources/src/main/java/internal/DefaultHelloWorld.java b/...type/src/main/resources/archetype-resources/src/main/java/internal/DefaultHelloWorld.java
@@ -22,7 +22,7 @@
 import ${package}.HelloWorld;
 import org.xwiki.component.annotation.Component;
 
-import javax.inject.Singleton;
+import jakarta.inject.Singleton;
 
 /**
  * Implementation of a {@code HelloWorld} component.