This is an automated email from the ASF dual-hosted git repository.

ntimofeev pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cayenne.git


The following commit(s) were added to refs/heads/master by this push:
     new cc56f6573 Minor code cleanup
cc56f6573 is described below

commit cc56f6573655330994c3b0383114a75ae577bd9f
Author: Nikita Timofeev <stari...@gmail.com>
AuthorDate: Fri Nov 1 17:47:34 2024 +0400

    Minor code cleanup
---
 .../cayenne/access/translator/select/OrderingAbstractStage.java   | 8 ++++++--
 .../cayenne/access/translator/select/TranslatorContext.java       | 4 +++-
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git 
a/cayenne/src/main/java/org/apache/cayenne/access/translator/select/OrderingAbstractStage.java
 
b/cayenne/src/main/java/org/apache/cayenne/access/translator/select/OrderingAbstractStage.java
index 545e59a5a..110f85b72 100644
--- 
a/cayenne/src/main/java/org/apache/cayenne/access/translator/select/OrderingAbstractStage.java
+++ 
b/cayenne/src/main/java/org/apache/cayenne/access/translator/select/OrderingAbstractStage.java
@@ -49,7 +49,11 @@ abstract class OrderingAbstractStage implements 
TranslationStage {
     }
 
     private boolean orderColumnAbsent(TranslatorContext context, Node 
orderingNode) {
-        return context.getResultNodeList().stream()
-                .noneMatch(result -> 
result.getNode().deepEquals(orderingNode));
+        for (ResultNodeDescriptor result : context.getResultNodeList()) {
+            if (result.getNode().deepEquals(orderingNode)) {
+                return false;
+            }
+        }
+        return true;
     }
 }
diff --git 
a/cayenne/src/main/java/org/apache/cayenne/access/translator/select/TranslatorContext.java
 
b/cayenne/src/main/java/org/apache/cayenne/access/translator/select/TranslatorContext.java
index 237cfac07..fcfdc3b15 100644
--- 
a/cayenne/src/main/java/org/apache/cayenne/access/translator/select/TranslatorContext.java
+++ 
b/cayenne/src/main/java/org/apache/cayenne/access/translator/select/TranslatorContext.java
@@ -205,7 +205,9 @@ public class TranslatorContext implements 
SQLGenerationContext {
     }
 
     boolean hasAggregate() {
-        if(hasAggregate != null) return hasAggregate;
+        if(hasAggregate != null) {
+            return hasAggregate;
+        }
 
         if(getQuery().getHavingQualifier() != null) {
             return (hasAggregate = true);

Reply via email to