From 7d06f20fc7b127da265bc9f6d7e852f106881d24 Mon Sep 17 00:00:00 2001 From: Radek Felcman Date: Tue, 14 May 2024 15:45:47 +0200 Subject: [PATCH] NPE in ExpressionOperator.printCollection - backport from 2.7 (#2141) Fixes #2136 Backport from #2137 Signed-off-by: Radek Felcman --- .../eclipse/persistence/expressions/ExpressionOperator.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/foundation/org.eclipse.persistence.core/src/main/java/org/eclipse/persistence/expressions/ExpressionOperator.java b/foundation/org.eclipse.persistence.core/src/main/java/org/eclipse/persistence/expressions/ExpressionOperator.java index 66d458d10da..6ede993480c 100644 --- a/foundation/org.eclipse.persistence.core/src/main/java/org/eclipse/persistence/expressions/ExpressionOperator.java +++ b/foundation/org.eclipse.persistence.core/src/main/java/org/eclipse/persistence/expressions/ExpressionOperator.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 1998, 2022 Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 1998, 2024 Oracle and/or its affiliates. All rights reserved. * Copyright (c) 2018, 2022 IBM Corporation. All rights reserved. * * This program and the accompanying materials are made available under the @@ -2387,8 +2387,7 @@ public void printCollection(List items, ExpressionSQLPrinter printer } String[] dbStrings = getDatabaseStrings(items.size()); - for (int i = 0; i < this.argumentIndices.length; i++) { - final int index = this.argumentIndices[i]; + for (final int index : this.argumentIndices) { Expression item = items.get(index); if ((this.selector == Ref) || ((this.selector == Deref) && (item.isObjectExpression()))) { DatabaseTable alias = item.aliasForTable(((ObjectExpression)item).getDescriptor().getTables().firstElement());