[midPoint-git] [Evolveum/midpoint] a8e888: Improve GeneralUpdate a bit

[mederly]

  Branch: refs/heads/tmp/provenance
  Home:   https://github.com/Evolveum/midpoint
  Commit: a8e8880fdd142248d11e58eb81e4100ad85bcd3b
      https://github.com/Evolveum/midpoint/commit/a8e8880fdd142248d11e58eb81e4100ad85bcd3b
  Author: Pavol Mederly <mederly at evolveum.com>
  Date:   2020-07-10 (Fri, 10 Jul 2020)

  Changed paths:
    M repo/repo-sql-impl-test/src/test/java/com/evolveum/midpoint/repo/sql/ModifyTest.java
    R repo/repo-sql-impl-test/src/test/resources/modify/user-adam-new-assignment-existing-id.xml
    M repo/repo-sql-impl/src/main/java/com/evolveum/midpoint/repo/sql/helpers/delta/GeneralUpdate.java
    M repo/repo-sql-impl/src/main/java/com/evolveum/midpoint/repo/sql/helpers/delta/ObjectDeltaUpdater.java
    M repo/repo-sql-impl/src/main/java/com/evolveum/midpoint/repo/sql/helpers/modify/DeltaUpdaterUtils.java

  Log Message:
  -----------
  Improve GeneralUpdate a bit

Minor changes in GeneralUpdate class e.g. swapping application
of add and delete values (now delete values come first, just like
in regular delta application). This is what has left from unsuccessful
attempt to make updater process container value metadata.

Related to MID-6275.


  Commit: 4a08872b170ed20f2dbadd697208248d0f74f1c5
      https://github.com/Evolveum/midpoint/commit/4a08872b170ed20f2dbadd697208248d0f74f1c5
  Author: Pavol Mederly <mederly at evolveum.com>
  Date:   2020-07-10 (Fri, 10 Jul 2020)

  Changed paths:
    M repo/repo-sql-impl-test/src/test/java/com/evolveum/midpoint/repo/sql/ModifyTest.java
    A repo/repo-sql-impl-test/src/test/java/com/evolveum/midpoint/repo/sql/StrangeModifyTest.java
    A repo/repo-sql-impl-test/src/test/resources/modify/user-bob.xml
    M repo/repo-sql-impl/src/main/java/com/evolveum/midpoint/repo/sql/data/common/container/RAssignment.java
    A repo/repo-sql-impl/src/main/java/com/evolveum/midpoint/repo/sql/helpers/delta/CollectionUpdate.java
    M repo/repo-sql-impl/src/main/java/com/evolveum/midpoint/repo/sql/helpers/delta/GeneralUpdate.java

  Log Message:
  -----------
  Implement replacing container values in repo

We can now replace containers in repo (for the time being
if using client-specified ID). The session.merge call was the key.

This commit also:
- fixes equals/hashCode for RAssignment in cases when ownerOid is null
- factors out collection-updating functionality to CollectionUpdate
class.

Related to MID-6275.


  Commit: cc97743cfe9cabeffe74dff63b64ed5e9d525600
      https://github.com/Evolveum/midpoint/commit/cc97743cfe9cabeffe74dff63b64ed5e9d525600
  Author: Pavol Mederly <mederly at evolveum.com>
  Date:   2020-07-11 (Sat, 11 Jul 2020)

  Changed paths:
    M repo/repo-sql-impl-test/src/test/java/com/evolveum/midpoint/repo/sql/ModifyTest.java
    M repo/repo-sql-impl/src/main/java/com/evolveum/midpoint/repo/sql/data/common/container/RAssignment.java
    M repo/repo-sql-impl/src/main/java/com/evolveum/midpoint/repo/sql/data/common/container/ROperationExecution.java
    M repo/repo-sql-impl/src/main/java/com/evolveum/midpoint/repo/sql/helpers/delta/CollectionUpdate.java
    A repo/repo-sql-impl/src/main/java/com/evolveum/midpoint/repo/sql/helpers/delta/ContainerCollectionUpdate.java
    M repo/repo-sql-impl/src/main/java/com/evolveum/midpoint/repo/sql/helpers/delta/GeneralUpdate.java

  Log Message:
  -----------
  Simplify [Container]CollectionUpdate

The code should be quite minimalistic now. It relies on repository
vs. XML consistency and tries to fail fast on any issues. Highly
experimental - we will see how it will behave in production.

Related to MID-6275.


Compare: https://github.com/Evolveum/midpoint/compare/e5dd3d4c58e7...cc97743cfe9c