Uploaded image for project: 'XWiki Platform'
  1. XWiki Platform
  2. XWIKI-6566

Changing the multiple select attribute of a ListProperty in an XWiki class occasionally causes a document to become unsavable.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Cannot Reproduce
    • Affects Version/s: 3.0
    • Fix Version/s: None
    • Component/s: Storage
    • Labels:
    • Difficulty:
      Unknown
    • Documentation:
      N/A
    • Documentation in Release Notes:
      N/A
    • Similar issues:

      Description

      This problem is essentially XWIKI-299 reincarnated.
      A ListProperty may exist in any of 3 tables. It may take the form of a DBListProperty (xwikilists), a ListProperty (xwikilergestrings), or a StringProperty (xwikistrings).

          public BaseProperty newProperty()
          {
              BaseProperty lprop;
      
              if (isRelationalStorage() && isMultiSelect()) {
                  lprop = new DBStringListProperty();
              } else if (isMultiSelect()) {
                  lprop = new StringListProperty();
              } else {
                  lprop = new StringProperty();
              }
      
              return lprop;
          }
      

      Changing the multi-select or relational attributes causes the location of the list to change and when the class is next saved, Hibernate panics because there is already a value in a different <joined-subclass>.

      The patch for XWIKI-299 modified all of the objects if the class when the class was modified so that when they were next loaded they would be correct. The patch for XWIKI-299 does not work if an object is saved using the old class after the new class is saved.

      Because of XWIKI-6170, it is possible to end up loading an old version after a new version of a document is saved. If one loads the old version of the class and uses that to create and save an object, the object property will be stored in the wrong table and the patch for XWIKI-299 will not fix it until the class is saved again.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                softec Denis Gervalle
                Reporter:
                calebjamesdelisle CalebJamesDeLisle
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: