Install prnMsg not displayed #172
Replies: 4 comments
-
I have done the change on the error message, and pushed the commit. I will need to think how to do the other change you suggest. Tim |
Beta Was this translation helpful? Give feedback.
-
@andrewcouling thinking about this there are a number of places in the installer where prnMsg() is called, and as header.php and footer.php are never used then the messages won't get displayed using the new system. Is it worth having a prnMsgInstaller() function? |
Beta Was this translation helpful? Give feedback.
-
All |
Beta Was this translation helpful? Give feedback.
-
Fixes applied, many thanks. tim |
Beta Was this translation helpful? Give feedback.
-
The
install/index.php
file makes an initial check for theconfig.php
file and is expected to exit after displaying a message. Thefunction prnMsg()
merely appends the message to an array to be displayed at the end of the script, but the very next statement isexit;
which makes sure that the script gets truncated thereat.Hence, replace line 156 of
install/index.php
:with
Furthermore, if the database exists and no tables exist in it, then, the script should be allowed to skip over db creation and
config.php
creation and proceed to database table creation.Beta Was this translation helpful? Give feedback.
All reactions