SetProperties does not allow removing a Computed value

[pobeaulieucl]

When trying to remove ComputedValues from a collection using arangodriver.SetCollectionPropertiesOptions, the operation does not succeed. Neither setting ComputedValues to nil nor an empty array results in the computed values being removed.

Example:

options := arangodriver.SetCollectionPropertiesOptions{
    ComputedValues: []arangodriver.ComputedValue{},
}

err = col.SetProperties(ctx, options)

Expected Behavior:

Setting ComputedValues to an empty array or nil should remove any existing computed values on the collection.

Actual Behavior:

Computed values remain unchanged, and no removal occurs.

[enikon]

Hi @pobeaulieucl,
Thank you for reporting the issue!

The issue has been fixed and is now available on the master branch.
In order to differentiate between nil - 'change to no value (0/empty list/empty text)' and nil - 'no change to a field' we use pointers. Nil pointer means no change and a pointer to nil means change the value of a property to nil.

Below is an example code to solve your issue. Notice that we use newly added V2 suffix for the names of new structures and methods.

options := arangodb.SetCollectionPropertiesOptionsV2{
	ComputedValues: &[]arangodb.ComputedValue{},
}

err = col.SetProperties(ctx, options)

Below is an example code that will skip setting the ComputedValue

options := arangodb.SetCollectionPropertiesOptionsV2{
	ComputedValues: &[]arangodb.ComputedValue{cv1},
}
err = col.SetProperties(ctx, options)
options = arangodb.SetCollectionPropertiesOptionsV2{
	JournalSize: utils.NewType(int64(512)),
	ComputedValues: nil
	// The line above can be skipped. Fields take 'no change to a field' nil by default.
}
err = col.SetProperties(ctx, options)
// col Properties is using cv1 as the ComputedValues and 512 as the JournalSize

Best regards,