MDL-42084 Core: Change set_time_limit to core_php_time_limit::raise everywhere
authorsam marshall <s.marshall@open.ac.uk>
Tue, 15 Oct 2013 12:22:19 +0000 (13:22 +0100)
committersam marshall <s.marshall@open.ac.uk>
Wed, 6 Nov 2013 16:40:01 +0000 (16:40 +0000)
Exceptions (places where set_time_limit is still left) are:

Third-party libraries:
  ADODB
  PHPMailer

Moodle core:
  PHPUnit - not relevant (CLI mode)

56 files changed:
admin/enrol.php
admin/tool/customlang/index.php
admin/tool/dbtransfer/locallib.php
admin/tool/generator/classes/backend.php
admin/tool/innodb/index.php
admin/tool/langimport/index.php
admin/tool/multilangupgrade/index.php
admin/tool/phpunit/webrunner.php
admin/tool/uploadcourse/classes/processor.php
admin/tool/uploaduser/index.php
admin/tool/uploaduser/picture.php
backup/controller/backup_controller.class.php
backup/controller/restore_controller.class.php
backup/util/helper/backup_cron_helper.class.php
blocks/rss_client/block_rss_client.php
calendar/lib.php
enrol/category/locallib.php
enrol/cohort/locallib.php
enrol/database/lib.php
enrol/flatfile/lib.php
enrol/imsenterprise/lib.php
enrol/ldap/lib.php
enrol/manual/lib.php
enrol/meta/locallib.php
enrol/self/lib.php
grade/import/csv/index.php
grade/import/xml/import.php
grade/import/xml/index.php
grade/report/grader/preferences.php
lib/adminlib.php
lib/badgeslib.php
lib/classes/session/manager.php
lib/cronlib.php
lib/enrollib.php
lib/externallib.php
lib/filelib.php
lib/modinfolib.php
lib/statslib.php
lib/tests/performance/filtersettingsperformancetester.php
lib/upgradelib.php
lib/webdavlib.php
mod/assign/feedback/file/importziplib.php
mod/assign/upgradelib.php
mod/chat/chatd.php
mod/chat/gui_header_js/jsupdated.php
mod/data/field/picture/field.class.php
mod/data/import.php
mod/forum/lib.php
mod/quiz/report/overview/report.php
question/classes/statistics/questions/calculator.php
question/engine/upgrade/upgradelib.php
question/format.php
report/security/index.php
repository/filepicker.php
repository/repository_ajax.php
repository/repository_callback.php

