Ok, I'll try to trap the error I found and fix the test to use a draft
module.


On Wed, Mar 5, 2014 at 10:02 AM, Calen Pennington
<notifications@github.com>wrote:

> In cms/djangoapps/contentstore/views/component.py:
>
> > @@ -370,6 +369,11 @@ def component_handler(request, usage_id, handler, suffix=''):
> >          log.info("XBlock %s attempted to access missing handler %r", descriptor, handler, exc_info=True)
> >          raise Http404
> >
> > -    modulestore().save_xmodule(descriptor)
> > +    # unintentional update to handle any side effects of handle call; so, request user didn't author
> > +    # the change
> > +    if getattr(descriptor, 'is_draft', False):
> > +        modulestore('draft').update_item(descriptor, None)
> > +    else:
>
> component_handler is only called by the editing view, which is only
> displayed when already in draft mode. preview_handler is used by the
> preview student view, which is displayed in either draft or published mode,
> but doesn't have permissions to edit settings or content.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/2738/files#r10302167>
> .
>