forked from moodle/moodle
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
MDL-37028 Integrity check for course modules and sections
This commit reinstates: commit 0bac49d Author: Marina Glancy <[email protected]> Date: Tue Sep 3 17:14:13 2013 +1000 MDL-37028 Integrity check for course modules and sections If section mentioned in 'orphaned' module does not exist module is added to the first available section. Also corrected whitespaces commit 1f0a9ce Author: Marina Glancy <[email protected]> Date: Mon Aug 12 14:06:48 2013 +1000 MDL-37028 Integrity check for course modules and sections created function, unittest, build-in quick integrity check on each call to rebuild_course_cache() also added CLI script
- Loading branch information
Damyon Wiese
committed
Sep 11, 2013
1 parent
111c649
commit 749ce98
Showing
3 changed files
with
385 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,125 @@ | ||
| <?php | ||
| // This file is part of Moodle - http://moodle.org/ | ||
| // | ||
| // Moodle is free software: you can redistribute it and/or modify | ||
| // it under the terms of the GNU General Public License as published by | ||
| // the Free Software Foundation, either version 3 of the License, or | ||
| // (at your option) any later version. | ||
| // | ||
| // Moodle is distributed in the hope that it will be useful, | ||
| // but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
| // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
| // GNU General Public License for more details. | ||
| // | ||
| // You should have received a copy of the GNU General Public License | ||
| // along with Moodle. If not, see <http://www.gnu.org/licenses/>. | ||
|
|
||
| /** | ||
| * This script fixed incorrectly deleted users. | ||
| * | ||
| * @package core | ||
| * @subpackage cli | ||
| * @copyright 2013 Marina Glancy | ||
| * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later | ||
| */ | ||
|
|
||
| define('CLI_SCRIPT', true); | ||
|
|
||
| require(__DIR__.'/../../config.php'); | ||
| require_once($CFG->libdir.'/clilib.php'); | ||
|
|
||
| // Get cli options. | ||
| list($options, $unrecognized) = cli_get_params( | ||
| array( | ||
| 'courses' => false, | ||
| 'fix' => false, | ||
| 'help' => false | ||
| ), | ||
| array( | ||
| 'h' => 'help', | ||
| 'c' => 'courses', | ||
| 'f' => 'fix' | ||
| ) | ||
| ); | ||
|
|
||
| if ($options['help'] || empty($options['courses'])) { | ||
| $help = | ||
| "Checks and fixes that course modules and sections reference each other correctly. | ||
| Compares DB fields course_sections.sequence and course_modules.section | ||
| checking that: | ||
| - course_sections.sequence contains each module id not more than once in the course | ||
| - for each moduleid from course_sections.sequence the field course_modules.section | ||
| refers to the same section id (this means course_sections.sequence is more | ||
| important if they are different) | ||
| - each module in the course is present in one of course_sections.sequence | ||
| - section sequences do not contain non-existing course modules | ||
| If there are any mismatches, the message is displayed. If --fix is specified, | ||
| the records in DB are corrected. | ||
| This script may run for a long time on big systems if called for all courses. | ||
| Avoid executing the script when another user may simultaneously edit any of the | ||
| courses being checked (recommended to run in mainenance mode). | ||
| Options: | ||
| -c, --courses List courses that need to be checked (comma-separated | ||
| values or * for all). Required | ||
| -f, --fix Fix the mismatches in DB. If not specified check only and | ||
| report problems to STDERR | ||
| -h, --help Print out this help | ||
| Example: | ||
| \$sudo -u www-data /usr/bin/php admin/cli/fix_course_sequence.php --courses=* | ||
| \$sudo -u www-data /usr/bin/php admin/cli/fix_course_sequence.php --courses=2,3,4 --fix | ||
| "; | ||
|
|
||
| echo $help; | ||
| die; | ||
| } | ||
|
|
||
| $courseslist = preg_split('/\s*,\s*/', $options['courses'], -1, PREG_SPLIT_NO_EMPTY); | ||
| if (in_array('*', $courseslist)) { | ||
| $where = ''; | ||
| $params = array(); | ||
| } else { | ||
| list($sql, $params) = $DB->get_in_or_equal($courseslist, SQL_PARAMS_NAMED, 'id'); | ||
| $where = 'WHERE id '. $sql; | ||
| } | ||
| $coursescount = $DB->get_field_sql('SELECT count(id) FROM {course} '. $where, $params); | ||
|
|
||
| if (!$coursescount) { | ||
| cli_error('No courses found'); | ||
| } | ||
| echo "Checking $coursescount courses...\n\n"; | ||
|
|
||
| require_once($CFG->dirroot. '/course/lib.php'); | ||
|
|
||
| $problems = array(); | ||
| $courses = $DB->get_fieldset_sql('SELECT id FROM {course} '. $where, $params); | ||
| foreach ($courses as $courseid) { | ||
| $errors = course_integrity_check($courseid, null, null, true, empty($options['fix'])); | ||
| if ($errors) { | ||
| if (!empty($options['fix'])) { | ||
| // Reset the course cache to make sure cache is recalculated next time the course is viewed. | ||
| $DB->upgrade_record('course', array('modinfo' => null, 'id' => $courseid)); | ||
| } | ||
| foreach ($errors as $error) { | ||
| cli_problem($error); | ||
| } | ||
| $problems[] = $courseid; | ||
| } else { | ||
| echo "Course [$courseid] is OK\n"; | ||
| } | ||
| } | ||
| if (!count($problems)) { | ||
| echo "\n...All courses are OK\n"; | ||
| } else { | ||
| if (!empty($options['fix'])) { | ||
| echo "\n...Found and fixed ".count($problems)." courses with problems". "\n"; | ||
| } else { | ||
| echo "\n...Found ".count($problems)." courses with problems. To fix run:\n"; | ||
| echo "\$sudo -u www-data /usr/bin/php admin/cli/fix_course_sequence.php --courses=".join(',', $problems)." --fix". "\n"; | ||
| } | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.