Improve SCORM tool to show message when item not found Fixing CT#253 requires testing

15 years ago · a7fcda267f
parent a6e215bbe8
commit a7fcda267f
2 changed files with 10 additions and 12 deletions
--- a/main/newscorm/blank.php
+++ b/main/newscorm/blank.php
@ -1,28 +1,21 @@
 <?php //$id: $
+/* For licensing terms, see /chamilo_license.txt */
 /**
 * Script that displays a blank page (with later a message saying why)
 * @package dokeos.learnpath
 * @author Yannick Warnier <ywarnier@beeznest.org>
 */
-/**
- * Script
- */
-

-$language_file[] = "learnpath";
+$language_file[] = array('learnpath','document');

 //flag to allow for anonymous user - needs to be set before global.inc.php
 $use_anonymous = true;

-require('../inc/global.inc.php');
-include_once('../inc/reduced_header.inc.php');
-
+require_once '../inc/global.inc.php';
+require_once '../inc/reduced_header.inc.php';
 ?>
-
 <body>
-
 <?php
-
 if (isset($_GET['error'])) {
 	switch($_GET['error']){
 		case 'document_deleted':
@ -33,6 +26,10 @@ if (isset($_GET['error'])) {
 			echo '<br /><br />';
 			Display::display_normal_message(get_lang('_prereq_not_complete'));
 			break;
+		case 'document_not_found':
+			echo '<br /><br />';
+			Display::display_normal_message(get_lang('FileNotFound'));
+			break;
 		default:
 			break;
 	}
@ -41,6 +38,5 @@ if (isset($_GET['error'])) {
 	Display::display_normal_message(get_lang('ExerciseFinished'));
 }
 ?>
-
 </body>
 </html>
--- a/main/newscorm/learnpath.class.php
+++ b/main/newscorm/learnpath.class.php
@ -3091,6 +3091,8 @@ class learnpath {
 								} else {
 									$file = $course_path . '/scorm/' . $lp_path . '/' . $decoded;
 								}
+							} else {
+								$file = 'blank.php?error=document_not_found';
 							}
 						}
 						//}else{