Issue 1755: Add support for non-breaking spaces

Signed-off-by: William Dazey <wadazey@us.ibm.com>

jpa/eclipselink.jpa.test.jse/src/it/java/org/eclipse/persistence/jpa/test/jpql/TestWhiteSpace.java

@@ -0,0 +1,65 @@
+/*
+ * Copyright (c) 2022 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2022 IBM Corporation. All rights reserved.
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the Eclipse Public License v. 2.0 which is available at
+ * http://www.eclipse.org/legal/epl-2.0,
+ * or the Eclipse Distribution License v. 1.0 which is available at
+ * http://www.eclipse.org/org/documents/edl-v10.php.
+ *
+ * SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
+ */
+package org.eclipse.persistence.jpa.test.jpql;
+
+import java.util.List;
+
+import jakarta.persistence.EntityManager;
+import jakarta.persistence.EntityManagerFactory;
+import jakarta.persistence.TypedQuery;
+
+import org.eclipse.persistence.jpa.test.framework.DDLGen;
+import org.eclipse.persistence.jpa.test.framework.Emf;
+import org.eclipse.persistence.jpa.test.framework.EmfRunner;
+import org.eclipse.persistence.jpa.test.jpql.model.NoResultEntity;
+import org.junit.Assert;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+
+@RunWith(EmfRunner.class)
+public class TestWhiteSpace {
+
+    @Emf(name = "noResultEMF", createTables = DDLGen.DROP_CREATE, classes = { NoResultEntity.class })
+    private EntityManagerFactory noResultEmf;
+
+    /**
+     * Complex test of the aggregate functions in JPQL. 
+     * For this test, there must be zero results in the entity table and the Entity state field 
+     * must be a primitive type.
+     * JPA 2.1 specification; Section 4.8.5 states aggregate functions (MIN, MAX, AVG, & SUM) 
+     * must return a result of NULL if there are no values to apply the aggregate function to
+     */
+    @Test
+    public void testNonBreakingSpace() {
+        EntityManager em = noResultEmf.createEntityManager();
+        try {
+        	String queryString = "SELECT n FROM\u00A0NoResultEntity n";
+            TypedQuery<NoResultEntity> checkQuery = em.createQuery(queryString, NoResultEntity.class);
+            List<NoResultEntity> checkResult = checkQuery.getResultList();
+            Assert.assertEquals("Entity table NoResultEntity must be empty for this test", 0, checkResult.size());
+
+        	queryString = "SELECT n FROM NoResultEntity n WHERE n.id =\u00A0:idparam";
+            checkQuery = em.createQuery(queryString, NoResultEntity.class);
+            checkQuery.setParameter("idparam", 22);
+            checkResult = checkQuery.getResultList();
+            Assert.assertEquals("Entity table NoResultEntity must be empty for this test", 0, checkResult.size());
+        } finally {
+            if (em.getTransaction().isActive()) {
+                em.getTransaction().rollback();
+            }
+            if (em.isOpen()) {
+                em.close();
+            }
+        }
+    }
+}

jpa/org.eclipse.persistence.jpa.jpql/src/main/java/org/eclipse/persistence/jpa/jpql/AbstractGrammarValidator.java

