You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2021/12/21 13:59:34 UTC

[wicket] branch upgrade-to-spring-6 created (now 3b6c455)

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

mgrigorov pushed a change to branch upgrade-to-spring-6
in repository https://gitbox.apache.org/repos/asf/wicket.git.


      at 3b6c455  WICKET-6940 Update Spring to 6.0.0-M1

This branch includes the following new commits:

     new 3b6c455  WICKET-6940 Update Spring to 6.0.0-M1

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


[wicket] 01/01: WICKET-6940 Update Spring to 6.0.0-M1

Posted by mg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

mgrigorov pushed a commit to branch upgrade-to-spring-6
in repository https://gitbox.apache.org/repos/asf/wicket.git

commit 3b6c455674c387a79b441f1742b2327cf3d80604
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
AuthorDate: Tue Dec 21 15:59:03 2021 +0200

    WICKET-6940 Update Spring to 6.0.0-M1
    
    Signed-off-by: Martin Tzvetanov Grigorov <mg...@apache.org>
---
 pom.xml                                                  | 16 +++++++++++++---
 .../wicket/spring/SpringWebApplicationFactory.java       |  4 ++--
 .../injection/annot/AnnotProxyFieldValueFactory.java     |  2 +-
 .../spring/injection/annot/SpringComponentInjector.java  |  4 ++--
 .../JakartaInjectAnnotProxyFieldValueFactoryTest.java    |  2 +-
 5 files changed, 19 insertions(+), 9 deletions(-)

diff --git a/pom.xml b/pom.xml
index 5dec096..4305f2a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -94,6 +94,16 @@
 		<url>http://git-wip-us.apache.org/repos/asf/wicket/repo?p=wicket.git</url>
 		<tag>HEAD</tag>
 	</scm>
+
+	<repositories>
+		<!-- TODO Wicket 10: Use stable release instead of a milestone one -->
+		<repository>
+			<id>repository.spring.milestone</id>
+			<name>Spring Milestone Repository</name>
+			<url>https://repo.spring.io/milestone</url>
+		</repository>
+	</repositories>
+
 	<modules>
 		<!-- wicket-eclipse-settings is separate released -->
 		<module>wicket</module>
@@ -130,8 +140,8 @@
 		<maven.compiler.optimize>true</maven.compiler.optimize>
 		<maven.compiler.showDeprecation>true</maven.compiler.showDeprecation>
 		<maven.compiler.showWarnings>true</maven.compiler.showWarnings>
-		<maven.compiler.source>11</maven.compiler.source>
-		<maven.compiler.target>11</maven.compiler.target>
+		<maven.compiler.source>17</maven.compiler.source>
+		<maven.compiler.target>17</maven.compiler.target>
 
 		<!-- Project Versions -->
 		<asm.version>9.2</asm.version>
@@ -172,7 +182,7 @@
 		<objenesis.version>3.2</objenesis.version>
 		<openjson.version>1.0.12</openjson.version>
 		<slf4j.version>2.0.0-alpha5</slf4j.version>
-		<spring.version>5.3.13</spring.version>
+		<spring.version>6.0.0-M1</spring.version>
 		<wagon-ssh-external.version>3.4.3</wagon-ssh-external.version>
 		<weld.version>4.0.2.Final</weld.version>
 		<velocity-engine-core.version>2.3</velocity-engine-core.version>
