Uploaded image for project: 'Magnolia UI'
  1. Magnolia UI
  2. MGNLUI-1376

JCR Multiple property values not supported yet.

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Critical Critical
    • 5.0
    • 5.0
    • None
    • None

      Currently JCR property defined as multiple is not supported
      <sv:property sv:name="excludeChannels" sv:type="String" sv:multiple="true">
      <sv:value>desktop</sv:value>
      <sv:value>smartphone</sv:value>
      </sv:property>

      JcrNodeAdapter should be able to:

      • retrieve this value and convert it as a set (Vaadin Property)
      • Item's used in fields (as property datasource) should be able to populate it
      • convert the Vaadin Property to JCR Values.

      In addition we should adapt the AbstractFieldBuilder. The following class will never call the getDefaultFieldType(...) as by default in the ConfiguredFieldDefinition the type is set by default to String.

      protected Class<?> getFieldType(FieldDefinition fieldDefinition) {
      if (StringUtils.isNotBlank(fieldDefinition.getType()))

      { return DefaultPropertyUtil.getFieldTypeClass(fieldDefinition.getType()); }

      return getDefaultFieldType(fieldDefinition);
      }

        Acceptance criteria

              ehechinger Eric Hechinger
              ehechinger Eric Hechinger
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Bug DoR
                  Task DoD