Support validating collection of objects on web controller method

[spring-projects-issues]

Will May opened SPR-12312 and commented

Add support for being able to validate a list of objects similar to the example below, where Foo is a class which has various JSR 303 annotations on its fields:

@RequestMapping(value = "/foo", method = RequestMethod.POST)
public void insertFoos(@Valid @RequestBody List<Foo> foos) {
  ...
}

I've managed to partially implement the functionality by extending the LocalValidatorFactoryBean and setting the nested path to the current list path ([i]) while calling super.validate in a loop. This almost works apart from the fact that the BeanWrapper cannot retrieve an invalid value if one fails validation and so throws an exception while trying to throw an exception.

So, in summary, the BeanWrapper needs to be able to work directly on lists rather than only being able to work on objects containing lists.

---

Affects: 4.0.7

Issue Links:

• @Validated support on Iterables (and implementors) [SPR-13428] #18007 @Validated support on Iterables (and implementors)
• Validate values in top-level Map parameters [SPR-14615] #19182 Validate values in top-level Map parameters

1 votes, 4 watchers

[spring-projects-issues]

Will May commented

It's possible that this might be a duplicate or a subtask of #6751 but that appears to be more to do with Map of objects rather than Collection of objects.

[spring-projects-issues]

Frédéric Camblor commented

+1 on this (just created a duplicate of the issue before finding this one -> #18007)

[spring-projects-issues]

mathsworld2001 commented

I recently came across this very same issue. Some at stackoverflow claims that this should already work as expected. I still cannot make this work. Can anyone confirm the current status of this ticket?

[spring-projects-issues]

Will May commented

Still doesn't work. See an example at https://github.com/wjam/spr-12312

[SumithraPrasad]

@Validated with MockMVC is not working.

@RunWith(MockitoJUnitRunner.class)
@Slf4j
public class TestController {
	private MockMvc mvc;

	@Mock
	private TestService service;

	@InjectMocks
	private Controller controller;

	private ObjectMapper mapper = new ObjectMapper();

	@Before
	public void setup() { // MockMvc standalone approach mvc =
		mvc = MockMvcBuilders.standaloneSetup(controller).setValidator(validator()).setControllerAdvice(new InvoiceServiceExceptionHandler())
				.build();
	}

	@Test
	public void whenNullValue_thenReturns400() throws JsonProcessingException, Exception {

		TestDTO testDTO = new TestDTO();
		testDTO.setId(null);

		ArrayList<TestDTO> testList = new ArrayList<TestDTO>(Arrays.asList(testDTO));

		String jsonTestList = mapper.writeValueAsString(testList);
		

		MvcResult responseMVC = mvc.perform(post("/v1/test").content(jsonTestList).header(HttpHeaders.CONTENT_TYPE,
				MediaType.APPLICATION_JSON)).andExpect(status().isBadRequest()).andReturn();

	}

[rstoyanchev]

@SumithraPrasad, I've edited your comment to improve the formatting. You might want to check out this Mastering Markdown guide for future reference. In particular avoid use of unquoted @ which are mentions on Github.

Also if it isn't supported to begin with, then it can't work with MockMvc.