castor
  1. castor
  2. CASTOR-3183

Property changes don't get persisted in extends hierarchy

    Details

    • Type: Bug Bug
    • Status: Resolved Resolved
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 1.3.3rc1
    • Fix Version/s: 1.3.3rc1
    • Component/s: JDO
    • Labels:
      None
    • Number of attachments :
      3
    1. patch_110808_C3183.txt
      14 kB
      Johannes Venzke
    2. patch-C3183-20110808.txt
      14 kB
      Ralf Joachim
    3. patch-C3183-20110809.txt
      39 kB
      Ralf Joachim

      Activity

      Hide
      Johannes Venzke added a comment -

      This testcase shows the bug.

      Show
      Johannes Venzke added a comment - This testcase shows the bug.
      Hide
      Ralf Joachim added a comment -

      Reduced test to bare minimum.

      Searching for the reason of the problem it apears, that no update on extended table gets executed as 'store' flag of all fields is false. Setting 'store' to true didn't help as 'index' of those fields is wrong. If also fixing the 'index' the test passes but then a bunch of other tests fail.

      Will try to search the history of sql info classes at which point this problem has been introduced to get an idea how to fix it.

      Show
      Ralf Joachim added a comment - Reduced test to bare minimum. Searching for the reason of the problem it apears, that no update on extended table gets executed as 'store' flag of all fields is false. Setting 'store' to true didn't help as 'index' of those fields is wrong. If also fixing the 'index' the test passes but then a bunch of other tests fail. Will try to search the history of sql info classes at which point this problem has been introduced to get an idea how to fix it.
      Hide
      Ralf Joachim added a comment -

      Checking history it apears that SQLStatementUpdate using new SQL info classes has never been able to update properties of extended classes. I therefore decided to revert back to previous version of SQLStatementUpdate till I had time to look into the problem in details. With this old version all tests pass.

      Show
      Ralf Joachim added a comment - Checking history it apears that SQLStatementUpdate using new SQL info classes has never been able to update properties of extended classes. I therefore decided to revert back to previous version of SQLStatementUpdate till I had time to look into the problem in details. With this old version all tests pass.
      Hide
      Ralf Joachim added a comment -

      Patch committed

      Show
      Ralf Joachim added a comment - Patch committed

        People

        • Assignee:
          Ralf Joachim
          Reporter:
          Johannes Venzke
        • Votes:
          0 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved: