Issue Details (XML | Word | Printable)

Key: MDL-9557
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Martin Dougiamas
Reporter: Zoltán Szarvas
Votes: 1
Watchers: 2
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

SCORM import blank screen (caused by set_time_limit())

Created: 24/Apr/07 07:23 PM   Updated: 11/Aug/08 05:38 PM
Return to search
Component/s: SCORM
Affects Version/s: 1.8
Fix Version/s: 1.8.1

Environment: PHP 5.2.0, MySql 5.0.27

Database: MySQL
Participants: aaron.danker@pixelphish.com, Martin Dougiamas and Zoltán Szarvas
Security Level: None
Resolved date: 06/Jun/07
Affected Branches: MOODLE_18_STABLE
Fixed Branches: MOODLE_18_STABLE


 Description  « Hide
I tried to import SCORM courses into Moodle. Lot of these courses could be imported, but several ones couldn't be imported. No error message, just a blank screen. After some debugging (I set $db->debug=true and I turned on the error reporting) I found that the error was occured because the 5 seconds timeout exceeded. In php.ini the timeout value was 30 seconds. I continued the debugging and found the timeout setting statement in the "moodle\mod\scorm\locallib.php" file in the scorm_delete_files($directory) function. The course was successfully imported after I removed the statement.

I think, this statement causes the error, because it overriddes the the default php.ini value. It restarts the max_execution_time counter, but only 5 seconds remain to finish any other operation after last set_time_limit() call. If the course is "big" this time value isn't sufficient.


 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
moodler committed 1 file to 'Moodle CVS' on branch 'MOODLE_18_STABLE' - 06/Jun/07 05:16 PM
Fix for MDL-9557 and MDL-9007 (increased timeout when deleting files)
MODIFY mod/scorm/locallib.php   Rev. 1.34.2.5    (+2 -2 lines)
moodler committed 1 file to 'Moodle CVS' - 06/Jun/07 05:19 PM
Merged fix for MDL-9077 and MDL-9557
MODIFY mod/scorm/locallib.php   Rev. 1.43    (+6 -7 lines)