zero could be a value

[rycks]

fix #32615 (i hope) :-)

there is a race condition with zero because empty(0) is true ... then we did not display the value

[rycks]

please wait i forget to do the "update/edit" part of the problem

but on clic on 'update' the value will become ''

[rycks]

the last commit do the job for edit step ... but there is something funny now when i try to update the row with "code zero" :

[hregis]

@rycks maybe the unique key for the field

[rycks]

@hregis No ... that is really a test :

now i have to find the reason why zero is prohibited and why some dictionnaries use that number :-)

[rycks]

@hregis ok but tinyint could store (int)0 as a value ... no ?

i dig into commit history, here is the origin of the test: 20 years before :-)

thanks to comments we could understand that race condition : zero and '' could not be desactivated entries ... that's not really what nowdays code do (without comment i can't imagine the real target of that code in current version) ...

[lvessiller-opendsi]

I'm ok with this fix.
@rycks
Have you something else to add for the error message when you update the dictionary line with "0" as code ?