|
Andrei Bautu made changes - 24/Jul/08 03:55 PM
We have this comment "we ignore other tables from the database since this can be a shared with other applications". People can have other tables in a Moodle database. We don't mind about them, because the will not export/import them.
Yep, but what about ones that start with mdl_ ?
you can safely ignore the mdl_pma_xxx tables used by phpmyadmin
 there is also one table from adodb without prefix that can be safely ignored I would say that if they are not in the install.xml files, then we don't care about them, even if their names start with mdl_. I can change this if you want to.
Petr Skoda made changes - 26/Aug/08 05:57 AM
Petr Skoda made changes - 26/Aug/08 05:57 AM
Petr Skoda committed 1 file to 'Moodle CVS' - 26/Aug/08 06:02 AM
in cvs with minor changes, it is a bit more strict
thanks!
Petr Skoda made changes - 26/Aug/08 06:03 AM
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Sub-task
Resolved
Minor
 |
|
All content on this web site is made available under the GNU General Public License, unless otherwise stated.
Couple things:
unset($dbfields[$fieldname]);
paired with:
foreach($dbfields as $fieldname=>$info) {
$errors[$tablename][] = 'Field ' . $fieldname . ' is unexpected.'
}
but for tables.
Other than that, it looks good to me.