Skip to content

Commit

Permalink
"MDL-14129, fix print_error call"
Browse files Browse the repository at this point in the history
  • Loading branch information
dongsheng committed Apr 30, 2008
1 parent 8f7198c commit 33aa572
Show file tree
Hide file tree
Showing 6 changed files with 16 additions and 11 deletions.
2 changes: 1 addition & 1 deletion calendar/event.php
Expand Up @@ -107,7 +107,7 @@
$title = get_string('deleteevent', 'calendar');
$event = get_record('event', 'id', $eventid);
if($event === false) {
print_error('Invalid event');
print_error('invalidevent');
}
if(!calendar_edit_event_allowed($event)) {
print_error('You are not authorized to do this');
Expand Down
18 changes: 9 additions & 9 deletions file.php
Expand Up @@ -28,36 +28,36 @@

// relative path must start with '/', because of backup/restore!!!
if (!$relativepath) {
print_error('No valid arguments supplied or incorrect server configuration');
print_error('invalidargorconf');
} else if ($relativepath{0} != '/') {
print_error('No valid arguments supplied, path does not start with slash!');
print_error('pathdoesnotstartslash');
}

$pathname = $CFG->dataroot.$relativepath;

// extract relative path components
$args = explode('/', trim($relativepath, '/'));
if (count($args) == 0) { // always at least courseid, may search for index.html in course root
print_error('No valid arguments supplied');
print_error('invalidarguments');
}

// security: limit access to existing course subdirectories
if (($args[0]!='blog') and (!$course = get_record_sql("SELECT * FROM {$CFG->prefix}course WHERE id='".(int)$args[0]."'"))) {
print_error('Invalid course ID');
print_error('invalidcourseid');
}

// security: prevent access to "000" or "1 something" directories
// hack for blogs, needs proper security check too
if (($args[0] != 'blog') and ($args[0] != $course->id)) {
print_error('Invalid course ID');
print_error('invalidcourseid');
}

// security: login to course if necessary
// Note: file.php always calls require_login() with $setwantsurltome=false
// in order to avoid messing redirects. MDL-14495
if ($args[0] == 'blog') {
if (empty($CFG->bloglevel)) {
print_error('Blogging is disabled!');
print_error('blogdisable', 'blog');
} else if ($CFG->bloglevel < BLOG_GLOBAL_LEVEL) {
require_login(0, true, null, false);
} else if ($CFG->forcelogin) {
Expand All @@ -78,7 +78,7 @@
// security: only editing teachers can access backups
if ((count($args) >= 2) and (strtolower($args[1]) == 'backupdata')) {
if (!has_capability('moodle/site:backup', get_context_instance(CONTEXT_COURSE, $course->id))) {
print_error('Access not allowed');
print_error('nopermissions');
} else {
$lifetime = 0; //disable browser caching for backups
}
Expand Down Expand Up @@ -108,7 +108,7 @@
$lifetime = 0; // do not cache assignments, students may reupload them
if (!has_capability('mod/assignment:grade', get_context_instance(CONTEXT_COURSE, $course->id))
and $args[4] != $USER->id) {
print_error('Access not allowed');
print_error('nopermissions');
}
}

Expand Down Expand Up @@ -152,7 +152,7 @@
"AND r.type = 'file' " .
"AND r.reference = '{$reference}'";
if (count_records_sql($sql)) {
print_error('Access not allowed');
print_error('nopermissions');
}
}

Expand Down
2 changes: 1 addition & 1 deletion index.php
Expand Up @@ -194,7 +194,7 @@
require_once($CFG->dirroot .'/mod/forum/lib.php');

if (! $newsforum = forum_get_course_forum($SITE->id, 'news')) {
print_error('Could not find or create a main news forum for the site');
print_error('cannotfindorcreateforum', 'forum');
}

if (!empty($USER->id)) {
Expand Down
1 change: 1 addition & 0 deletions lang/en_utf8/debug.php
Expand Up @@ -15,6 +15,7 @@
$string['erroroccur'] = 'An error has occurred during this process';
$string['fixsetting'] = 'Please fix your settings in config.php: <p>You have:</p> <p>\$CFG->dirroot = \"$a[0]\";</p> <p>but it should be:</p> <p>\$CFG->dirroot = \"$a[1]\"</p>';
$string['notables'] = 'No Tables!';
$string['nomodules'] = 'No modules found!!';
$string['phpvaroff'] = 'The PHP server variable \'$a[0]\' should be Off - $a[1]';
$string['phpvaron'] = 'The PHP server variable \'$a[0]\' is not turned On - $a[1]';
$string['prefixcannotbeempty'] = 'Table prefix \"$a[0]\" cannot be empty for your target DB ($a[1])';
Expand Down
3 changes: 3 additions & 0 deletions lang/en_utf8/error.php
Expand Up @@ -86,10 +86,12 @@
$string['guestnoeditprofileother'] = 'The guest user profile cannot be edited';
$string['invalidaction'] = 'Invalid action parameter.';
$string['invalidarguments'] = 'No valid arguments supplied';
$string['invalidargorconf'] = 'No valid arguments supplied or incorrect server configuration';
$string['invalidaccessparameter'] = 'Invalid access parameter.';
$string['invalidcourse'] = 'Invalid course';
$string['invalidcourseid'] = 'You are tring to use an invalid course ID: ($a)';
$string['invalidcoursemodule'] = 'Bad course module ID';
$string['invalidevent'] = 'Invalid event';
$string['invalidcoursenameshort'] = 'Invalid short course name';
$string['invalidevent'] = 'Invalid event';
$string['invalidfieldname'] = '\"$a\" is not a valid field name';
Expand Down Expand Up @@ -147,6 +149,7 @@
$string['onlyeditingteachers'] = 'Only editing teachers can do that.';
$string['onlyeditown'] = 'You can only edit your own information';
$string['pagenotexist'] = 'An unusual error occurred (tried to reach a page that doesn\'t exist).';
$string['pathdoesnotstartslash'] = 'No valid arguments supplied, path does not start with slash!';
$string['pleasereport'] = 'If you have time, please let us know what you were trying to do when the error occurred:';
$string['pluginrequirementsnotmet'] = 'Plugin \"$a->pluginname\" ($a->pluginversion) could not be installed. It requires a newer version of Moodle (currently you are using $a->currentmoodle, you need $a->requiremoodle).';
$string['processingstops'] = 'Processing stops here. Remaining records ignored.';
Expand Down
1 change: 1 addition & 0 deletions lang/en_utf8/forum.php
Expand Up @@ -29,6 +29,7 @@
$string['cannotviewpostyet'] = 'You cannot read other students questions in this discussion yet because you haven\'t posted';
$string['cannotadddiscussion'] = 'Adding discussions to this forum requires group membership.';
$string['cannotadddiscussionall'] = 'You do not have permission to add a new discussion topic for all participants.';
$string['cannotfindorcreateforum'] = 'Could not find or create a main news forum for the site';
$string['cleanreadtime'] = 'Mark old posts as read hour';
$string['configcleanreadtime'] = 'The hour of the day to clean old posts from the \'read\' table.';
$string['configdisplaymode'] = 'The default display mode for discussions if one isn\'t set.';
Expand Down

0 comments on commit 33aa572

Please sign in to comment.