Yeah you are correct. It is a bit different than the command you had. On
the other hand it lets you choose between completely different module
stores. It seems to me to be a bit cleaner since we don't have to modify
the Django settings.


On Fri, Oct 11, 2013 at 2:54 PM, chrisndodge <notifications@github.com>wrote:

> In lms/djangoapps/courseware/management/commands/dump_course_structure.py:
>
> > +    option_list = BaseCommand.option_list + (
> > +        make_option('--modulestore',
> > +                    action='store',
> > +                    default='default',
> > +                    help='Name of the modulestore'),
> > +    )
> > +
> > +    def handle(self, *args, **options):
> > +        if len(args) != 1:
> > +            raise CommandError("course_id not specified")
> > +
> > +        # Get the modulestore
> > +
> > +        try:
> > +            name = options['modulestore']
> > +            store = modulestore(name)
>
> Question: this will require that there is a full definition of modulestore
> configuration in an env .py file. right? That should be fine, but it's
> slightly different than the way that I'd done in the past - I just
> overloaded the 'db' property
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/1289/files#r6923637>
> .
>