index 642d93b..5f26eff 100644 (file)
@@ -108,7 +108,7 @@ switch ($action) {
         echo $OUTPUT->header();
 
         // This may take a long time.
-        set_time_limit(0);
+        core_php_time_limit::raise();
 
         // Disable plugin to prevent concurrent cron execution.
         unset($enabled[$enrol]);
index 431a911..4d120ff 100644 (file)
@@ -55,7 +55,7 @@ if ($action === 'checkout') {
     $progressbar->create();         // prints the HTML code of the progress bar
 
     // we may need a bit of extra execution time and memory here
-    @set_time_limit(HOURSECS);
+    core_php_time_limit::raise(HOURSECS);
     raise_memory_limit(MEMORY_EXTRA);
     tool_customlang_utils::checkout($lng, $progressbar);
 
index c7e622f..f37f62a 100644 (file)
@@ -50,7 +50,7 @@ require_once($CFG->libdir.'/dtllib.php');
  * @return does not return, calls die()
  */
 function tool_dbtransfer_export_xml_database($description, $mdb) {
-    @set_time_limit(0);
+    core_php_time_limit::raise();
 
     \core\session\manager::write_close(); // Release session.
 
@@ -77,7 +77,7 @@ function tool_dbtransfer_export_xml_database($description, $mdb) {
  * @return void
  */
 function tool_dbtransfer_transfer_database(moodle_database $sourcedb, moodle_database $targetdb, progress_trace $feedback = null) {
-    @set_time_limit(0);
+    core_php_time_limit::raise();
 
     \core\session\manager::write_close(); // Release session.
 
index da931db..4299599 100644 (file)
@@ -177,7 +177,7 @@ abstract class tool_generator_backend {
 
         // Update time limit so PHP doesn't time out.
         if (!CLI_SCRIPT) {
-            set_time_limit(120);
+            core_php_time_limit::raise(120);
         }
     }
 
index 06f4f6a..6e99834 100644 (file)
@@ -57,7 +57,7 @@ if (data_submitted() and $confirm and confirm_sesskey()) {
 
     echo $OUTPUT->notification('Please be patient and wait for this to complete...', 'notifysuccess');
 
-    set_time_limit(0);
+    core_php_time_limit::raise();
 
     foreach ($rs as $table) {
         $DB->set_debug(true);
index 82b5cc2..36b416f 100644 (file)
@@ -69,7 +69,7 @@ $notice_ok    = array();
 $notice_error = array();
 
 if (($mode == INSTALLATION_OF_SELECTED_LANG) and confirm_sesskey() and !empty($pack)) {
-    set_time_limit(0);
+    core_php_time_limit::raise();
     make_temp_directory('');
     make_upload_directory('lang');
 
@@ -125,7 +125,7 @@ if ($mode == DELETION_OF_SELECTED_LANG and !empty($uninstalllang)) {
 }
 
 if ($mode == UPDATE_ALL_LANG) {
-    set_time_limit(0);
+    core_php_time_limit::raise();
 
     $installer = new lang_installer();
 
index 04dd691..6b4ce5c 100644 (file)
@@ -59,7 +59,7 @@ echo $OUTPUT->box_start();
 
 /// Turn off time limits, sometimes upgrades can be slow.
 
-@set_time_limit(0);
+core_php_time_limit::raise();
 
 echo '<strong>Progress:</strong>';
 $i = 0;
index f52bf6b..b565ae1 100644 (file)
@@ -38,7 +38,7 @@ if (!$CFG->debugdeveloper) {
     error('Not available on production sites, sorry.');
 }
 
-set_time_limit(60*30);
+core_php_time_limit::raise(60*30);
 
 $oldcwd = getcwd();
 $code = 0;
index 7113ded..8963f7e 100644 (file)
@@ -195,7 +195,7 @@ class tool_uploadcourse_processor {
         $errors = 0;
 
         // We will most certainly need extra time and memory to process big files.
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_EXTRA);
 
         // Loop over the CSV lines.
@@ -335,7 +335,7 @@ class tool_uploadcourse_processor {
         $tracker->start();
 
         // We might need extra time and memory depending on the number of rows to preview.
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_EXTRA);
 
         // Loop over the CSV lines.
index 82f7ccc..4480278 100644 (file)
@@ -36,7 +36,7 @@ require_once('user_form.php');
 $iid         = optional_param('iid', '', PARAM_INT);
 $previewrows = optional_param('previewrows', 10, PARAM_INT);
 
-@set_time_limit(60*60); // 1 hour should be enough
+core_php_time_limit::raise(60*60); // 1 hour should be enough
 raise_memory_limit(MEMORY_HUGE);
 
 require_login();
index fe3d8e5..03091aa 100644 (file)
@@ -72,7 +72,7 @@ if ($formdata = $mform->get_data()) {
         // Large files are likely to take their time and memory. Let PHP know
         // that we'll take longer, and that the process should be recycled soon
         // to free up memory.
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_EXTRA);
 
         // Create a unique temporary directory, to process the zip file
index f38d263..345831c 100644 (file)
@@ -307,7 +307,7 @@ class backup_controller extends base_controller {
      */
     public function execute_plan() {
         // Basic/initial prevention against time/memory limits
-        set_time_limit(1 * 60 * 60); // 1 hour for 1 course initially granted
+        core_php_time_limit::raise(1 * 60 * 60); // 1 hour for 1 course initially granted
         raise_memory_limit(MEMORY_EXTRA);
         // If this is not a course backup, inform the plan we are not
         // including all the activities for sure. This will affect any
index 3349a18..f4d3958 100644 (file)
@@ -315,7 +315,7 @@ class restore_controller extends base_controller {
 
     public function execute_plan() {
         // Basic/initial prevention against time/memory limits
-        set_time_limit(1 * 60 * 60); // 1 hour for 1 course initially granted
+        core_php_time_limit::raise(1 * 60 * 60); // 1 hour for 1 course initially granted
         raise_memory_limit(MEMORY_EXTRA);
         // If this is not a course restore, inform the plan we are not
         // including all the activities for sure. This will affect any
@@ -349,7 +349,7 @@ class restore_controller extends base_controller {
             throw new restore_controller_exception('cannot_precheck_wrong_status', $this->status);
         }
         // Basic/initial prevention against time/memory limits
-        set_time_limit(1 * 60 * 60); // 1 hour for 1 course initially granted
+        core_php_time_limit::raise(1 * 60 * 60); // 1 hour for 1 course initially granted
         raise_memory_limit(MEMORY_EXTRA);
         $this->precheck = restore_prechecks_helper::execute_prechecks($this, $droptemptablesafter);
         if (!array_key_exists('errors', $this->precheck)) { // No errors, can be executed
@@ -420,7 +420,7 @@ class restore_controller extends base_controller {
         require_once($CFG->dirroot . '/backup/util/helper/convert_helper.class.php');
 
         // Basic/initial prevention against time/memory limits
-        set_time_limit(1 * 60 * 60); // 1 hour for 1 course initially granted
+        core_php_time_limit::raise(1 * 60 * 60); // 1 hour for 1 course initially granted
         raise_memory_limit(MEMORY_EXTRA);
         $this->progress->start_progress('Backup format conversion');
 
index 914e4ab..6e13b42 100644 (file)
@@ -111,7 +111,7 @@ abstract class backup_cron_automated_helper {
             cron_trace_time_and_memory();
 
             // This could take a while!
-            @set_time_limit(0);
+            core_php_time_limit::raise();
             raise_memory_limit(MEMORY_EXTRA);
 
             $nextstarttime = backup_cron_automated_helper::calculate_next_automated_backup($admin->timezone, $now);
index e2ab922..f8c5993 100644 (file)
             mtrace('    ' . $rec->url . ' ', '');
             // Fetch the rss feed, using standard simplepie caching
             // so feeds will be renewed only if cache has expired
-            @set_time_limit(60);
+            core_php_time_limit::raise(60);
 
             $feed =  new moodle_simplepie();
             // set timeout for longer than normal to be agressive at
index c11141e..be9f906 100644 (file)
@@ -3025,7 +3025,7 @@ function calendar_import_icalendar_events($ical, $courseid, $subscriptionid = nu
     $updatecount = 0;
 
     // Large calendars take a while...
-    set_time_limit(300);
+    core_php_time_limit::raise(300);
 
     // Mark all events in a subscription with a zero timestamp.
     if (!empty($subscriptionid)) {
index 404ff8f..2439955 100644 (file)
@@ -141,7 +141,7 @@ function enrol_category_sync_full(progress_trace $trace) {
     }
 
     // We may need a lot of time here.
-    @set_time_limit(0);
+    core_php_time_limit::raise();
 
     $plugin = enrol_get_plugin('category');
 
index e07c20b..ec9f9cf 100644 (file)
@@ -167,7 +167,7 @@ function enrol_cohort_sync(progress_trace $trace, $courseid = NULL) {
     }
 
     // Unfortunately this may take a long time, this script can be interrupted without problems.
-    @set_time_limit(0);
+    core_php_time_limit::raise();
     raise_memory_limit(MEMORY_HUGE);
 
     $trace->output('Starting user enrolment synchronisation...');
index cde6d62..b2aa363 100644 (file)
@@ -307,7 +307,7 @@ class enrol_database_plugin extends enrol_plugin {
         }
 
         // We may need a lot of memory here.
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_HUGE);
 
         $table            = $this->get_config('remoteenroltable');
@@ -625,7 +625,7 @@ class enrol_database_plugin extends enrol_plugin {
         $trace->output('Starting course synchronisation...');
 
         // We may need a lot of memory here.
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_HUGE);
 
         if (!$extdb = $this->db_init()) {
index e0117b9..d30a3a6 100644 (file)
@@ -229,7 +229,7 @@ class enrol_flatfile_plugin extends enrol_plugin {
         global $CFG, $DB;
 
         // We may need more memory here.
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_HUGE);
 
         $filelocation = $this->get_config('location');
index 5528e55..34a0442 100644 (file)
@@ -94,7 +94,7 @@ class enrol_imsenterprise_plugin extends enrol_plugin {
 
         $fileisnew = false;
         if ( file_exists($filename) ) {
-            @set_time_limit(0);
+            core_php_time_limit::raise();
             $starttime = time();
 
             $this->log_line('----------------------------------------------------------------------');
index fa84fe3..b4bf0b9 100644 (file)
@@ -147,7 +147,7 @@ class enrol_ldap_plugin extends enrol_plugin {
         }
 
         // We may need a lot of memory here
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_HUGE);
 
         // Get enrolments for each type of role.
@@ -311,7 +311,7 @@ class enrol_ldap_plugin extends enrol_plugin {
         $ldap_pagedresults = ldap_paged_results_supported($this->get_config('ldap_version'));
 
         // we may need a lot of memory here
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_HUGE);
 
         $oneidnumber = null;
index 10e12e1..805ed05 100644 (file)
@@ -303,7 +303,7 @@ class enrol_manual_plugin extends enrol_plugin {
         }
 
         // Unfortunately this may take a long time, execution can be interrupted safely here.
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_HUGE);
 
         $trace->output('Verifying manual enrolment expiration...');
index 5c10d79..84b6107 100644 (file)
@@ -255,7 +255,7 @@ function enrol_meta_sync($courseid = NULL, $verbose = false) {
     }
 
     // unfortunately this may take a long time, execution can be interrupted safely
-    @set_time_limit(0);
+    core_php_time_limit::raise();
     raise_memory_limit(MEMORY_HUGE);
 
     if ($verbose) {
index fd99646..d64281a 100644 (file)
@@ -468,7 +468,7 @@ class enrol_self_plugin extends enrol_plugin {
         }
 
         // Unfortunately this may take a long time, execution can be interrupted safely here.
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_HUGE);
 
         $trace->output('Verifying self-enrolments...');
index 8cdc537..07d6e2e 100644 (file)
@@ -85,7 +85,7 @@ if (!$iid) {
         // Large files are likely to take their time and memory. Let PHP know
         // that we'll take longer, and that the process should be recycled soon
         // to free up memory.
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_EXTRA);
 
         // Use current (non-conflicting) time stamp.
@@ -183,7 +183,7 @@ if ($formdata = $mform2->get_data()) {
     // Large files are likely to take their time and memory. Let PHP know
     // that we'll take longer, and that the process should be recycled soon
     // to free up memory.
-    @set_time_limit(0);
+    core_php_time_limit::raise();
     raise_memory_limit(MEMORY_EXTRA);
 
     $csvimport->init();
index 92c69fa..7151a35 100644 (file)
@@ -43,7 +43,7 @@ require_capability('gradeimport/xml:view', $context);
 // Large files are likely to take their time and memory. Let PHP know
 // that we'll take longer, and that the process should be recycled soon
 // to free up memory.
-@set_time_limit(0);
+core_php_time_limit::raise();
 raise_memory_limit(MEMORY_EXTRA);
 
 $text = download_file_content($url);
index de16555..f9a2a5a 100644 (file)
@@ -47,7 +47,7 @@ if ($data = $mform->get_data()) {
     // Large files are likely to take their time and memory. Let PHP know
     // that we'll take longer, and that the process should be recycled soon
     // to free up memory.
-    @set_time_limit(0);
+    core_php_time_limit::raise();
     raise_memory_limit(MEMORY_EXTRA);
 
     if ($text = $mform->get_file_content('userfile')) {
index 6d2d717..cfba41b 100644 (file)
@@ -22,7 +22,7 @@
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
-set_time_limit(0);
+core_php_time_limit::raise();
 require_once '../../../config.php';
 require_once $CFG->libdir . '/gradelib.php';
 require_once '../../lib.php';
index 074ad64..449a88e 100644 (file)
@@ -126,7 +126,7 @@ function uninstall_plugin($type, $name) {
     global $CFG, $DB, $OUTPUT;
 
     // This may take a long time.
-    @set_time_limit(0);
+    core_php_time_limit::raise();
 
     // Recursively uninstall all subplugins first.
     $subplugintypes = core_component::get_plugin_types_with_subplugins();
@@ -6728,7 +6728,7 @@ function db_replace($search, $replace) {
                         'block_instances', '');
 
     // Turn off time limits, sometimes upgrades can be slow.
-    @set_time_limit(0);
+    core_php_time_limit::raise();
 
     if (!$tables = $DB->get_tables() ) {    // No tables yet at all.
         return false;
index 26d4447..2542c09 100644 (file)
@@ -426,7 +426,7 @@ class badge {
         $awards = 0;
 
         // Raise timelimit as this could take a while for big web sites.
-        set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_HUGE);
 
         // For site level badges, get all active site users who can earn this badge and haven't got it yet.
index c5e764c..6743cdc 100644 (file)
@@ -614,7 +614,7 @@ class manager {
         global $CFG, $DB;
 
         // This may take a long time...
-        set_time_limit(0);
+        \core_php_time_limit::raise();
 
         $maxlifetime = $CFG->sessiontimeout;
 
index 552398b..18a0e19 100644 (file)
@@ -49,7 +49,7 @@ function cron_run() {
         set_debugging(DEBUG_DEVELOPER, true);
     }
 
-    set_time_limit(0);
+    core_php_time_limit::raise();
     $starttime = microtime();
 
     // Increase memory limit
@@ -287,7 +287,7 @@ function cron_run() {
                         mtrace("... used " . (microtime(1) - $pre_time) . " seconds");
                     }
                     // Reset possible changes by modules to time_limit. MDL-11597
-                    @set_time_limit(0);
+                    core_php_time_limit::raise();
                     mtrace("done.");
                 }
             }
@@ -314,7 +314,7 @@ function cron_run() {
                         $DB->set_field('block', 'lastcron', $timenow, array('id'=>$block->id));
                     }
                     // Reset possible changes by blocks to time_limit. MDL-11597
-                    @set_time_limit(0);
+                    core_php_time_limit::raise();
                     mtrace('done.');
                 }
             }
@@ -493,7 +493,7 @@ function cron_run() {
                     }
                 }
             }
-            @set_time_limit(0);
+            core_php_time_limit::raise();
         } else {
             mtrace('Next stats run after:'. userdate($timetocheck));
         }
@@ -575,7 +575,7 @@ function cron_execute_plugin_type($plugintype, $description = null) {
                 round(microtime(true) - $pre_time, 2) . " seconds)");
 
         set_config('lastcron', time(), $component);
-        @set_time_limit(0);
+        core_php_time_limit::raise();
     }
 
     if ($description) {
index df880d8..30b930a 100644 (file)
@@ -2036,7 +2036,7 @@ abstract class enrol_plugin {
         // Unfortunately this may take a long time, it should not be interrupted,
         // otherwise users get duplicate notification.
 
-        @set_time_limit(0);
+        core_php_time_limit::raise();
         raise_memory_limit(MEMORY_HUGE);
 
 
index 698a9ef..6f7a001 100644 (file)
@@ -149,7 +149,7 @@ class external_api {
      */
     public static function set_timeout($seconds=360) {
         $seconds = ($seconds < 300) ? 300 : $seconds;
-        set_time_limit($seconds);
+        core_php_time_limit::raise($seconds);
     }
 
     /**
index f0253fc..cc525b6 100644 (file)
@@ -2693,7 +2693,7 @@ function byteserving_send_file($handle, $mimetype, $ranges, $filesize) {
 
         fseek($handle, $ranges[0][1]);
         while (!feof($handle) && $length > 0) {
-            @set_time_limit(60*60); //reset time limit to 60 min - should be enough for 1 MB chunk
+            core_php_time_limit::raise(60*60); //reset time limit to 60 min - should be enough for 1 MB chunk
             $buffer = fread($handle, ($chunksize < $length ? $chunksize : $length));
             echo $buffer;
             flush();
@@ -2722,7 +2722,7 @@ function byteserving_send_file($handle, $mimetype, $ranges, $filesize) {
             echo $range[0];
             fseek($handle, $range[1]);
             while (!feof($handle) && $length > 0) {
-                @set_time_limit(60*60); //reset time limit to 60 min - should be enough for 1 MB chunk
+                core_php_time_limit::raise(60*60); //reset time limit to 60 min - should be enough for 1 MB chunk
                 $buffer = fread($handle, ($chunksize < $length ? $chunksize : $length));
                 echo $buffer;
                 flush();
index 576f460..0395c82 100644 (file)
@@ -2077,7 +2077,7 @@ function rebuild_course_cache($courseid=0, $clearonly=false) {
         $select = array('id'=>$courseid);
     } else {
         $select = array();
-        @set_time_limit(0);  // this could take a while!   MDL-10954
+        core_php_time_limit::raise();  // this could take a while!   MDL-10954
     }
 
     $rs = $DB->get_recordset("course", $select,'','id,'.join(',', course_modinfo::$cachedfields));
index 1f72d68..84e9a5a 100644 (file)
@@ -208,7 +208,7 @@ function stats_cron_daily($maxdays=1) {
         }
 
         $days++;
-        @set_time_limit($timeout - 200);
+        core_php_time_limit::raise($timeout - 200);
 
         if ($days > 1) {
             // move the lock
@@ -677,7 +677,7 @@ function stats_cron_weekly() {
 
     $weeks = 0;
     while ($now > $nextstartweek) {
-        @set_time_limit($timeout - 200);
+        core_php_time_limit::raise($timeout - 200);
         $weeks++;
 
         if ($weeks > 1) {
@@ -820,7 +820,7 @@ function stats_cron_monthly() {
 
     $months = 0;
     while ($now > $nextstartmonth) {
-        @set_time_limit($timeout - 200);
+        core_php_time_limit::raise($timeout - 200);
         $months++;
 
         if ($months > 1) {
index 6eec7b6..ae9fea6 100644 (file)
@@ -138,7 +138,7 @@ function simple_get_record_by_id($context) {
 }
 
 function run_tests($function, $contexts, $numcalls, $basetime) {
-    set_time_limit(120);
+    core_php_time_limit::raise(120);
     $startime = microtime(true);
     for ($j = 0; $j < $numcalls; $j++) {
         $function($contexts[array_rand($contexts)]);
@@ -158,7 +158,7 @@ function print_result_line($duration, $basetime, $numcalls, $action1, $action2 =
 
 function populate_test_database($syscontext, $numcategories, $numcourses, $nummodules, $numoverrides, $numconfigs) {
     global $DB, $OUTPUT;
-    set_time_limit(600);
+    core_php_time_limit::raise(600);
     $syscontext->id = $DB->insert_record('context', $syscontext);
 
     // Category contexts.
index f9eda59..3dcb0c3 100644 (file)
@@ -156,9 +156,9 @@ function upgrade_set_timeout($max_execution_time=300) {
 
     if (CLI_SCRIPT) {
         // there is no point in timing out of CLI scripts, admins can stop them if necessary
-        set_time_limit(0);
+        core_php_time_limit::raise();
     } else {
-        set_time_limit($max_execution_time);
+        core_php_time_limit::raise($max_execution_time);
     }
     set_config('upgraderunning', $expected_end); // keep upgrade locked until this time
 }
@@ -401,7 +401,7 @@ function upgrade_plugins($type, $startcallback, $endcallback, $verbose) {
 
     foreach ($plugs as $plug=>$fullplug) {
         // Reset time so that it works when installing a large number of plugins
-        set_time_limit(600);
+        core_php_time_limit::raise(600);
         $component = clean_param($type.'_'.$plug, PARAM_COMPONENT); // standardised plugin name
 
         // check plugin dir is valid name
@@ -1463,7 +1463,7 @@ function install_core($version, $verbose) {
     make_writable_directory($CFG->dataroot.'/muc', true);
 
     try {
-        set_time_limit(600);
+        core_php_time_limit::raise(600);
         print_upgrade_part_start('moodle', true, $verbose); // does not store upgrade running flag
 
         $DB->get_manager()->install_from_xmldb_file("$CFG->libdir/db/install.xml");
@@ -1521,7 +1521,7 @@ function upgrade_core($version, $verbose) {
         // Pre-upgrade scripts for local hack workarounds.
         $preupgradefile = "$CFG->dirroot/local/preupgrade.php";
         if (file_exists($preupgradefile)) {
-            set_time_limit(0);
+            core_php_time_limit::raise();
             require($preupgradefile);
             // Reset upgrade timeout to default.
             upgrade_set_timeout();
index 96b4458..1f7b641 100644 (file)
@@ -161,7 +161,7 @@ class webdav_client {
         // let's try to open a socket
         $this->_error_log('open a socket connection');
         $this->sock = fsockopen($this->_socket . $this->_server, $this->_port, $this->_errno, $this->_errstr, $this->_socket_timeout);
-        set_time_limit(30);
+        core_php_time_limit::raise(30);
         if (is_resource($this->sock)) {
             socket_set_blocking($this->sock, true);
             $this->_connection_closed = false;
index c3fd443..c59477f 100644 (file)
@@ -167,7 +167,7 @@ class assignfeedback_file_zip_importer {
         raise_memory_limit(MEMORY_EXTRA);
 
         $packer = get_file_packer('application/zip');
-        @set_time_limit(ASSIGNFEEDBACK_FILE_MAXFILEUNZIPTIME);
+        core_php_time_limit::raise(ASSIGNFEEDBACK_FILE_MAXFILEUNZIPTIME);
 
         return $packer->extract_to_storage($zipfile,
                                     $contextid,
@@ -218,7 +218,7 @@ class assignfeedback_file_zip_importer {
     public function import_zip_files($assignment, $fileplugin) {
         global $CFG, $PAGE, $DB;
 
-        @set_time_limit(ASSIGNFEEDBACK_FILE_MAXFILEUNZIPTIME);
+        core_php_time_limit::raise(ASSIGNFEEDBACK_FILE_MAXFILEUNZIPTIME);
         $packer = get_file_packer('application/zip');
 
         $feedbackfilesupdated = 0;
index 3b416a5..1b26b56 100644 (file)
@@ -61,7 +61,7 @@ class assign_upgrade_manager {
               return false;
         }
 
-        @set_time_limit(ASSIGN_MAX_UPGRADE_TIME_SECS);
+        core_php_time_limit::raise(ASSIGN_MAX_UPGRADE_TIME_SECS);
 
         // Get the module details.
         $oldmodule = $DB->get_record('modules', array('name'=>'assignment'), '*', MUST_EXIST);
index 567a95d..9f56fbb 100644 (file)
@@ -62,7 +62,7 @@ if(!empty($safemode)) {
     die("Error: Cannot run with PHP safe_mode = On. Turn off safe_mode in php.ini.\n");
 }
 
-@set_time_limit (0);
+core_php_time_limit::raise(0);
 error_reporting(E_ALL);
 
 function chat_empty_connection() {
index 22bdb58..9048004 100644 (file)
@@ -25,7 +25,7 @@ require('../lib.php');
 
 // we are going to run for a long time
 // avoid being terminated by php
-@set_time_limit(0);
+core_php_time_limit::raise();
 
 $chat_sid      = required_param('chat_sid',          PARAM_ALPHANUM);
 $chat_lasttime = optional_param('chat_lasttime',  0, PARAM_INT);
index bf66839..b3e378d 100644 (file)
@@ -210,7 +210,7 @@ class data_field_picture extends data_field_base {
                     if ($thumbfile = $fs->get_file($this->context->id, 'mod_data', 'content', $content->id, '/', 'thumb_'.$content->content)) {
                         $thumbfile->delete();
                     }
-                    @set_time_limit(300);
+                    core_php_time_limit::raise(300);
                     // Might be slow!
                     $this->update_thumbnail($content, $file);
                 }
index 9b7425b..8da044f 100644 (file)
@@ -94,7 +94,7 @@ if (!$formdata = $form->get_data()) {
     // Large files are likely to take their time and memory. Let PHP know
     // that we'll take longer, and that the process should be recycled soon
     // to free up memory.
-    @set_time_limit(0);
+    core_php_time_limit::raise();
     raise_memory_limit(MEMORY_EXTRA);
 
     $iid = csv_import_reader::get_new_iid('moddata');
index 0477a1a..3ed756f 100644 (file)
@@ -581,7 +581,7 @@ function forum_cron() {
 
         foreach ($users as $userto) {
 
-            @set_time_limit(120); // terminate if processing of any account takes longer than 2 minutes
+            core_php_time_limit::raise(120); // terminate if processing of any account takes longer than 2 minutes
 
             mtrace('Processing user '.$userto->id);
 
@@ -816,7 +816,7 @@ function forum_cron() {
 
     mtrace('Starting digest processing...');
 
-    @set_time_limit(300); // terminate if not able to fetch all digests in 5 minutes
+    core_php_time_limit::raise(300); // terminate if not able to fetch all digests in 5 minutes
 
     if (!isset($CFG->digestmailtimelast)) {    // To catch the first time
         set_config('digestmailtimelast', 0);
@@ -894,7 +894,7 @@ function forum_cron() {
             // Data collected, start sending out emails to each user
             foreach ($userdiscussions as $userid => $thesediscussions) {
 
-                @set_time_limit(120); // terminate if processing of any account takes longer than 2 minutes
+                core_php_time_limit::raise(120); // terminate if processing of any account takes longer than 2 minutes
 
                 cron_setup_user();
 
@@ -938,7 +938,7 @@ function forum_cron() {
 
                 foreach ($thesediscussions as $discussionid) {
 
-                    @set_time_limit(120);   // to be reset for each post
+                    core_php_time_limit::raise(120);   // to be reset for each post
 
                     $discussion = $discussions[$discussionid];
                     $forum      = $forums[$discussion->forum];
index 5f62bc1..20c3424 100644 (file)
@@ -340,7 +340,7 @@ class quiz_overview_report extends quiz_attempts_report {
     protected function regrade_attempt($attempt, $dryrun = false, $slots = null) {
         global $DB;
         // Need more time for a quiz with many questions.
-        set_time_limit(300);
+        core_php_time_limit::raise(300);
 
         $transaction = $DB->start_delegated_transaction();
 
index 55bb803..340adb4 100644 (file)
@@ -76,7 +76,7 @@ class calculator {
      * @return array containing two arrays calculated[] and calculated_for_subquestion[].
      */
     public function calculate($qubaids) {
-        set_time_limit(0);
+        \core_php_time_limit::raise();
 
         list($lateststeps, $summarks, $summarksavg) = $this->get_latest_steps($qubaids);
 
index 438bf74..f24e30d 100644 (file)
@@ -73,7 +73,7 @@ class question_engine_attempt_upgrader {
     }
 
     protected function prevent_timeout() {
-        set_time_limit(300);
+        core_php_time_limit::raise(300);
         if ($this->doingbackup) {
             return;
         }
index 7dfe330..14f2483 100644 (file)
@@ -290,7 +290,7 @@ class qformat_default {
         global $USER, $CFG, $DB, $OUTPUT;
 
         // reset the timer in case file upload was slow
-        set_time_limit(0);
+        core_php_time_limit::raise();
 
         // STAGE 1: Parse the file
         echo $OUTPUT->notification(get_string('parsingquestions', 'question'), 'notifysuccess');
@@ -359,7 +359,7 @@ class qformat_default {
         foreach ($questions as $question) {   // Process and store each question
 
             // reset the php timeout
-            set_time_limit(0);
+            core_php_time_limit::raise();
 
             // check for category modifiers
             if ($question->qtype == 'category') {
index 7d356f0..7385211 100644 (file)
@@ -42,7 +42,7 @@ if (array_search($issue, $issues, true) === false) {
 
 // we may need a bit more memory and this may take a long time to process
 raise_memory_limit(MEMORY_EXTRA);
-@set_time_limit(0);
+core_php_time_limit::raise();
 
 // Print the header.
 admin_externalpage_setup('reportsecurity', '', null, '', array('pagelayout'=>'report'));
index 4004655..aff254a 100644 (file)
@@ -30,7 +30,7 @@ require_once('../config.php');
 require_once($CFG->libdir.'/filelib.php');
 require_once('lib.php');
 /// Wait as long as it takes for this script to finish
-set_time_limit(0);
+core_php_time_limit::raise();
 
 require_sesskey();
 require_login();
index ebb1c31..bd2c762 100644 (file)
@@ -87,7 +87,7 @@ if (!empty($course)) {
 $maxbytes = get_user_max_upload_file_size($context, $CFG->maxbytes, $coursemaxbytes, $maxbytes);
 
 // Wait as long as it takes for this script to finish
-set_time_limit(0);
+core_php_time_limit::raise();
 
 // These actions all occur on the currently active repository instance
 switch ($action) {
index 3abcbca..f1e1be5 100644 (file)
@@ -40,7 +40,7 @@ header('Cache-Control: no-cache, must-revalidate');
 header('Expires: Sat, 26 Jul 1997 05:00:00 GMT');
 
 /// Wait as long as it takes for this script to finish
-set_time_limit(0);
+core_php_time_limit::raise();
 
 /// Get repository instance information
 $sql = 'SELECT i.name, i.typeid, r.type, i.contextid FROM {repository} r, {repository_instances} i '.