Christina, can you confirm this? If so, then I can refactor this - as well
as updating the existing test cases.


On Fri, Jul 12, 2013 at 12:22 PM, Calen Pennington <notifications@github.com
> wrote:

> In common/lib/xmodule/xmodule/xml_module.py:
>
> >      return json.dumps(value, cls=EdxJSONEncoder)
> >
> >
> > +def serialize_string_literal(
> value):
> > +    """
> > +    Assert that the value is a base string and - if it is - simply return it
> > +    """
> > +    if not isinstance(value, basestring):
> > +        raise Exception('Value {0} is not of type basestring!'.format(value))
> > +
> > +    return value
>
> serialize_field should really only be for serializing fields to xml,
> which is exactly when we want this behavior.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/387/files#r5168720>
> .
>