Completed
Push — 1.11.x ( 518476...344d9e )
by José
55:02 queued 28:13
created
main/work/edit_work.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 
31 31
 $htmlHeadXtra[] = to_javascript_work();
32 32
 $interbreadcrumb[] = array(
33
-    'url' => api_get_path(WEB_CODE_PATH) . 'work/work.php?' . api_get_cidreq(),
33
+    'url' => api_get_path(WEB_CODE_PATH).'work/work.php?'.api_get_cidreq(),
34 34
     'name' => get_lang('StudentPublications')
35 35
 );
36 36
 $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Edit'));
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 $form = new FormValidator(
39 39
     'edit_dir',
40 40
     'post',
41
-    api_get_path(WEB_CODE_PATH) . 'work/edit_work.php?id=' . $workId . '&' . api_get_cidreq()
41
+    api_get_path(WEB_CODE_PATH).'work/edit_work.php?id='.$workId.'&'.api_get_cidreq()
42 42
 );
43 43
 $form->addElement('header', get_lang('Edit'));
44 44
 
Please login to merge, or discard this patch.
main/work/upload_corrections.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -5,7 +5,7 @@  discard block
 block discarded – undo
5 5
 use Symfony\Component\Finder\Finder;
6 6
 
7 7
 require_once '../inc/global.inc.php';
8
-$current_course_tool  = TOOL_STUDENTPUBLICATION;
8
+$current_course_tool = TOOL_STUDENTPUBLICATION;
9 9
 
10 10
 api_protect_course_script(true);
11 11
 
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
         $zip = new PclZip($_FILES['file']['tmp_name']);
71 71
 
72 72
         // Check the zip content (real size and file extension)
73
-        $zipFileList = (array)$zip->listContent();
73
+        $zipFileList = (array) $zip->listContent();
74 74
 
75 75
         $realSize = 0;
