Issue Details (XML | Word | Printable)

Key: MDL-19346
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Eloy Lafuente (stronk7)
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

Discrepancies between plugins declared in get_plugin_types() and detected by get_db_directories()

Created: 30/May/09 11:29 PM   Updated: 16/Jun/09 06:16 PM
Return to search
Component/s: Database SQL/XMLDB, Installation
Affects Version/s: 2.0
Fix Version/s: 2.0

Issue Links:
Relates

Database: Any
Participants: Eloy Lafuente (stronk7)
Security Level: None
QA Assignee: Petr Skoda
Difficulty: Moderate
Resolved date: 30/May/09
Affected Branches: MOODLE_20_STABLE
Fixed Branches: MOODLE_20_STABLE


 Description  « Hide
Right now we have one core function:

get_plugin_types()

that returns all the plugins supported by moodle. It's used in upgrade code to look for upgrades under all plugins.

In the other side, we have the function:

get_db_directories()

that returns all the directories where we can found database install/upgrade scripts (used by the xmldb editor, database manager...)

And right now, both functions are 100% separate, having to keep such lists maintained manually.

Solution: Let's modify get_db_directories() in order to use get_plugin_types() to know about all the dirs being candidates to have install/upgrade/xmldb editor... scripts recognising the same plugins always.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Eloy Lafuente (stronk7) added a comment - 30/May/09 11:34 PM
Done, now get_db_directories() is using get_plugin_types(), so the latter is the only place where supported plugins and paths need to be defined.

Resolving as fixed. Ciao