19 lines
867 B
Markdown
19 lines
867 B
Markdown
This plugin not only locks pages but ensures too a user is logged in. This
|
|
seems to me redundant with signedit. I propose [removing the if block that
|
|
calls needsignin ].
|
|
|
|
> That was added because the most typical reason for being unable to edit a
|
|
> page is that you are not logged in. And without the jump to logging the
|
|
> user in, there is no way for the user to log in, without navigating away
|
|
> from the page they were trying to edit. --[[Joey]]
|
|
|
|
>> Ok, but the problem is that when you don't want any signin form you end up
|
|
>> with a lone login button. That might happend if you lock pages only on IP
|
|
>> adresses, if you use another cookie from another webapp...
|
|
|
|
>> That happends to me and I had to reimplement lockedit in my private auth
|
|
>> plugin.
|
|
|
|
>> Perhaps you could return undef on that case and let another plugin do the
|
|
>> needsignin call ? -- [[Jogo]]
|