Revert commit 0f38c28

The fix is not how the issue needs to be addressed.

See gh-34121

Author: rstoyanchev

spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/ServletRequestDataBinderIntegrationTests.java

@@ -19,6 +19,7 @@
 import java.util.List;
 import java.util.Map;
 
+import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
 import org.springframework.test.context.junit.jupiter.web.SpringJUnitWebConfig;
@@ -56,6 +57,7 @@ void postArray(WebApplicationContext wac) throws Exception {
 				.andExpect(content().string("valueB"));
 	}
 
+	@Disabled("see gh-34121")
 	@Test // gh-34121
 	void postArrayWithEmptyIndex(WebApplicationContext wac) throws Exception {
 		MockMvc mockMvc = webAppContextSetup(wac).build();
@@ -86,6 +88,7 @@ void postList(WebApplicationContext wac) throws Exception {
 				.andExpect(content().string("valueB"));
 	}
 
+	@Disabled("see gh-34121")
 	@Test // gh-34121
 	void postListWithEmptyIndex(WebApplicationContext wac) throws Exception {
 		MockMvc mockMvc = webAppContextSetup(wac).build();

spring-web/src/main/java/org/springframework/web/bind/ServletRequestDataBinder.java

@@ -244,10 +244,6 @@ public final Object resolveValue(String name, Class<?> paramType) {
 		@Nullable
 		protected Object getRequestParameter(String name, Class<?> type) {
 			Object value = this.request.getParameterValues(name);
-			if (value == null && !name.endsWith("[]") &&
-					(List.class.isAssignableFrom(type) || type.isArray())) {
-				value = this.request.getParameterValues(name + "[]");
-			}
 			return (ObjectUtils.isArray(value) && Array.getLength(value) == 1 ? Array.get(value, 0) : value);
 		}
 

spring-web/src/test/java/org/springframework/web/bind/ServletRequestDataBinderTests.java

@@ -93,32 +93,6 @@ void testFieldPrefixCausesFieldResetWithIgnoreUnknownFields() {
 		assertThat(target.isPostProcessed()).isFalse();
 	}
 
-	@Test
-	public void testFieldWithArrayIndex() {
-		TestBean target = new TestBean();
-		ServletRequestDataBinder binder = new ServletRequestDataBinder(target);
-		binder.setIgnoreUnknownFields(false);
-
-		MockHttpServletRequest request = new MockHttpServletRequest();
-		request.addParameter("stringArray[0]", "ONE");
-		request.addParameter("stringArray[1]", "TWO");
-		binder.bind(request);
-		assertThat(target.getStringArray()).containsExactly("ONE", "TWO");
-	}
-
-	@Test
-	public void testFieldWithEmptyArrayIndex() {
-		TestBean target = new TestBean();
-		ServletRequestDataBinder binder = new ServletRequestDataBinder(target);
-		binder.setIgnoreUnknownFields(false);
-
-		MockHttpServletRequest request = new MockHttpServletRequest();
-		request.addParameter("stringArray[]", "ONE");
-		request.addParameter("stringArray[]", "TWO");
-		binder.bind(request);
-		assertThat(target.getStringArray()).containsExactly("ONE", "TWO");
-	}
-
 	@Test
 	void testFieldDefault() {
 		TestBean target = new TestBean();