| 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/!