Passed
Push — 1.10.x ( 04397c...5e25f1 )
by Angel Fernando Quiroz
181:14 queued 130:00
created
main/newscorm/lp_ajax_initialize.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -80,11 +80,11 @@  discard block
 block discarded – undo
80 80
     if (!empty($myistring)) {
81 81
         $myistring = substr($myistring, 1);
82 82
     }
83
-	// Obtention des donnees d'objectifs
83
+    // Obtention des donnees d'objectifs
84 84
 
85
-	$mycoursedb = Database::get_course_table(TABLE_LP_IV_OBJECTIVE);
85
+    $mycoursedb = Database::get_course_table(TABLE_LP_IV_OBJECTIVE);
86 86
     $course_id = api_get_course_int_id();
87
-	$mylp_iv_id = $mylpi->db_item_view_id;
87
+    $mylp_iv_id = $mylpi->db_item_view_id;
88 88
 
89 89
     $phpobjectives = array();
90 90
 
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
             $phpobjectives[] = $row;
99 99
         }
100 100
     }
101
-	$myobjectives = json_encode($phpobjectives);
101
+    $myobjectives = json_encode($phpobjectives);
102 102
 
103 103
     $return .=
104 104
             "olms.score=".$myscore.";" .
Please login to merge, or discard this patch.
main/newscorm/lp_add_audio.php 1 patch
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,10 +1,10 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 /* For licensing terms, see /license.txt */
3 3
 /**
4
- * This is a learning path creation and player tool in Chamilo - previously
5
- * @author Julio Montoya  - Improving the list of templates
6
- * @package chamilo.learnpath
7
- */
4
+     * This is a learning path creation and player tool in Chamilo - previously
5
+     * @author Julio Montoya  - Improving the list of templates
6
+     * @package chamilo.learnpath
7
+     */
8 8
 
9 9
 $this_section = SECTION_COURSES;
10 10
 
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 
25 25
 // Using the resource linker as a tool for adding resources to the learning path.
26 26
 if ($action == 'add' && $type == 'learnpathitem') {
27
-     $htmlHeadXtra[] = "<script>
27
+        $htmlHeadXtra[] = "<script>
28 28
      window.location=\"../resourcelinker/resourcelinker.php?source_id=5&action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no\";
29 29
      </script>";
30 30
 }
Please login to merge, or discard this patch.
main/newscorm/blank.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,10 +1,10 @@
 block discarded – undo
1 1
 <?php
2 2
 /* For licensing terms, see /license.txt */
3 3
 /**
4
- * Script that displays a blank page (with later a message saying why)
5
- * @package chamilo.learnpath
6
- * @author Yannick Warnier <[email protected]>
7
- */
4
+     * Script that displays a blank page (with later a message saying why)
5
+     * @package chamilo.learnpath
6
+     * @author Yannick Warnier <[email protected]>
7
+     */
8 8
 // Flag to allow for anonymous user - needs to be set before global.inc.php.
9 9
 $use_anonymous = true;
10 10
 require_once '../inc/global.inc.php';
Please login to merge, or discard this patch.
main/newscorm/lp_move_item.php 1 patch
Indentation   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -23,23 +23,23 @@
 block discarded – undo
23 23
 $htmlHeadXtra[] = '<script type="text/javascript">'.
24 24
 $_SESSION['oLP']->get_js_dropdown_array() .
25 25
 'function load_cbo(id){' ."\n" .
26
-  'if (!id) {return false;}'.
27
-  'var cbo = document.getElementById(\'previous\');' .
28
-  'for(var i = cbo.length - 1; i > 0; i--) {' .
26
+    'if (!id) {return false;}'.
27
+    'var cbo = document.getElementById(\'previous\');' .
28
+    'for(var i = cbo.length - 1; i > 0; i--) {' .
29 29
     'cbo.options[i] = null;' .
