Passed
Push — 1.10.x ( f9cbcd...83c791 )
by Yannick
657:30 queued 614:41
created
main/resourcelinker/resourcelinker.php 1 patch
Spacing   +39 added lines, -39 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 use ChamiloSession as Session;
18 18
 
19 19
 include ('../inc/global.inc.php');
20
-$this_section=SECTION_COURSES;
20
+$this_section = SECTION_COURSES;
21 21
 
22 22
 api_protect_course_script();
23 23
 
@@ -110,11 +110,11 @@  discard block
 block discarded – undo
110 110
 	$max_temp2 = $row[0];
111 111
 	if ($max_temp2 > $max_temp)
112 112
 	{
113
-		$order = $max_temp2 +1;
113
+		$order = $max_temp2 + 1;
114 114
 	}
115 115
 	else
116 116
 	{
117
-		$order = $max_temp +1;
117
+		$order = $max_temp + 1;
118 118
 	}
119 119
 
120 120
 	$sql = "INSERT INTO $tbl_learnpath_chapter "."(learnpath_id,chapter_name,chapter_description,parent_chapter_id,display_order) "." VALUES "."($learnpath_id, '$title', '$description', $chapter_id, $order )";
@@ -186,21 +186,21 @@  discard block
 block discarded – undo
186 186
 		//calculating the last order of the items of this chapter
187 187
 		$sql = "SELECT MAX(display_order) FROM $tbl_learnpath_item WHERE chapter_id=$chapter_id";
188 188
 		$result = Database::query($sql);
189
-		if(Database::num_rows($result)==0){
189
+		if (Database::num_rows($result) == 0) {
190 190
 			$lastorder_item = 0;
191
-		}else{
191
+		} else {
192 192
 			$row = Database::fetch_array($result);
193 193
 			$lastorder_item = ($row[0]);
194 194
 		}
195 195
 		$sql = "SELECT MAX(display_order) FROM $tbl_learnpath_chapter WHERE parent_chapter_id=$chapter_id";
196 196
 		$result = Database::query($sql);
197
-		if(Database::num_rows($result)==0){
197
+		if (Database::num_rows($result) == 0) {
198 198
 			$lastorder_chapter = 0;
199
-		}else{
199
+		} else {
200 200
 			$row = Database::fetch_array($result);
201 201
 			$lastorder_chapter = ($row[0]);
202 202
 		}
203
-		$lastorder = ($lastorder_chapter>$lastorder_item?$lastorder_chapter+1:$lastorder_item+1);
203
+		$lastorder = ($lastorder_chapter > $lastorder_item ? $lastorder_chapter + 1 : $lastorder_item + 1);
204 204
 
205 205
 		foreach ($addedresource as $addedresource_item)
206 206
 		{
@@ -251,8 +251,8 @@  discard block
 block discarded – undo
251 251
 				$addedresourceassigned[$i] = 1;
252 252
 				$resource_added = true;
253 253
 			}
254
-			$i ++;
255
-			$lastorder ++;
254
+			$i++;
255
+			$lastorder++;
256 256
 		}
257 257
 		//$_SESSION['addedresource']=null;
258 258
 		//$_SESSION['addedresourceid']=null;
@@ -294,34 +294,34 @@  discard block
 block discarded – undo
294 294
 				$url = "../calendar/agenda.php?action=add";
295 295
 			}
296 296
 			$originaltoolname = get_lang("Agenda");
297
-			$breadcrumbelement = array ("url" => $url, "name" => $originaltoolname);
297
+			$breadcrumbelement = array("url" => $url, "name" => $originaltoolname);
298 298
 			session_unregister('from_learnpath');
299 299
 			unset ($from_learnpath);
300 300
 			break;
301 301
 		case "2" : // coming from forum: new topic
302 302
 			$url = "../phpbb/newtopic.php?forum=$source_forum&md5=$md5";
303 303
 			$originaltoolname = get_lang("ForumAddNewTopic");
304
-			$breadcrumbelement = array ("url" => $url, "name" => $originaltoolname);
304
+			$breadcrumbelement = array("url" => $url, "name" => $originaltoolname);
305 305
 			session_unregister('from_learnpath');
306 306
 			unset ($from_learnpath);
307 307
 			break;
308 308
 		case "3" : // coming from forum: edit topic
309 309
 			$url = "../phpbb/editpost.php?post_id=$post_id&topic=$topic&forum=$forum&md5=$md5&originalresource=no";
310 310
 			$originaltoolname = get_lang("ForumEditTopic");
311
-			$breadcrumbelement = array ("url" => $url, "name" => $originaltoolname);
311
+			$breadcrumbelement = array("url" => $url, "name" => $originaltoolname);
312 312
 			session_unregister('from_learnpath');
313 313
 			unset ($from_learnpath);
314 314
 			break;
315 315
 		case "4" : // coming from exercises: edit topic
316 316
 			$url = "../exercice/admin.php?modifyAnswers=$modifyAnswers";
317 317
 			$originaltoolname = get_lang("ExerciseAnswers");
318
-			$breadcrumbelement = array ("url" => $url, "name" => $originaltoolname);
318
+			$breadcrumbelement = array("url" => $url, "name" => $originaltoolname);
319 319
 			session_unregister('from_learnpath');
320 320
 			unset ($from_learnpath);
321 321
 			break;
322 322
 		case "5" : // coming from learning path
323 323
 			$from_learnpath = 'yes';
324
-			Session::write('from_learnpath',$from_learnpath);
324
+			Session::write('from_learnpath', $from_learnpath);
325 325
 			break;
326 326
 			/*************************************** end add [email protected] *********************************/
327 327
 
@@ -329,7 +329,7 @@  discard block
 block discarded – undo
329 329
 	// We do not come from the learning path. We store the name of the tool & url in a session.
330 330
 	if ($from_learnpath != 'yes')
331 331
 	{
332
-		if (!$_SESSION["origintoolurl"] OR $_SESSION["origintoolurl"]<>$interbreadcrumb["url"])
332
+		if (!$_SESSION["origintoolurl"] OR $_SESSION["origintoolurl"] <> $interbreadcrumb["url"])
333 333
 		{
334 334
 			$_SESSION["origintoolurl"] = $breadcrumbelement["url"];
335 335
 			$_SESSION["origintoolname"] = $breadcrumbelement["name"];
@@ -344,7 +344,7 @@  discard block
 block discarded – undo
344 344
 if ($from_learnpath != 'yes')
345 345
 {
346 346
 	$nameTools = get_lang('AddResource');
347
-	$interbreadcrumb[] = array ("url" => $_SESSION["origintoolurl"], "name" => $_SESSION["origintoolname"]);
347
+	$interbreadcrumb[] = array("url" => $_SESSION["origintoolurl"], "name" => $_SESSION["origintoolname"]);
348 348
 }
349 349
 else
350 350
 {
@@ -360,9 +360,9 @@  discard block
 block discarded – undo
360 360
 
361 361
 	$from_learnpath = 'yes';
362 362
 	session_register('from_learnpath');
363
-	$interbreadcrumb[] = array ("url" => "../scorm/scormdocument.php", "name" => get_lang('LearningPath'));
364
-	$interbreadcrumb[] = array ("url" => "../learnpath/learnpath_handler.php?learnpath_id=$learnpath_id", "name" => "{$therow['learnpath_name']}");
365
-	$interbreadcrumb[] = array ("url" => api_get_self()."?action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no", "name" => "{$therow2['chapter_name']}");
363
+	$interbreadcrumb[] = array("url" => "../scorm/scormdocument.php", "name" => get_lang('LearningPath'));
364
+	$interbreadcrumb[] = array("url" => "../learnpath/learnpath_handler.php?learnpath_id=$learnpath_id", "name" => "{$therow['learnpath_name']}");
365
+	$interbreadcrumb[] = array("url" => api_get_self()."?action=$action&learnpath_id=$learnpath_id&chapter_id=$chapter_id&originalresource=no", "name" => "{$therow2['chapter_name']}");
366 366
 
367 367
 }
368 368
 
@@ -387,13 +387,13 @@  discard block
 block discarded – undo
387 387
 // we retrieve the tools that are active.
388 388
 // We use this to check which resources a student may add (only the modules that are active)
389 389
 // see http://www.dokeos.com/forum/viewtopic.php?t=4858
390
-$active_modules=array();
390
+$active_modules = array();
391 391
 $tool_table = Database::get_course_table(TABLE_TOOL_LIST);
392
-$sql_select_active="SELECT * FROM $tool_table WHERE visibility='1'";
393
-$result_select_active=Database::query($sql_select_active);
394
-while ($row=Database::fetch_array($result_select_active))
392
+$sql_select_active = "SELECT * FROM $tool_table WHERE visibility='1'";
393
+$result_select_active = Database::query($sql_select_active);
394
+while ($row = Database::fetch_array($result_select_active))
395 395
 {
396
-	$active_modules[]=$row['name'];
396
+	$active_modules[] = $row['name'];
397 397
 }
398 398
 
399 399
 
@@ -408,7 +408,7 @@  discard block
 block discarded – undo
408 408
           <td width="26%"><b><?php echo get_lang('CourseResources'); ?></b></td>
409 409
         </tr>
410 410
         <?php
411
-        if (api_is_allowed_to_edit() OR in_array(TOOL_DOCUMENT,$active_modules))
411
+        if (api_is_allowed_to_edit() OR in_array(TOOL_DOCUMENT, $active_modules))
412 412
         {
413 413
         ?>
414 414
         <tr>
@@ -416,7 +416,7 @@  discard block
 block discarded – undo
416 416
         </tr>
417 417
         <?php
418 418
         }
419
-        if (api_is_allowed_to_edit() OR in_array(TOOL_CALENDAR_EVENT,$active_modules))
419
+        if (api_is_allowed_to_edit() OR in_array(TOOL_CALENDAR_EVENT, $active_modules))
420 420
         {
421 421
         ?>
422 422
         <tr>
@@ -424,7 +424,7 @@  discard block
 block discarded – undo
424 424
         </tr>
425 425
         <?php
426 426
         }
427
-        if (api_is_allowed_to_edit() OR in_array(TOOL_ANNOUNCEMENT,$active_modules))
427
+        if (api_is_allowed_to_edit() OR in_array(TOOL_ANNOUNCEMENT, $active_modules))
428 428
         {
429 429
         ?>
430 430
         <tr>
@@ -432,7 +432,7 @@  discard block
 block discarded – undo
432 432
         </tr>
433 433
         <?php
434 434
         }
435
-        if (api_is_allowed_to_edit() OR in_array(TOOL_BB_FORUM,$active_modules))
435
+        if (api_is_allowed_to_edit() OR in_array(TOOL_BB_FORUM, $active_modules))
436 436
         {
437 437
         ?>
438 438
         <tr>
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
         </tr>
441 441
         <?php
442 442
         }
443
-        if (api_is_allowed_to_edit() OR in_array(TOOL_LINK,$active_modules))
443
+        if (api_is_allowed_to_edit() OR in_array(TOOL_LINK, $active_modules))
444 444
         {
445 445
         ?>
446 446
         <tr>
@@ -448,7 +448,7 @@  discard block
 block discarded – undo
448 448
         </tr>
449 449
         <?php
450 450
         }
451
-        if (api_is_allowed_to_edit() OR in_array(TOOL_QUIZ,$active_modules))
451
+        if (api_is_allowed_to_edit() OR in_array(TOOL_QUIZ, $active_modules))
452 452
         {
453 453
         ?>
454 454
         <tr>
@@ -468,7 +468,7 @@  discard block
 block discarded – undo
468 468
 		<tr>
469 469
           <td width="26%"><b><?php echo get_lang('ExportableCourseResources'); ?></b></td>
470 470
         </tr>
471
-<?php if ($multi_level_learnpath === true ) { ?>
471
+<?php if ($multi_level_learnpath === true) { ?>
472 472
         <tr>
473 473
           <td><?php echo "<a href=\"".api_get_self()."?content=chapter&action=$action&id=$id&learnpath_id=$learnpath_id&chapter_id=$chapter_id&source_forum=$source_forum&originalresource=no\">".get_lang('Chapter')."</a>"; ?></td>
474 474
         </tr>
@@ -589,10 +589,10 @@  discard block
 block discarded – undo
589 589
 // Agenda items -->
590 590
 if ($content == "Agenda")
591 591
 {
592
-	$TABLEAGENDA 			= Database::get_course_table(TABLE_AGENDA);
593
-	$TABLE_ITEM_PROPERTY 	= Database::get_course_table(TABLE_ITEM_PROPERTY);
592
+	$TABLEAGENDA = Database::get_course_table(TABLE_AGENDA);
593
+	$TABLE_ITEM_PROPERTY = Database::get_course_table(TABLE_ITEM_PROPERTY);
594 594
 
595
-	$sql="SELECT agenda.*, toolitemproperties.*
595
+	$sql = "SELECT agenda.*, toolitemproperties.*
596 596
 					FROM ".$TABLEAGENDA." agenda, ".$TABLE_ITEM_PROPERTY." toolitemproperties
597 597
 					WHERE agenda.id = toolitemproperties.ref
598 598
 					AND toolitemproperties.tool='".TOOL_CALENDAR_EVENT."'
@@ -638,7 +638,7 @@  discard block
 block discarded – undo
638 638
 // 2. we come to the resource linker for the first time (documents = default). In this case it can only be shown if
639 639
 //  			a. one is a teacher (documents can be shown even if the tool is inactive)
640 640
 //				b. one is a student AND the documents tool is active. Student cannot add documents if the documents tool is inactive (teacher can do this)
641
-if ($content == "Document" OR (empty($content) AND (api_is_allowed_to_edit() OR in_array(TOOL_DOCUMENT,$active_modules))) AND !$_GET['showresources'])
641
+if ($content == "Document" OR (empty($content) AND (api_is_allowed_to_edit() OR in_array(TOOL_DOCUMENT, $active_modules))) AND !$_GET['showresources'])
642 642
 {
643 643
 	// setting variables for file locations
644 644
 	$baseServDir = $_configuration['root_sys'];
@@ -698,7 +698,7 @@  discard block
 block discarded – undo
698 698
 {
699 699
 	$TBL_FORUMS 		= Database::get_course_table(TABLE_FORUM);
700 700
 	$TBL_CATAGORIES 	= Database::get_course_table(TABLE_FORUM_CATEGORY);
701
-	$TBL_FORUMTOPICS 	= Database::get_course_table(TABLE_FORUM_POST);
701
+	$TBL_FORUMTOPICS = Database::get_course_table(TABLE_FORUM_POST);
702 702
 	$tbl_posts 			= Database::get_course_table(TABLE_FORUM_POST);
703 703
 	$tbl_posts_text 	= Database::get_course_table(TOOL_FORUM_POST_TEXT_TABLE);
704 704
 
@@ -883,9 +883,9 @@  discard block
 block discarded – undo
883 883
 ?>
884 884
 	</tr>
885 885
     <tr>
886
-      <td><?php if ($is_allowedToEdit) {echo get_lang('AddToLinks');} ?></td>
886
+      <td><?php if ($is_allowedToEdit) {echo get_lang('AddToLinks'); } ?></td>
887 887
       <td>
888
-  	  <?php if ($is_allowedToEdit){?>
888
+  	  <?php if ($is_allowedToEdit) {?>
889 889
 	  <select name="add_2_links" id="add_2_links">
890 890
       <option value="niet toevoegen" selected="selected">-<?php echo get_lang('DontAdd'); ?>-</option>
891 891
 	  <option value="0"><?php echo get_lang('MainCategory'); ?></option>
Please login to merge, or discard this patch.
main/template/default/auth/courses_list.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 <!-- Actions: The menu with the different options in cathe course management -->
18 18
 <div id="actions" class="actions">
19 19
     <?php if ($action != 'createcoursecategory') { ?>
20
-	<a href="<?php echo api_get_self(); ?>?action=createcoursecategory"><?php echo Display::return_icon('new_folder.png', get_lang('CreateCourseCategory'),'','32'); ?></a>
20
+	<a href="<?php echo api_get_self(); ?>?action=createcoursecategory"><?php echo Display::return_icon('new_folder.png', get_lang('CreateCourseCategory'), '', '32'); ?></a>
21 21
     <?php } ?>
22 22
 </div>
23 23
 
@@ -49,26 +49,26 @@  discard block
 block discarded – undo
49 49
             $max_category_key = count($user_course_categories);
50 50
             if ($action != 'unsubscribe') { ?>
51 51
                 <a href="courses.php?action=sortmycourses&amp;categoryid=<?php echo $row['id']; ?>&amp;sec_token=<?php echo $stok; ?>#category<?php echo $row['id']; ?>">
52
-                <?php echo Display::display_icon('edit.png', get_lang('Edit'),'',22); ?>
52
+                <?php echo Display::display_icon('edit.png', get_lang('Edit'), '', 22); ?>
53 53
                 </a>
54 54
 
55 55
                 <?php if ($row['id'] != $user_course_categories[0]['id']) { ?>
56 56
                         <a href="courses.php?action=<?php echo $action ?>&amp;move=up&amp;category=<?php echo $row['id']; ?>&amp;sec_token=<?php echo $stok; ?>">
57
-                        <?php echo Display::return_icon('up.png', get_lang('Up'),'',22); ?>
57
+                        <?php echo Display::return_icon('up.png', get_lang('Up'), '', 22); ?>
58 58
                         </a>
59 59
                 <?php } else { ?>
60
-                    <?php echo Display::return_icon('up_na.png', get_lang('Up'),'',22); ?>
60
+                    <?php echo Display::return_icon('up_na.png', get_lang('Up'), '', 22); ?>
61 61
                <?php } ?>
62 62
 
63 63
                 <?php if ($row['id'] != $user_course_categories[$max_category_key - 1]['id']) { ?>
64 64
                     <a href="courses.php?action=<?php echo $action; ?>&amp;move=down&amp;category=<?php echo $row['id']; ?>&amp;sec_token=<?php echo $stok; ?>">
65
-                    <?php echo Display::return_icon('down.png', get_lang('Down'),'',22); ?>
65
+                    <?php echo Display::return_icon('down.png', get_lang('Down'), '', 22); ?>
66 66
                     </a>
67 67
                 <?php } else { ?>
68
-                    <?php echo Display::return_icon('down_na.png', get_lang('Down'),'',22); ?>
68
+                    <?php echo Display::return_icon('down_na.png', get_lang('Down'), '', 22); ?>
69 69
                 <?php } ?>
70 70
                 <a href="courses.php?action=deletecoursecategory&amp;id=<?php echo $row['id']; ?>&amp;sec_token=<?php echo $stok; ?>">
71
-                <?php echo Display::display_icon('delete.png', get_lang('Delete'), array('onclick' => "javascript: if (!confirm('".addslashes(api_htmlentities(get_lang("CourseCategoryAbout2bedeleted"), ENT_QUOTES, api_get_system_encoding()))."')) return false;"),22) ?>
71
+                <?php echo Display::display_icon('delete.png', get_lang('Delete'), array('onclick' => "javascript: if (!confirm('".addslashes(api_htmlentities(get_lang("CourseCategoryAbout2bedeleted"), ENT_QUOTES, api_get_system_encoding()))."')) return false;"), 22) ?>
72 72
                 </a>
73 73
             <?php }
74 74
             echo '<br /><br />';
@@ -122,35 +122,35 @@  discard block
 block discarded – undo
122 122
                     <div style="float:left;width:110px;">
123 123
                     <?php
124 124
                         if (api_get_setting('show_courses_descriptions_in_catalog') == 'true') {
125
-                            $icon_title = get_lang('CourseDetails') . ' - ' . $course['title'];
125
+                            $icon_title = get_lang('CourseDetails').' - '.$course['title'];
126 126
                     ?>
127 127
                     <a href="<?php echo api_get_path(WEB_CODE_PATH); ?>inc/ajax/course_home.ajax.php?a=show_course_information&code=<?php echo $course['code'] ?>" data-title="<?php echo $icon_title ?>" title="<?php echo $icon_title ?>" class="ajax">
128
-                        <?php echo Display::return_icon('info.png', $icon_title,'','22') ?>
128
+                        <?php echo Display::return_icon('info.png', $icon_title, '', '22') ?>
129 129
                        <?php } ?>
130 130
                     </a>
131 131
 
132 132
                     <?php if (isset($_GET['edit']) && $course['code'] == $_GET['edit']) { ?>
133
-                          <?php echo Display::display_icon('edit_na.png', get_lang('Edit'),'',22); ?>
133
+                          <?php echo Display::display_icon('edit_na.png', get_lang('Edit'), '', 22); ?>
134 134
                     <?php } else { ?>
135 135
                         <a href="courses.php?action=<?php echo $action; ?>&amp;edit=<?php echo $course['code']; ?>&amp;sec_token=<?php echo $stok; ?>">
136
-                            <?php echo Display::display_icon('edit.png', get_lang('Edit'),'',22); ?>
136
+                            <?php echo Display::display_icon('edit.png', get_lang('Edit'), '', 22); ?>
137 137
                             </a>
138 138
                     <?php } ?>
139 139
 
140 140
                     <?php if ($key > 0) { ?>
141 141
                         <a href="courses.php?action=<?php echo $action; ?>&amp;move=up&amp;course=<?php echo $course['code']; ?>&amp;category=<?php echo $course['user_course_cat']; ?>&amp;sec_token=<?php echo $stok; ?>">
142
-                        <?php echo Display::display_icon('up.png', get_lang('Up'),'',22); ?>
142
+                        <?php echo Display::display_icon('up.png', get_lang('Up'), '', 22); ?>
143 143
                         </a>
144 144
                     <?php } else { ?>
145
-                        <?php echo Display::display_icon('up_na.png', get_lang('Up'),'',22); ?>
145
+                        <?php echo Display::display_icon('up_na.png', get_lang('Up'), '', 22); ?>
146 146
                     <?php } ?>
147 147
 
148 148
                     <?php if ($key < $number_of_courses - 1) { ?>
149 149
                         <a href="courses.php?action=<?php echo $action; ?>&amp;move=down&amp;course=<?php echo $course['code']; ?>&amp;category=<?php echo $course['user_course_cat']; ?>&amp;sec_token=<?php echo $stok; ?>">
150
-                        <?php echo Display::display_icon('down.png', get_lang('Down'),'',22); ?>
150
+                        <?php echo Display::display_icon('down.png', get_lang('Down'), '', 22); ?>
151 151
                         </a>
152 152
                     <?php } else { ?>
153
-                        <?php echo Display::display_icon('down_na.png', get_lang('Down'),'',22); ?>
153
+                        <?php echo Display::display_icon('down_na.png', get_lang('Down'), '', 22); ?>
154 154
                     <?php } ?>
155 155
 
156 156
                   </div>
@@ -221,36 +221,36 @@  discard block
 block discarded – undo
221 221
             <div style="float:left; width:110px">
222 222
             <?php
223 223
             if (api_get_setting('show_courses_descriptions_in_catalog') == 'true') {
224
-            $icon_title = get_lang('CourseDetails') . ' - ' . $course['title'];
224
+            $icon_title = get_lang('CourseDetails').' - '.$course['title'];
225 225
             ?>
226 226
             <a href="<?php echo api_get_path(WEB_CODE_PATH); ?>inc/ajax/course_home.ajax.php?a=show_course_information&code=<?php echo $course['code'] ?>" data-title="<?php echo $icon_title ?>" title="<?php echo $icon_title ?>" class="ajax">
227
-                <?php echo Display::return_icon('info.png', $icon_title, '','22'); ?>
227
+                <?php echo Display::return_icon('info.png', $icon_title, '', '22'); ?>
228 228
             </a>
229 229
             <?php } ?>
230 230
 
231 231
              <?php if (isset($_GET['edit']) && $course['code'] == $_GET['edit']) {   ?>
232
-                    <?php echo Display::display_icon('edit_na.png', get_lang('Edit'),'',22); ?>
232
+                    <?php echo Display::display_icon('edit_na.png', get_lang('Edit'), '', 22); ?>
233 233
                   <?php } else { ?>
234 234
                     <a href="courses.php?action=<?php echo $action; ?>&amp;edit=<?php echo $course['code']; ?>&amp;sec_token=<?php echo $stok; ?>">
235
-                    <?php echo Display::display_icon('edit.png', get_lang('Edit'),'',22); ?>
235
+                    <?php echo Display::display_icon('edit.png', get_lang('Edit'), '', 22); ?>
236 236
                     </a>
237 237
              <?php } ?>
238 238
 
239 239
             <!-- up /down icons-->
240 240
             <?php if ($key > 0) { ?>
241 241
                     <a href="courses.php?action=<?php echo $action; ?>&amp;move=up&amp;course=<?php echo $course['code']; ?>&amp;category=<?php echo $course['user_course_cat']; ?>&amp;sec_token=<?php echo $stok; ?>">
242
-                    <?php echo Display::display_icon('up.png', get_lang('Up'),'',22) ?>
242
+                    <?php echo Display::display_icon('up.png', get_lang('Up'), '', 22) ?>
243 243
                     </a>
244 244
             <?php } else {
245
-                    echo Display::display_icon('up_na.png', get_lang('Up'),'',22);
245
+                    echo Display::display_icon('up_na.png', get_lang('Up'), '', 22);
246 246
                   }
247 247
 
248 248
                   if ($key < $number_of_courses - 1) { ?>
249 249
                     <a href="courses.php?action=<?php echo $action; ?>&amp;move=down&amp;course=<?php echo $course['code']; ?>&amp;category=<?php echo $course['user_course_cat']; ?>&amp;sec_token=<?php echo $stok; ?>">
250
-                    <?php echo Display::display_icon('down.png', get_lang('Down'),'',22); ?>
250
+                    <?php echo Display::display_icon('down.png', get_lang('Down'), '', 22); ?>
251 251
                     </a>
252 252
             <?php } else {
253
-                    echo Display::display_icon('down_na.png', get_lang('Down'),'',22);
253
+                    echo Display::display_icon('down_na.png', get_lang('Down'), '', 22);
254 254
                   }?>
255 255
                 </div>
256 256
                  <div style="float:left; margin-right:10px;">
Please login to merge, or discard this patch.
main/template/default/auth/categories_list.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@
 block discarded – undo
16 16
 <!-- Actions: The menu with the different options in cathe course management -->
17 17
 <div id="actions" class="actions">
18 18
     <a href="<?php echo api_get_self() ?>?action=sortmycourses">
19
-        <?php echo Display::return_icon('back.png', get_lang('Back'),'','32'); ?>
19
+        <?php echo Display::return_icon('back.png', get_lang('Back'), '', '32'); ?>
20 20
     </a>
21 21
 </div>
22 22
 
Please login to merge, or discard this patch.
main/template/default/auth/courses_categories.php 1 patch
Spacing   +24 added lines, -29 removed lines patch added patch discarded remove patch
@@ -15,17 +15,12 @@  discard block
 block discarded – undo
15 15
 
16 16
 $showCourses = CoursesAndSessionsCatalog::showCourses();
17 17
 $showSessions = CoursesAndSessionsCatalog::showSessions();
18
-$pageCurrent = isset($pageCurrent) ? $pageCurrent :
19
-    isset($_GET['pageCurrent']) ? intval($_GET['pageCurrent']) :
20
-        1;
21
-$pageLength = isset($pageLength) ? $pageLength :
22
-    isset($_GET['pageLength']) ? intval($_GET['pageLength']) :
23
-        10;
18
+$pageCurrent = isset($pageCurrent) ? $pageCurrent : isset($_GET['pageCurrent']) ? intval($_GET['pageCurrent']) : 1;
19
+$pageLength = isset($pageLength) ? $pageLength : isset($_GET['pageLength']) ? intval($_GET['pageLength']) : 10;
24 20
 $pageTotal = intval(ceil(intval($countCoursesInCategory) / $pageLength));
25 21
 $cataloguePagination = $pageTotal > 1 ?
26
-    getCataloguePagination($pageCurrent, $pageLength, $pageTotal) :
27
-    '';
28
-$search_term = isset($search_term) ? $search_term :null;
22
+    getCataloguePagination($pageCurrent, $pageLength, $pageTotal) : '';
23
+$search_term = isset($search_term) ? $search_term : null;
29 24
 
30 25
 if ($showSessions && isset($_POST['date'])) {
31 26
     $date = $_POST['date'];
@@ -66,7 +61,7 @@  discard block
 block discarded – undo
66 61
             $el.prepend('<?php echo Display::display_icon('nolines_minus.gif'); ?>');
67 62
 
68 63
             $.ajax({
69
-                url: '<?php echo api_get_path(WEB_AJAX_PATH) . 'course.ajax.php' ?>',
64
+                url: '<?php echo api_get_path(WEB_AJAX_PATH).'course.ajax.php' ?>',
70 65
                 type: 'GET',
71 66
                 dataType: 'json',
72 67
                 data: {
@@ -130,13 +125,13 @@  discard block
 block discarded – undo
130 125
         <?php
131 126
 
132 127
             $webAction = api_get_path(WEB_CODE_PATH).'auth/courses.php';
133
-            $action = (!empty($_REQUEST['action'])?Security::remove_XSS($_REQUEST['action']):'display_courses');
134
-            $pageLength = (!empty($_REQUEST['pageLength'])?intval($_REQUEST['pageLength']):10);
135
-            $pageCurrent = (!empty($_REQUEST['pageCurrent'])?intval($_REQUEST['pageCurrent']):1);
128
+            $action = (!empty($_REQUEST['action']) ? Security::remove_XSS($_REQUEST['action']) : 'display_courses');
129
+            $pageLength = (!empty($_REQUEST['pageLength']) ? intval($_REQUEST['pageLength']) : 10);
130
+            $pageCurrent = (!empty($_REQUEST['pageCurrent']) ? intval($_REQUEST['pageCurrent']) : 1);
136 131
             $form = '<form action="'.$webAction.'" method="GET" class="form-horizontal">';
137
-            $form .= '<input type="hidden" name="action" value="' . $action . '">';
138
-            $form .= '<input type="hidden" name="pageCurrent" value="' . $pageCurrent . '">';
139
-            $form .= '<input type="hidden" name="pageLength" value="' . $pageLength . '">';
132
+            $form .= '<input type="hidden" name="action" value="'.$action.'">';
133
+            $form .= '<input type="hidden" name="pageCurrent" value="'.$pageCurrent.'">';
134
+            $form .= '<input type="hidden" name="pageLength" value="'.$pageLength.'">';
140 135
             $form .= '<div class="form-group">';
141 136
             $form .= '<div class="col-sm-12">';
142 137
             $form .= '<select name="category_code" onchange="submit();" class="selectpicker show-tick form-control">';
@@ -145,11 +140,11 @@  discard block
 block discarded – undo
145 140
                 $categoryCode = $category['code'];
146 141
                 $countCourse = $category['count_courses'];
147 142
 
148
-                $form .= '<option '. ($categoryCode == $codeType? 'selected="selected" ':'') .' value="' . $category['code'] . '">' . $category['name'] . ' ( '. $countCourse .' ) </option>';
143
+                $form .= '<option '.($categoryCode == $codeType ? 'selected="selected" ' : '').' value="'.$category['code'].'">'.$category['name'].' ( '.$countCourse.' ) </option>';
149 144
                 if (!empty($browse_course_categories[$categoryCode])) {
150
-                    foreach ($browse_course_categories[$categoryCode] as $subCategory){
145
+                    foreach ($browse_course_categories[$categoryCode] as $subCategory) {
151 146
                         $subCategoryCode = $subCategory['code'];
152
-                        $form .= '<option '. ($subCategoryCode == $codeType ? 'selected="selected" ':'') .' value="' . $subCategory['code'] . '"> ---' . $subCategory['name'] . ' ( '. $subCategory['count_courses'] .' ) </option>';
147
+                        $form .= '<option '.($subCategoryCode == $codeType ? 'selected="selected" ' : '').' value="'.$subCategory['code'].'"> ---'.$subCategory['name'].' ( '.$subCategory['count_courses'].' ) </option>';
153 148
                     }
154 149
                 }
155 150
             }
@@ -212,7 +207,7 @@  discard block
 block discarded – undo
212 207
             $course_subscribe_allowed = ($course['subscribe'] == 1);
213 208
             $course_unsubscribe_allowed = ($course['unsubscribe'] == 1);
214 209
             $count_connections = $course['count_connections'];
215
-            $creation_date = substr($course['creation_date'],0,10);
210
+            $creation_date = substr($course['creation_date'], 0, 10);
216 211
 
217 212
             $icon_title = null;
218 213
             $html = null;
@@ -328,9 +323,9 @@  discard block
 block discarded – undo
328 323
     $ajax_url = api_get_path(WEB_AJAX_PATH).'course.ajax.php?a=add_course_vote';
329 324
     $teachers = CourseManager::get_teacher_list_from_course_code_to_string($course['code']);
330 325
     $rating = Display::return_rating_system('star_'.$course['real_id'], $ajax_url.'&course_id='.$course['real_id'], $course['point_info']);
331
-    $html .=  '<h4 class="title"><a href="' . $linkCourse . '">' . cut($title, 60) . '</a></h4>';
326
+    $html .= '<h4 class="title"><a href="'.$linkCourse.'">'.cut($title, 60).'</a></h4>';
332 327
     $html .= '<div class="teachers">'.$teachers.'</div>';
333
-    $html .= '<div class="ranking">'. $rating . '</div>';
328
+    $html .= '<div class="ranking">'.$rating.'</div>';
334 329
 
335 330
     return $html;
336 331
 }
@@ -345,8 +340,8 @@  discard block
 block discarded – undo
345 340
     $title = $course['title'];
346 341
     $html = '';
347 342
     if (api_get_setting('show_courses_descriptions_in_catalog') == 'true') {
348
-        $html = '<a data-title="' . $title . '" class="ajax btn btn-default btn-sm" href="'.api_get_path(WEB_CODE_PATH).'inc/ajax/course_home.ajax.php?a=show_course_information&code='.$course['code'].'" title="' . get_lang('Description') . '">' .
349
-        Display::returnFontAwesomeIcon('info-circle') . '</a>';
343
+        $html = '<a data-title="'.$title.'" class="ajax btn btn-default btn-sm" href="'.api_get_path(WEB_CODE_PATH).'inc/ajax/course_home.ajax.php?a=show_course_information&code='.$course['code'].'" title="'.get_lang('Description').'">'.
344
+        Display::returnFontAwesomeIcon('info-circle').'</a>';
350 345
     }
351 346
 
352 347
     return $html;
@@ -358,7 +353,7 @@  discard block
 block discarded – undo
358 353
  */
359 354
 function return_goto_button($course)
360 355
 {
361
-    $html = ' <a class="btn btn-default btn-sm" title="' . get_lang('GoToCourse') . '" href="'.api_get_course_url($course['code']).'">'.
356
+    $html = ' <a class="btn btn-default btn-sm" title="'.get_lang('GoToCourse').'" href="'.api_get_course_url($course['code']).'">'.
362 357
     Display::returnFontAwesomeIcon('share').'</a>';
363 358
 
364 359
     return $html;
@@ -392,8 +387,8 @@  discard block
 block discarded – undo
392 387
  */
393 388
 function return_register_button($course, $stok, $code, $search_term)
394 389
 {
395
-    $html = ' <a class="btn btn-success btn-sm" title="' . get_lang('Subscribe') . '" href="'.api_get_self().'?action=subscribe_course&sec_token='.$stok.'&subscribe_course='.$course['code'].'&search_term='.$search_term.'&category_code='.$code.'">' .
396
-    Display::returnFontAwesomeIcon('sign-in') . '</a>';
390
+    $html = ' <a class="btn btn-success btn-sm" title="'.get_lang('Subscribe').'" href="'.api_get_self().'?action=subscribe_course&sec_token='.$stok.'&subscribe_course='.$course['code'].'&search_term='.$search_term.'&category_code='.$code.'">'.
391
+    Display::returnFontAwesomeIcon('sign-in').'</a>';
397 392
     return $html;
398 393
 }
399 394
 
@@ -406,7 +401,7 @@  discard block
 block discarded – undo
406 401
  */
407 402
 function return_unregister_button($course, $stok, $search_term, $code)
408 403
 {
409
-    $html = ' <a class="btn btn-danger btn-sm" title="' . get_lang('Unsubscribe') . '" href="'. api_get_self().'?action=unsubscribe&sec_token='.$stok.'&unsubscribe='.$course['code'].'&search_term='.$search_term.'&category_code='.$code.'">' .
410
-    Display::returnFontAwesomeIcon('sign-out') . '</a>';
404
+    $html = ' <a class="btn btn-danger btn-sm" title="'.get_lang('Unsubscribe').'" href="'.api_get_self().'?action=unsubscribe&sec_token='.$stok.'&unsubscribe='.$course['code'].'&search_term='.$search_term.'&category_code='.$code.'">'.
405
+    Display::returnFontAwesomeIcon('sign-out').'</a>';
411 406
     return $html;
412 407
 }
Please login to merge, or discard this patch.
main/upload/upload_ppt.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
         if (in_array(strtolower(pathinfo($_FILES['user_file']['name'], PATHINFO_EXTENSION)), $allowed_extensions)) {
22 22
             require_once api_get_path(SYS_CODE_PATH).'newscorm/lp_upload.php';
23 23
             if (isset($o_ppt) && $first_item_id != 0) {
24
-                if (api_get_setting('search_enabled')=='true') {
24
+                if (api_get_setting('search_enabled') == 'true') {
25 25
                     require_once api_get_path(LIBRARY_PATH).'specific_fields_manager.lib.php';
26 26
                     $specific_fields = get_specific_field_list();
27 27
                     foreach ($specific_fields as $specific_field) {
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
     api_not_allowed(true);
61 61
 }
62 62
 
63
-$interbreadcrumb[]= array ("url"=>"../newscorm/lp_controller.php?action=list", "name"=> get_lang("Doc"));
63
+$interbreadcrumb[] = array("url"=>"../newscorm/lp_controller.php?action=list", "name"=> get_lang("Doc"));
64 64
 
65 65
 $nameTools = get_lang("OogieConversionPowerPoint");
66 66
 Display :: display_header($nameTools);
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 $form->addElement('file', 'user_file', array('<img src="../img/powerpoint_big.gif" />', $div_upload_limit));
79 79
 $form->addElement('checkbox', 'take_slide_name', '', get_lang('TakeSlideName'));
80 80
 if (api_get_setting('search_enabled') == 'true') {
81
-    require_once(api_get_path(LIBRARY_PATH) . 'specific_fields_manager.lib.php');
81
+    require_once(api_get_path(LIBRARY_PATH).'specific_fields_manager.lib.php');
82 82
     $specific_fields = get_specific_field_list();
83 83
     $form->addElement('checkbox', 'index_document', '', get_lang('SearchFeatureDoIndexDocument'));
84 84
     $form->addElement('select_language', 'language', get_lang('SearchFeatureDocumentLanguage'));
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 $form->addButtonUpload(get_lang('ConvertToLP'), 'convert');
91 91
 $form->addElement('hidden', 'ppt2lp', 'true');
92 92
 $form->add_real_progress_bar(md5(rand(0, 10000)), 'user_file', 1, true);
93
-$defaults = array('take_slide_name'=>'checked="checked"','index_document'=>'checked="checked"');
93
+$defaults = array('take_slide_name'=>'checked="checked"', 'index_document'=>'checked="checked"');
94 94
 $form->setDefaults($defaults);
95 95
 
96 96
 // display the form
Please login to merge, or discard this patch.
main/upload/upload_word.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@  discard block
 block discarded – undo
10 10
 
11 11
 include '../inc/global.inc.php';
12 12
 
13
-$form_style= '<style>
13
+$form_style = '<style>
14 14
 .row {
15 15
     width: 200px;
16 16
 }
@@ -28,20 +28,20 @@  discard block
 block discarded – undo
28 28
 </script>';
29 29
 $htmlHeadXtra[] = $form_style;
30 30
 
31
-if (api_get_setting('search_enabled')=='true') {
31
+if (api_get_setting('search_enabled') == 'true') {
32 32
     $specific_fields = get_specific_field_list();
33 33
 }
34 34
 
35 35
 if (isset($_POST['convert'])) {
36 36
     $cwdir = getcwd();
37 37
     if (isset($_FILES['user_file'])) {
38
-        $allowed_extensions = array('doc','docx','odt','txt','sxw','rtf');
39
-        if (in_array(strtolower(pathinfo($_FILES['user_file']['name'],PATHINFO_EXTENSION)),$allowed_extensions)) {
38
+        $allowed_extensions = array('doc', 'docx', 'odt', 'txt', 'sxw', 'rtf');
39
+        if (in_array(strtolower(pathinfo($_FILES['user_file']['name'], PATHINFO_EXTENSION)), $allowed_extensions)) {
40 40
             require('../newscorm/lp_upload.php');
41 41
             if (isset($o_doc) && $first_item_id != 0) {
42 42
                 // Search-related section
43
-                if (api_get_setting('search_enabled')=='true') {
44
-                    require_once(api_get_path(LIBRARY_PATH) . 'specific_fields_manager.lib.php');
43
+                if (api_get_setting('search_enabled') == 'true') {
44
+                    require_once(api_get_path(LIBRARY_PATH).'specific_fields_manager.lib.php');
45 45
                     $specific_fields = get_specific_field_list();
46 46
 
47 47
                     foreach ($specific_fields as $specific_field) {
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
     api_not_allowed(true);
78 78
 }
79 79
 
80
-$interbreadcrumb[]= array ("url"=>"../newscorm/lp_controller.php?action=list", "name"=> get_lang("Doc"));
80
+$interbreadcrumb[] = array("url"=>"../newscorm/lp_controller.php?action=list", "name"=> get_lang("Doc"));
81 81
 $nameTools = get_lang("WoogieConversionPowerPoint");
82 82
 Display :: display_header($nameTools);
83 83
 
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
          border-color: #4171B5;
99 99
          color: #000;";
100 100
 
101
-$s_style_error="border-width: 1px;
101
+$s_style_error = "border-width: 1px;
102 102
          border-style: solid;
103 103
          margin-left: 0;
104 104
          margin-top: 10px;
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
 
123 123
 // build the form
124 124
 
125
-$form -> addElement ('html','<br>');
125
+$form -> addElement('html', '<br>');
126 126
 
127 127
 $div_upload_limit = '&nbsp;&nbsp;'.get_lang('UploadMaxSize').' : '.ini_get('post_max_size');
128 128
 
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
         <!-- BEGIN error --><br /><span class="form_error">{error}</span><!-- END error -->
136 136
 </div>
137 137
 EOT;
138
-$renderer->setElementTemplate($user_file_template,'user_file');
138
+$renderer->setElementTemplate($user_file_template, 'user_file');
139 139
 
140 140
 // set template for other elements
141 141
 $user_file_template =
@@ -147,16 +147,16 @@  discard block
 block discarded – undo
147 147
 EOT;
148 148
 $renderer->setCustomElementTemplate($user_file_template);
149 149
 
150
-$form -> addElement ('file', 'user_file','<img src="../img/word_big.gif" align="absbottom" />');
151
-if (api_get_setting('search_enabled')=='true') {
152
-    $form -> addElement ('checkbox', 'index_document','', get_lang('SearchFeatureDoIndexDocument'));
153
-    $form -> addElement ('html','<br />');
154
-    $form -> addElement ('html', get_lang('SearchFeatureDocumentLanguage').': &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;'. api_get_languages_combo());
155
-    $form -> addElement ('html','<div class="sub-form">');
150
+$form -> addElement('file', 'user_file', '<img src="../img/word_big.gif" align="absbottom" />');
151
+if (api_get_setting('search_enabled') == 'true') {
152
+    $form -> addElement('checkbox', 'index_document', '', get_lang('SearchFeatureDoIndexDocument'));
153
+    $form -> addElement('html', '<br />');
154
+    $form -> addElement('html', get_lang('SearchFeatureDocumentLanguage').': &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;'.api_get_languages_combo());
155
+    $form -> addElement('html', '<div class="sub-form">');
156 156
     foreach ($specific_fields as $specific_field) {
157
-        $form -> addElement ('text', $specific_field['code'], $specific_field['name'].' : ');
157
+        $form -> addElement('text', $specific_field['code'], $specific_field['name'].' : ');
158 158
     }
159
-    $form -> addElement ('html','</div>');
159
+    $form -> addElement('html', '</div>');
160 160
 }
161 161
 
162 162
 /*
@@ -164,11 +164,11 @@  discard block
 block discarded – undo
164 164
  * $form -> addElement ('radio', 'split_steps',null, get_lang('SplitStepsPerPage'),'per_page');
165 165
  * $form -> addElement ('radio', 'split_steps',null, get_lang('SplitStepsPerChapter'),'per_chapter');
166 166
  */
167
-$form -> addElement ('hidden', 'split_steps','per_page');
168
-$form -> addElement ('submit', 'convert', get_lang('ConvertToLP'), 'class="convert_button"');
169
-$form -> addElement ('hidden', 'woogie', 'true');
170
-$form -> add_real_progress_bar(md5(rand(0,10000)), 'user_file', 1, true);
171
-$defaults = array('split_steps'=>'per_page','index_document'=>'checked="checked"');
167
+$form -> addElement('hidden', 'split_steps', 'per_page');
168
+$form -> addElement('submit', 'convert', get_lang('ConvertToLP'), 'class="convert_button"');
169
+$form -> addElement('hidden', 'woogie', 'true');
170
+$form -> add_real_progress_bar(md5(rand(0, 10000)), 'user_file', 1, true);
171
+$defaults = array('split_steps'=>'per_page', 'index_document'=>'checked="checked"');
172 172
 $form -> setDefaults($defaults);
173 173
 
174 174
 // display the form
Please login to merge, or discard this patch.
main/upload/index.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -42,14 +42,14 @@
 block discarded – undo
42 42
 
43 43
 
44 44
 //$is_allowed_to_edit = api_is_allowed_to_edit();
45
-$is_allowed_to_edit = api_is_allowed_to_edit(null,true);
46
-if(!$is_allowed_to_edit){
45
+$is_allowed_to_edit = api_is_allowed_to_edit(null, true);
46
+if (!$is_allowed_to_edit) {
47 47
 	api_not_allowed(true);
48 48
 }
49 49
 
50
-$courseDir = $_course['path'] . "/document";
50
+$courseDir = $_course['path']."/document";
51 51
 $sys_course_path = api_get_path(SYS_COURSE_PATH);
52
-$base_work_dir = $sys_course_path . $courseDir;
52
+$base_work_dir = $sys_course_path.$courseDir;
53 53
 $noPHP_SELF = true;
54 54
 $max_filled_space = DocumentManager::get_course_quota();
55 55
 
Please login to merge, or discard this patch.
main/upload/upload.scorm.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
         $dialogtype = 'error';
34 34
     } else {
35 35
         if (api_get_setting('search_enabled') == 'true') {
36
-            require_once api_get_path(LIBRARY_PATH) . 'specific_fields_manager.lib.php';
36
+            require_once api_get_path(LIBRARY_PATH).'specific_fields_manager.lib.php';
37 37
             $specific_fields = get_specific_field_list();
38 38
 
39 39
             foreach ($specific_fields as $specific_field) {
@@ -58,5 +58,5 @@  discard block
 block discarded – undo
58 58
         $dialogtype = 'confirmation';
59 59
     }
60 60
 }
61
-header('location: ../newscorm/lp_controller.php?action=list&dialog_box=' . $msg . '&dialogtype=' . $dialogtype);
61
+header('location: ../newscorm/lp_controller.php?action=list&dialog_box='.$msg.'&dialogtype='.$dialogtype);
62 62
 exit;
Please login to merge, or discard this patch.
main/upload/upload.document.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -9,9 +9,9 @@  discard block
 block discarded – undo
9 9
  * @author Yannick Warnier <[email protected]>
10 10
  */
11 11
 
12
-$courseDir = $_course['path'] . "/document";
12
+$courseDir = $_course['path']."/document";
13 13
 $sys_course_path = api_get_path(SYS_COURSE_PATH);
14
-$base_work_dir = $sys_course_path . $courseDir;
14
+$base_work_dir = $sys_course_path.$courseDir;
15 15
 $noPHP_SELF = true;
16 16
 $max_filled_space = DocumentManager::get_course_quota();
17 17
 
@@ -34,14 +34,14 @@  discard block
 block discarded – undo
34 34
 
35 35
 $nameTools = get_lang('UplUploadDocument');
36 36
 $interbreadcrumb[] = array(
37
-    "url" => "./document.php?curdirpath=" . urlencode(
37
+    "url" => "./document.php?curdirpath=".urlencode(
38 38
             $path
39
-        ) . $req_gid,
39
+        ).$req_gid,
40 40
     "name" => $langDocuments
41 41
 );
42 42
 Display::display_header($nameTools, "Doc");
43 43
 //show the title
44
-api_display_tool_title($nameTools . $add_group_to_title);
44
+api_display_tool_title($nameTools.$add_group_to_title);
45 45
 
46 46
 /**
47 47
  * Process
@@ -71,11 +71,11 @@  discard block
 block discarded – undo
71 71
         	$ct = '';
72 72
         	if ($new_comment) $ct .= ", comment='$new_comment'";
73 73
         	if ($new_title)   $ct .= ", title='$new_title'";
74
-        	Database::query("UPDATE $table_document SET" . substr($ct, 1) ." WHERE id = '$docid'");
74
+        	Database::query("UPDATE $table_document SET".substr($ct, 1)." WHERE id = '$docid'");
75 75
     	}
76 76
         //check for missing images in html files
77 77
         $missing_files = check_for_missing_files($base_work_dir.$_POST['curdirpath'].$new_path);
78
-        if ($missing_files)  {
78
+        if ($missing_files) {
79 79
             //show a form to upload the missing files
80 80
             Display::display_normal_message(
81 81
                 build_missing_files_form(
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
     if ($number_of_uploaded_images > 0) {
95 95
         //we could also create a function for this, I'm not sure...
96 96
         //create a directory for the missing files
97
-        $img_directory = str_replace('.','_',$_POST['related_file']."_files");
97
+        $img_directory = str_replace('.', '_', $_POST['related_file']."_files");
98 98
         $folderData = create_unexisting_directory(
99 99
             $_course,
100 100
             $_user['user_id'],
@@ -120,15 +120,15 @@  discard block
 block discarded – undo
120 120
         replace_img_path_in_html_file(
121 121
             $_POST['img_file_path'],
122 122
             $paths_to_replace_in_file,
123
-            $base_work_dir . $_POST['related_file']
123
+            $base_work_dir.$_POST['related_file']
124 124
         );
125 125
         //update parent folders
126
-        item_property_update_on_folder($_course,$_POST['curdirpath'],$_user['user_id']);
126
+        item_property_update_on_folder($_course, $_POST['curdirpath'], $_user['user_id']);
127 127
     }
128 128
 }
129 129
 //they want to create a directory
130
-if (isset($_POST['create_dir']) && $_POST['dirname']!='') {
131
-	$added_slash = ($path=='/')?'':'/';
130
+if (isset($_POST['create_dir']) && $_POST['dirname'] != '') {
131
+	$added_slash = ($path == '/') ? '' : '/';
132 132
 	$dir_name = $path.$added_slash.api_replace_dangerous_char($_POST['dirname']);
133 133
     $created_dir = create_unexisting_directory(
134 134
         $_course,
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
 	//create the form that asks for the directory name
153 153
 	$new_folder_text = '<form action="'.api_get_self().'" method="POST">';
154 154
 	$new_folder_text .= '<input type="hidden" name="curdirpath" value="'.$path.'"/>';
155
-	$new_folder_text .= get_lang('NewDir') .' ';
155
+	$new_folder_text .= get_lang('NewDir').' ';
156 156
 	$new_folder_text .= '<input type="text" name="dirname"/>';
157 157
 	$new_folder_text .= '<input type="submit" name="create_dir" value="'.get_lang('Ok').'"/>';
158 158
 	$new_folder_text .= '</form>';
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 	<p>
164 164
         <a href="<?php echo api_get_self(); ?>?path=<?php echo $path; ?>&amp;createdir=1">
165 165
             <img src="../img/new_folder.gif" border="0" align="absmiddle" alt ="" />
166
-            <?php echo(get_lang('CreateDir'));?>
166
+            <?php echo(get_lang('CreateDir')); ?>
167 167
         </a>
168 168
     </p>
169 169
 <?php
@@ -186,11 +186,11 @@  discard block
 block discarded – undo
186 186
 </td>
187 187
 </tr>
188 188
     <tr>
189
-    <td><?php echo get_lang('Title');?></td>
189
+    <td><?php echo get_lang('Title'); ?></td>
190 190
     <td><input type="text" size="20" name="title" style="width:300px;"></td>
191 191
     </tr>
192 192
     <tr>
193
-    <td valign="top"><?php echo get_lang('Comment');?></td>
193
+    <td valign="top"><?php echo get_lang('Comment'); ?></td>
194 194
     <td><textarea rows="3" cols="20" name="comment" wrap="virtual" style="width:300px;"></textarea></td>
195 195
     </tr>
196 196
     <tr>
@@ -198,22 +198,22 @@  discard block
 block discarded – undo
198 198
 <?php echo get_lang('Options'); ?>
199 199
 </td>
200 200
 <td>
201
-- <input type="checkbox" name="unzip" value="1" onclick="check_unzip()"/> <?php echo(get_lang('Uncompress'));?><br/>
202
-- <?php echo (get_lang('UplWhatIfFileExists'));?><br/>
203
-&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="nothing" title="<?php echo (get_lang('UplDoNothingLong'));?>" checked="checked"/>  <?php echo (get_lang('UplDoNothing'));?><br/>
204
-&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="overwrite" title="<?php echo (get_lang('UplOverwriteLong'));?>"/> <?php echo (get_lang('UplOverwrite'));?><br/>
205
-&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="rename" title="<?php echo (get_lang('UplRenameLong'));?>"/> <?php echo (get_lang('UplRename'));?>
201
+- <input type="checkbox" name="unzip" value="1" onclick="check_unzip()"/> <?php echo(get_lang('Uncompress')); ?><br/>
202
+- <?php echo (get_lang('UplWhatIfFileExists')); ?><br/>
203
+&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="nothing" title="<?php echo (get_lang('UplDoNothingLong')); ?>" checked="checked"/>  <?php echo (get_lang('UplDoNothing')); ?><br/>
204
+&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="overwrite" title="<?php echo (get_lang('UplOverwriteLong')); ?>"/> <?php echo (get_lang('UplOverwrite')); ?><br/>
205
+&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="rename" title="<?php echo (get_lang('UplRenameLong')); ?>"/> <?php echo (get_lang('UplRename')); ?>
206 206
 
207 207
 </td>
208 208
 </tr>
209 209
 </table>
210 210
 
211
-<input type="submit" value="<?php echo(get_lang('Ok'));?>">
211
+<input type="submit" value="<?php echo(get_lang('Ok')); ?>">
212 212
 </form>
213 213
 <!-- end upload form -->
214 214
 
215 215
  <!-- so they can get back to the documents   -->
216
- <p><?php echo (get_lang('Back'));?> <?php echo (get_lang('To'));?> <a href="document.php?curdirpath=<?php echo $path; ?>"><?php echo (get_lang('DocumentsOverview'));?></a></p>
216
+ <p><?php echo (get_lang('Back')); ?> <?php echo (get_lang('To')); ?> <a href="document.php?curdirpath=<?php echo $path; ?>"><?php echo (get_lang('DocumentsOverview')); ?></a></p>
217 217
 <?php
218 218
 
219 219
 Display::display_footer();
Please login to merge, or discard this patch.