castor

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. Text File
    patch_110808_C3183.txt
    08/Aug/11 7:17 AM
    14 kB
    Johannes Venzke
  2. Text File
    patch-C3183-20110808.txt
    08/Aug/11 11:27 AM
    14 kB
    Ralf Joachim
  3. Text File
    patch-C3183-20110809.txt
    08/Aug/11 5:22 PM
    39 kB
    Ralf Joachim

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Johannes Venzke added a comment -

This testcase shows the bug.

Show
Johannes Venzke added a comment - This testcase shows the bug.
Hide
Permalink
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
Permalink
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
Permalink
Ralf Joachim added a comment -

Patch committed

Show
Ralf Joachim added a comment - Patch committed

People

  • Assignee:
    Ralf Joachim
    Reporter:
    Johannes Venzke
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: