I had issues with extended_login_form

Line 89, "request = self.auth.environment.request", was throwing an
exception.  Since the example was passing in request it seemed weird
it wasn't being used.  So I changed init

    def __init__(self,
                 request,
                 auth,
                 alt_login_form,
                 signals=[],
                 login_arg = 'login'
                 ):
        self.request = request
        ...

Then I changed line 89 (now 91) to this:

request = self.request

This seems to have fixed the problem.

-Rip

On Jul 28, 8:17 pm, keitheis <keith...@gmail.com> wrote:
> Okay, if you have any question or suggestion please tell me. :)
>
> On Jul 29, 4:46 am, mdipierro <mdipie...@cs.depaul.edu> wrote:
>
>
>
> > No. like inclusion in web2py_src.zip as it is.
>
> > On Jul 28, 11:42 am, keitheis <keith...@gmail.com> wrote:
>
> > > like inclusion_login_form.py ?
>
> > > On Jul 28, 12:50 am, mdipierro <mdipie...@cs.depaul.edu> wrote:
>
> > > > +1. I am thinking about inclusion
>
> > > > On Jul 27, 11:37 am, keitheis <keith...@gmail.com> wrote:
>
> > > > > No problem.
>
> > > > > It extends the built-in login form with external login form.
>
> > > > > Like the bottom of the 
> > > > > slicehttp://www.web2pyslices.com/main/slices/take_slice/28
>
> > > > > Pichttp://www.web2pyslices.com/main/static/share/images/rpxembedlocal.JPG
>
> > > > > Maybe I should name it embedded_auth_form or something else.
>
> > > > > Hope it's not too confusing.
>
> > > > > On Jul 27, 8:59 pm, mdipierro <mdipie...@cs.depaul.edu> wrote:
>
> > > > > > Sorry for my late replay.
>
> > > > > > Can you explain whatextended_login_formdoes?
>
> > > > > > On Jul 25, 12:52 am, keitheis <keith...@gmail.com> wrote:> Hi,
>
> > > > > > > I made theOpenIDsupport for sahana eden. And thanks to Fran Boon, 
> > > > > > > I
> > > > > > > did it most on web2py/gluon/contrib/login_methods . Here is the 
> > > > > > > code:
>
> > > > > > >https://code.launchpad.net/~keitheis/web2py/openid
>
> > > > > > > Fran said that in the long run it's better to have it maintained 
> > > > > > > up
> > > > > > > here.
> > > > > > > So I send it here for the feedback.
>
> > > > > > > Kind Regards,
> > > > > > > Keith Yang

Reply via email to