You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by se...@apache.org on 2013/08/26 14:55:39 UTC
svn commit: r1517531 - in /cxf/trunk/rt/frontend/jaxrs/src:
main/java/org/apache/cxf/jaxrs/model/ClassResourceInfo.java
test/java/org/apache/cxf/jaxrs/model/ClassResourceInfoTest.java
Author: sergeyb
Date: Mon Aug 26 12:55:39 2013
New Revision: 1517531
URL: http://svn.apache.org/r1517531
Log:
[CXF-5065] Minor update for subresources not to lose their own Produces/Consumes annotations
Modified:
cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/model/ClassResourceInfo.java
cxf/trunk/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/model/ClassResourceInfoTest.java
Modified: cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/model/ClassResourceInfo.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/model/ClassResourceInfo.java?rev=1517531&r1=1517530&r2=1517531&view=diff
==============================================================================
--- cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/model/ClassResourceInfo.java (original)
+++ cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/model/ClassResourceInfo.java Mon Aug 26 12:55:39 2013
@@ -251,24 +251,24 @@ public class ClassResourceInfo extends B
}
public List<MediaType> getProduceMime() {
- if (parent == null) {
- if (producesTypes != null) {
- return JAXRSUtils.parseMediaTypes(producesTypes);
- }
- return JAXRSUtils.getProduceTypes(
- AnnotationUtils.getClassAnnotation(getServiceClass(), Produces.class));
+ if (producesTypes != null) {
+ return JAXRSUtils.parseMediaTypes(producesTypes);
+ }
+ Produces produces = AnnotationUtils.getClassAnnotation(getServiceClass(), Produces.class);
+ if (produces != null || parent == null) {
+ return JAXRSUtils.getProduceTypes(produces);
} else {
return parent.getProduceMime();
}
}
public List<MediaType> getConsumeMime() {
- if (parent == null) {
- if (consumesTypes != null) {
- return JAXRSUtils.parseMediaTypes(consumesTypes);
- }
- return JAXRSUtils.getConsumeTypes(
- AnnotationUtils.getClassAnnotation(getServiceClass(), Consumes.class));
+ if (consumesTypes != null) {
+ return JAXRSUtils.parseMediaTypes(consumesTypes);
+ }
+ Consumes consumes = AnnotationUtils.getClassAnnotation(getServiceClass(), Consumes.class);
+ if (consumes != null || parent == null) {
+ return JAXRSUtils.getConsumeTypes(consumes);
} else {
return parent.getConsumeMime();
}
Modified: cxf/trunk/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/model/ClassResourceInfoTest.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/model/ClassResourceInfoTest.java?rev=1517531&r1=1517530&r2=1517531&view=diff
==============================================================================
--- cxf/trunk/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/model/ClassResourceInfoTest.java (original)
+++ cxf/trunk/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/model/ClassResourceInfoTest.java Mon Aug 26 12:55:39 2013
@@ -93,6 +93,24 @@ public class ClassResourceInfoTest exten
}
}
+ @Produces("test/foo")
+ static class TestClassWithProduces extends TestClass1 {
+ @GET
+ public void getIt() {
+
+ }
+
+ @Path("/same")
+ public TestClassWithProduces getThis() {
+ return this;
+ }
+
+ @Path("sub")
+ public TestClass3 getTestClass3() {
+ return new TestClass3();
+ }
+ }
+
static class TestClass3 {
@Context HttpServletRequest req;
@Context HttpServletResponse res;
@@ -219,6 +237,16 @@ public class ClassResourceInfoTest exten
}
@Test
+ public void testSubresourceWithProduces() {
+ ClassResourceInfo parent = ResourceUtils.createClassResourceInfo(
+ TestClass2.class, TestClass2.class, true, true);
+ ClassResourceInfo c = ResourceUtils.createClassResourceInfo(
+ TestClassWithProduces.class, TestClassWithProduces.class, true, true);
+ c.setParent(parent);
+ assertEquals("test/foo", c.getProduceMime().get(0).toString());
+ }
+
+ @Test
public void testNameBindings() {
Application app = new TestApplication();
JAXRSServerFactoryBean bean = ResourceUtils.createApplication(app, true, true);