Repository: olingo-odata4 Updated Branches: refs/heads/master 80d046b31 -> 14278ac18
[OLINGO-936] clean-up Signed-off-by: Christian Amend <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/olingo-odata4/repo Commit: http://git-wip-us.apache.org/repos/asf/olingo-odata4/commit/20a00704 Tree: http://git-wip-us.apache.org/repos/asf/olingo-odata4/tree/20a00704 Diff: http://git-wip-us.apache.org/repos/asf/olingo-odata4/diff/20a00704 Branch: refs/heads/master Commit: 20a00704d8c51a08c615dba6f3286ea6d53d2f45 Parents: 80d046b Author: Klaus Straubinger <[email protected]> Authored: Thu Sep 15 13:54:36 2016 +0200 Committer: Christian Amend <[email protected]> Committed: Thu Sep 15 14:05:51 2016 +0200 ---------------------------------------------------------------------- .../olingo/fit/tecsvc/client/BasicITCase.java | 3 --- .../olingo/fit/tecsvc/client/ConformanceITCase.java | 2 -- .../olingo/fit/tecsvc/http/BasicStreamITCase.java | 1 - .../client/core/serialization/JsonDeserializer.java | 1 - .../server/core/debug/ExpressionJsonVisitor.java | 9 ++++++++- .../core/serializer/json/ODataJsonSerializer.java | 2 +- .../core/serializer/xml/ODataXmlSerializer.java | 2 +- .../tecsvc/processor/TechnicalEntityProcessor.java | 16 +++++++--------- .../expression/ExpressionVisitorImpl.java | 9 ++++++++- 9 files changed, 25 insertions(+), 20 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/olingo-odata4/blob/20a00704/fit/src/test/java/org/apache/olingo/fit/tecsvc/client/BasicITCase.java ---------------------------------------------------------------------- diff --git a/fit/src/test/java/org/apache/olingo/fit/tecsvc/client/BasicITCase.java b/fit/src/test/java/org/apache/olingo/fit/tecsvc/client/BasicITCase.java index 63f7fa0..8cef37d 100644 --- a/fit/src/test/java/org/apache/olingo/fit/tecsvc/client/BasicITCase.java +++ b/fit/src/test/java/org/apache/olingo/fit/tecsvc/client/BasicITCase.java @@ -69,8 +69,6 @@ import org.apache.olingo.client.api.domain.ClientObjectFactory; import org.apache.olingo.client.api.domain.ClientPrimitiveValue; import org.apache.olingo.client.api.domain.ClientProperty; import org.apache.olingo.client.api.domain.ClientServiceDocument; -import org.apache.olingo.client.api.domain.ClientSingleton; -import org.apache.olingo.client.api.domain.ClientValuable; import org.apache.olingo.client.api.domain.ClientValue; import org.apache.olingo.client.api.edm.xml.Reference; import org.apache.olingo.client.api.edm.xml.XMLMetadata; @@ -88,7 +86,6 @@ import org.apache.olingo.commons.api.ex.ODataError; import org.apache.olingo.commons.api.format.ContentType; import org.apache.olingo.commons.api.http.HttpHeader; import org.apache.olingo.commons.api.http.HttpStatusCode; -import static org.junit.Assert.assertNotNull; import org.junit.Ignore; import org.junit.Test; http://git-wip-us.apache.org/repos/asf/olingo-odata4/blob/20a00704/fit/src/test/java/org/apache/olingo/fit/tecsvc/client/ConformanceITCase.java ---------------------------------------------------------------------- diff --git a/fit/src/test/java/org/apache/olingo/fit/tecsvc/client/ConformanceITCase.java b/fit/src/test/java/org/apache/olingo/fit/tecsvc/client/ConformanceITCase.java index 7ac81a4..831499a 100644 --- a/fit/src/test/java/org/apache/olingo/fit/tecsvc/client/ConformanceITCase.java +++ b/fit/src/test/java/org/apache/olingo/fit/tecsvc/client/ConformanceITCase.java @@ -35,8 +35,6 @@ import org.apache.olingo.commons.api.edm.FullQualifiedName; import org.apache.olingo.commons.api.format.ContentType; import org.apache.olingo.commons.api.http.HttpHeader; import org.apache.olingo.commons.api.http.HttpStatusCode; -import static org.apache.olingo.fit.tecsvc.client.AbstractTecSvcITCase.SERVICE_NAMESPACE; -import static org.apache.olingo.fit.tecsvc.client.AbstractTecSvcITCase.SERVICE_URI; import static org.junit.Assert.assertEquals; import org.junit.Test; import static org.junit.Assert.assertNotNull; http://git-wip-us.apache.org/repos/asf/olingo-odata4/blob/20a00704/fit/src/test/java/org/apache/olingo/fit/tecsvc/http/BasicStreamITCase.java ---------------------------------------------------------------------- diff --git a/fit/src/test/java/org/apache/olingo/fit/tecsvc/http/BasicStreamITCase.java b/fit/src/test/java/org/apache/olingo/fit/tecsvc/http/BasicStreamITCase.java index 33eb4f5..8935f97 100644 --- a/fit/src/test/java/org/apache/olingo/fit/tecsvc/http/BasicStreamITCase.java +++ b/fit/src/test/java/org/apache/olingo/fit/tecsvc/http/BasicStreamITCase.java @@ -70,7 +70,6 @@ public class BasicStreamITCase extends AbstractBaseTestITCase { assertEquals(ContentType.APPLICATION_XML, ContentType.create(connection.getHeaderField(HttpHeader.CONTENT_TYPE))); final String content = IOUtils.toString(connection.getInputStream()); -System.out.println(content); assertTrue(content.contains("<m:element>[email protected]</m:element>" + "<m:element>[email protected]</m:element>" + "<m:element>[email protected]</m:element>")); http://git-wip-us.apache.org/repos/asf/olingo-odata4/blob/20a00704/lib/client-core/src/main/java/org/apache/olingo/client/core/serialization/JsonDeserializer.java ---------------------------------------------------------------------- diff --git a/lib/client-core/src/main/java/org/apache/olingo/client/core/serialization/JsonDeserializer.java b/lib/client-core/src/main/java/org/apache/olingo/client/core/serialization/JsonDeserializer.java index 8775f17..3691d8b 100644 --- a/lib/client-core/src/main/java/org/apache/olingo/client/core/serialization/JsonDeserializer.java +++ b/lib/client-core/src/main/java/org/apache/olingo/client/core/serialization/JsonDeserializer.java @@ -30,7 +30,6 @@ import java.util.Set; import java.util.regex.Matcher; import java.util.regex.Pattern; -import org.apache.commons.io.IOUtils; import org.apache.commons.lang3.StringUtils; import org.apache.olingo.client.api.data.ResWrap; import org.apache.olingo.client.api.serialization.ODataDeserializer; http://git-wip-us.apache.org/repos/asf/olingo-odata4/blob/20a00704/lib/server-core/src/main/java/org/apache/olingo/server/core/debug/ExpressionJsonVisitor.java ---------------------------------------------------------------------- diff --git a/lib/server-core/src/main/java/org/apache/olingo/server/core/debug/ExpressionJsonVisitor.java b/lib/server-core/src/main/java/org/apache/olingo/server/core/debug/ExpressionJsonVisitor.java index b93bb3f..cff320f 100644 --- a/lib/server-core/src/main/java/org/apache/olingo/server/core/debug/ExpressionJsonVisitor.java +++ b/lib/server-core/src/main/java/org/apache/olingo/server/core/debug/ExpressionJsonVisitor.java @@ -29,7 +29,14 @@ import org.apache.olingo.server.api.uri.UriResource; import org.apache.olingo.server.api.uri.UriResourceLambdaAll; import org.apache.olingo.server.api.uri.UriResourceLambdaAny; import org.apache.olingo.server.api.uri.UriResourcePartTyped; -import org.apache.olingo.server.api.uri.queryoption.expression.*; +import org.apache.olingo.server.api.uri.queryoption.expression.BinaryOperatorKind; +import org.apache.olingo.server.api.uri.queryoption.expression.Expression; +import org.apache.olingo.server.api.uri.queryoption.expression.ExpressionVisitException; +import org.apache.olingo.server.api.uri.queryoption.expression.ExpressionVisitor; +import org.apache.olingo.server.api.uri.queryoption.expression.Literal; +import org.apache.olingo.server.api.uri.queryoption.expression.Member; +import org.apache.olingo.server.api.uri.queryoption.expression.MethodKind; +import org.apache.olingo.server.api.uri.queryoption.expression.UnaryOperatorKind; import java.util.List; http://git-wip-us.apache.org/repos/asf/olingo-odata4/blob/20a00704/lib/server-core/src/main/java/org/apache/olingo/server/core/serializer/json/ODataJsonSerializer.java ---------------------------------------------------------------------- diff --git a/lib/server-core/src/main/java/org/apache/olingo/server/core/serializer/json/ODataJsonSerializer.java b/lib/server-core/src/main/java/org/apache/olingo/server/core/serializer/json/ODataJsonSerializer.java index 3d6aa68..57af7a9 100644 --- a/lib/server-core/src/main/java/org/apache/olingo/server/core/serializer/json/ODataJsonSerializer.java +++ b/lib/server-core/src/main/java/org/apache/olingo/server/core/serializer/json/ODataJsonSerializer.java @@ -683,7 +683,7 @@ public class ODataJsonSerializer extends AbstractODataSerializer { throws IOException, SerializerException{ json.writeStartObject(); String derivedName = property.getType(); - final EdmComplexType resolvedType = resolveComplexType(metadata, (EdmComplexType) type, derivedName); + final EdmComplexType resolvedType = resolveComplexType(metadata, type, derivedName); if (!isODataMetadataNone && !resolvedType.equals(type) || isODataMetadataFull) { json.writeStringField(Constants.JSON_TYPE, "#" + property.getType()); } http://git-wip-us.apache.org/repos/asf/olingo-odata4/blob/20a00704/lib/server-core/src/main/java/org/apache/olingo/server/core/serializer/xml/ODataXmlSerializer.java ---------------------------------------------------------------------- diff --git a/lib/server-core/src/main/java/org/apache/olingo/server/core/serializer/xml/ODataXmlSerializer.java b/lib/server-core/src/main/java/org/apache/olingo/server/core/serializer/xml/ODataXmlSerializer.java index 06bb13d..0f0a673 100644 --- a/lib/server-core/src/main/java/org/apache/olingo/server/core/serializer/xml/ODataXmlSerializer.java +++ b/lib/server-core/src/main/java/org/apache/olingo/server/core/serializer/xml/ODataXmlSerializer.java @@ -1307,7 +1307,7 @@ public class ODataXmlSerializer extends AbstractODataSerializer { || invalidCharacterReplacement == null || isUniCode == null || !isUniCode) { return value; } - String s = (String) value; + String s = value; StringBuilder result = null; for (int i = 0; i < s.length(); i++) { char c = s.charAt(i); http://git-wip-us.apache.org/repos/asf/olingo-odata4/blob/20a00704/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/processor/TechnicalEntityProcessor.java ---------------------------------------------------------------------- diff --git a/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/processor/TechnicalEntityProcessor.java b/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/processor/TechnicalEntityProcessor.java index c547992..e6b625e 100644 --- a/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/processor/TechnicalEntityProcessor.java +++ b/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/processor/TechnicalEntityProcessor.java @@ -624,22 +624,20 @@ public class TechnicalEntityProcessor extends TechnicalProcessor private void addTo(String name, Object data, List<Property> properties) { int pos = 0; for (Property property : properties) { - if(property.isComplex()) { - final List<ComplexValue> cvs; - if(property.isCollection()) { - cvs = (List<ComplexValue>) property.asCollection(); - } else { - cvs = Collections.singletonList(property.asComplex()); - } + if (property.isComplex()) { + @SuppressWarnings("unchecked") + final List<ComplexValue> cvs = property.isCollection() ? + (List<ComplexValue>) property.asCollection() : + Collections.singletonList(property.asComplex()); for (ComplexValue cv : cvs) { final List<Property> value = cv.getValue(); - if(value != null) { + if (value != null) { addTo(name, data, value); } } } - if(name.equals(property.getName())) { + if (name.equals(property.getName())) { properties.remove(pos); final String old = property.getValue().toString(); String newValue = (old == null ? "": old) + data.toString(); http://git-wip-us.apache.org/repos/asf/olingo-odata4/blob/20a00704/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/processor/queryoptions/expression/ExpressionVisitorImpl.java ---------------------------------------------------------------------- diff --git a/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/processor/queryoptions/expression/ExpressionVisitorImpl.java b/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/processor/queryoptions/expression/ExpressionVisitorImpl.java index 3748ab8..475925c 100644 --- a/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/processor/queryoptions/expression/ExpressionVisitorImpl.java +++ b/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/processor/queryoptions/expression/ExpressionVisitorImpl.java @@ -38,7 +38,14 @@ import org.apache.olingo.server.api.uri.UriParameter; import org.apache.olingo.server.api.uri.UriResource; import org.apache.olingo.server.api.uri.UriResourceFunction; import org.apache.olingo.server.api.uri.UriResourceProperty; -import org.apache.olingo.server.api.uri.queryoption.expression.*; +import org.apache.olingo.server.api.uri.queryoption.expression.BinaryOperatorKind; +import org.apache.olingo.server.api.uri.queryoption.expression.Expression; +import org.apache.olingo.server.api.uri.queryoption.expression.ExpressionVisitException; +import org.apache.olingo.server.api.uri.queryoption.expression.ExpressionVisitor; +import org.apache.olingo.server.api.uri.queryoption.expression.Literal; +import org.apache.olingo.server.api.uri.queryoption.expression.Member; +import org.apache.olingo.server.api.uri.queryoption.expression.MethodKind; +import org.apache.olingo.server.api.uri.queryoption.expression.UnaryOperatorKind; import org.apache.olingo.server.tecsvc.data.DataProvider; import org.apache.olingo.server.tecsvc.processor.queryoptions.expression.operand.TypedOperand; import org.apache.olingo.server.tecsvc.processor.queryoptions.expression.operand.UntypedOperand;
