search

panique / huge

Simple user-authentication solution, embedded into a small framework.
2.14k stars 788 forks source link

password verify not compatible with PHP Version 5.4.7 #461

Closed rajsquid closed 10 years ago

rajsquid commented 10 years ago

I think inbuilt function password_verify inside login model is not compatible with php 5.4.7 as it runs under 5.5.x but not under 5.4

Is there any other way around.

Thanks.

GrahamCampbell commented 10 years ago

Yes it is. https://github.com/ircmaxell/password_compat.

rajsquid commented 10 years ago

Ahh...Sorry mate just overlooked that link.

@GrahamCampbell Thanks.I will go through it now.

rajsquid commented 10 years ago

All good thanks a lot.

Seeking for a suggestion. I put the password.php under libs folder and calling it in index.php as require 'application/libs/password.php';

Do you think this is the right way or is there any better way to do this.

panique commented 10 years ago

@rajsquid He man, you dont need to put anything anywhere :), Simply follow the install tutorial and it will work fine

rajsquid commented 10 years ago

Sir you need to copy that file somewhere in the folder and call it otherwise it won't work

GrahamCampbell commented 10 years ago

I find it strange that people can come here and not know about composer.

panique commented 10 years ago

@GrahamCampbell Totally agree! And even if somebody has its first day with PHP, well, then the tutorial gives enough links & infos to go to the smaller versions, read into composer, etc. ... It's really weird that so many people seriously skip the readmes, skip the instlal tutorials, skip basically everything and spend days trying to find out how to install everything. Super-strange!

GrahamCampbell commented 10 years ago

GitHub needs a "did you read the readme" warning before somebody summits their first issue to a repo. I'm actually going to contact support and ask for this...

panique commented 10 years ago

@GrahamCampbell !!!!!!!!!!!!!!!!!! SO TRUE !!! This is indeed in general issue in society: Most (?) humans have problems recognizing the point of "maybe I should read the manual now" and rather spend lots of time trying to find out how things work (and then quit frustrated), even if a quick research or RTFM would be much quicker. To be fair: This project doesn't really have a manual, but the readme answers nearly every question I ever got from users. This is really interesting, and the rise of technology in every part of life will make this topic even bigger in the future.

Excellent subject for a master thesis or something...

rajsquid commented 10 years ago

Sorry guys once again for opening a issue without knowing it well enough.Actually I have a bit different scenario and that's why I have chosen not to install external stuffs via composer. My app is very very simple and I don't need captcha or any social logins.For future improvements I am thinking to write a small code snippet where you can tell the app which feature to turn off or on via config. :)

rajsquid commented 10 years ago

Now the 'after effect' shock is that I am missing vendor/autoload.php script. :(

panique commented 10 years ago

@rajsquid Okay, just start over from zero and do exactly like the install tutorials says (and then everything will work fine). vendor/autoload.php is automatically generated by Composer. It's not possible to skip tutorial steps, this wouldnt work!

Alternativly, start with the smaller versions of the script.

rajsquid commented 10 years ago

Thanks @panique I have done a fresh install(with composer) and it went well.

panique commented 10 years ago

@rajsquid :+1: