This started entirely in the template, using shoppingcart.Order.has_items().  I decided to make a utility function to unit test the anonymous user case, and because of the "user" element decided to put it in student.  I'm not opposed to having it in shopping cart thought, so I'll move it.
On Sep 25, 2013, at 7:46 AM, Diana Huang <notifications@github.com> wrote:

> In common/djangoapps/student/views.py:
> 
> > @@ -1374,3 +1388,14 @@ def change_email_settings(request):
> >          track.views.server_track(request, "change-email-settings", {"receive_emails": "no", "course": course_id}, page='dashboard')
> >  
> >      return HttpResponse(json.dumps({'success': True}))
> > +
> > +
> > +def _user_cart_has_items(user):
> I'm not a fan of putting this here. Why in student.views instead of in the shoppingcart djangoapp?
> 
> —
> Reply to this email directly or view it on GitHub.
>