homepageredux: groups page 2 causes 500 error

VERIFIED FIXED in 2013-06-27

Status

Participation Infrastructure
Phonebook
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: hoosteeno, Unassigned)

Tracking

other
2013-06-27

Details

(Whiteboard: [kb=998206][homepageredux])

When you visit this page you get a 500:
https://mozillians-dev.allizom.org/en-US/groups/?page=2

The issue, according to :sancus, is "it's passing in empty string for order_by if none is set in the form".

The traceback is here:


    Traceback (most recent call last):
     
      File "/data/www/mozillians-dev.allizom.org/mozillians/vendor/src/django/django/core/handlers/base.py", line 111, in get_response
        response = callback(request, *callback_args, **callback_kwargs)
     
      File "/usr/lib64/python2.6/site-packages/newrelic-1.10.2.38/newrelic/api/object_wrapper.py", line 220, in __call__
        self._nr_instance, args, kwargs)
     
      File "/usr/lib64/python2.6/site-packages/newrelic-1.10.2.38/newrelic/hooks/framework_django.py", line 475, in wrapper
        return wrapped(*args, **kwargs)
     
      File "/data/www/mozillians-dev.allizom.org/mozillians/apps/groups/views.py", line 29, in index
        query = query.order_by(sort_form.cleaned_data['sort'], 'name')
     
      File "/data/www/mozillians-dev.allizom.org/mozillians/vendor/src/django/django/db/models/query.py", line 756, in order_by
        obj.query.add_ordering(*field_names)
     
      File "/data/www/mozillians-dev.allizom.org/mozillians/vendor/src/django/django/db/models/sql/query.py", line 1686, in add_ordering
        raise FieldError('Invalid order_by arguments: %s' % errors)
     
    FieldError: Invalid order_by arguments: [u'']
Whiteboard: [kb=998206]
https://github.com/mozilla/mozillians/commit/0f829042825140b0499901f78ec8af7f5d59cbe9
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Whiteboard: [kb=998206] → [kb=998206][homepageredux]
Target Milestone: --- → 2013-06-27
QA verified on dev and stage, I'm unable to repro the the 500 error in comment 0.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.