Uploaded image for project: 'OpenJPA'
  1. OpenJPA
  2. OPENJPA-2018

Cannot bind String[] to ParameterExpression for path.in(parameter)

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.0
    • Fix Version/s: 2.3.0
    • Component/s: jpa

      Description

      Given the following code:

      User user = new User("Dave", "Matthews", "foo@bar.de");
      em.persist(user);
      em.flush();
      
      CriteriaBuilder builder = em.getCriteriaBuilder();
      
      CriteriaQuery<User> criteria = builder.createQuery(User.class);
      Root<User> root = criteria.from(User.class);
      criteria.where(root.get("firstname").in(builder.parameter(String[].class)));
      
      TypedQuery<User> query = em.createQuery(criteria);
      for (ParameterExpression parameter : criteria.getParameters()) {
        query.setParameter(parameter, new String[] {"Dave", "Carter"});
      }
      
      List<User> result = query.getResultList();
      assertThat(result.isEmpty(), is(false));
      

      I get a

      <openjpa-2.0.0-r422266:935683 nonfatal user error> org.apache.openjpa.persistence.ArgumentException: The specified parameter of type "class [Ljava.lang.String;" is not a valid query parameter.
      

      Using Collection as ParameterExpression type and binding the parameters via Arrays.asList(...) works fine.

        Attachments

        1. OPENJPA-2018-with-array.patch
          21 kB
          Romain Manni-Bucau
        2. OPENJPA-2018-test-update.patch
          6 kB
          Romain Manni-Bucau
        3. OPENJPA-2018-test2.patch
          4 kB
          Mark Struberg
        4. openjpa-2018.zip
          6 kB
          Oliver Gierke
        5. OPENJPA-2018.patch
          15 kB
          Romain Manni-Bucau
        6. missingpatch.patch
          7 kB
          Romain Manni-Bucau

          Activity

            People

            • Assignee:
              struberg Mark Struberg
              Reporter:
              oliver.gierke Oliver Gierke
            • Votes:
              1 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: