search

tgpfeiffer / restful-openerp

36 stars 13 forks source link

Empty dates and datetime should be False instead of None #11

Open arielmol opened 9 years ago

arielmol commented 9 years ago

Line 567:

[567]            if c.attrib["type"] in ("char", "selection", "text", "datetime"):
                    fields[tagname] = c.text
                    if fields[tagname] == None:
                        fields[tagname] = False
tgpfeiffer commented 9 years ago

Can you explain why that should be so?

arielmol commented 9 years ago

Because it fails otherwise. Try yourself.

tgpfeiffer commented 9 years ago

Could you elaborate on what "it fails" means? For example, you get errors while loading, while storing, the program crashes... etc.? Since I have never run into this problem, I cannot really "try myself" if I don't know what problem I am actually looking for.