Skip to content
This repository has been archived by the owner on Apr 8, 2023. It is now read-only.

$this->getCode not compatible with PHP7 #32

Open
jfm-so opened this issue Sep 7, 2017 · 4 comments
Open

$this->getCode not compatible with PHP7 #32

jfm-so opened this issue Sep 7, 2017 · 4 comments
Assignees

Comments

@jfm-so
Copy link
Owner

jfm-so commented Sep 7, 2017

"In the classes/User.php file, you use this->getCode() which is a builtin method in php7 at least. This causes 2 factor authentication not to work. I rectified this in my instance by changing it to getCodePin() as the function/method name, but to each his own." - TW (via eMail)

@jfm-so jfm-so self-assigned this Sep 7, 2017
@jfm-so jfm-so closed this as completed Sep 7, 2017
@jfm-so
Copy link
Owner Author

jfm-so commented Sep 7, 2017

Didn't mean to close. piWallet is not officially compatible with PHP7 to begin with, but I will make this change.

@jfm-so jfm-so reopened this Sep 7, 2017
@NicoDFS
Copy link

NicoDFS commented Jul 12, 2018

Is piWallet now working with PHP7, I've been trying to setup using ubuntu 16.04 with PHP7 and Nginx as the web server but having some issues. Do I need to start over with PHP 5 and use apache2?

@jfm-so
Copy link
Owner Author

jfm-so commented Jul 13, 2018

Please use PHP5 for the time being - and I don't plan on having any future compatibility with NGINX or WAMP, for that matter.

@NicoDFS
Copy link

NicoDFS commented Jul 13, 2018

yup, reverting back to php5.6 and apache2 did the trick, thanks

@jfm-so jfm-so closed this as completed Jul 19, 2018
@jfm-so jfm-so reopened this Jul 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants