|
|
| Participants: |
Eloy Lafuente (stronk7) and Petr Skoda
|
| Security Level: |
None
|
| Resolved date: |
22/May/08
|
| Affected Branches: |
MOODLE_20_STABLE
|
| Fixed Branches: |
MOODLE_20_STABLE
|
|
Right now prefix rules are executed by the xxx_yyy_moodle_database constructor.
That causes 2 problems:
1) force ALL db to fulfil the rules (not only Moodle DB, but auth/enrol....)
2) print_error() output is 100% broken, because a lot of stuff hasn't been defined (lang, themes....)
IMO those checks should continue where they are now (admin/index.php), simply preventing any Moodle installation, but not making the dml/ddl stuff dependent of that.
Ciao :-)
|
|
Description
|
Right now prefix rules are executed by the xxx_yyy_moodle_database constructor.
That causes 2 problems:
1) force ALL db to fulfil the rules (not only Moodle DB, but auth/enrol....)
2) print_error() output is 100% broken, because a lot of stuff hasn't been defined (lang, themes....)
IMO those checks should continue where they are now (admin/index.php), simply preventing any Moodle installation, but not making the dml/ddl stuff dependent of that.
Ciao :-) |
Show » |
committed 1 file to 'Moodle CVS' - 18/May/08 04:46 AM
Revert the prefix check to admin/index.php. MDL-14877
|
|
|
committed 3 files to 'Moodle CVS' - 18/May/08 04:51 AM
non empty prefixes aren't a driver limitation, but a Moodle one. MDL-14877
|
|
|
|
Sub-task
Resolved
Major
