Issue Details (XML | Word | Printable)

Key: MDL-14864
Type: Sub-task Sub-task
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Petr Skoda
Reporter: Dongsheng Cai
Votes: 0
Watchers: 2
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle
MDL-14679

Installer script is broken

Created: 16/May/08 01:12 PM   Updated: 20/May/08 02:07 AM
Return to search
Component/s: Installation
Affects Version/s: 2.0
Fix Version/s: 2.0

File Attachments: 1. Text File installer_and_other_5.patch (64 kB)
2. Text File installer_and_other_6.patch (64 kB)

Environment: Ubuntu, PHP5, Mysql 5, Apache2

Database: MySQL
Participants: Dongsheng Cai, Eloy Lafuente (stronk7) and Petr Skoda
Security Level: None
QA Assignee: Dongsheng Cai
Resolved date: 20/May/08
Affected Branches: MOODLE_20_STABLE
Fixed Branches: MOODLE_20_STABLE


 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
Petr Skoda added a comment - 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

Petr Skoda added a comment - 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

Petr Skoda added a comment - 19/May/08 05:55 AM
passing to Eloy

Petr Skoda added a comment - 19/May/08 04:18 PM
fresh patch

Eloy Lafuente (stronk7) added a comment - 20/May/08 02:07 AM
Reviewed, tested under MySQL, 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!