30
- '}' ."\n" .
31
-  'var k=0;' .
32
-  'for(var i = 1; i <= child_name[id].length; i++){' ."\n" .
33
-  '  cbo.options[i] = new Option(child_name[id][i-1], child_value[id][i-1]);' ."\n" .
34
-  '  k=i;' ."\n" .
35
-  '}' ."\n" .
36
-  'cbo.options[k].selected = true;'."\n" .
30
+    '}' ."\n" .
31
+    'var k=0;' .
32
+    'for(var i = 1; i <= child_name[id].length; i++){' ."\n" .
33
+    '  cbo.options[i] = new Option(child_name[id][i-1], child_value[id][i-1]);' ."\n" .
34
+    '  k=i;' ."\n" .
35
+    '}' ."\n" .
36
+    'cbo.options[k].selected = true;'."\n" .
37 37
 '}'."\n" .
38 38
 '$().ready(function() {'."\n" .
39
-  'if ($(\'#previous\')) {'."\n" .
39
+    'if ($(\'#previous\')) {'."\n" .
40 40
     'if(\'parent is\'+$(\'#idParent\').val()) {'.
41
-      'load_cbo($(\'#idParent\').val());'."\n" .
42
-  '}}'."\n" .
41
+        'load_cbo($(\'#idParent\').val());'."\n" .
42
+    '}}'."\n" .
43 43
 '});</script>'."\n" ;
44 44
 
45 45
 /* Constants and variables */
Please login to merge, or discard this patch.
main/newscorm/lp_impress.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@
 block discarded – undo
36 36
     api_get_group_id()
37 37
 );
38 38
 if (!api_is_allowed_to_edit(null, true) && intval($visibility) == 0 ) {
39
-     api_not_allowed();
39
+        api_not_allowed();
40 40
 }
41 41
 