76 76
         foreach ($zipFileList as & $this_content) {
Please login to merge, or discard this patch.
main/work/download_comment_file.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
         if (Security::check_abs_path(
43 43
             $workData['file_path'],
44
-            api_get_path(SYS_COURSE_PATH) . api_get_course_path() . '/'
44
+            api_get_path(SYS_COURSE_PATH).api_get_course_path().'/'
45 45
         )
46 46
         ) {
47 47
             DocumentManager::file_send_for_download(
Please login to merge, or discard this patch.
main/work/add_user.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -74,8 +74,8 @@  discard block
 block discarded – undo
74 74
         $usersAdded[] = $myUserId;
75 75
         $userInfo = api_get_user_info($myUserId);
76 76
         $url = api_get_path(WEB_CODE_PATH).'work/add_user.php?action=delete&id='.$workId.'&user_id='.$myUserId;
77
-        $link = Display::url('<em class="fa fa-trash"></em> ' . get_lang('Delete'), $url, array('class' => 'btn btn-danger btn-sm'));
78
-        echo '<li class="list-group-item">' . $userInfo['complete_name_with_username'] . '<div class="pull-right">' . $link . '</div></li>';
77
+        $link = Display::url('<em class="fa fa-trash"></em> '.get_lang('Delete'), $url, array('class' => 'btn btn-danger btn-sm'));
78
+        echo '<li class="list-group-item">'.$userInfo['complete_name_with_username'].'<div class="pull-right">'.$link.'</div></li>';
79 79
     }
80 80
     echo '</ul>';
81 81
 }
@@ -108,8 +108,8 @@  discard block
 block discarded – undo
108 108
     foreach ($userToAddList as $user) {
109 109
         $userName = api_get_person_name($user['firstname'], $user['lastname']).' ('.$user['username'].') ';
110 110
         $url = api_get_path(WEB_CODE_PATH).'work/add_user.php?action=add&id='.$workId.'&user_id='.$user['user_id'];
111
-        $link = Display::url('<em class="fa fa-plus"></em> ' . get_lang('Add'), $url, array('class' => 'btn btn-primary btn-sm'));
112
-        echo '<li class="list-group-item">' . $userName . '<div class="pull-right"> ' . $link . '</div></li>';
111
+        $link = Display::url('<em class="fa fa-plus"></em> '.get_lang('Add'), $url, array('class' => 'btn btn-primary btn-sm'));
112
+        echo '<li class="list-group-item">'.$userName.'<div class="pull-right"> '.$link.'</div></li>';
113 113
     }
114 114
     echo '</ul>';
115 115
 } else {
Please login to merge, or discard this patch.
main/work/upload_from_template.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@
 block discarded – undo
4 4
 use ChamiloSession as Session;
5 5
 
6 6
 require_once '../inc/global.inc.php';
7
-$current_course_tool  = TOOL_STUDENTPUBLICATION;
7
+$current_course_tool = TOOL_STUDENTPUBLICATION;
8 8
 
9 9
 api_protect_course_script(true);
10 10
 
Please login to merge, or discard this patch.
main/coursecopy/copy_course.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@  discard block
 block discarded – undo
7 7
 
8 8
 // Setting the global file that gets the general configuration, the databases, the languages, ...
9 9
 require_once '../inc/global.inc.php';
10
-$current_course_tool  = TOOL_COURSE_MAINTENANCE;
10
+$current_course_tool = TOOL_COURSE_MAINTENANCE;
11 11
 api_protect_course_script(true);
12 12
 
13 13
 // Including additional libraries
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
         $form->addGroup($group, '', get_lang('SameFilename'));
121 121
         $form->add_progress_bar();
122 122
         $form->addButtonSave(get_lang('CopyCourse'));
123
-        $form->setDefaults(array('copy_option' =>'select_items','same_file_name_option' => FILE_OVERWRITE));
123
+        $form->setDefaults(array('copy_option' =>'select_items', 'same_file_name_option' => FILE_OVERWRITE));
124 124
 
125 125
         // Add Security token
126 126
         $token = Security::get_token();
Please login to merge, or discard this patch.
main/coursecopy/import_backup.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -98,21 +98,21 @@  discard block
 block discarded – undo
98 98
         $cr->set_file_option($_POST['same_file_name_option']);
99 99
         $cr->restore();
100 100
         Display::display_normal_message(get_lang('ImportFinished'));
101
-        echo '<a class="btn btn-default" href="' . api_get_path(WEB_COURSE_PATH) . api_get_course_path() . '/index.php">' . get_lang('CourseHomepage') . '</a>';
101
+        echo '<a class="btn btn-default" href="'.api_get_path(WEB_COURSE_PATH).api_get_course_path().'/index.php">'.get_lang('CourseHomepage').'</a>';
102 102
     } else {
103 103
         if (!$error) {
104 104
             Display::display_warning_message(get_lang('NoResourcesInBackupFile'));
105
-            echo '<a class="btn btn-default" href="import_backup.php?' . api_get_cidreq() . '">' . get_lang('TryAgain') . '</a>';
105
+            echo '<a class="btn btn-default" href="import_backup.php?'.api_get_cidreq().'">'.get_lang('TryAgain').'</a>';
106 106
         } elseif ($filename === false) {
107 107
             Display::display_error_message(get_lang('ArchivesDirectoryNotWriteableContactAdmin'));
108
-            echo '<a class="btn btn-default" href="import_backup.php?' . api_get_cidreq() . '">' . get_lang('TryAgain') . '</a>';
108
+            echo '<a class="btn btn-default" href="import_backup.php?'.api_get_cidreq().'">'.get_lang('TryAgain').'</a>';
109 109
         } else {
110 110
             if ($filename == '') {
111 111
                 Display::display_error_message(get_lang('SelectBackupFile'));
112
-                echo '<a class="btn btn-default" href="import_backup.php?' . api_get_cidreq() . '">' . get_lang('TryAgain') . '</a>';
112
+                echo '<a class="btn btn-default" href="import_backup.php?'.api_get_cidreq().'">'.get_lang('TryAgain').'</a>';
113 113
             } else {
114 114
                 Display::display_error_message(get_lang('UploadError'));
115
-                echo '<a class="btn btn-default" href="import_backup.php?' . api_get_cidreq() . '">' . get_lang('TryAgain') . '</a>';
115
+                echo '<a class="btn btn-default" href="import_backup.php?'.api_get_cidreq().'">'.get_lang('TryAgain').'</a>';
116 116
             }
117 117
         }
118 118
     }
