Added check for non-nullable property when deleting a record

[joaojacome]

Solves #7999

[beberlei]

This needs to be fixed in doctrine/persistence instead. We already have a TypedNoDefaultReflectionProperty overwrite for ReflectionProperty there and it only needs an addition to setValue() to handle the NULL case as unset(). The problem is your unset() code here only works on public properties, it will cause an error on protected or private ones.

A workaround needs to be found to unset any kind of visibility property. The following is a prototype using closure and binding to this, but it doesn't seem to work: https://gist.github.com/beberlei/f28142a1b15f011bbcd0156c7aa64430

[beberlei]

Have the right approach thanks to @lcobucci now, updated the gist.

[prof-web-coding]

@joaojacome Will you update the code according to @beberlei 's input?

[joaojacome]

@joaojacome Will you update the code according to @beberlei 's input?

I could, but I think @beberlei fixed it here: doctrine/reflection#34

[prof-web-coding]

Thanks. It does not work for me. I have a private int $id; in an Doctrine\ORM\Mapping\Entity and I'm gettingError: Cannot access private property xxx::$id. It seems that the newly added setValue in doctrine/reflection#34 is not called, with or without 23b57f6