http://localhost/ticket/1339 <p> I ran into a bug with the size field on resources. </p> <ul><li>It would not accept an empty value from form (IMO this clearly equates to null/None) </li><li>This could be fixed via using ignore_empty instead of ignore_missing </li><li>However using this means there was no way to empty the field (e.g. i may just want to set the size field back to null not just change to another value) </li><li>similar issues could arise around other fields (such as last_modified ...) </li></ul><p> To solve this i create an int_converter temporarily in logic/schema.py (this is almost certainly the wrong place). But I think it raises a bigger issue about the conversion layer and how it works. </p> en-us http://assets.okfn.org/p/ckan/img/ckan_logo_shortname.png http://localhost/ticket/1339 Trac 0.12.3 rgrp Tue, 13 Sep 2011 20:58:00 GMT http://localhost/ticket/1339#comment:1 http://localhost/ticket/1339#comment:1 <ul> <li><strong>description</strong> modified (<a href="/ticket/1339?action=diff&amp;version=1">diff</a>) </li> </ul> Ticket kindly Wed, 14 Sep 2011 14:30:07 GMT http://localhost/ticket/1339#comment:2 http://localhost/ticket/1339#comment:2 <ul> <li><strong>cc</strong> <em>rgrp</em> added </li> </ul> <p> I have fixed the isodata and made a slightly modified int_converter for this case. In the correct place and raises invalids on not being an int. cset:a4af115116bb </p> <p> The thinking was that the input of these fields would be through the api so the empty string case did not arise. These should clearly be converted to None (Null). </p> <p> What issues in general? Having done this lots of times before you always end up needing to write your own little validators as the standard ones never do what you want. Thats the point of them. Look in ckan/lib/validators if you need examples. </p> <p> So what you did was correct... </p> Ticket kindly Wed, 14 Sep 2011 15:42:34 GMT http://localhost/ticket/1339#comment:3 http://localhost/ticket/1339#comment:3 <ul> <li><strong>status</strong> changed from <em>new</em> to <em>closed</em> </li> <li><strong>resolution</strong> set to <em>fixed</em> </li> </ul> Ticket