@@ -1,5 +1,6 @@
 /*
  * Copyright (c) 2006, 2022 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2022 IBM Corporation. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v. 2.0 which is available at
@@ -3033,7 +3034,7 @@ public void visit(DateTime expression) {
                 int endPosition = startPosition;
 
                 for (int index = 1; index < length; index++) {
-                    if (Character.isWhitespace(dateTime.charAt(index))) {
+                    if (ExpressionTools.isWhiteSpace(dateTime.charAt(index))) {
                         break;
                     }
                     endPosition++;
@@ -3051,7 +3052,7 @@ else if (!dateTime.startsWith("{d '") &&
                 for (int index = 1; index < length; index++) {
                     startPosition++;
 
-                    if (Character.isWhitespace(dateTime.charAt(index))) {
+                    if (ExpressionTools.isWhiteSpace(dateTime.charAt(index))) {
                         break;
                     }
                 }

jpa/org.eclipse.persistence.jpa.jpql/src/main/java/org/eclipse/persistence/jpa/jpql/ExpressionTools.java

@@ -1,5 +1,6 @@
 /*
- * Copyright (c) 2006, 2021 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2006, 2022 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2022 IBM Corporation. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v. 2.0 which is available at
@@ -239,6 +240,17 @@ public static boolean isQuote(char character) {
                character == '\"';
     }
 
+    /**
+     * Determines whether the given character is whitespace. Supports non-breaking space characters
+     *
+     * @param character The character to check if it's whitespace
+     * @return <code>true</code> if the given character is either whitespace or non-breaking space; <code>false</code> otherwise
+     */
+    public static boolean isWhiteSpace(char character) {
+        return Character.isWhitespace(character) ||
+               Character.isSpaceChar(character);
+    }
+
     /**
      * Returns the given string literal by wrapping it with single quotes. Any single quote within
      * the string will be escaped with another single quote.
@@ -292,8 +304,8 @@ public static void reposition(CharSequence query1, int[] positions, CharSequence
             char character1 = Character.toUpperCase(query1.charAt(index1));
             char character2 = Character.toUpperCase(query2.charAt(index2));
 
-            boolean whitespace1 = Character.isWhitespace(character1);
-            boolean whitespace2 = Character.isWhitespace(character2);
+            boolean whitespace1 = ExpressionTools.isWhiteSpace(character1);
+            boolean whitespace2 = ExpressionTools.isWhiteSpace(character2);
 
             if (character1 != character2) {
 
@@ -459,7 +471,7 @@ public static boolean stringIsEmpty(CharSequence text) {
         }
 
         for (int i = text.length(); i-- > 0;) {
-            if (!Character.isWhitespace(text.charAt(i))) {
+            if (!ExpressionTools.isWhiteSpace(text.charAt(i))) {
                 return false;
             }
         }

jpa/org.eclipse.persistence.jpa.jpql/src/main/java/org/eclipse/persistence/jpa/jpql/WordParser.java

@@ -1,5 +1,6 @@
 /*
- * Copyright (c) 2006, 2021 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2006, 2022 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2022 IBM Corporation. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v. 2.0 which is available at
@@ -229,12 +230,12 @@ public boolean isTail() {
      * @return <code>true</code> if the character can be part of a word; <code>false</code> if it is
      * not an alphanumeric character, which usually means is a whitespace, a delimiter or an
      * arithmetic symbol
-     * @see Character#isWhitespace(char)
+     * @see ExpressionTools.isWhiteSpace(char)
      * @see #isArithmeticSymbol(char)
      * @see #isDelimiter(char)
      */
     public boolean isWordSeparator(char character) {
-        return Character.isWhitespace(character) ||
+        return ExpressionTools.isWhiteSpace(character) ||
                isDelimiter(character)            ||
                isArithmeticSymbol(character);
     }
@@ -311,9 +312,9 @@ public String moveForwardIgnoreWhitespace(CharSequence word) {
             // Handle testing
             // 1. "... GROUP    BY" and "GROUP BY"
             // 2. "... GROUP\nBY" and "GROUP BY"
-            if (Character.isWhitespace(c1)) {
+            if (ExpressionTools.isWhiteSpace(c1)) {
 
-                if (Character.isWhitespace(c2)) {
+                if (ExpressionTools.isWhiteSpace(c2)) {
                     sb.append(' ');
                     continue;
                 }
@@ -397,10 +398,7 @@ public int partialWordStartPosition(int position) {
         for (int index = position; --index >= 0; ) {
             char character = text.charAt(index);
 
-            if (Character.isWhitespace(character) ||
-                isDelimiter(character) ||
-                isArithmeticSymbol(character)) {
-
+            if (isWordSeparator(character)) {
                 break;
             }
 
@@ -613,7 +611,7 @@ public int skipLeadingWhitespace() {
         while (cursor < length) {
 
             char character = text.charAt(cursor);
-            if (!Character.isWhitespace(character)) {
+            if (!ExpressionTools.isWhiteSpace(character)) {
                 break;
             }
 
@@ -788,9 +786,9 @@ public boolean startsWithIdentifier(CharSequence identifier, int position) {
             // Handle testing
             // 1. "... GROUP    BY" and "GROUP BY"
             // 2. "... GROUP\nBY" and "GROUP BY"
-            if (Character.isWhitespace(c1)) {
+            if (ExpressionTools.isWhiteSpace(c1)) {
 
-                if (Character.isWhitespace(c2)) {
+                if (ExpressionTools.isWhiteSpace(c2)) {
                     continue;
                 }
 
@@ -964,7 +962,7 @@ public int whitespaceCount(int position) {
         for (int index = position; index < length; index++) {
             char character = text.charAt(index);
 
-            if (!Character.isWhitespace(character)) {
+            if (!ExpressionTools.isWhiteSpace(character)) {
                 return index - position;
             }
         }
@@ -1147,7 +1145,7 @@ else if (isWordSeparator(character)) {
                 continue;
             }
 
-            if (Character.isWhitespace(character) ||
+            if (ExpressionTools.isWhiteSpace(character) ||
                 isDelimiter(character) ||
                 character == '>' ||
                character == '<' ||

jpa/org.eclipse.persistence.jpa.jpql/src/main/java/org/eclipse/persistence/jpa/jpql/tools/AbstractContentAssistVisitor.java

@@ -1,5 +1,6 @@
 /*
  * Copyright (c) 2006, 2022 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2022 IBM Corporation. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v. 2.0 which is available at
@@ -439,7 +440,7 @@ else if (isValidVersion(identifier)) {
 
                     char character = wordParser.character(position - fragment.length() - 1);
 
-                    if (Character.isWhitespace(character)) {
+                    if (ExpressionTools.isWhiteSpace(character)) {
                         proposals.addIdentifier(identifier);
                         break;
                     }