ProcessLoginPersistent

[adrian]

@Craig - thanks for your work on this. If a cronjob is run it throws this error:

PHP Notice: Undefined index: HTTP_USER_AGENT

Would you mind disabling this module when called via the CLI, eg. PHP_SAPI !== 'cli'

Thanks!

[adrian]

Also a bug:

$this->destroyLogins($user);

needs to be:

$this->destroyLogins($this->wire('user'));

 

[adrian]

Actually, I think it needs to be:

$this->destroyLogins($this->wire('user')->id);

because I am getting this error:

HP Notice: Object of class ProcessWire\User could not be converted to int in /var/www/html/site/assets/cache/FileCompiler/site/modules/LoginPersist/LoginPersist.module:400

[teppo]

Hey @adrian! I'm pretty sure that those issues were already fixed. Could you check that you've got 2.1.0 installed?

[adrian]

4 minutes ago, teppo said:

Hey @adrian! I'm pretty sure that those issues were already fixed. Could you check that you've got 2.1.0 installed?

Thanks @teppo - I actually noticed there was a new version the other day and used the PW Upgrades module to update, but for some reason it's not actually updating the module (I just tried again) - I assumed it had worked. 

Anyway, I manually updated to 2.1.0 and you're correct of course - it seems like they have been fixed.

[Craig]

1 hour ago, adrian said:

used the PW Upgrades module to update, but for some reason it's not actually updating the module (I just tried again) - I assumed it had worked. 

Hi! Glad you've got it updated (in the end)... is the problem you had with PW Upgrades, or something I've done (or not done!) with the LoginPersist module?

[adrian]

1 minute ago, Craig said:

is the problem you had with PW Upgrades, or something I've done (or not done!) with the LoginPersist module?

Honestly not sure - maybe the permissions on that file got messed up? I expect it was something locally specific.

[Craig]

Ah OK! Thanks. All's well that ends well :) 

[Arne]

It seems that the latest version links to the V1.0.0 zip file. Both from updating within module as the PW module page download button.
I've downloaded the correct zip file from Github for now.

[da²]

Hello @Craig

I don't know if you get notifications from Github issues, so I notice you here that your module has a huge bug, it just doesn't manage cookie expiration date.

Greetings, and thanks for your work.

 

[Craig]

Thanks @da²! Cookie expiration time issue has now been fixed and version 2.3.0 released.

[Andy]

On 10/24/2019 at 8:10 PM, tires said:

Hi!

Great Module.

I just want to have a "remember me" checkbox in the login form.
So the user can decide ...
Ho can i add this?