Uploaded image for project: 'Jackrabbit Content Repository'
  1. Jackrabbit Content Repository
  2. JCR-325

docview roundtripping does not work with multivalue non-string properties

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: 0.9, 1.0
    • Fix Version/s: None
    • Component/s: xml
    • Labels:
      None
    • Environment:
      jackrabbit r379292

      Description

      when exporting a multivalue property with docview, the property values are serialized to a space delimited list in the xml attributes:
      for example:

      <?xml version="1.0" encoding="UTF-8"?>
      .
      .
      <testNode
      jcr:primaryType="refTest"
      refs="b5c12524-5446-4c1a-b024-77f623680271 7b4d4e6f-9515-47d8-a77c-b4beeaf469bc"
      />

      the refTest nodetype was:

      [refTest]

      • refs (reference) multiple

      importing this docview fails with: javax.jcr.ValueFormatException: not a valid UUID format
      this is due to the fact, that the space delimited list is not exploded anymore. actually this code is commented:

      org.apache.jackrabbit.core.xml.DocViewImportHandler, lines 191 - 200:
      /*
      // @todo should attribute value be interpreted as LIST type (i.e. multi-valued property)?
      String[] strings = Text.explode(attrValue, ' ', true);
      propValues = new Value[strings.length];
      for (int j = 0; j < strings.length; j++)

      { // decode encoded blanks in value strings[j] = Text.replace(strings[j], "_x0020_", " "); propValues[j] = InternalValue.create(strings[j]); }

      */

      i haven't tested, but i assume this also fails for all other non-string property types.

        Attachments

        1. xml-refactoring.patch
          69 kB
          Jukka Zitting
        2. namespace-context.patch
          21 kB
          Jukka Zitting

          Issue Links

            Activity

              People

              • Assignee:
                jukkaz Jukka Zitting
                Reporter:
                tripod Tobias Bocanegra
              • Votes:
                1 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: