Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-8834

ContentMap doesn't consider the possibility that some reference options are arrays

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Neutral
    • 6.3.0, 6.2.36
    • 6.2.29
    • None
    • None
    • Yes
    • Nucleus 38
    • 2
    • Yes

    Description

      When the ContentMap class is used for retrieve a REFERENCE property as jcr:predecessor it will fail because the get method won't consider it as multivalue.

      It would be better to include at the else if (type == PropertyType.REFERENCE) statement another check like  "if (prop.isMultiple())" so for those cases like jcr:predecessors, it won't thrown error as:

      jcr:predecessors is a multi-valued property, so it's values can only be retrieved as an array

      The groovy script to simulate the bug: scripts.getMultiReferencesProperty.xml

      Steps:

      1. Open Groovy app and import the script
      2. Open and run the script

      Actual result:

      logs:

      WARN  info.magnolia.jcr.util.ContentMap                 : Failed to retrieve jcr:predecessors on node /test with property /test/jcr:predecessors is a multi-valued property, so it's values can only be retrieved as an array

      Expected result:

      support multivalue for jcr:predecessor propertives on ContentMap

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            is duplicated by

            Improvement - An improvement or enhancement to an existing feature or task. MGNLEE-761 ContentMap#get method doesn't consider the PropertyType.REFERENCE type as array

            • Neutral -
            • Closed

            Improvement - An improvement or enhancement to an existing feature or task. MGNLEE-766 ContentMap#get method doesn't consider the PropertyType.REFERENCE type as array

            • Neutral -
            • Closed

            Improvement - An improvement or enhancement to an existing feature or task. MGNLEE-767 ContentMap#get method doesn't consider the PropertyType.REFERENCE type as array

            • Neutral -
            • Closed

            Improvement - An improvement or enhancement to an existing feature or task. MGNLEE-775 ContentMap#get method doesn't consider the PropertyType.REFERENCE type as array

            • Neutral -
            • Closed

            Activity

              People

                thien.quach Quach Hao Thien
                ccantalapiedra Carlos Cantalapiedra
                Nucleus
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:
                  Work Started:

                  Checklists

                    Task DoD

                    Time Tracking

                      Estimated:
                      Original Estimate - Not Specified
                      Not Specified
                      Remaining:
                      Remaining Estimate - Not Specified
                      Not Specified
                      Logged:
                      Time Spent - 2d 3h
                      2d 3h