I think we're safe here, since UDF fields handle their own
serialization/deserialization. There's also a test that verifies you can't
create audits with bad data.


On Tue, Sep 3, 2013 at 10:26 AM, Justin Walgran <notifications@github.com>wrote:

> In opentreemap/treemap/audit.py:
>
> > @@ -852,7 +875,10 @@ def _deserialize_value(self, value):
> >
> >      @property
> >      def clean_current_value(self):
> > -        return self._deserialize_value(self.current_value)
> > +        if self.field.startswith('udf:'):
> > +            return self.current_value
> > +        else:
> > +            return self._deserialize_value(self.current_value)
>
> Can/should we skip passing sub-field values through _deserialize_value?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/azavea/OTM2/pull/268/files#r6126905>
> .
>



-- 
Adam Hinz
Software Engineer, Azavea
hinz.adam@gmail.com | c: 608-445-8391 | Twitter: @ahinz