#617 - AnnotatedParametersParameterAccessor now considers @AliasFor declarations.

We're now using SynthesizingMethodParameter to correctly support annotation attributes that use @AliasFor.

Original pull request: #621.

Author: gregturn

src/main/java/org/springframework/hateoas/core/MethodParameters.java

@@ -24,6 +24,7 @@
 import org.springframework.core.DefaultParameterNameDiscoverer;
 import org.springframework.core.MethodParameter;
 import org.springframework.core.ParameterNameDiscoverer;
+import org.springframework.core.annotation.SynthesizingMethodParameter;
 import org.springframework.util.Assert;
 import org.springframework.util.ConcurrentReferenceHashMap;
 
@@ -151,7 +152,7 @@ public List<MethodParameter> getParametersWith(Class<? extends Annotation> annot
 	 * 
 	 * @author Oliver Gierke
 	 */
-	private static class AnnotationNamingMethodParameter extends MethodParameter {
+	private static class AnnotationNamingMethodParameter extends SynthesizingMethodParameter {
 
 		private final AnnotationAttribute attribute;
 		private String name;

src/test/java/org/springframework/hateoas/mvc/ControllerLinkBuilderUnitTest.java

@@ -580,6 +580,16 @@ public void considersOptionalWithValueMethodParameterOptional() {
 		assertThat(link.getHref(), endsWith("?value=1"));
 	}
 
+	/**
+	 * @see #617
+	 */
+	@Test
+	public void alternativePathVariableParameter() {
+
+		Link link = linkTo(methodOn(ControllerWithMethods.class).methodWithAlternatePathVariable("bar")).withSelfRel();
+		assertThat(link.getHref(), is("http://localhost/something/bar/foo"));
+	}
+
 	private static UriComponents toComponents(Link link) {
 		return UriComponentsBuilder.fromUriString(link.expand().getHref()).build();
 	}
@@ -647,6 +657,11 @@ HttpEntity<Void> methodWithMultiValueRequestParams(@PathVariable String id, @Req
 			return null;
 		}
 
+		@RequestMapping(value = "/{id}/foo")
+		HttpEntity<Void> methodWithAlternatePathVariable(@PathVariable(name = "id") String otherId) {
+			return null;
+		}
+
 		@RequestMapping(value = "/foo")
 		HttpEntity<Void> methodForOptionalNextPage(@RequestParam(required = false) Integer offset) {
 			return null;

src/test/java/org/springframework/hateoas/mvc/DummyInvocationUtilsUnitTest.java

@@ -15,7 +15,11 @@
  */
 package org.springframework.hateoas.mvc;
 
+import static org.hamcrest.CoreMatchers.*;
+import static org.hamcrest.MatcherAssert.*;
+
 import org.junit.Test;
+import org.springframework.hateoas.Link;
 import org.springframework.hateoas.TestUtils;
 import org.springframework.hateoas.core.DummyInvocationUtils;
 import org.springframework.http.HttpEntity;
@@ -30,10 +34,17 @@
 public class DummyInvocationUtilsUnitTest extends TestUtils {
 
 	@Test
-	public void test() {
+	public void pathVariableWithDefaultParameter() {
+
+		Link link = ControllerLinkBuilder.linkTo(DummyInvocationUtils.methodOn(SampleController.class).someMethod(1L)).withSelfRel();
+		assertThat(link.getHref(), is("http://localhost/sample/1/foo"));
+	}
 
-		ControllerLinkBuilder.linkTo(DummyInvocationUtils.methodOn(SampleController.class).someMethod(1L));
+	@Test
+	public void pathVariableWithNameParameter() {
 
+		Link link = ControllerLinkBuilder.linkTo(DummyInvocationUtils.methodOn(SampleController.class).someOtherMethod(2L)).withSelfRel();
+		assertThat(link.getHref(), is("http://localhost/sample/2/bar"));
 	}
 
 	@RequestMapping("/sample")
@@ -43,5 +54,10 @@ static class SampleController {
 		HttpEntity<Void> someMethod(@PathVariable("id") Long id) {
 			return new ResponseEntity<Void>(HttpStatus.OK);
 		}
+
+		@RequestMapping("/{otherName}/bar")
+		HttpEntity<Void> someOtherMethod(@PathVariable(name = "otherName") Long id) {
+			return new ResponseEntity<Void>(HttpStatus.OK);
+		}
 	}
 }