IRC logs of #tryton for Saturday, 2012-04-28

chat.freenode.net #tryton log beginning Sat Apr 28 00:00:01 CEST 2012
2012-04-28 16:15 <cedk> smarro: about issue2572, I don't think it was something supported
2012-04-28 16:45 <smarro> cedk: your patch at review340003 works great
2012-04-28 16:47 <cedk> smarro: ok but I'm pretty sure it was broken since some days
2012-04-28 16:47 <cedk> smarro: and it is not linked to the changeset you pointed
2012-04-28 16:49 <smarro> cedk: we did not realize a bug in savannah https://savannah.gnu.org/bugs/?36218 (i didn't get the mail of the bug...)
2012-04-28 16:50 <smarro> cedk: because in Health, we use this kind of declaration... .and before we had no problems with this
2012-04-28 16:51 <smarro> cedk: but if i remove that changeset, it works fine
2012-04-28 16:52 <smarro> cedk: i made a little module just to check that, i can upload to bug if you want...
2012-04-28 18:05 -!- smarro_(~sebastian@190.105.90.39) has left #tryton
2012-04-28 18:37 <cedk> smarro: it is really strange because the changeset has nothing to do with ordering
2012-04-28 18:38 <smarro> cedk: yes, it's strange
2012-04-28 18:57 <meanmicio> cedk : ping
2012-04-28 19:11 <cedk> meanmicio: pong
2012-04-28 19:12 <meanmicio> cedk: It looks like the unique contraint in non-required fields do not accept a null value in 2.4
2012-04-28 19:12 <cedk> meanmicio: don't understand
2012-04-28 19:14 <meanmicio> cedk: I have a "ref" field (Social Security Number - SSN) that is a non-required Char field, but, when filled, it looks for uniqness
2012-04-28 19:15 <meanmicio> cedk: until 2.2, the field could be empty.
2012-04-28 19:15 <cedk> meanmicio: it is the database job
2012-04-28 19:15 <cedk> meanmicio: ok, it is because it is fill with empty string
2012-04-28 19:16 <meanmicio> cedk : I created a constraint 'UNIQUE(ref)'
2012-04-28 19:16 <meanmicio> cedk : but this did not happen in 2.2
2012-04-28 19:16 <cedk> meanmicio: yes because before the behavior was not consistent
2012-04-28 19:17 <cedk> meanmicio: we can not represent a NULL char in the client
2012-04-28 19:17 <meanmicio> cedk : let me double check in this version
2012-04-28 19:17 <meanmicio> cedk : maybe I put a space char without noticing before saving
2012-04-28 19:18 <cedk> meanmicio: no the client set by default empty string for char fields
2012-04-28 19:19 <meanmicio> cedk : hmm
2012-04-28 19:20 <meanmicio> cedk : yes. Actually I can confirm it. The party created associated to the company has an empty SSN, but since it's not created by the client, it does not give an error
2012-04-28 19:21 <meanmicio> cedk : how can we find a workaround '
2012-04-28 19:21 <meanmicio> cedk : to this issue ?
2012-04-28 19:21 <meanmicio> cedk : because it will be a very common situation
2012-04-28 19:26 <cedk> meanmicio: I think you can “clean” values in create/write
2012-04-28 19:29 <meanmicio> cedk : But probably it would be better to have it in the Tryton kernel
2012-04-28 19:30 <meanmicio> cedk : Because that would happen in every "empty" field that has a unique constraint
2012-04-28 19:34 <cedk> meanmicio: if you have idea on how to display in the client of course
2012-04-28 19:40 <meanmicio> cedk : but how was represented in the previous version ?
2012-04-28 19:52 <cedk> meanmicio: it was not
2012-04-28 19:53 <cedk> we have the same problem as we had with numeric fields
2012-04-28 19:53 <cedk> but we did not yet find a solution
2012-04-28 19:56 <cedk> indeed the problem is to know per Char field what is the value to be stored for empty widget
2012-04-28 19:56 <cedk> until we have a clean solution, it must be managed by override of create/write
2012-04-28 19:57 <meanmicio> cedk : ok. I'm positive we'll find a way around soon.
2012-04-28 19:59 <meanmicio> cedk : I agree that in numeric values, it's now way better not to store a "0"
2012-04-28 20:06 <meanmicio> cedk : Specially in clinical settings. I'm checking now how it looks the patient clinical evaluation. Much better

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!