ParasBotadra / django-email-confirmation

Automatically exported from code.google.com/p/django-email-confirmation
0 stars 0 forks source link

r43 breaks reverse(), thus breaks sending email confirmations #7

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
In the calls to reverse() added in r43, the second argument to reverse() is
given as [confirmation_key] where it should be a keyword argument,
args=[confirmation_key].  The second positional argument to reverse()
should be the name of a URLconf.  Because this is passed to get_resolver,
which is memoized, the use of a list here results in a very confusing
"lists are unhashable" error.

Adding args= in both calls to reverse() fixes the problem.

Original issue reported on code.google.com by carl.j.meyer on 21 Jan 2009 at 6:14

GoogleCodeExporter commented 8 years ago
Patch attached.

Original comment by carl.j.meyer on 21 Jan 2009 at 6:15

Attachments:

GoogleCodeExporter commented 8 years ago
Fixed in r44

Original comment by leidel on 21 Jan 2009 at 8:02