Login view mixup

[aellerton]

Login view mixup

[aellerton]

Hi Brad,
That's a strange one. Did it render the fields in that order out of the
box, or were there any changes you made locally at all?
A

On 17 Aug 2016 9:58 AM, "Brad Jimenez" [email protected] wrote:

I have really enjoyed this template, but i'm having some issues with the
view on the Login page. I'm very new to python and Django, so i'm a little
confused as to how I could fix this. I traced the url back into the allauth
directory inside the virtual environment and found where the login field
order was referenced, but changing this had no effect at all. I've never
tried to change anything close to this file location so i'm not sure what I
would do exactly. The good news is that it still works perfectly!

I am on Linux Ubuntu 14.04 using
Django==1.8.1
django-allauth==0.22.0
django-bootstrap3==7.0.1

[image: djangologin]
https://cloud.githubusercontent.com/assets/17770993/17721861/14e34330-63f3-11e6-8b61-4a81c1a51bba.png

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#16, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7Eoo0KD7AkJrnTYIyVL484_0CQxOv8ks5qgmq7gaJpZM4JmBUl
.

[CadBrad]

This is how it came out of the box.

The other thing is that the Signup page looks perfect, with email, password, and password again.

[aellerton]

Thanks.

Could be that allauth has had various releases since I last updated the
code.

I'll try to get time to pull the latest and see if that affects it...

On 17 Aug 2016 10:08 AM, "Brad Jimenez" [email protected] wrote:

This is how it came out of the box.

The other thing is that the Signup page looks perfect, with email,
password, and password again.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#16 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA7EonESjAxHua96TFebChditjUJ4DBnks5qgm0lgaJpZM4JmBUl
.

[CadBrad]

Sounds great thanks so much!

Fixed:

in allauth/utils.py:

Replace the following function w/ what's below:

def set_form_field_order(form, fields_order):
    import collections
    if isinstance(form.fields, SortedDict):
        form.fields = collections.OrderedDict((f, form.fields[f])
                                  for f in fields_order)
    else:
        # Python 2.7+
        from collections import OrderedDict
        assert isinstance(form.fields, OrderedDict)
        form.fields = collections.OrderedDict((f, form.fields[f])
                                  for f in fields_order)

[CadBrad]

Works perfect for me, thanks for the speedy reply.

[aellerton]

@chabura do you mean you needed to modify the allauth source itself?

On 18 August 2016 at 03:06, Brad Jimenez [email protected] wrote:

Works perfect for me, thanks for the speedy reply.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#16 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA7EoucE3fTjQ19ObpBpYsJ7asxA0hoaks5qg1vTgaJpZM4JmBUl
.

yes