That was leftover from my attempt to create a meaningful exception
__unicode__ method. I'll remove it.


On Mon, Sep 23, 2013 at 10:22 AM, Calen Pennington <notifications@github.com
> wrote:

> In common/lib/xmodule/xmodule/modulestore/exceptions.py:
>
> > @@ -28,7 +28,14 @@ class NoPathToItem(Exception):
> >
> >
> >  class DuplicateItemError(Exception):
> > -    pass
> > +    """
> > +    Attempted to create an item which already exists.
> > +    """
> > +    def __init__(self, element_id, store=None, collection=None):
> > +        super(DuplicateItemError, self).__init__()
> > +        self.element_id = element_id
> > +        self.store = store or {'db': 'unset'}
>
> This is a bit strange. Why not just leave self.store as None? That seems
> like a much clearer way of indicating that it was unset.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/1029/files#r6515681>
> .
>