42 42
 if (empty($_SESSION['oLP'])) {
Please login to merge, or discard this patch.
main/newscorm/scormOrganization.class.php 1 patch
Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -38,14 +38,14 @@  discard block
 block discarded – undo
38 38
                     //if ($first_item->type == XML_ELEMENT_NODE) this is already check prior to the call to this function.
39 39
                     $children = $element->childNodes;
40 40
                     foreach ($children as $child) {
41
-                         switch ($child->nodeType) {
41
+                            switch ($child->nodeType) {
42 42
                             case XML_ELEMENT_NODE:
43 43
                                 switch ($child->tagName) {
44 44
                                     case 'item':
45 45
                                          $oItem = new scormItem('manifest', $child);
46
-                                         if ($oItem->identifier != '') {
46
+                                            if ($oItem->identifier != '') {
47 47
                                             $this->items[$oItem->identifier] = $oItem;
48
-                                         }
48
+                                            }
49 49
                                         break;
50 50
                                     case 'metadata':
51 51
                                         $this->metadata = new scormMetadata('manifest', $child);
@@ -58,24 +58,24 @@  discard block
 block discarded – undo
58 58
                                         break;
59 59
                                 }
60 60
                                 break;
61
-                             case XML_TEXT_NODE:
61
+                                case XML_TEXT_NODE:
62 62
                                  break;
63
-                         }
63
+                            }
64 64
                     }
65 65
 
66 66
                     if ($element->hasAttributes()) {
67
-                         $attributes = $element->attributes;
68
-                         //$keep_href = '';
69
-                         foreach ($attributes as $attrib) {
70
-                             switch ($attrib->name) {
71
-                                 case 'identifier':
67
+                            $attributes = $element->attributes;
68
+                            //$keep_href = '';
69
+                            foreach ($attributes as $attrib) {
70
+                                switch ($attrib->name) {
71
+                                    case 'identifier':
72 72
                                      $this->identifier = $attrib->value;
73
-                                     break;
74
-                                 case 'structure':
73
+                                        break;
74
+                                    case 'structure':
75 75
                                      $this->structure = $attrib->value;
76
-                                     break;
77
-                             }
78
-                         }
76
+                                        break;
77
+                                }
78
+                            }
79 79
                     }
80 80
 
81 81
                     return true;
Please login to merge, or discard this patch.
main/newscorm/lp_view_item.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 
65 65
 // Using the resource linker as a tool for adding resources to the learning path.
66 66
 if ($action == 'add' && $type == 'learnpathitem') {
67
-     $htmlHeadXtra[] = "<script> window.location=\"../resourcelinker/resourcelinker.php?source_id=5&action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no\"; </script>";
67
+        $htmlHeadXtra[] = "<script> window.location=\"../resourcelinker/resourcelinker.php?source_id=5&action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no\"; </script>";
68 68
 }
69 69
 if ((!$is_allowed_to_edit) || ($isStudentView)) {
70 70
     error_log('New LP - User not authorized in lp_view_item.php');
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
 // Theme calls
99 99
 $show_learn_path = true;
100 100
 if (isset($_SESSION['oLP']) && is_object($_SESSION['oLP'])) {
101
-	$lp_theme_css = $_SESSION['oLP']->get_theme();
101
+    $lp_theme_css = $_SESSION['oLP']->get_theme();
102 102
 }
103 103
 
104 104
 if ($mode == 'fullpage') {
Please login to merge, or discard this patch.
main/newscorm/scormMetadata.class.php 1 patch
Indentation   +47 added lines, -47 removed lines patch added patch discarded remove patch
@@ -39,60 +39,60 @@
 block discarded – undo
39 39
                 case 'manifest': // Do the same as the default.
40 40
                      $children = $element->childNodes;
41 41
                     foreach ($children as $child) {
42
-                         switch ($child->nodeType) {
43
-                             case XML_ELEMENT_NODE:
42
+                            switch ($child->nodeType) {
43
+                                case XML_ELEMENT_NODE:
44 44
                                 // Could be 'lom', 'schema', 'schemaversion' or 'location'.
45 45
                                  switch ($child->tagName) {
46
-                                     case 'lom':
46
+                                        case 'lom':
47 47
                                          $childchildren = $child->childNodes;
48
-                                         foreach ($childchildren as $childchild) {
49
-                                             $this->lom = $childchild->nodeValue;
50
-                                         }
51
-                                         break;
52
-                                     case 'schema':
48
+                                            foreach ($childchildren as $childchild) {
49
+                                                $this->lom = $childchild->nodeValue;
50
+                                            }
51
+                                            break;
52
+                                        case 'schema':
53 53
                                          $childchildren = $child->childNodes;
54
-                                         foreach ($childchildren as $childchild) {
55
-                                             // There is generally only one child here.
56
-                                             $this->schema = $childchild->nodeValue;
57
-                                         }
58
-                                         break;
59
-                                     case 'schemaversion':
54
+                                            foreach ($childchildren as $childchild) {
55
+                                                // There is generally only one child here.
56
+                                                $this->schema = $childchild->nodeValue;
57
+                                            }
58
+                                            break;
59
+                                        case 'schemaversion':
60 60
                                          $childchildren = $child->childNodes;
61
-                                         foreach ($childchildren as $childchild) {
62
-                                             // There is generally only one child here.
63
-                                             $this->schemaversion = $childchild->nodeValue;
64
-                                         }
65
-                                         break;
66
-                                     case 'location':
61
+                                            foreach ($childchildren as $childchild) {
62
+                                                // There is generally only one child here.
63
+                                                $this->schemaversion = $childchild->nodeValue;
64
+                                            }
65
+                                            break;
66
+                                        case 'location':
67 67
                                          $childchildren = $child->childNodes;
68
-                                         foreach ($childchildren as $childchild) {
69
-                                             // There is generally only one child here.
70
-                                             $this->location = $childchild->nodeValue;
71
-                                         }
72
-                                         break;
73
-                                 }
74
-                                 break;
75
-                             case XML_TEXT_NODE:
68
+                                            foreach ($childchildren as $childchild) {
69
+                                                // There is generally only one child here.
70
+                                                $this->location = $childchild->nodeValue;
71
+                                            }
72
+                                            break;
73
+                                    }
74
+                                    break;
75
+                                case XML_TEXT_NODE:
76 76
                                  if (trim($child->textContent) != '') {
77
-                                     if (count($children == 1)) {
78
-                                         // If this is the only child at this level and it is a content... save differently.
79
-                                         $this->text = $child->textContent;
80
-                                     } else {
81
-                                         $this->text[$element->tagName] = $child->textContent;
82
-                                     }
83
-                                 }
84
-                                 break;
85
-                         }
86
-                     }
87
-                     $attributes = $element->attributes;
88
-                     //$keep_href = '';
89
-                     if (is_array($attributes)) {
90
-                         foreach ($attributes as $attrib) {
91
-                             if (trim($attrib->value) != ''){
92
-                                 $this->attribs[$attrib->name] = $attrib->value;
93
-                             }
94
-                         }
95
-                     }
77
+                                        if (count($children == 1)) {
78
+                                            // If this is the only child at this level and it is a content... save differently.
79
+                                            $this->text = $child->textContent;
80
+                                        } else {
81
+                                            $this->text[$element->tagName] = $child->textContent;
82
+                                        }
83
+                                    }
84
+                                    break;
85
+                            }
86
+                        }
87
+                        $attributes = $element->attributes;
88
+                        //$keep_href = '';
89
+                        if (is_array($attributes)) {
90
+                            foreach ($attributes as $attrib) {
91
+                                if (trim($attrib->value) != ''){
92
+                                    $this->attribs[$attrib->name] = $attrib->value;
93
+                                }
94
+                            }
95
+                        }
96 96
                     return true;
