BUG: Cannot unset defaultValue via CreateNodeAggregateWithNode::$initialPropertyValues

[mhsdesign]

While the change #4322 improved the unset handling of properties a lot, it introduced a regression.

Using the following NodeType

'Flowpack.NodeTemplates:Content.Properties':
  properties:
    unsetValueWithDefault:
      defaultValue: true
      type: boolean

and creating it but using

PropertyValuesToWrite::fromArray(["unsetValueWithDefault" => null])

would originally set unsetValueWithDefault to NULL in the database. This was the behaviour in Neos 8.3 and Neos 9 before the mentioned change. Now when fetching the nodes properties we get the following result - the default value is still present:

-        "unsetValueWithDefault": null
+        "unsetValueWithDefault": true

The reason for that is that we filter out the unsets when creating a node (under the impression that no one would do this ^^):

neos-development-collection/Neos.ContentRepository.Core/Classes/Feature/NodeCreation/NodeCreation.php

Line 88 in 7db53b6

$command->initialPropertyValues->withoutUnsets(),

A workaround for now is to followup with a SetNodeProperties command

[kitsunet]

nvm deleted my previous comment I thought unsetsValueWithDefault is a feature here^^ Still seems right behavior?

[mhsdesign]

Hmmm i dont think its right if a followup SetNodeProperties can unset that property like i did here: (because i had to precise tests 😂 and couldnt let it slip)

https://github.com/Flowpack/Flowpack.NodeTemplates/blob/4d1ccfa71674054fe2189d2e984a8c86609d3491/Classes/Domain/NodeCreation/NodeCreationService.php#L81-L92

[kitsunet]

Wait what? I don't think you should be able to unset default values, you can overwrite them with other values but not remove them?

[mhsdesign]

😂 hahah yes sounds legit ... i just wondered a year ago what the behaviour might be and observed that its possible and put it in stone via the node template tests :D
I guess there might be people relying on this ... eg setting an text field back to empty (which is null effectively by the neos ui)