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 2016/04/07 19:14:19 UTC

[1/2] cxf git commit: [CXF-6861] Prototyping JAXBElementTypedProvider

Repository: cxf
Updated Branches:
  refs/heads/master 53d7ad07c -> 55976e149


[CXF-6861] Prototyping JAXBElementTypedProvider


Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/ade81e37
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/ade81e37
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/ade81e37

Branch: refs/heads/master
Commit: ade81e37b5172085b35b92f33a28f99832c968da
Parents: 7cccdcf
Author: Sergey Beryozkin <sb...@gmail.com>
Authored: Thu Apr 7 18:13:30 2016 +0100
Committer: Sergey Beryozkin <sb...@gmail.com>
Committed: Thu Apr 7 18:13:30 2016 +0100

----------------------------------------------------------------------
 .../provider/JAXBElementTypedProvider.java      | 48 +++++++++++++++++++
 .../cxf/jaxrs/provider/ProviderFactory.java     |  3 +-
 .../cxf/jaxrs/provider/ProviderFactoryTest.java | 50 ++++++++++----------
 3 files changed, 75 insertions(+), 26 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf/blob/ade81e37/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/JAXBElementTypedProvider.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/JAXBElementTypedProvider.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/JAXBElementTypedProvider.java
new file mode 100644
index 0000000..388035f
--- /dev/null
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/JAXBElementTypedProvider.java
@@ -0,0 +1,48 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package org.apache.cxf.jaxrs.provider;
+
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.OutputStream;
+import java.lang.annotation.Annotation;
+import java.lang.reflect.Type;
+
+import javax.ws.rs.core.MediaType;
+import javax.ws.rs.core.MultivaluedMap;
+import javax.ws.rs.ext.MessageBodyReader;
+import javax.ws.rs.ext.MessageBodyWriter;
+import javax.xml.bind.JAXBElement;
+
+@SuppressWarnings("rawtypes")
+public class JAXBElementTypedProvider extends JAXBElementProvider<JAXBElement>  
+    implements MessageBodyWriter<JAXBElement>, MessageBodyReader<JAXBElement> {
+    
+    public JAXBElement readFrom(Class<JAXBElement> type, Type genericType, Annotation[] anns, MediaType mt, 
+        MultivaluedMap<String, String> headers, InputStream is) 
+        throws IOException {
+        return super.readFrom(type, genericType, anns, mt, headers, is);
+    }
+    public void writeTo(JAXBElement<?> obj, Class<?> cls, Type genericType, Annotation[] anns,  
+        MediaType m, MultivaluedMap<String, Object> headers, OutputStream os) 
+        throws IOException {
+        super.writeTo(obj, cls, genericType, anns, m, headers, os);
+    }
+}

