I guess I was wondering, is possible under any conceivable circumstances
that that value will be unset? But if it's not, then no worries. :)


On Thu, Sep 19, 2013 at 2:54 PM, Vik Paruchuri <notifications@github.com>wrote:

> In
> common/lib/xmodule/xmodule/open_ended_grading_classes/self_assessment_module.py:
>
> > @@ -218,7 +214,7 @@ def save_assessment(self, data, _system):
> >              return self.out_of_sync_error(data)
> >
> >          try:
> > -            score = int(data['assessment'])
> > +            score = int(data.get('assessment'))
>
> Assessment is the score the student gave themselves. I can't think of a
> case where we would want a default for that.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/1068/files#r6474500>
> .
>