@@ -142,10 +142,10 @@  discard block
 block discarded – undo
142 142
         CourseSelectForm::display_form($course, $hiddenFields);
143 143
     } elseif ($filename === false) {
144 144
         Display::display_error_message(get_lang('ArchivesDirectoryNotWriteableContactAdmin'));
145
-        echo '<a class="btn btn-default" href="import_backup.php?' . api_get_cidreq() . '">' . get_lang('TryAgain') . '</a>';
145
+        echo '<a class="btn btn-default" href="import_backup.php?'.api_get_cidreq().'">'.get_lang('TryAgain').'</a>';
146 146
     } else {
147 147
         Display::display_warning_message(get_lang('NoResourcesInBackupFile'));
148
-        echo '<a class="btn btn-default" href="import_backup.php?' . api_get_cidreq() . '">' . get_lang('TryAgain') . '</a>';
148
+        echo '<a class="btn btn-default" href="import_backup.php?'.api_get_cidreq().'">'.get_lang('TryAgain').'</a>';
149 149
     }
150 150
 } else {
151 151
     $user = api_get_user_info();
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
157 157
     $form = new FormValidator(
158 158
         'import_backup_form',
159 159
         'post',
160
-        api_get_path(WEB_CODE_PATH) . 'coursecopy/import_backup.php?' . api_get_cidreq(),
160
+        api_get_path(WEB_CODE_PATH).'coursecopy/import_backup.php?'.api_get_cidreq(),
161 161
         '',
162 162
         array('enctype' => 'multipart/form-data')
163 163
     );
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
         );
190 190
         $options['null'] = '-';
191 191
         foreach ($backups as $index => $backup) {
192
-            $options[$backup['file']] = $backup['course_code'] . ' (' . $backup['date'] . ')';
192
+            $options[$backup['file']] = $backup['course_code'].' ('.$backup['date'].')';
193 193
         }
194 194
         $form->addElement(
195 195
             'select',
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
             'radio',
208 208
             '',
209 209
             '',
210
-            '<i>' . get_lang('NoBackupsAvailable') . '</i>',
210
+            '<i>'.get_lang('NoBackupsAvailable').'</i>',
211 211
             '',
212 212
             'disabled="true"'
213 213
         );
Please login to merge, or discard this patch.
main/search/index.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@
 block discarded – undo
7 7
  */
8 8
 
9 9
 require_once '../inc/global.inc.php';
10
-$this_section =  SECTION_COURSES;
10
+$this_section = SECTION_COURSES;
11 11
 
12 12
 if (extension_loaded('xapian')) {
13 13
 	require '../newscorm/lp_list_search.php';
Please login to merge, or discard this patch.
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@
 block discarded – undo
10 10
 $this_section =  SECTION_COURSES;
11 11
 
12 12
 if (extension_loaded('xapian')) {
13
-	require '../lp/lp_list_search.php';
13
+    require '../lp/lp_list_search.php';
14 14
 } else {
15 15
     Display::display_header(get_lang('Search'));
16 16
     Display::display_error_message(get_lang('SearchXapianModuleNotInstalled'));
Please login to merge, or discard this patch.
main/social/home.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -101,8 +101,8 @@
 block discarded – undo
101 101
         $group_url = "group_view.php?id=$id";
102 102
 
103 103
         $result['name'] = '<div class="group-name">'.Display::url(
104
-                          api_ucwords(cut($result['name'], 40, true)), $group_url)
105
-                          .'</div><div class="count-username">'.
104
+                            api_ucwords(cut($result['name'], 40, true)), $group_url)
105
+                            .'</div><div class="count-username">'.
106 106
                             Display::returnFontAwesomeIcon('user').$result['count'].'</div>';
107 107
 
108 108
         $picture = $userGroup->get_picture_group(
Please login to merge, or discard this patch.
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -20,8 +20,8 @@  discard block
 block discarded – undo
20 20
 api_block_anonymous_users();
21 21
 
22 22
 if (api_get_setting('allow_social_tool') != 'true') {
23
-    $url = api_get_path(WEB_CODE_PATH) . 'auth/profile.php';
24
-    header('Location: ' . $url);
23
+    $url = api_get_path(WEB_CODE_PATH).'auth/profile.php';
24
+    header('Location: '.$url);
25 25
     exit;
26 26
 }
27 27
 
@@ -45,10 +45,10 @@  discard block
 block discarded – undo
45 45
     $allowed_picture_types = api_get_supported_image_extensions();
46 46
     $form->addRule(
47 47
         'picture',
48
-        get_lang('OnlyImagesAllowed') . ' (' . implode(
48
+        get_lang('OnlyImagesAllowed').' ('.implode(
49 49
             ',',
50 50
             $allowed_picture_types
51
-        ) . ')',
51
+        ).')',
52 52
         'filetype',
53 53
         $allowed_picture_types
54 54
     );
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
                 $sql = "UPDATE $table_user
69 69
                         SET 
70 70
                             picture_uri = '$new_picture' 
71
-                        WHERE user_id =  " . api_get_user_id();
71
+                        WHERE user_id =  ".api_get_user_id();
72 72
 
73 73
                 $result = Database::query($sql);
74 74
             }
@@ -95,9 +95,9 @@  discard block
 block discarded – undo
95 95
         $result['name'] = Security::remove_XSS($result['name'], STUDENT, true);
96 96
 
97 97
         if ($result['count'] == 1) {
98
-            $result['count'] = '1 ' . get_lang('Member');
98
+            $result['count'] = '1 '.get_lang('Member');
99 99
         } else {
100
-            $result['count'] = $result['count'] . ' ' . get_lang('Members');
100
+            $result['count'] = $result['count'].' '.get_lang('Members');
101 101
         }
102 102
 
103 103
         $group_url = "group_view.php?id=$id";
@@ -114,9 +114,9 @@  discard block
 block discarded – undo
114 114
             GROUP_IMAGE_SIZE_BIG
115 115
         );
116 116
 
117
-        $result['picture'] = '<img class="img-responsive" src="' . $picture['file'] . '" />';
118
-        $group_actions = '<div class="group-more"><a href="groups.php?#tab_browse-2">' . get_lang('SeeMore') . '</a></div>';
119
-        $group_info= '<div class="description"><p>' . cut($result['description'], 120, true) . "</p></div>";
117
+        $result['picture'] = '<img class="img-responsive" src="'.$picture['file'].'" />';
118
+        $group_actions = '<div class="group-more"><a href="groups.php?#tab_browse-2">'.get_lang('SeeMore').'</a></div>';
119
+        $group_info = '<div class="description"><p>'.cut($result['description'], 120, true)."</p></div>";
120 120
         $groups_newest[] = array(
121 121
             Display::url(
122 122
                 $result['picture'],
@@ -143,9 +143,9 @@  discard block
 block discarded – undo
143 143
     $group_url = "group_view.php?id=$id";
144 144
 
145 145
     if ($result['count'] == 1) {
146
-        $result['count'] = '1 ' . get_lang('Member');
146
+        $result['count'] = '1 '.get_lang('Member');
147 147
     } else {
148
-        $result['count'] = $result['count'] . ' ' . get_lang('Members');
148
+        $result['count'] = $result['count'].' '.get_lang('Members');
149 149
     }
150 150
     $result['name'] = '<div class="group-name">'.Display::url(
151 151
             api_ucwords(cut($result['name'], 40, true)), $group_url)
@@ -157,42 +157,42 @@  discard block
 block discarded – undo
157 157
         null,
158 158
         GROUP_IMAGE_SIZE_BIG
159 159
     );
160
-    $result['picture_uri'] = '<img class="img-responsive" src="' . $picture['file'] . '" />';
161
-    $group_actions = '<div class="group-more" ><a href="groups.php?#tab_browse-3">' . get_lang('SeeMore') . '</a></div>';
162
-    $group_info= '<div class="description"><p>' . cut($result['description'], 120, true) . "</p></div>";
160
+    $result['picture_uri'] = '<img class="img-responsive" src="'.$picture['file'].'" />';
161
+    $group_actions = '<div class="group-more" ><a href="groups.php?#tab_browse-3">'.get_lang('SeeMore').'</a></div>';
162
+    $group_info = '<div class="description"><p>'.cut($result['description'], 120, true)."</p></div>";
163 163
     $groups_pop[] = array(
164 164
         Display::url($result['picture_uri'], $group_url),
165
-        $result['name'],$group_info. $group_actions
165
+        $result['name'], $group_info.$group_actions
166 166
     );
167 167
 }
