Actually, I didn't write this code.  Seems like @rocha did (and he probably didn't intend to instrument PaidCourseRegistration).  Mind if I just remove it and leave just the log?

On Sep 26, 2013, at 11:02 AM, David Ormsbee <notifications@github.com> wrote:

> In lms/djangoapps/shoppingcart/models.py:
> 
> > @@ -301,8 +363,9 @@ def purchased_callback(self):
> >  
> >          CourseEnrollment.enroll(user=self.user, course_id=self.course_id, mode=self.mode)
> >  
> > -        log.info("Enrolled {0} in paid course {1}, paid ${2}".format(self.user.email, self.course_id, self.line_cost))
> > -        org, course_num, run = self.course_id.split("/")
> > +        log.info("Enrolled {0} in paid course {1}, paid ${2}"
> > +                 .format(self.user.email, self.course_id, self.line_cost))  # pylint: disable=E1101
> > +        org, course_num, run = self.course_id.split("/")                    # pylint: disable=E1101
> Please make the course object and use its attributes instead of relying on course_ids having a particular format (the current convention may change in the not too distant future).
> 
> —
> Reply to this email directly or view it on GitHub.
>