You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by re...@apache.org on 2020/01/24 00:38:59 UTC
[cxf] branch 3.3.x-fixes updated (24138a2 -> 59c47a6)
This is an automated email from the ASF dual-hosted git repository.
reta pushed a change to branch 3.3.x-fixes
in repository https://gitbox.apache.org/repos/asf/cxf.git.
from 24138a2 [CXF-7952]MTOM + schema validation creates invalid SOAP FAULT response
new 5a2525c CXF-8200: Use name() instead of toString() for enums when generating WADL
new 59c47a6 Recording .gitmergeinfo Changes
The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.gitmergeinfo | 7 +++++++
.../org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java | 2 +-
.../wadl/{AbstractStore.java => BookOrderEnum.java} | 17 +++++++++--------
.../java/org/apache/cxf/jaxrs/model/wadl/BookStore.java | 1 +
.../apache/cxf/jaxrs/model/wadl/WadlGeneratorTest.java | 4 +++-
5 files changed, 21 insertions(+), 10 deletions(-)
copy rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/{AbstractStore.java => BookOrderEnum.java} (78%)
[cxf] 02/02: Recording .gitmergeinfo Changes
Posted by re...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
reta pushed a commit to branch 3.3.x-fixes
in repository https://gitbox.apache.org/repos/asf/cxf.git
commit 59c47a66c7a584bbae945af8aeecf6e8644ed17d
Author: reta <dr...@gmail.com>
AuthorDate: Thu Jan 23 19:38:20 2020 -0500
Recording .gitmergeinfo Changes
---
.gitmergeinfo | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/.gitmergeinfo b/.gitmergeinfo
index 71f30a7..3e7822f 100644
--- a/.gitmergeinfo
+++ b/.gitmergeinfo
@@ -112,10 +112,14 @@ B fab7fc9e2dfbe32cc9988b16f2aa03e4bfff28e3
B fcd22cd026bc94a95fa19210df193df753afe4d1
M 09ddfdeaef6f32537dba23fa6d7ef36992b3217b
M 160f50ed9bd9391b61417716b3a67653c9e81a98
+M 180578d01f361d77ce7f42dd760fa8d5f245ce7e
M 2275d49772676ac72bc14ebbc3173719cd72e635
M 325ed0436bc9cfa9150b3504a54b5911b47e3266
M 3b119ca6630f1384b93cd718d0cd95ccabc81e91
+M 3dfe11c866073ff93951dac2b94e0f8e8e464b27
+M 464e41a12dea11a00c86a370b31d7a7f009cb8e9
M 4699995c6ef392a98615382a5824dda0d1bbe9bf
+M 49c1ca37a066dbfe0f4f32313da7ee4cdce61a15
M 528b35178ec31860cf5ad2095d6f3cc3c951d914
M 55abc6f8416fd81f26627fa42a37f903f9ef7965
M 589e321622b72ae57fab910d76b3e4fb0c160979
@@ -125,6 +129,7 @@ M 61aeaff2d747379c64cdad40f82cf51e3bfaecf6
M 6b7e50b87d74dc6b7d1e830b373c7e30c04e14a2
M 7b50181ebc445bb84d3463b284643432399bc0f8
M 7ef814556d727c147d6f625cbb1170edfd24a752
+M 95c3c37092751b1d00b6f2c26642f18ebcbba878
M 9c462e9bd614d529f0a2fd86490243ea1c858651
M a4fc3a0462f09dd6c6c920385b67e621d7e36399
M a61ca44ab50c1a3fc20fbaafe89b372809e0f49b
@@ -139,5 +144,7 @@ M d8619b13cd9ff375590946c1349f6fe374959f30
M d875ceee917237f5c344df989947aff025d45d4f
M e1b81269089d6e7cb53e949a9fde629d37bce421
M e72ad770c9a8f4439b93e2c19e61c43562ac5877
+M f22022b1f094efdcc81b63d9f3f10b85894fcdb8
M f3e2a52faec1210b1a45eaa36e3812ca03a4969d
+M f5899862b61bc39a7e2bb7945071ebfa3e6a8153
M f6ee8880f961974da0aa2406d415627f6a382aaa
[cxf] 01/02: CXF-8200: Use name() instead of toString() for enums
when generating WADL
Posted by re...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
reta pushed a commit to branch 3.3.x-fixes
in repository https://gitbox.apache.org/repos/asf/cxf.git
commit 5a2525c2c537040b31fd7646644c49bd9b59d505
Author: reta <dr...@gmail.com>
AuthorDate: Thu Jan 23 19:16:16 2020 -0500
CXF-8200: Use name() instead of toString() for enums when generating WADL
(cherry picked from commit 1fb179ff05138dcd34a21bca20dc5fe381d56301)
---
.../apache/cxf/jaxrs/model/wadl/WadlGenerator.java | 2 +-
.../apache/cxf/jaxrs/model/wadl/BookOrderEnum.java | 34 ++++++++++++++++++++++
.../org/apache/cxf/jaxrs/model/wadl/BookStore.java | 1 +
.../cxf/jaxrs/model/wadl/WadlGeneratorTest.java | 4 ++-
4 files changed, 39 insertions(+), 2 deletions(-)
diff --git a/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java b/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java
index 1827e81..229bea0 100644
--- a/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java
+++ b/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java
@@ -1013,7 +1013,7 @@ public class WadlGenerator implements ContainerRequestFilter {
try {
Method m = enumClass.getMethod("values", new Class<?>[] {});
Object[] values = (Object[])m.invoke(null, new Object[] {});
- m = enumClass.getMethod("toString", new Class<?>[] {});
+ m = enumClass.getMethod("name", new Class<?>[] {});
for (Object o : values) {
String str = (String)m.invoke(o, new Object[] {});
sb.append("<option value=\"").append(str).append("\"/>");
diff --git a/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/BookOrderEnum.java b/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/BookOrderEnum.java
new file mode 100644
index 0000000..0328d68
--- /dev/null
+++ b/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/BookOrderEnum.java
@@ -0,0 +1,34 @@
+/**
+ * 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.model.wadl;
+
+public enum BookOrderEnum {
+ INVOICED("Invoiced"),
+ NOT_INVOICED("Not invoiced");
+
+ private final String desc;
+
+ BookOrderEnum(String desc) {
+ this.desc = desc;
+ }
+
+ public String toString() {
+ return desc;
+ }
+}
diff --git a/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/BookStore.java b/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/BookStore.java
index 98a37f5..fa913a5 100644
--- a/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/BookStore.java
+++ b/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/BookStore.java
@@ -125,6 +125,7 @@ public class BookStore extends AbstractStore<Book> implements BookDescription {
@CookieParam("cid") int cookieId,
@QueryParam("provider.bar") int queryParam,
@QueryParam("bookstate") BookEnum state,
+ @QueryParam("orderstatus") BookOrderEnum status,
@QueryParam("a") List<String> queryList,
@Context HttpHeaders headers,
@Description("InputBook")
diff --git a/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/WadlGeneratorTest.java b/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/WadlGeneratorTest.java
index 4a5cbf3..30a9da3 100644
--- a/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/WadlGeneratorTest.java
+++ b/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/WadlGeneratorTest.java
@@ -568,11 +568,13 @@ public class WadlGeneratorTest {
checkDocs(requestEls.get(0), "", "Request", "");
- verifyParameters(requestEls.get(0), 4,
+ verifyParameters(requestEls.get(0), 5,
new Param("hid", "header", "xs:int"),
new Param("provider.bar", "query", "xs:int"),
new Param("bookstate", "query", "xs:string",
new HashSet<>(Arrays.asList("NEW", "USED", "OLD"))),
+ new Param("orderstatus", "query", "xs:string",
+ new HashSet<>(Arrays.asList("INVOICED", "NOT_INVOICED"))),
new Param("a", "query", "xs:string", true));
verifyXmlJsonRepresentations(requestEls.get(0), book2El, "InputBook");