Cayenne
  1. Cayenne
  2. CAY-1306

Binding Collection in Parameters for SQLTemplate is not working

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.5
    • Component/s: Core Library
    • Labels:
      None

      Description

      I'm trying to bind Collection in SQLTemplate query.
      Like in example from this page: http://cayenne.apache.org/doc20/scripting-sqltemplate.html
      From section "Binding Parameters - #bind Directive" -> "Collection example"
      Sadly it's not working... I got an Exception...

      Glad to say that in 3.0 branch this feature is working...

      1. CAY-1306.patch
        5 kB
        Evgeny Ryabitskiy
      2. CAY-1306.patch
        5 kB
        Evgeny Ryabitskiy
      3. CAY-1306_testv3.patch
        4 kB
        Evgeny Ryabitskiy
      4. CAY-1306_testv2.patch
        4 kB
        Evgeny Ryabitskiy

        Activity

        Hide
        Evgeny Ryabitskiy added a comment -

        CAY-1306_testv2.patch - is a JUnit for 2.0 branch where it's fails.

        Show
        Evgeny Ryabitskiy added a comment - CAY-1306 _testv2.patch - is a JUnit for 2.0 branch where it's fails.
        Hide
        Evgeny Ryabitskiy added a comment -

        CAY-1306_testv3.patch - is a JUnit for 3.0 branch (deferents only in formating)
        this JUnit is passed.

        Show
        Evgeny Ryabitskiy added a comment - CAY-1306 _testv3.patch - is a JUnit for 3.0 branch (deferents only in formating) this JUnit is passed.
        Hide
        Evgeny Ryabitskiy added a comment -

        JUnit for 3.0 can be applied now - it is a constraint for regression.

        For 2.0 I will post a solution on this weak. I know where is a problem but don't have enough time right now

        Show
        Evgeny Ryabitskiy added a comment - JUnit for 3.0 can be applied now - it is a constraint for regression. For 2.0 I will post a solution on this weak. I know where is a problem but don't have enough time right now
        Hide
        Evgeny Ryabitskiy added a comment -

        Done.
        Fixed with small code changes.
        No need to pass this changes to next versions.
        Bind directive is rewritten in 3.0

        Show
        Evgeny Ryabitskiy added a comment - Done. Fixed with small code changes. No need to pass this changes to next versions. Bind directive is rewritten in 3.0
        Hide
        Andrus Adamchik added a comment -

        so, am I correct to change the fix verion of this to just 2.0.5?

        Show
        Andrus Adamchik added a comment - so, am I correct to change the fix verion of this to just 2.0.5?
        Hide
        Evgeny Ryabitskiy added a comment -

        Fixed. Going to commit.

        Show
        Evgeny Ryabitskiy added a comment - Fixed. Going to commit.
        Hide
        Evgeny Ryabitskiy added a comment -

        Fixed in 2.0 branch, add Test.
        Also I add this test to 3.1 branch to prevent regression

        Show
        Evgeny Ryabitskiy added a comment - Fixed in 2.0 branch, add Test. Also I add this test to 3.1 branch to prevent regression

          People

          • Assignee:
            Evgeny Ryabitskiy
            Reporter:
            Evgeny Ryabitskiy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development