Are you saying that groups won't get the error that groupdicts got claiming
_id.org was an illegal key?


On Mon, Aug 12, 2013 at 9:09 AM, Calen Pennington
<notifications@github.com>wrote:

> In common/lib/xmodule/xmodule/modulestore/split_mongo/split.py:
>
> > @@ -1238,3 +1528,56 @@ def _update_head(self, index_entry, branch, new_id):
> >          self.course_index.update(
> >              {"_id": index_entry["_id"]},
> >              {"$set": {"versions.{}".format(branch): new_id}})
> > +
> > +    def _interpret_location_course_id(self, course_id, location):
> > +        """
> > +        Take the old style course id (org/course/run) and return a dict for querying the mapping table.
> > +        If the course_id is empty, it uses location, but this may result in an inadequate id.
> > +
> > +        :param course_id: old style 'org/course/run' id from Location.course_id where Location.category = 'course'
> > +        :param location: a Location object which may be to a module or a course. Provides partial info
> > +        if course_id is omitted.
> > +        """
> > +        if course_id:
> > +            # re doesn't allow ?P<_id.org> and ilk
> > +            m = re.match(r'([^/]+)/([^/]+)/([^/]+)', course_id)
> > +            return dict(zip(['_id.org', '_id.course', '_id.name'], m.groups()))
>
> You could actually just capture the groups as (?P<_id.org>...) and then
> just use m.groups() directly
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/601/files#r5708683>
> .
>