Ah makes sense. Just sets my Spidey senses off to see gemeral Exception
being caught.
On Aug 11, 2013 9:59 PM, "Ned Batchelder" <notifications@github.com> wrote:

> In xblock/core.py:
>
> > @@ -730,12 +730,16 @@ def __repr__(self):
> >          # pylint: disable=E1101
> >          attrs = []
> >          for field in self.fields:
> > -            value = getattr(self, field.name)
> > -            if isinstance(value, basestring):
> > -                value = value.strip()
> > -                if len(value) > 40:
> > -                    value = value[:37] + "..."
> > -            attrs.append(" %s=%r" % (field.name, value))
> > +            try:
> > +                value = getattr(self, field.name)
> > +            except Exception:
>
> I want to make sure that whatever problem occurs, it doesn't stop the
> repr() from producing a string. So I catch Exception. Did you want to catch
> more or less?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/XBlock/pull/62/files#r5702140>
> .
>