bug report #registration

1 view
Skip to first unread message

hsk

unread,
Aug 18, 2009, 5:56:14 PM8/18/09
to CNProg open discussion
I found a bug under django_authopenid.views in register function.

The last line for the register function (around line 367 ish)

return render('authopenid/complete.html', {
'form1': form1,
'form2': form2,
'provider':providers[provider_name],
'nickname': nickname,
'email': email
}, context_instance=RequestContext(request))


the 'provider' should be
'provider': provider_logo

(you'll see why if you read a few lines of code above line 367. I
didn't copy that part on this message)
So, right now, you can't register with a LiveJournal, Technorati,
Wordpress, etc.

cheers,

HS

Mike Chen

unread,
Aug 19, 2009, 10:10:12 AM8/19/09
to cnp...@googlegroups.com
Thanks for reporting this bug of django_authopenid.
We'll fix this asap.

Regards,
-Mike

hsk

unread,
Aug 19, 2009, 4:46:01 PM8/19/09
to CNProg open discussion
No problem... thank you and the community for this amazing piece of
work.

I gotta admit, this thing is pretty awesome and can't imagine the
amount of work it would have taken me if I tried to build something
like this from scratch.

another small bug (for ppl using Django 1.0.2 final)

line 1128 forum/views.py under user_stats
if settings.DJANGO_VERSION < 1.1:

I think that should be changed to '1.1' since settings.DJANGO_VERSION
will have (at least for me a string containing '1.0.2 final')
when you compare '1.0.2 final' < 1.1, it's going to be false whatever
the version it is. As a result, ppl with Django 1.1 version or higher
won't notice it.

cheers!

HS
On Aug 19, 10:10 am, Mike Chen <cha...@gmail.com> wrote:
> Thanks for reporting this bug of django_authopenid.We'll fix this asap.
>
> Regards,
> -Mike
Reply all
Reply to author
Forward
0 new messages