cdodge/update-opaque-key: update user social stats to use the legacy cou...

...rseId format since that is what the comment service expects
Apros is the only caller here.

Sent from my iPhone

On Aug 6, 2014, at 4:52 PM, Matt Drayer <notifications@github.com> wrote:

> In lms/djangoapps/api_manager/users/views.py:
> 
> > @@ -1073,7 +1075,10 @@ def get(self, request, user_id, course_id):  # pylint: disable=W0613,W0221
> >              return Response({}, status.HTTP_404_NOT_FOUND)
> >  
> >          comment_user = CommentUser.from_django_user(user)
> > -        comment_user.course_id = course_id
> > +
> > +        # comment service uses old style of keys
> > +        course_key = CourseKey.from_string(course_id)
> Are we sure that course_id is always a new-style identifier? If it's not you'll get an opaque keys InvalidKeyError when you try to run the from_string() method
> 
> —
> Reply to this email directly or view it on GitHub.