Uploaded image for project: 'MidPoint'
  1. MidPoint
  2. MID-6686

Consolidation of PCVs with IDs does not work

    XMLWordPrintable

Details

    • Bug
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • None
    • backlog
    • Model, Prism
    • Internal

    Description

      Values are considered equivalent even if they are not:

      2020-10-29 16:15:11,791 [MODEL] [pool-7-thread-5] TRACE (com.evolveum.midpoint.model.impl.lens.IvwoConsolidator): Consolidating value equivalence class (49 of 63):
      Equivalence class members: (1)
          id=50
            subtype: [ x ]
            extension: 
                recStatus: d
                uuser: ...
            metadata: (...) 
            targetRef: (...)
            activation: 
                effectiveStatus: ENABLED
      Plus origins: []
      Zero origins: (1)
          ItemValueWithOrigin:
            itemValue:
              id=50
                subtype: [ x ]
                extension: 
                    recStatus: d
                    uuser: ...
                metadata: (...) 
                targetRef: (...)
                activation: 
                    effectiveStatus: ENABLED
            mapping: ...
            construction: null
      Minus origins: (1)
          ItemValueWithOrigin:
            itemValue:
              id=50
                subtype: [ x ]
                extension: 
                    recStatus: a
                metadata: (...) 
                targetRef: (...)
                activation: 
                    effectiveStatus: ENABLED
            mapping: ...
            construction: null
      Presence in existing item: (1)
          id=50
            subtype: [ x ]
            extension: 
                recStatus: a
            metadata: (...) 
            targetRef: (...)
            activation: 
                effectiveStatus: ENABLED
      Presence in apriori plus: []
      Presence in apriori minus: []

      Note that the values have different content: recStatus is different (a vs d), uuser is different. But the PCVs fall into one equivalence class, which is wrong.

      The problem lies most probably here:

      https://github.com/Evolveum/midpoint/blob/3483096551d407c607e412d23938034a5c81ca37/model/model-impl/src/main/java/com/evolveum/midpoint/model/impl/lens/IvwoConsolidator.java#L1200-L1231

      We shouldn't automatically consider values with matching IDs to be equivalent.

      Attachments

        Issue Links

          Activity

            People

              mederly Pavol Mederly
              mederly Pavol Mederly
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated: