Uploaded image for project: 'CMS'
  1. CMS
  2. CMS-11904

Rework DefaultContentType._computePropertiesReferences(ModelItemContainer)

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 4.8.0
    • 4.8.0
    • None
    • None
    • 4.8.0 M8

      org.ametys.cms.contenttype.DefaultContentType._computePropertiesReferences(ModelItemContainer) method controls validity of ElementRefProperty only.

      It checks the item path and add properties references to compute indexing references (only used to alert the user when saving a content with external properties).

      But on properties, we should do this on all properties without knowing their instancied class. We can check it with two methods:

      • boolean isValid(ContentType)
      • Map<String, List<String>> or List<String> getReferences()

          [CMS-11904] Rework DefaultContentType._computePropertiesReferences(ModelItemContainer)

          Caroline Bayle made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]
          Caroline Bayle made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Ready to merge [ 10208 ] New: Resolved [ 5 ]
          Bérénice Maurel made changes -
          Assignee Original: Bérénice Maurel [ bmaurel ] New: Caroline Bayle [ cbayle ]
          Bérénice Maurel made changes -
          Status Original: Code review OK [ 10206 ] New: Ready to merge [ 10208 ]
          Bérénice Maurel made changes -
          Status Original: Available for review [ 10005 ] New: Code review OK [ 10206 ]
          Caroline Bayle made changes -
          Développeur New: Caroline Bayle [ cbayle ]
          Caroline Bayle made changes -
          Assignee Original: Caroline Bayle [ cbayle ] New: Bérénice Maurel [ bmaurel ]
          Caroline Bayle made changes -
          Status Original: In Progress [ 3 ] New: Available for review [ 10005 ]
          Caroline Bayle made changes -
          Status Original: Reopened [ 4 ] New: In Progress [ 3 ]
          Caroline Bayle made changes -
          Resolution Original: Fixed [ 1 ]
          Status Original: Resolved [ 5 ] New: Reopened [ 4 ]

            cbayle Caroline Bayle
            bmaurel Bérénice Maurel
            Caroline Bayle Caroline Bayle
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: