Passed
Push — 1.10.x ( d9a04b...aeb152 )
by Yannick
123:06 queued 74:04
created
main/webservices/example_fill_users_fields.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
 $tuserfv = Database::get_main_table(TABLE_EXTRA_FIELD_VALUES);
22 22
 $sql = "SELECT user_id, username FROM $tuser ORDER BY user_id";
23 23
 $res = Database::query($sql);
24
-while($row = Database::fetch_array($res)) {
24
+while ($row = Database::fetch_array($res)) {
25 25
   $sql2 = "INSERT INTO $tuserfv (item_id, field_id, value)
26 26
            VALUES (".$row['user_id'].", 11,'".$row['username']."')";
27 27
   $res2 = Database::query($sql2);
Please login to merge, or discard this patch.
main/calendar/ical_export.php 1 patch
Spacing   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -43,15 +43,15 @@  discard block
 block discarded – undo
43 43
 $event = $agenda->get_event($id);
44 44
 
45 45
 if (!empty($event)) {
46
-	define('ICAL_LANG',api_get_language_isocode());
46
+	define('ICAL_LANG', api_get_language_isocode());
47 47
 
48 48
     $ical = new vcalendar();
49
-    $ical->setConfig('unique_id',api_get_path(WEB_PATH));
50
-    $ical->setProperty( 'method', 'PUBLISH' );
51
-    $ical->setConfig('url',api_get_path(WEB_PATH));
49
+    $ical->setConfig('unique_id', api_get_path(WEB_PATH));
50
+    $ical->setProperty('method', 'PUBLISH');
51
+    $ical->setConfig('url', api_get_path(WEB_PATH));
52 52
     $vevent = new vevent();
53 53
 
54
-    switch($_GET['class']) {
54
+    switch ($_GET['class']) {
55 55
         case 'public':
56 56
             $vevent->setClass('PUBLIC');
57 57
             break;
@@ -69,44 +69,44 @@  discard block
 block discarded – undo
69 69
     $event['start_date'] = api_get_local_time($event['start_date']);
70 70
     $event['end_date'] = api_get_local_time($event['end_date']);
71 71
 
72
-    switch($type) {
72
+    switch ($type) {
73 73
         case 'personal':
74 74
         case 'platform':
75
-            $vevent->setProperty( 'summary', api_convert_encoding($event['title'],'UTF-8', $charset));
76
-            if(empty($event['start_date'])){header('location:'.Security::remove_XSS($_SERVER['HTTP_REFERER']));}
77
-            list($y,$m,$d,$h,$M,$s) = preg_split('/[\s:-]/',$event['start_date']);
78
-            $vevent->setProperty('dtstart',array('year'=>$y,'month'=>$m,'day'=>$d,'hour'=>$h,'min'=>$M,'sec'=>$s));
79
-            if(empty($event['end_date'])) {
80
-                $y2=$y;$m2=$m;$d2=$d;$h2=$h;$M2=$M+15;$s2=$s;
81
-                if($M2>60){$M2=$M2-60;$h2+=1;}
75
+            $vevent->setProperty('summary', api_convert_encoding($event['title'], 'UTF-8', $charset));
76
+            if (empty($event['start_date'])) {header('location:'.Security::remove_XSS($_SERVER['HTTP_REFERER'])); }
77
+            list($y, $m, $d, $h, $M, $s) = preg_split('/[\s:-]/', $event['start_date']);
78
+            $vevent->setProperty('dtstart', array('year'=>$y, 'month'=>$m, 'day'=>$d, 'hour'=>$h, 'min'=>$M, 'sec'=>$s));
79
+            if (empty($event['end_date'])) {
80
+                $y2 = $y; $m2 = $m; $d2 = $d; $h2 = $h; $M2 = $M + 15; $s2 = $s;
81
+                if ($M2 > 60) {$M2 = $M2 - 60; $h2 += 1; }
82 82
             } else {
83
-                list($y2,$m2,$d2,$h2,$M2,$s2) = preg_split('/[\s:-]/',$event['end_date']);
83
+                list($y2, $m2, $d2, $h2, $M2, $s2) = preg_split('/[\s:-]/', $event['end_date']);
84 84
             }
85
-            $vevent->setProperty('dtend',array('year'=>$y2,'month'=>$m2,'day'=>$d2,'hour'=>$h2,'min'=>$M2,'sec'=>$s2));
85
+            $vevent->setProperty('dtend', array('year'=>$y2, 'month'=>$m2, 'day'=>$d2, 'hour'=>$h2, 'min'=>$M2, 'sec'=>$s2));
86 86
             //$vevent->setProperty( 'LOCATION', get_lang('Unknown') ); // property name - case independent
87
-            $vevent->setProperty( 'description', api_convert_encoding($event['description'],'UTF-8',$charset));
87
+            $vevent->setProperty('description', api_convert_encoding($event['description'], 'UTF-8', $charset));
88 88
             //$vevent->setProperty( 'comment', 'This is a comment' );
89 89
             //$user = api_get_user_info($event['user']);
90 90
             //$vevent->setProperty('organizer',$user['mail']);
91 91
             //$vevent->setProperty('attendee',$user['mail']);
92 92
             //$vevent->setProperty( 'rrule', array( 'FREQ' => 'WEEKLY', 'count' => 4));// occurs also four next weeks
93
-            $ical->setConfig('filename',$y.$m.$d.$h.$M.$s.'-'.rand(1,1000).'.ics');
94
-            $ical->setComponent ($vevent); // add event to calendar
93
+            $ical->setConfig('filename', $y.$m.$d.$h.$M.$s.'-'.rand(1, 1000).'.ics');
94
+            $ical->setComponent($vevent); // add event to calendar
95 95
             $ical->returnCalendar();
96 96
             break;
97 97
         case 'course':
98
-            $vevent->setProperty( 'summary', api_convert_encoding($event['title'],'UTF-8',$charset));
99
-            if(empty($event['start_date'])){header('location:'.Security::remove_XSS($_SERVER['HTTP_REFERER']));}
100
-            list($y,$m,$d,$h,$M,$s) = preg_split('/[\s:-]/',$event['start_date']);
101
-            $vevent->setProperty('dtstart',array('year'=>$y,'month'=>$m,'day'=>$d,'hour'=>$h,'min'=>$M,'sec'=>$s));
98
+            $vevent->setProperty('summary', api_convert_encoding($event['title'], 'UTF-8', $charset));
99
+            if (empty($event['start_date'])) {header('location:'.Security::remove_XSS($_SERVER['HTTP_REFERER'])); }
100
+            list($y, $m, $d, $h, $M, $s) = preg_split('/[\s:-]/', $event['start_date']);
101
+            $vevent->setProperty('dtstart', array('year'=>$y, 'month'=>$m, 'day'=>$d, 'hour'=>$h, 'min'=>$M, 'sec'=>$s));
102 102
             if (empty($event['end_date'])) {
103
-                $y2=$y;$m2=$m;$d2=$d;$h2=$h;$M2=$M+15;$s2=$s;
104
-                if($M2>60){$M2=$M2-60;$h2+=1;}
103
+                $y2 = $y; $m2 = $m; $d2 = $d; $h2 = $h; $M2 = $M + 15; $s2 = $s;
104
+                if ($M2 > 60) {$M2 = $M2 - 60; $h2 += 1; }
105 105
             } else {
106
-                list($y2,$m2,$d2,$h2,$M2,$s2) = preg_split('/[\s:-]/',$event['end_date']);
106
+                list($y2, $m2, $d2, $h2, $M2, $s2) = preg_split('/[\s:-]/', $event['end_date']);
107 107
             }
108
-            $vevent->setProperty('dtend',array('year'=>$y2,'month'=>$m2,'day'=>$d2,'hour'=>$h2,'min'=>$M2,'sec'=>$s2));
109
-            $vevent->setProperty( 'description', api_convert_encoding($event['description'],'UTF-8',$charset));
108
+            $vevent->setProperty('dtend', array('year'=>$y2, 'month'=>$m2, 'day'=>$d2, 'hour'=>$h2, 'min'=>$M2, 'sec'=>$s2));
109
+            $vevent->setProperty('description', api_convert_encoding($event['description'], 'UTF-8', $charset));
110 110
             //$vevent->setProperty( 'comment', 'This is a comment' );
111 111
             //$user = api_get_user_info($event['user']);
112 112
             //$vevent->setProperty('organizer',$user['mail']);
@@ -120,8 +120,8 @@  discard block
 block discarded – undo
120 120
                 $vevent->setProperty('rrule',array('FREQ'=>$freq,'UNTIL'=>array('year'=>$e_y,'month'=>$e_m,'day'=>$e_d),'INTERVAL'=>'1'));
121 121
             }*/
122 122
             //$vevent->setProperty( 'rrule', array( 'FREQ' => 'WEEKLY', 'count' => 4));// occurs also four next weeks
123
-            $ical->setConfig('filename',$y.$m.$d.$h.$M.$s.'-'.rand(1,1000).'.ics');
124
-            $ical->setComponent ($vevent); // add event to calendar
123
+            $ical->setConfig('filename', $y.$m.$d.$h.$M.$s.'-'.rand(1, 1000).'.ics');
124
+            $ical->setComponent($vevent); // add event to calendar
125 125
             $ical->returnCalendar();
126 126
             break;
127 127
         default:
Please login to merge, or discard this patch.
main/calendar/agenda_js.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@
 block discarded – undo
36 36
 if (isset($_REQUEST['cidReq']) && !empty($_REQUEST['cidReq'])) {
37 37
     if ($_REQUEST['cidReq'] == -1) {
38 38
         // When is out of the course tool (e.g My agenda)
39
-        header('Location: ' . api_get_self());
39
+        header('Location: '.api_get_self());
40 40
         exit;
41 41
     } else {
42 42
         $type = 'course';
Please login to merge, or discard this patch.
main/calendar/agenda_list.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@  discard block
 block discarded – undo
7 7
 require_once '../inc/global.inc.php';
8 8
 
9 9
 $interbreadcrumb[] = array(
10
-    'url' => api_get_path(WEB_CODE_PATH) . "calendar/agenda_js.php",
10
+    'url' => api_get_path(WEB_CODE_PATH)."calendar/agenda_js.php",
11 11
     'name' => get_lang('Agenda')
12 12
 );
13 13
 
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 
30 30
 if (!empty($currentCourseId) && $currentCourseId != -1) {
31 31
     // Agenda is inside a course tool
32
-    $url = api_get_self() . '?' . api_get_cidreq();
32
+    $url = api_get_self().'?'.api_get_cidreq();
33 33
     $this_section = SECTION_COURSES;
34 34
 } else {
35 35
     // Agenda is out of the course tool (e.g personal agenda)
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
     if (!empty($events)) {
46 46
         foreach ($events as &$event) {
47 47
             $courseId = isset($event['course_id']) ? $event['course_id'] : '';
48
-            $event['url'] = api_get_self() . '?cid=' . $courseId . '&type=' . $event['type'];
48
+            $event['url'] = api_get_self().'?cid='.$courseId.'&type='.$event['type'];
49 49
         }
50 50
     }
51 51
 }
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
                 // personal and admin do not have visibility property
77 77
             }
78 78
         }
79
-        header('Location: '. api_get_self());
79
+        header('Location: '.api_get_self());
80 80
         exit;
81 81
     }
82 82
 }
Please login to merge, or discard this patch.
main/document/record_audio_wami.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -33,9 +33,9 @@  discard block
 block discarded – undo
33 33
 $dir = $document_data['path'];
34 34
 
35 35
 //make some vars
36
-$wamidir=$dir;
37
-if($wamidir=="/"){
38
- $wamidir="";
36
+$wamidir = $dir;
37
+if ($wamidir == "/") {
38
+ $wamidir = "";
39 39
 }
40 40
 $wamiurlplay = api_get_path(WEB_COURSE_PATH).api_get_course_path().'/document'.$wamidir."/";
41 41
 $groupId = api_get_group_id();
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 
70 70
 //groups //TODO: clean
71 71
 if (!empty($groupId)) {
72
-	$interbreadcrumb[] = array ("url" => "../group/group_space.php?".api_get_cidreq(), "name" => get_lang('GroupSpace'));
72
+	$interbreadcrumb[] = array("url" => "../group/group_space.php?".api_get_cidreq(), "name" => get_lang('GroupSpace'));
73 73
 	$noPHP_SELF = true;
74 74
 	$group = GroupManager :: get_group_properties($groupId);
75 75
 	$path = explode('/', $dir);
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
 }
86 86
 
87 87
 if (!($is_allowed_to_edit || $_SESSION['group_member_with_upload_rights'] ||
88
-	DocumentManager::is_my_shared_folder(api_get_user_id(), Security::remove_XSS($dir),api_get_session_id()))) {
88
+	DocumentManager::is_my_shared_folder(api_get_user_id(), Security::remove_XSS($dir), api_get_session_id()))) {
89 89
 	api_not_allowed(true);
90 90
 }
91 91
 
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
 // Interbreadcrumb for the current directory root path
104 104
 $counter = 0;
105 105
 if (isset($document_data['parents'])) {
106
-    foreach($document_data['parents'] as $document_sub_data) {
106
+    foreach ($document_data['parents'] as $document_sub_data) {
107 107
         //fixing double group folder in breadcrumb
108 108
         if (api_get_group_id()) {
109 109
             if ($counter == 0) {
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 
122 122
 Display :: display_header($nameTools, 'Doc');
123 123
 echo '<div class="actions">';
124
-		echo '<a href="document.php?id='.$document_id.'">'.Display::return_icon('back.png',get_lang('BackTo').' '.get_lang('DocumentsOverview'),'',ICON_SIZE_MEDIUM).'</a>';
124
+		echo '<a href="document.php?id='.$document_id.'">'.Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('DocumentsOverview'), '', ICON_SIZE_MEDIUM).'</a>';
125 125
 echo '</div>';
126 126
 ?>
127 127
 <!-- swfobject is a commonly used library to embed Flash content https://ajax.googleapis.com/ajax/libs/swfobject/2.2/ -->
Please login to merge, or discard this patch.
main/document/download_scorm.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
 
42 42
 $doc_url = str_replace(array('../', '\\..', '\\0', '..\\'), array('', '', '', ''), $doc_url); //echo $doc_url;
43 43
 
44
-if (strpos($doc_url,'../') OR strpos($doc_url,'/..')) {
44
+if (strpos($doc_url, '../') OR strpos($doc_url, '/..')) {
45 45
     $doc_url = '';
46 46
 }
47 47
 
Please login to merge, or discard this patch.
main/document/edit_odf.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 $web_odf_supported_files = DocumentManager::get_web_odf_extension_list();
35 35
 
36 36
 if (in_array(strtolower($pathinfo['extension']), $web_odf_supported_files)) {
37
-    $show_web_odf  = true;
37
+    $show_web_odf = true;
38 38
 }
39 39
 
40 40
 $file_url_web = api_get_path(WEB_COURSE_PATH).$_course['path'].'/document'.$header_file;
@@ -85,8 +85,8 @@  discard block
 block discarded – undo
85 85
 echo Display::display_header('');
86 86
 
87 87
 echo '<div class="actions">';
88
-echo '<a href="document.php?id='.$parent_id.'">'.Display::return_icon('back.png',get_lang('BackTo').' '.get_lang('DocumentsOverview'),'',ICON_SIZE_MEDIUM).'</a>';
89
-echo '<a href="edit_document.php?'.api_get_cidreq().'&id='.$document_id.$req_gid.'&origin=editodf">'.Display::return_icon('edit.png',get_lang('Rename').'/'.get_lang('Comments'),'',ICON_SIZE_MEDIUM).'</a>';
88
+echo '<a href="document.php?id='.$parent_id.'">'.Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('DocumentsOverview'), '', ICON_SIZE_MEDIUM).'</a>';
89
+echo '<a href="edit_document.php?'.api_get_cidreq().'&id='.$document_id.$req_gid.'&origin=editodf">'.Display::return_icon('edit.png', get_lang('Rename').'/'.get_lang('Comments'), '', ICON_SIZE_MEDIUM).'</a>';
90 90
 echo '</div>';
91 91
 
92 92
 // echo '<div id="odf"></div>';
Please login to merge, or discard this patch.
main/document/webcam_clip.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -32,9 +32,9 @@  discard block
 block discarded – undo
32 32
 $dir = $document_data['path'];
33 33
 
34 34
 //make some vars
35
-$webcamdir=$dir;
36
-if($webcamdir=="/"){
37
- $webcamdir="";
35
+$webcamdir = $dir;
36
+if ($webcamdir == "/") {
37
+ $webcamdir = "";
38 38
 }
39 39
 
40 40
 
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
 $groupId = api_get_group_id();
69 69
 
70 70
 if (!empty($groupId)) {
71
-	$interbreadcrumb[] = array ("url" => "../group/group_space.php?".api_get_cidreq(), "name" => get_lang('GroupSpace'));
71
+	$interbreadcrumb[] = array("url" => "../group/group_space.php?".api_get_cidreq(), "name" => get_lang('GroupSpace'));
72 72
 	$noPHP_SELF = true;
73 73
 	$group = GroupManager :: get_group_properties($groupId);
74 74
 	$path = explode('/', $dir);
@@ -77,14 +77,14 @@  discard block
 block discarded – undo
77 77
 	}
78 78
 }
79 79
 
80
-$interbreadcrumb[] = array ("url" => "./document.php?id=".$document_id."&".api_get_cidreq(), "name" => get_lang('Documents'));
80
+$interbreadcrumb[] = array("url" => "./document.php?id=".$document_id."&".api_get_cidreq(), "name" => get_lang('Documents'));
81 81
 
82 82
 if (!$is_allowed_in_course) {
83 83
 	api_not_allowed(true);
84 84
 }
85 85
 
86 86
 if (!($is_allowed_to_edit || $_SESSION['group_member_with_upload_rights'] ||
87
-	DocumentManager::is_my_shared_folder(api_get_user_id(), Security::remove_XSS($dir),api_get_session_id()))) {
87
+	DocumentManager::is_my_shared_folder(api_get_user_id(), Security::remove_XSS($dir), api_get_session_id()))) {
88 88
 	api_not_allowed(true);
89 89
 }
90 90
 
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 // Interbreadcrumb for the current directory root path
103 103
 $counter = 0;
104 104
 if (isset($document_data['parents'])) {
105
-    foreach($document_data['parents'] as $document_sub_data) {
105
+    foreach ($document_data['parents'] as $document_sub_data) {
106 106
         //fixing double group folder in breadcrumb
107 107
         if (api_get_group_id()) {
108 108
             if ($counter == 0) {
@@ -116,12 +116,12 @@  discard block
 block discarded – undo
116 116
 }
117 117
 
118 118
 //make some vars
119
-$webcamuserid=api_get_user_id();
119
+$webcamuserid = api_get_user_id();
120 120
 
121 121
 Display :: display_header($nameTools, 'Doc');
122 122
 echo '<div class="actions">';
123 123
 echo '<a href="document.php?id='.$document_id.'">'.
124
-	Display::return_icon('back.png',get_lang('BackTo').' '.get_lang('DocumentsOverview'),'',ICON_SIZE_MEDIUM).'</a>';
124
+	Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('DocumentsOverview'), '', ICON_SIZE_MEDIUM).'</a>';
125 125
 echo '</div>';
126 126
 ?>
127 127
 
Please login to merge, or discard this patch.
main/document/edit_document.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
 }
126 126
 
127 127
 $editorConfig = array(
128
-    'ToolbarSet' => (api_is_allowed_to_edit(null, true) ? 'Documents' :'DocumentsStudent'),
128
+    'ToolbarSet' => (api_is_allowed_to_edit(null, true) ? 'Documents' : 'DocumentsStudent'),
129 129
     'Width' => '100%',
130 130
     'Height' => '400',
131 131
     'cols-size' => [2, 10, 0],
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
     $editorConfig['BaseHref']             = api_get_path(WEB_COURSE_PATH).$_course['path'].'/document'.$dir;
145 145
 }
146 146
 
147
-$is_allowed_to_edit = api_is_allowed_to_edit(null, true) || $_SESSION['group_member_with_upload_rights']||
147
+$is_allowed_to_edit = api_is_allowed_to_edit(null, true) || $_SESSION['group_member_with_upload_rights'] ||
148 148
 	DocumentManager::is_my_shared_folder(api_get_user_id(), $dir, $sessionId);
149 149
 $noPHP_SELF = true;
150 150
 
@@ -168,14 +168,14 @@  discard block
 block discarded – undo
168 168
         "name" => get_lang('Documents'),
169 169
     );
170 170
 } else {
171
-    $interbreadcrumb[]= array('url' => '../gradebook/'.$_SESSION['gradebook_dest'], 'name' => get_lang('Gradebook'));
171
+    $interbreadcrumb[] = array('url' => '../gradebook/'.$_SESSION['gradebook_dest'], 'name' => get_lang('Gradebook'));
172 172
 }
173 173
 
174 174
 // Interbreadcrumb for the current directory root path
175 175
 if (empty($document_data['parents'])) {
176 176
     $interbreadcrumb[] = array('url' => '#', 'name' => $document_data['title']);
177 177
 } else {
178
-    foreach($document_data['parents'] as $document_sub_data) {
178
+    foreach ($document_data['parents'] as $document_sub_data) {
179 179
         if ($document_data['title'] == $document_sub_data['title']) {
180 180
             continue;
181 181
         }
@@ -359,7 +359,7 @@  discard block
 block discarded – undo
359 359
 								$dir,
360 360
 								api_get_user_id()
361 361
 							);
362
-							header('Location: document.php?id=' . $document_data['parent_id'] . '&' . api_get_cidreq() . ($is_certificate_mode?'&curdirpath=/certificates&selectcat=1':''));
362
+							header('Location: document.php?id='.$document_data['parent_id'].'&'.api_get_cidreq().($is_certificate_mode ? '&curdirpath=/certificates&selectcat=1' : ''));
363 363
 							exit;
364 364
 						} else {
365 365
                             Display::addFlash(Display::return_message(get_lang('Impossible'), 'warning'));
@@ -402,7 +402,7 @@  discard block
 block discarded – undo
402 402
 /*	Display user interface */
403 403
 
404 404
 // Display the header
405
-$nameTools = get_lang('EditDocument') . ': '.Security::remove_XSS($document_data['title']);
405
+$nameTools = get_lang('EditDocument').': '.Security::remove_XSS($document_data['title']);
406 406
 Display::display_header($nameTools, 'Doc');
407 407
 
408 408
 $document_info = api_get_item_property_info(
@@ -484,7 +484,7 @@  discard block
 block discarded – undo
484 484
 	$form->addElement('textarea', 'comment', get_lang('Comment'), ['cols-size' => [2, 10, 0]]);
485 485
 
486 486
 	if ($owner_id == api_get_user_id() || api_is_platform_admin()) {
487
-		$checked =& $form->addElement('checkbox', 'readonly', null, get_lang('ReadOnly'));
487
+		$checked = & $form->addElement('checkbox', 'readonly', null, get_lang('ReadOnly'));
488 488
 		if ($readonly == 1) {
489 489
 			$checked->setChecked(true);
490 490
 		}
@@ -524,14 +524,14 @@  discard block
 block discarded – undo
524 524
 		foreach ($all_information_by_create_certificate[0] as $info_value) {
525 525
 			$str_info .= $info_value.'<br/>';
526 526
 		}
527
-		$create_certificate=get_lang('CreateCertificateWithTags');
527
+		$create_certificate = get_lang('CreateCertificateWithTags');
528 528
 		Display::display_normal_message(
529 529
 			$create_certificate.': <br /><br />'.$str_info,
530 530
 			false
531 531
 		);
532 532
 	}
533 533
 
534
-	if ($extension=='svg' && !api_browser_support('svg') && api_get_setting('enabled_support_svg') == 'true'){
534
+	if ($extension == 'svg' && !api_browser_support('svg') && api_get_setting('enabled_support_svg') == 'true') {
535 535
 		Display::display_warning_message(get_lang('BrowserDontSupportsSVG'));
536 536
 	}
537 537
     echo '<div class="row" style="overflow:hidden">
@@ -584,7 +584,7 @@  discard block
 block discarded – undo
584 584
 }
585 585
 
586 586
 //return button back to
587
-function show_return($document_id, $path, $call_from_tool='', $slide_id=0, $is_certificate_mode=false)
587
+function show_return($document_id, $path, $call_from_tool = '', $slide_id = 0, $is_certificate_mode = false)
588 588
 {
589 589
     global $parent_id;
590 590
 	echo '<div class="actions">';
@@ -592,26 +592,26 @@  discard block
 block discarded – undo
592 592
     $url = api_get_path(WEB_CODE_PATH).'document/document.php?'.api_get_cidreq().'&id='.$parent_id;
593 593
 
594 594
 	if ($is_certificate_mode) {
595
-		echo '<a href="document.php?curdirpath='.Security::remove_XSS($_GET['curdirpath']).'&selectcat=' . Security::remove_XSS($_GET['selectcat']).'">'.
596
-            Display::return_icon('back.png',get_lang('Back').' '.get_lang('To').' '.get_lang('CertificateOverview'),'',ICON_SIZE_MEDIUM).'</a>';
597
-	} elseif($call_from_tool=='slideshow') {
595
+		echo '<a href="document.php?curdirpath='.Security::remove_XSS($_GET['curdirpath']).'&selectcat='.Security::remove_XSS($_GET['selectcat']).'">'.
596
+            Display::return_icon('back.png', get_lang('Back').' '.get_lang('To').' '.get_lang('CertificateOverview'), '', ICON_SIZE_MEDIUM).'</a>';
597
+	} elseif ($call_from_tool == 'slideshow') {
598 598
 		echo '<a href="'.api_get_path(WEB_PATH).'main/document/slideshow.php?slide_id='.$slide_id.'&curdirpath='.Security::remove_XSS(urlencode($_GET['curdirpath'])).'">'.
599
-            Display::return_icon('slideshow.png', get_lang('BackTo').' '.get_lang('ViewSlideshow'),'',ICON_SIZE_MEDIUM).'</a>';
600
-	} elseif($call_from_tool=='editdraw') {
599
+            Display::return_icon('slideshow.png', get_lang('BackTo').' '.get_lang('ViewSlideshow'), '', ICON_SIZE_MEDIUM).'</a>';
600
+	} elseif ($call_from_tool == 'editdraw') {
601 601
 		echo '<a href="'.$url.'">'.
602
-            Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('DocumentsOverview'),'',ICON_SIZE_MEDIUM).'</a>';
602
+            Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('DocumentsOverview'), '', ICON_SIZE_MEDIUM).'</a>';
603 603
 		echo '<a href="javascript:history.back(1)">'.Display::return_icon('draw.png', get_lang('BackTo').' '.get_lang('Draw'), array(), 32).'</a>';
604
-	} elseif($call_from_tool=='editodf') {
604
+	} elseif ($call_from_tool == 'editodf') {
605 605
         echo '<a href="'.$url.'">'.
606
-            Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('DocumentsOverview'),'',ICON_SIZE_MEDIUM).'</a>';
606
+            Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('DocumentsOverview'), '', ICON_SIZE_MEDIUM).'</a>';
607 607
         echo '<a href="javascript:history.back(1)">'.Display::return_icon('draw.png', get_lang('BackTo').' '.get_lang('Write'), array(), 32).'</a>';
608
-    } elseif($call_from_tool=='editpaint'){
608
+    } elseif ($call_from_tool == 'editpaint') {
609 609
 		echo '<a href="'.$url.'">'.
610 610
             Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('DocumentsOverview'), array(), ICON_SIZE_MEDIUM).'</a>';
611 611
 		echo '<a href="javascript:history.back(1)">'.Display::return_icon('paint.png', get_lang('BackTo').' '.get_lang('Paint'), array(), 32).'</a>';
612 612
 	} else {
613 613
 		echo '<a href="'.$url.'">'.
614
-            Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('DocumentsOverview'),'',ICON_SIZE_MEDIUM).'</a>';
614
+            Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('DocumentsOverview'), '', ICON_SIZE_MEDIUM).'</a>';
615 615
 	}
616 616
 	echo '</div>';
617 617
 }
Please login to merge, or discard this patch.