A little feedback before I bug log them :P
First, MySQLi needed, could be a tad clearer for us dumber lot.
Warning: file_put_contents(/home/xj45/public_html/forum/db_last_error.php): failed to open stream: Permission denied in /home/xj45/public_html/forum/Settings.php on line 192
If we're going to need db_last_error.php it should be part of the initial permissions check on install and fixable then.
Warning: is_readable(): open_basedir restriction in effect. File(/dev/urandom) is not within the allowed path(s): (/home/xj45:/usr/lib/php:/usr/local/lib/php:/tmp) in /home/xj45/public_html/forum/sources/ext/PasswordHash.php on line 51
@is_readable? may be better?
Why is MySQLi needed?
http://it1.php.net/manual/en/function.mysql-db-query.php
Do you see the terribly big red warning that the mysql is going to be dropped? :P
Meant to say though, its a very friendly install :)
Oh I didn't read the first line... lol
Sorry.
There was a similar report about this, we need to check the dependancy up front and complain more nicely.
Maybe we should just include that empty file in the install package and be done with it.
Probably yes
Meh. Big fuss about nothing. :P
db_last_error here: https://github.com/emanuele45/Dialogo/commit/c0013d7ae65710450cf465285a9bf5414d5edfbe
phpass here: https://github.com/emanuele45/Dialogo/commit/84aa4bcf13a78652b02226aa260cf237c42cce46
Thanks !
There is still the MySQLi thing pending.
Nice :D
Did anyone remember if the MySQLi warning was improved?
Just tried it with without 'i' ...
So the message still needs improvement, plus its not inserting in the supported scheme in to the string
hmm...
I think that this:
if (count($db_missing) === count($db))
should be:
if (count($db_missing) === count($databases))
Apparently this is enough to trigger the proper sprintf.