97 97
                     //break;
98 98
             }
Please login to merge, or discard this patch.
main/newscorm/lp_controller.php 1 patch
Indentation   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -538,17 +538,17 @@  discard block
 block discarded – undo
538 538
                 if (isset($_REQUEST['activate_start_date_check']) &&
539 539
                     $_REQUEST['activate_start_date_check'] == 1
540 540
                 ) {
541
-                	$publicated_on = $_REQUEST['publicated_on'];
541
+                    $publicated_on = $_REQUEST['publicated_on'];
542 542
                 } else {
543
-                	$publicated_on = null;
543
+                    $publicated_on = null;
544 544
                 }
545 545
 
546 546
                 if (isset($_REQUEST['activate_end_date_check']) &&
547 547
                     $_REQUEST['activate_end_date_check'] == 1
548 548
                 ) {
549
-                	$expired_on = $_REQUEST['expired_on'];
549
+                    $expired_on = $_REQUEST['expired_on'];
550 550
                 } else {
551
-                	$expired_on = null;
551
+                    $expired_on = null;
552 552
                 }
553 553
 
554 554
                 $new_lp_id = learnpath::add_lp(
@@ -937,9 +937,9 @@  discard block
 block discarded – undo
937 937
             $_SESSION['oLP']->setSubscribeUsers($_REQUEST['subscribe_users']);
938 938
 
939 939
             if (isset($_REQUEST['activate_start_date_check']) && $_REQUEST['activate_start_date_check'] == 1) {
940
-            	$publicated_on  = $_REQUEST['publicated_on'];
940
+                $publicated_on  = $_REQUEST['publicated_on'];
941 941
             } else {
942
-            	$publicated_on = null;
942
+                $publicated_on = null;
943 943
             }
944 944
 
945 945
             if (isset($_REQUEST['activate_end_date_check']) && $_REQUEST['activate_end_date_check'] == 1) {
@@ -1151,7 +1151,7 @@  discard block
 block discarded – undo
1151 1151
         } elseif ($mode == 'embedded') {
1152 1152
             $_SESSION['oLP']->mode = 'embedded';
1153 1153
         } elseif ($mode == 'embedframe') {
1154
-        	$_SESSION['oLP']->mode = 'embedframe';
1154
+            $_SESSION['oLP']->mode = 'embedframe';
1155 1155
         } elseif ($mode == 'impress') {
1156 1156
             $_SESSION['oLP']->mode = 'impress';
1157 1157
         }
@@ -1189,11 +1189,11 @@  discard block
 block discarded – undo
1189 1189
 		require 'lp_list.php';
1190 1190
 		break;
1191 1191
      */
1192
-	case 'switch_attempt_mode':
1193
-		if($debug>0) error_log('New LP - switch_reinit action triggered',0);
1194
-		if(!$lp_found){ error_log('New LP - No learnpath given for switch',0); require 'lp_list.php'; }
1195
-		$_SESSION['refresh'] = 1;
1196
-		$_SESSION['oLP']->switch_attempt_mode();
1192
+    case 'switch_attempt_mode':
1193
+        if($debug>0) error_log('New LP - switch_reinit action triggered',0);
1194
+        if(!$lp_found){ error_log('New LP - No learnpath given for switch',0); require 'lp_list.php'; }
1195
+        $_SESSION['refresh'] = 1;
1196
+        $_SESSION['oLP']->switch_attempt_mode();
1197 1197
         require 'lp_list.php';
1198 1198
         break;
1199 1199
     case 'switch_scorm_debug':
Please login to merge, or discard this patch.