http://git-wip-us.apache.org/repos/asf/cxf/blob/ade81e37/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ProviderFactory.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ProviderFactory.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ProviderFactory.java
index c0bdd1f..f89c6fb 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ProviderFactory.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ProviderFactory.java
@@ -134,7 +134,8 @@ public abstract class ProviderFactory {
                      new FormEncodingProvider<Object>(),
                      new StringTextProvider(),
                      new PrimitiveTextProvider<Object>(),
-                     createProvider(JAXB_PROVIDER_NAME),
+                     new JAXBElementProvider<Object>(),
+                     new JAXBElementTypedProvider(),
                      new MultipartProvider());
         Object prop = factory.getBus().getProperty("skip.default.json.provider.registration");
         if (!PropertyUtils.isTrue(prop)) {

http://git-wip-us.apache.org/repos/asf/cxf/blob/ade81e37/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/ProviderFactoryTest.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/ProviderFactoryTest.java b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/ProviderFactoryTest.java
index 2f66a70..8da87d5 100644
--- a/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/ProviderFactoryTest.java
+++ b/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/provider/ProviderFactoryTest.java
@@ -96,9 +96,9 @@ public class ProviderFactoryTest extends Assert {
         WildcardReader2 reader2 = new WildcardReader2();
         pf.registerUserProvider(reader2);
         List<ProviderInfo<MessageBodyReader<?>>> readers = pf.getMessageReaders();
-        assertEquals(10, readers.size());
-        assertSame(reader1, readers.get(6).getProvider());
-        assertSame(reader2, readers.get(7).getProvider());
+        assertEquals(11, readers.size());
+        assertSame(reader1, readers.get(7).getProvider());
+        assertSame(reader2, readers.get(8).getProvider());
     }
     
     @Test
@@ -124,13 +124,13 @@ public class ProviderFactoryTest extends Assert {
         
         // writers
         List<ProviderInfo<MessageBodyWriter<?>>> writers = pf.getMessageWriters();
-        assertEquals(8, writers.size());
-        Object lastWriter = writers.get(7).getProvider();
+        assertEquals(9, writers.size());
+        Object lastWriter = writers.get(8).getProvider();
         assertTrue(lastWriter instanceof StringTextProvider);
         //readers
         List<ProviderInfo<MessageBodyReader<?>>> readers = pf.getMessageReaders();
-        assertEquals(8, readers.size());
-        Object lastReader = readers.get(7).getProvider();
+        assertEquals(9, readers.size());
+        Object lastReader = readers.get(8).getProvider();
         assertTrue(lastReader instanceof StringTextProvider);
     }
     @Test
@@ -154,13 +154,13 @@ public class ProviderFactoryTest extends Assert {
         
         // writers
         List<ProviderInfo<MessageBodyWriter<?>>> writers = pf.getMessageWriters();
-        assertEquals(8, writers.size());
-        Object lastWriter = writers.get(7).getProvider();
+        assertEquals(9, writers.size());
+        Object lastWriter = writers.get(8).getProvider();
         assertTrue(lastWriter instanceof StringTextProvider);
         //readers
         List<ProviderInfo<MessageBodyReader<?>>> readers = pf.getMessageReaders();
-        assertEquals(8, readers.size());
-        Object lastReader = readers.get(7).getProvider();
+        assertEquals(9, readers.size());
+        Object lastReader = readers.get(8).getProvider();
         assertTrue(lastReader instanceof StringTextProvider);
     }
     @SuppressWarnings("rawtypes")
@@ -188,13 +188,13 @@ public class ProviderFactoryTest extends Assert {
         
         // writers
         List<ProviderInfo<MessageBodyWriter<?>>> writers = pf.getMessageWriters();
-        assertEquals(8, writers.size());
-        Object lastWriter = writers.get(7).getProvider();
+        assertEquals(9, writers.size());
+        Object lastWriter = writers.get(8).getProvider();
         assertFalse(lastWriter instanceof StringTextProvider);
         //readers
         List<ProviderInfo<MessageBodyReader<?>>> readers = pf.getMessageReaders();
-        assertEquals(8, readers.size());
-        Object lastReader = readers.get(7).getProvider();
+        assertEquals(9, readers.size());
+        Object lastReader = readers.get(8).getProvider();
         assertTrue(lastReader instanceof StringTextProvider);
     }
     @Test
@@ -221,13 +221,13 @@ public class ProviderFactoryTest extends Assert {
         
         // writers
         List<ProviderInfo<MessageBodyWriter<?>>> writers = pf.getMessageWriters();
-        assertEquals(8, writers.size());
-        Object lastWriter = writers.get(7).getProvider();
+        assertEquals(9, writers.size());
+        Object lastWriter = writers.get(8).getProvider();
         assertTrue(lastWriter instanceof StringTextProvider);
         //readers
         List<ProviderInfo<MessageBodyReader<?>>> readers = pf.getMessageReaders();
-        assertEquals(8, readers.size());
-        Object lastReader = readers.get(7).getProvider();
+        assertEquals(9, readers.size());
+        Object lastReader = readers.get(8).getProvider();
         assertFalse(lastReader instanceof StringTextProvider);
     }
     
@@ -240,9 +240,9 @@ public class ProviderFactoryTest extends Assert {
         ProviderFactory pf = ServerProviderFactory.createInstance(bus);
         pf.registerUserProvider(wc2);
         List<ProviderInfo<MessageBodyReader<?>>> readers = pf.getMessageReaders();
-        assertEquals(10, readers.size());
-        assertSame(wc2, readers.get(6).getProvider());
-        assertSame(wc1, readers.get(7).getProvider());
+        assertEquals(11, readers.size());
+        assertSame(wc2, readers.get(7).getProvider());
+        assertSame(wc1, readers.get(8).getProvider());
     }
     @Test
     public void testCustomProviderSortingWithBus2() {
@@ -253,9 +253,9 @@ public class ProviderFactoryTest extends Assert {
         ProviderFactory pf = ServerProviderFactory.createInstance(bus);
         pf.registerUserProvider(wc1);
         List<ProviderInfo<MessageBodyReader<?>>> readers = pf.getMessageReaders();
-        assertEquals(10, readers.size());
-        assertSame(wc1, readers.get(6).getProvider());
-        assertSame(wc2, readers.get(7).getProvider());
+        assertEquals(11, readers.size());
+        assertSame(wc1, readers.get(7).getProvider());
+        assertSame(wc2, readers.get(8).getProvider());
     }
     
     @Test


[2/2] cxf git commit: Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/cxf

Posted by se...@apache.org.
Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/cxf


Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/55976e14
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/55976e14
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/55976e14

Branch: refs/heads/master
Commit: 55976e149cb082d6b0fc1e35dc13f10406204b94
Parents: ade81e3 53d7ad0
Author: Sergey Beryozkin <sb...@gmail.com>
Authored: Thu Apr 7 18:14:00 2016 +0100
Committer: Sergey Beryozkin <sb...@gmail.com>
Committed: Thu Apr 7 18:14:00 2016 +0100

----------------------------------------------------------------------
 .../oauth2/filters/OAuthRequestFilter.java      |  2 +-
 .../jaxrs/JAXRSClientServerSpringBookTest.java  | 35 ++++++++++----------
 2 files changed, 19 insertions(+), 18 deletions(-)
----------------------------------------------------------------------