That's like finding an old price tag at a supermarket!  =)  Usually they honor the old price tag rather than the one on the item, so I think the same behavior would be okay here.

Also note that I'm refactoring some of this code into CourseMode.models

On Sep 25, 2013, at 12:09 PM, Jay Zoldak <notifications@github.com> wrote:

> In lms/djangoapps/courseware/views.py:
> 
> > @@ -604,10 +605,31 @@ def course_about(request, course_id):
> >      show_courseware_link = (has_access(request.user, course, 'load') or
> >                              settings.MITX_FEATURES.get('ENABLE_LMS_MIGRATION'))
> >  
> > +    # Note: this is a flow for payment for course registration, not the Verified Certificate flow.
> > +    registration_prices = []
> > +    in_cart = False
> > +    reg_then_add_to_cart_link = ""
> > +    if settings.MITX_FEATURES.get('ENABLE_PAID_COURSE_REGISTRATION'):
> > +        registration_prices = (CourseMode.objects
> > +                               .filter(course_id=course_id,
> > +                                       currency=settings.PAID_COURSE_REGISTRATION_CURRENCY[0],
> > +                                       mode_slug=CourseMode.DEFAULT_MODE_SLUG,)
> > +                               .values_list('min_price', flat=True))
> Edge cases here - if you change the prices after a student already has the course in their shopping cart, then they will still be able to purchase with the old price.
> 
> —
> Reply to this email directly or view it on GitHub.
>