Update CMISSessionFacade.java Hi, this piece of code causes two calls in a row for content stream. Result of document.getContentStream() has to be stored and used where is needed, not to call document.getContentStream() in condition and than again in return statement.
Project: http://git-wip-us.apache.org/repos/asf/camel/repo Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/5f734e81 Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/5f734e81 Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/5f734e81 Branch: refs/heads/camel-2.19.x Commit: 5f734e810c574fc79418cb30ee41967d25060189 Parents: 9d41626 Author: Tomáš Burian <tomas.bur...@lundegaard.eu> Authored: Wed May 31 15:06:01 2017 +0200 Committer: Claus Ibsen <davscl...@apache.org> Committed: Thu Jun 1 09:11:48 2017 +0200 ---------------------------------------------------------------------- .../java/org/apache/camel/component/cmis/CMISSessionFacade.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/camel/blob/5f734e81/components/camel-cmis/src/main/java/org/apache/camel/component/cmis/CMISSessionFacade.java ---------------------------------------------------------------------- diff --git a/components/camel-cmis/src/main/java/org/apache/camel/component/cmis/CMISSessionFacade.java b/components/camel-cmis/src/main/java/org/apache/camel/component/cmis/CMISSessionFacade.java index 7b47e69..d143d7c 100644 --- a/components/camel-cmis/src/main/java/org/apache/camel/component/cmis/CMISSessionFacade.java +++ b/components/camel-cmis/src/main/java/org/apache/camel/component/cmis/CMISSessionFacade.java @@ -182,8 +182,9 @@ public class CMISSessionFacade { public InputStream getContentStreamFor(QueryResult item) { Document document = getDocument(item); - if (document != null && document.getContentStream() != null) { - return document.getContentStream().getStream(); + ContentStream contentStream = document.getContentStream(); + if (document != null && contentStream != null) { + return contentStream.getStream(); } return null; }
