Improve documentation explaining how to enable MatrixVariable in Spring MVC [SPR-11331]

[spring-projects-issues]

leebm opened SPR-11331 and commented

example:

@RequestMapping(value = "/hello/{mes}", method = RequestMethod.GET)
	public String hello(
			@PathVariable String mes, Model model, 
			@MatrixVariable int age) {
		model.addAttribute("mes", mes);
		model.addAttribute("age", age);
		return "hello";
	}

url: http://localhost/s3/hello/hi;age=21
this url can't be matched.

---

Affects: 3.2.6

Issue Links:

• MatrixVariables do not work [SPR-11036] #15664 MatrixVariables do not work

Referenced from: commits cfa78a9, 175aa86, 98174e1

Backported to: 3.2.7

[spring-projects-issues]

Stéphane Nicoll commented

I just replicated this code in a test case and it works with master (4.0.1.BUILD-SNAPSHOT).

I am guessing that if you are referring to @MatrixVariable, the same code works without it (i.e. by removing the age parameter and trying to resolve http://localhost/s3/hello/hi). If that's the case, could you please provide the version of Spring that you are using?

Thanks!

[spring-projects-issues]

leebm commented

thanks yours comment!

i'm try this url "http://localhost/s3/hello/hi",but it's still not work.

i'm change to this: @MatrixVariable(defaultValue = "1") int age, url can matched. but "age=21",this age value is not binded.

i'm use spring version 3.2.6.RELEASE.

[spring-projects-issues]

Stéphane Nicoll commented

This might be a duplicate of #15664. If you're not using JavaConfig, please make sure that you have enabled the feature:

<mvc:annotation-driven enableMatrixVariables="true"> ...

If that still does not work for you, I might need a bit more info and some representative logs with the full stacktrace that you get.

Thanks!

[spring-projects-issues]

leebm commented

use this config,it's worked ok!

i read docs/3.2.6.RELEASE part of 17.3.2 Matrix Variables,but any content with this config.if enableMatrixVariables default
false,should has some tip?

thank you very much! :)

[spring-projects-issues]

Stéphane Nicoll commented

Yes, you are right, the documentation definitely needs an update in that area. Thanks.

[spring-projects-issues]

Stéphane Nicoll commented

See PR #447

[spring-projects-issues]

leebm commented

ye,it's nice!