fix: Add check for callable in BooleanBone

[ArneGudermann]

Fix for #1178

[sveneberth]

That's not really useful. A callable defaultValue should be called at runtime:

viur-core/src/viur/core/bones/base.py

Line 358 in 40d4e82

return self.defaultValue(skeletonInstance, self)

otherwise I could BooleanBone(defaultValue=my_func()) directly ...

[ArneGudermann]

Yes, you are right. The whole thing with the default value needs to be revised... . I don't know if we should wait for #1154. Or is it enough to allow callables for now?

Missed conversation.

[sveneberth]

Yes, you are right. The whole thing with the default value needs to be revised... . I don't know if we should wait for #1154. Or is it enough to allow callables for now?

Is okay

my_bone = BooleanBone(...)
my_bone.defaultValue = lambda skel, bone: ...

does the job for now too

[phorward]

Ok,what's with this PR now? It's still flagged for ViUR-core 3.6. Can we close it?

[sveneberth]

Ok,what's with this PR now? It's still flagged for ViUR-core 3.6. Can we close it?

As it's not really solving the problem and it's possible to workaround with a monkey patch I would recommend to close this too and wait for 3.7.