Skip to content

Commit

Permalink
MDL-50890 enrol_flatfile: Correcting typo and return
Browse files Browse the repository at this point in the history
Would be safer to return nothing, execute methods don't have
a return. I understand this return 2 was inherited from the
previous cron function.
  • Loading branch information
David Monllao committed Sep 16, 2015
1 parent fd1b399 commit 6bc490e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion enrol/flatfile/classes/task/flatfile_sync_task.php
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ public function execute() {
require_once($CFG->dirroot . '/enrol/flatfile/lib.php');

if (!enrol_is_enabled('flatfile')) {
return 2;
return;
}

// Instance of enrol_flatfile_plugin.
Expand Down
2 changes: 1 addition & 1 deletion enrol/flatfile/cli/sync.php
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
* $ php admin/tool/task/cli/schedule_task.php -h
*
* Execute task:
* $ sudo -u www-data /usr/bin/php admin/tool/task/cli/scheduled_task.php /
* $ sudo -u www-data /usr/bin/php admin/tool/task/cli/schedule_task.php /
* --execute=\\enrol_flatfile\\task\\flatfile_sync_task
*
* @package enrol_flatfile
Expand Down

0 comments on commit 6bc490e

Please sign in to comment.