django 1.5 compatibility

Bug #1170355 reported by Richard Marko
32
This bug affects 6 people
Affects Status Importance Assigned to Milestone
django-openid-auth
Fix Released
Undecided
Unassigned
django-openid-auth (Ubuntu)
Fix Released
High
Unassigned

Bug Description

Hello,

django-openid-auth seems to work fine with Django 1.5 after {% url openid-logo %} is replaced with {% url 'openid-logo' %} in templates/openid/login.html.

There seem to be some minor test failures as well:

./manage.py test django_openid_auth

.FF
======================================================================
FAIL: test_admin_site_with_openid_login_authenticated_non_staff (django_openid_auth.tests.test_admin.SiteAdminTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/django_openid_auth/tests/test_admin.py", line 74, in test_admin_site_with_openid_login_authenticated_non_staff
    response.content, 'Missing error message in response')
AssertionError: Missing error message in response

======================================================================
FAIL: test_admin_site_with_openid_login_non_authenticated_user (django_openid_auth.tests.test_admin.SiteAdminTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/django_openid_auth/tests/test_admin.py", line 82, in test_admin_site_with_openid_login_non_authenticated_user
    self.assertEqual(302, response.status_code)
AssertionError: 302 != 404

----------------------------------------------------------------------
Ran 61 tests in 1.696s

FAILED (failures=2)

Revision history for this message
Luke Faraone (lfaraone) wrote :

This was fixed by a patch provided by asb in Debian.

Changed in django-openid-auth (Ubuntu):
status: New → Fix Released
importance: Undecided → High
Changed in django-openid-auth:
status: New → Confirmed
Revision history for this message
Luke Faraone (lfaraone) wrote :
Revision history for this message
Andrew Starr-Bochicchio (andrewsomething) wrote :
Changed in django-openid-auth:
status: Confirmed → Fix Committed
Changed in django-openid-auth:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.