diff --git a/wicket-spring/src/main/java/org/apache/wicket/spring/SpringWebApplicationFactory.java b/wicket-spring/src/main/java/org/apache/wicket/spring/SpringWebApplicationFactory.java
index 61e0e36..f5b1476 100644
--- a/wicket-spring/src/main/java/org/apache/wicket/spring/SpringWebApplicationFactory.java
+++ b/wicket-spring/src/main/java/org/apache/wicket/spring/SpringWebApplicationFactory.java
@@ -140,7 +140,7 @@ public class SpringWebApplicationFactory implements IWebApplicationFactory
 	{
 		ServletContext servletContext = filter.getFilterConfig().getServletContext();
 
-		WebApplicationContext applicationContext = WebApplicationContextUtils.getWebApplicationContext(new javax.servlet.ServletContext.Impl(servletContext));
+		WebApplicationContext applicationContext = WebApplicationContextUtils.getWebApplicationContext(servletContext);
 
 		if (getContextConfigLocation(filter) != null)
 		{
@@ -204,7 +204,7 @@ public class SpringWebApplicationFactory implements IWebApplicationFactory
 	{
 		webApplicationContext = newApplicationContext();
 		webApplicationContext.setParent(parent);
-		webApplicationContext.setServletContext(new javax.servlet.ServletContext.Impl(filter.getFilterConfig().getServletContext()));
+		webApplicationContext.setServletContext(filter.getFilterConfig().getServletContext());
 		webApplicationContext.setConfigLocation(getContextConfigLocation(filter));
 
 		postProcessWebApplicationContext(webApplicationContext, filter);
diff --git a/wicket-spring/src/main/java/org/apache/wicket/spring/injection/annot/AnnotProxyFieldValueFactory.java b/wicket-spring/src/main/java/org/apache/wicket/spring/injection/annot/AnnotProxyFieldValueFactory.java
index 57caa9b..14d6f93 100644
--- a/wicket-spring/src/main/java/org/apache/wicket/spring/injection/annot/AnnotProxyFieldValueFactory.java
+++ b/wicket-spring/src/main/java/org/apache/wicket/spring/injection/annot/AnnotProxyFieldValueFactory.java
@@ -287,7 +287,7 @@ public class AnnotProxyFieldValueFactory implements IFieldValueFactory
 			msg.append("More than one bean of type [");
 			msg.append(clazz.getName());
 			msg.append("] found, you have to specify the name of the bean ");
-			msg.append("(@SpringBean(name=\"foo\")) or (@Named(\"foo\") if using @javax.inject classes) in order to resolve this conflict. ");
+			msg.append("(@SpringBean(name=\"foo\")) or (@Named(\"foo\") if using @jakarta.inject classes) in order to resolve this conflict. ");
 			msg.append("Matched beans: ");
 			msg.append(Strings.join(",", names));
 			throw new IllegalStateException(msg.toString());
diff --git a/wicket-spring/src/main/java/org/apache/wicket/spring/injection/annot/SpringComponentInjector.java b/wicket-spring/src/main/java/org/apache/wicket/spring/injection/annot/SpringComponentInjector.java
index 02f27d2..7d08c44 100644
--- a/wicket-spring/src/main/java/org/apache/wicket/spring/injection/annot/SpringComponentInjector.java
+++ b/wicket-spring/src/main/java/org/apache/wicket/spring/injection/annot/SpringComponentInjector.java
@@ -69,7 +69,7 @@ public class SpringComponentInjector extends Injector
 	/**
 	 * Constructor used when spring application context is declared in the spring standard way and
 	 * can be located through
-	 * {@link WebApplicationContextUtils#getRequiredWebApplicationContext(javax.servlet.ServletContext)}.
+	 * {@link WebApplicationContextUtils#getRequiredWebApplicationContext(jakarta.servlet.ServletContext)}.
 	 * 
 	 * @param webapp
 	 *            wicket web application
@@ -172,7 +172,7 @@ public class SpringComponentInjector extends Injector
 		ApplicationContext context = webapp.getMetaData(CONTEXT_KEY);
 		if (context == null)
 		{
-			context = WebApplicationContextUtils.getRequiredWebApplicationContext(new javax.servlet.ServletContext.Impl(webapp.getServletContext()));
+			context = WebApplicationContextUtils.getRequiredWebApplicationContext(webapp.getServletContext());
 		}
 		return context;
 	}
diff --git a/wicket-spring/src/test/java/org/apache/wicket/spring/injection/annot/JakartaInjectAnnotProxyFieldValueFactoryTest.java b/wicket-spring/src/test/java/org/apache/wicket/spring/injection/annot/JakartaInjectAnnotProxyFieldValueFactoryTest.java
index eb4dbdc..b4bf77a 100644
--- a/wicket-spring/src/test/java/org/apache/wicket/spring/injection/annot/JakartaInjectAnnotProxyFieldValueFactoryTest.java
+++ b/wicket-spring/src/test/java/org/apache/wicket/spring/injection/annot/JakartaInjectAnnotProxyFieldValueFactoryTest.java
@@ -25,7 +25,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.junit.jupiter.api.Assertions.fail;
 
 /**
- * Tests for Spring injection with {@literal @javax.inject.Inject} annotation
+ * Tests for Spring injection with {@literal @jakarta.inject.Inject} annotation
  */
 public class JakartaInjectAnnotProxyFieldValueFactoryTest extends AnnotProxyFieldValueFactoryTest
 {