[#MDL-14864] Installer script is broken

Moodle

MDL-14679

Installer script is broken

Created: 16/May/08 01:12 PM Updated: 20/May/08 02:07 AM

Component/s:

Installation

Affects Version/s:

2.0

Fix Version/s:

2.0

File Attachments:	1. installer_and_other_5.patch (64 kb) 2. installer_and_other_6.patch (64 kb)
Environment:	Ubuntu, PHP5, Mysql 5, Apache2

Database:	MySQL
Participants:	Dongsheng Cai, Eloy Lafuente (stronk7) and Petr Škoda
Security Level:	None
QA Assignee:	Dongsheng Cai

Description

« Hide

When I installing a new moodle (HEAD), the installation stopped in environment checking page, it said:

Checking your environment ...

We are checking if the various components of your system meet the system requirements

Fatal error: Call to a member function setup_is_unicodedb() on a non-object in /var/www/m20/lib/environmentlib.php on line 707

All

Comments

Change History

Version Control

Sort Order:

Ascending order - Click to sort in descending order

[ Permlink | « Hide ]

Petr Škoda - 16/May/08 02:40 PM

yes, the installer does not work yet, it is still heavily depending on adodb and odl dmllib
workaround is to create config.php manually

[ Permlink | « Hide ]

Petr Škoda - 19/May/08 05:55 AM

sending a patch that solves several issues:
* installer works again (db creation still missing though)
* the prefix checks moved to database classes again
* detection of missing php extensions added
* new support for $external (non-moodle) databases
* added driver specific options
* connect info moved from constructor to connect() method
* fixed print_error() before db connected and themes setup
* other minor changes and API improvements

[ Permlink | « Hide ]

Petr Škoda - 19/May/08 05:55 AM

passing to Eloy ;-)

[ Permlink | « Hide ]

Petr Škoda - 19/May/08 04:18 PM

fresh patch

[ Permlink | « Hide ]

Eloy Lafuente (stronk7) - 20/May/08 02:07 AM

Reviewed, tested under MySQL(i), PostgreSQL and MSSQL.

I only have:

- moved some PHP docs from __construct() to connect()
- added some TODO comments (plz, check them in commit message)

Resolving this as fixed. Thanks!