Repository: camel Updated Branches: refs/heads/master ab7d838b4 -> af8c3283a
Fixed CS. Break up to smaller methods. Project: http://git-wip-us.apache.org/repos/asf/camel/repo Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/b8e2604d Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/b8e2604d Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/b8e2604d Branch: refs/heads/master Commit: b8e2604d92c9ef7454a67452a4154791433762a6 Parents: ab7d838 Author: Claus Ibsen <davscl...@apache.org> Authored: Fri Mar 4 13:51:57 2016 +0100 Committer: Claus Ibsen <davscl...@apache.org> Committed: Fri Mar 4 13:51:57 2016 +0100 ---------------------------------------------------------------------- .../apache/camel/swagger/RestSwaggerReader.java | 230 ++++++++++--------- 1 file changed, 119 insertions(+), 111 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/camel/blob/b8e2604d/components/camel-swagger-java/src/main/java/org/apache/camel/swagger/RestSwaggerReader.java ---------------------------------------------------------------------- diff --git a/components/camel-swagger-java/src/main/java/org/apache/camel/swagger/RestSwaggerReader.java b/components/camel-swagger-java/src/main/java/org/apache/camel/swagger/RestSwaggerReader.java index bc01e37..e3e65ef 100644 --- a/components/camel-swagger-java/src/main/java/org/apache/camel/swagger/RestSwaggerReader.java +++ b/components/camel-swagger-java/src/main/java/org/apache/camel/swagger/RestSwaggerReader.java @@ -151,6 +151,10 @@ public class RestSwaggerReader { appendModels(clazz, swagger); } + doParseVerbs(swagger, rest, camelContextId, verbs, pathAsTag); + } + + private void doParseVerbs(Swagger swagger, RestDefinition rest, String camelContextId, List<VerbDefinition> verbs, String pathAsTag) { // used during gathering of apis List<Path> paths = new ArrayList<>(); @@ -291,128 +295,132 @@ public class RestSwaggerReader { } // enrich with configured response messages from the rest-dsl - for (RestOperationResponseMsgDefinition msg : verb.getResponseMsgs()) { - Response response = null; - if (op.getResponses() != null) { - response = op.getResponses().get(msg.getCode()); - } - if (response == null) { - response = new Response(); - } - if (ObjectHelper.isNotEmpty(msg.getResponseModel())) { - Property prop = modelTypeAsProperty(msg.getResponseModel(), swagger); - response.setSchema(prop); - } - response.setDescription(msg.getMessage()); - - // add headers - if (msg.getHeaders() != null) { - for (RestOperationResponseHeaderDefinition header : msg.getHeaders()) { - String name = header.getName(); - String type = header.getDataType(); - if ("string".equals(type)) { - StringProperty sp = new StringProperty(); - sp.setName(name); - sp.setDescription(header.getDescription()); - if (header.getAllowableValues() != null) { - sp.setEnum(header.getAllowableValues()); + doParseResponseMessages(swagger, verb, op); + + // add path + swagger.path(opPath, path); + } + } + + private void doParseResponseMessages(Swagger swagger, VerbDefinition verb, Operation op) { + for (RestOperationResponseMsgDefinition msg : verb.getResponseMsgs()) { + Response response = null; + if (op.getResponses() != null) { + response = op.getResponses().get(msg.getCode()); + } + if (response == null) { + response = new Response(); + } + if (ObjectHelper.isNotEmpty(msg.getResponseModel())) { + Property prop = modelTypeAsProperty(msg.getResponseModel(), swagger); + response.setSchema(prop); + } + response.setDescription(msg.getMessage()); + + // add headers + if (msg.getHeaders() != null) { + for (RestOperationResponseHeaderDefinition header : msg.getHeaders()) { + String name = header.getName(); + String type = header.getDataType(); + if ("string".equals(type)) { + StringProperty sp = new StringProperty(); + sp.setName(name); + sp.setDescription(header.getDescription()); + if (header.getAllowableValues() != null) { + sp.setEnum(header.getAllowableValues()); + } + response.addHeader(name, sp); + } else if ("int".equals(type) || "integer".equals(type)) { + IntegerProperty ip = new IntegerProperty(); + ip.setName(name); + ip.setDescription(header.getDescription()); + + List<Integer> values; + if (!header.getAllowableValues().isEmpty()) { + values = new ArrayList<Integer>(); + for (String text : header.getAllowableValues()) { + values.add(Integer.valueOf(text)); } - response.addHeader(name, sp); - } else if ("int".equals(type) || "integer".equals(type)) { - IntegerProperty ip = new IntegerProperty(); - ip.setName(name); - ip.setDescription(header.getDescription()); - - List<Integer> values; - if (!header.getAllowableValues().isEmpty()) { - values = new ArrayList<Integer>(); - for (String text : header.getAllowableValues()) { - values.add(Integer.valueOf(text)); - } - ip.setEnum(values); + ip.setEnum(values); + } + response.addHeader(name, ip); + } else if ("long".equals(type)) { + LongProperty lp = new LongProperty(); + lp.setName(name); + lp.setDescription(header.getDescription()); + + List<Long> values; + if (!header.getAllowableValues().isEmpty()) { + values = new ArrayList<Long>(); + for (String text : header.getAllowableValues()) { + values.add(Long.valueOf(text)); } - response.addHeader(name, ip); - } else if ("long".equals(type)) { - LongProperty lp = new LongProperty(); - lp.setName(name); - lp.setDescription(header.getDescription()); - - List<Long> values; - if (!header.getAllowableValues().isEmpty()) { - values = new ArrayList<Long>(); - for (String text : header.getAllowableValues()) { - values.add(Long.valueOf(text)); - } - lp.setEnum(values); + lp.setEnum(values); + } + response.addHeader(name, lp); + } else if ("float".equals(type)) { + FloatProperty lp = new FloatProperty(); + lp.setName(name); + lp.setDescription(header.getDescription()); + + List<Float> values; + if (!header.getAllowableValues().isEmpty()) { + values = new ArrayList<Float>(); + for (String text : header.getAllowableValues()) { + values.add(Float.valueOf(text)); } - response.addHeader(name, lp); - } else if ("float".equals(type)) { - FloatProperty lp = new FloatProperty(); - lp.setName(name); - lp.setDescription(header.getDescription()); - - List<Float> values; - if (!header.getAllowableValues().isEmpty()) { - values = new ArrayList<Float>(); - for (String text : header.getAllowableValues()) { - values.add(Float.valueOf(text)); - } - lp.setEnum(values); + lp.setEnum(values); + } + response.addHeader(name, lp); + } else if ("double".equals(type)) { + DoubleProperty dp = new DoubleProperty(); + dp.setName(name); + dp.setDescription(header.getDescription()); + + List<Double> values; + if (!header.getAllowableValues().isEmpty()) { + values = new ArrayList<Double>(); + for (String text : header.getAllowableValues()) { + values.add(Double.valueOf(text)); } - response.addHeader(name, lp); - } else if ("double".equals(type)) { - DoubleProperty dp = new DoubleProperty(); - dp.setName(name); - dp.setDescription(header.getDescription()); - - List<Double> values; - if (!header.getAllowableValues().isEmpty()) { - values = new ArrayList<Double>(); - for (String text : header.getAllowableValues()) { - values.add(Double.valueOf(text)); - } - dp.setEnum(values); + dp.setEnum(values); + } + response.addHeader(name, dp); + } else if ("boolean".equals(type)) { + BooleanProperty bp = new BooleanProperty(); + bp.setName(name); + bp.setDescription(header.getDescription()); + response.addHeader(name, bp); + } else if ("array".equals(type)) { + ArrayProperty ap = new ArrayProperty(); + ap.setName(name); + ap.setDescription(header.getDescription()); + if (header.getArrayType() != null) { + if (header.getArrayType().equalsIgnoreCase("string")) { + ap.setItems(new StringProperty()); } - response.addHeader(name, dp); - } else if ("boolean".equals(type)) { - BooleanProperty bp = new BooleanProperty(); - bp.setName(name); - bp.setDescription(header.getDescription()); - response.addHeader(name, bp); - } else if ("array".equals(type)) { - ArrayProperty ap = new ArrayProperty(); - ap.setName(name); - ap.setDescription(header.getDescription()); - if (header.getArrayType() != null) { - if (header.getArrayType().equalsIgnoreCase("string")) { - ap.setItems(new StringProperty()); - } - if (header.getArrayType().equalsIgnoreCase("int") || header.getArrayType().equalsIgnoreCase("integer")) { - ap.setItems(new IntegerProperty()); - } - if (header.getArrayType().equalsIgnoreCase("long")) { - ap.setItems(new LongProperty()); - } - if (header.getArrayType().equalsIgnoreCase("float")) { - ap.setItems(new FloatProperty()); - } - if (header.getArrayType().equalsIgnoreCase("double")) { - ap.setItems(new DoubleProperty()); - } - if (header.getArrayType().equalsIgnoreCase("boolean")) { - ap.setItems(new BooleanProperty()); - } + if (header.getArrayType().equalsIgnoreCase("int") || header.getArrayType().equalsIgnoreCase("integer")) { + ap.setItems(new IntegerProperty()); + } + if (header.getArrayType().equalsIgnoreCase("long")) { + ap.setItems(new LongProperty()); + } + if (header.getArrayType().equalsIgnoreCase("float")) { + ap.setItems(new FloatProperty()); + } + if (header.getArrayType().equalsIgnoreCase("double")) { + ap.setItems(new DoubleProperty()); + } + if (header.getArrayType().equalsIgnoreCase("boolean")) { + ap.setItems(new BooleanProperty()); } - response.addHeader(name, ap); } + response.addHeader(name, ap); } } - - op.addResponse(msg.getCode(), response); } - // add path - swagger.path(opPath, path); + op.addResponse(msg.getCode(), response); } }