168 168
 
169
-$list=count($groups_newest);
169
+$list = count($groups_newest);
170 170
 $social_group_block = null;
171 171
 if ($list > 0) {
172 172
     $social_group_block .= '<div class="list-group-newest">';
173
-    $social_group_block .= '<div class="group-title">' . get_lang('Newest') . '</div>';
174
-    for ($i = 0;$i < $list; $i++) {
175
-        $social_group_block.='<div class="row">';
176
-        $social_group_block.='<div class="col-md-3">' . $groups_newest[$i][0] . '</div>';
177
-        $social_group_block.='<div class="col-md-9">' . $groups_newest[$i][1];
178
-        $social_group_block.= $groups_newest[$i][2] . '</div>';
179
-        $social_group_block.="</div>";
173
+    $social_group_block .= '<div class="group-title">'.get_lang('Newest').'</div>';
174
+    for ($i = 0; $i < $list; $i++) {
175
+        $social_group_block .= '<div class="row">';
176
+        $social_group_block .= '<div class="col-md-3">'.$groups_newest[$i][0].'</div>';
177
+        $social_group_block .= '<div class="col-md-9">'.$groups_newest[$i][1];
178
+        $social_group_block .= $groups_newest[$i][2].'</div>';
179
+        $social_group_block .= "</div>";
180 180
     }
181
-    $social_group_block.= "</div>";
181
+    $social_group_block .= "</div>";
182 182
 }
183
-$list=count($groups_pop);
183
+$list = count($groups_pop);
184 184
 if ($list > 0) {
185 185
     $social_group_block .= '<div class="list-group-newest">';
186
-    $social_group_block .= '<div class="group-title">' . get_lang('Popular') . '</div>';
187
-
188
-    for ($i = 0;$i < $list; $i++) {
189
-        $social_group_block.='<div class="row">';
190
-        $social_group_block.='<div class="col-md-3">' . $groups_pop[$i][0] . '</div>';
191
-        $social_group_block.='<div class="col-md-9">' . $groups_pop[$i][1];
192
-        $social_group_block.= $groups_pop[$i][2] . '</div>';
193
-        $social_group_block.="</div>";
186
+    $social_group_block .= '<div class="group-title">'.get_lang('Popular').'</div>';
187
+
188
+    for ($i = 0; $i < $list; $i++) {
189
+        $social_group_block .= '<div class="row">';
190
+        $social_group_block .= '<div class="col-md-3">'.$groups_pop[$i][0].'</div>';
191
+        $social_group_block .= '<div class="col-md-9">'.$groups_pop[$i][1];
192
+        $social_group_block .= $groups_pop[$i][2].'</div>';
193
+        $social_group_block .= "</div>";
194 194
     }
195
-    $social_group_block.= "</div>";
195
+    $social_group_block .= "</div>";
196 196
 }
197 197
 // My friends
198 198
 $friend_html = SocialManager::listMyFriendsBlock(
Please login to merge, or discard this patch.