Uploaded image for project: 'Integrated'
  1. Integrated
  2. INTEGRATED-1422

Still unchanged documents in content history

    Details

    • Sprint:
    • Team:
      Developers - on the back
    • Epic Link:
    • Story Points:
      3
    • Max. hours:
      13
    • Min. hours:
      11

      Description

      Background

      In INTEGRATED‌-1331 changes have been made to avoid storing content history changes for null values for new variables. However, this is still a problem.

      Requirements

      • Empty values
        • Values that are not set, are set with "null", so this problem is not present for the first level. This does however not work for values with multiple levels, so this check should be recursive.
      // collection: content_history
{
	"_id" : "4dffab413a57354c88f91933e25e1507",
	"contentId" : "3907af99b3003b3c922989e076c59bd6",
	"contentType" : "company",
	"contentClass" : "Zkn\\Bundle\\ContentBundle\\Document\\Relation\\Company",
	"action" : "update",
	"date" : ISODate("2018-11-15T15:32:03.485+01:00"),
	"changeSet" : {
		"phonenumbers" : [
			[
				{
					"number" : "0612345678"
				}
			],
			[
				{
					"type" : null,
					"number" : "0612345678"
				}
			]
		],
		"address" : [
			{
				"street" : "Straat",
				"houseNumber" : "13",
				"zipCode" : "1000AB"
			},
			{
				"street" : "Straat",
				"houseNumber" : "13",
				"houseNumberAddition" : null,
				"zipCode" : "1000AB"
			}
		]
	}
}

        Attachments

          Issue links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. INTEGRATED-1331 Don't save non-changes to content history

          • Major - Major loss of function.
          • Closed

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                marijn Marijn Otte
                Code reviewer:
                Ger Jan van den Bosch (Inactive)
                Developer:
                Michael Jongman (Inactive)
                Product owner:
                Marijn Otte
                Client:
                Integrated Marijn
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:

                  OTM