Issue Details (XML | Word | Printable)

Key: MDL-14877
Type: Sub-task Sub-task
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Petr Skoda
Reporter: Eloy Lafuente (stronk7)
Votes: 0
Watchers: 0
Operations

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

prefix check shouldn't be perfomed by moodle_database classes

Created: 17/May/08 06:55 AM   Updated: 22/May/08 08:27 AM
Return to search
Component/s: Database SQL/XMLDB
Affects Version/s: 2.0
Fix Version/s: 2.0

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


 Description  « Hide
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 :-)

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Eloy Lafuente (stronk7) made changes - 18/May/08 04:51 AM
Field Original Value New Value
Resolution Fixed [ 1 ]
Status Open [ 1 ] Resolved [ 5 ]
Assignee Petr Skoda [ skodak ] Eloy Lafuente (stronk7) [ stronk7 ]
QA Assignee skodak
Petr Skoda made changes - 18/May/08 05:56 AM
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Eloy Lafuente (stronk7) made changes - 22/May/08 08:26 AM
Status Reopened [ 4 ] Resolved [ 5 ]
QA Assignee skodak
Resolution Fixed [ 1 ]
Eloy Lafuente (stronk7) made changes - 22/May/08 08:27 AM
Assignee Eloy Lafuente (stronk7) [ stronk7 ] Petr Skoda [ skodak ]