698548 - [traceback] ValueError: invalid literal for int() with base 10: 'x'

Status: VERIFIED FIXED

(Participation Infrastructure :: Phonebook, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

https://mozillians-dev.allizom.org/en-US/search?q=%40mozilla.com&limit=x&page=1:

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 "/data/www/mozillians-dev.allizom.org/mozillians/vendor/src/django/django/contrib/auth/decorators.py", line 19, in _wrapped_view
    return view_func(request, *args, **kwargs)

  File "/data/www/mozillians-dev.allizom.org/mozillians/apps/phonebook/views.py", line 42, in wrapped
    return f(request, *args, **kwargs)

  File "/data/www/mozillians-dev.allizom.org/mozillians/apps/phonebook/views.py", line 233, in search
    limit = int(request.GET.get('limit'))

ValueError: invalid literal for int() with base 10: 'x'

Comment 1

[[github robot]]

Commits pushed to https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/d35f8e3bd03b8ee07575c66ebb71d08ea774bb1c
fixed invalid limit param; fix bug 698548

https://github.com/mozilla/mozillians/commit/9b056b606e5760dae11f8df9dc039904260d1eed
Merge pull request #48 from ednapiranha/bug698548

fixed invalid limit param; fix bug 698548

Comment 2

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED; thanks for the quick turnaround!