Passed
Push — 1.10.x ( d9a04b...aeb152 )
by Yannick
123:06 queued 74:04
created
main/work/student_work.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 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
 
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 // Only a teachers page.
30 30
 
31 31
 if (!empty($group_id)) {
32
-    $group_properties  = GroupManager :: get_group_properties($group_id);
32
+    $group_properties = GroupManager :: get_group_properties($group_id);
33 33
     $show_work = false;
34 34
 
35 35
     if (api_is_allowed_to_edit(false, true)) {
Please login to merge, or discard this patch.
main/work/downloadfolder.inc.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@  discard block
 block discarded – undo
9 9
 
10 10
 $work_id = $_GET['id'];
11 11
 require_once '../inc/global.inc.php';
12
-$current_course_tool  = TOOL_STUDENTPUBLICATION;
12
+$current_course_tool = TOOL_STUDENTPUBLICATION;
13 13
 $_course = api_get_course_info();
14 14
 
15 15
 // Protection
@@ -178,10 +178,10 @@  discard block
 block discarded – undo
178 178
 if (!empty($files)) {
179 179
     $fileName = api_replace_dangerous_char($work_data['title']);
180 180
     // Logging
181
-    Event::event_download($fileName .'.zip (folder)');
181
+    Event::event_download($fileName.'.zip (folder)');
182 182
 
183 183
     //start download of created file
184
-    $name = $fileName .'.zip';
184
+    $name = $fileName.'.zip';
185 185
     if (Security::check_abs_path($temp_zip_file, api_get_path(SYS_ARCHIVE_PATH))) {
186 186
         DocumentManager::file_send_for_download($temp_zip_file, true, $name);
187 187
         @unlink($temp_zip_file);
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/work/work_list_all.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 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
 
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 $user_id = api_get_user_id();
45 45
 
46 46
 if (!empty($group_id)) {
47
-    $group_properties  = GroupManager :: get_group_properties($group_id);
47
+    $group_properties = GroupManager :: get_group_properties($group_id);
48 48
     $show_work = false;
49 49
 
50 50
     if (api_is_allowed_to_edit(false, true)) {
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
     $actionsLeft .= '<a href="'.api_get_path(WEB_CODE_PATH).'work/add_document.php?'.api_get_cidreq().'&id='.$workId.'">';
162 162
     $actionsLeft .= Display::return_icon('new_document.png', get_lang('AddDocument'), '', ICON_SIZE_MEDIUM).'</a>';
163 163
 
164
-    $actionsLeft .=  '<a href="'.api_get_path(WEB_CODE_PATH).'work/add_user.php?'.api_get_cidreq().'&id='.$workId.'">';
164
+    $actionsLeft .= '<a href="'.api_get_path(WEB_CODE_PATH).'work/add_user.php?'.api_get_cidreq().'&id='.$workId.'">';
165 165
     $actionsLeft .= Display::return_icon('addworkuser.png', get_lang('AddUsers'), '', ICON_SIZE_MEDIUM).'</a>';
166 166
 
167 167
     $actionsLeft .= '<a href="'.api_get_path(WEB_CODE_PATH).'work/work_list_all.php?'.api_get_cidreq().'&id='.$workId.'&action=export_pdf">';
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 }
185 185
 
186 186
 
187
-echo Display::toolbarAction('toolbar-worklist', array( 0 => $actionsLeft), 1);
187
+echo Display::toolbarAction('toolbar-worklist', array(0 => $actionsLeft), 1);
188 188
 
189 189
 if (!empty($my_folder_data['title'])) {
190 190
     echo Display::page_subheader($my_folder_data['title']);
Please login to merge, or discard this patch.
main/coursecopy/create_backup.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -72,8 +72,8 @@  discard block
 block discarded – undo
72 72
 
73 73
     $zip_file = CourseArchiver::write_course($course);
74 74
     Display::display_confirmation_message(get_lang('BackupCreated'));
75
-    echo '<br /><a class="btn btn-primary btn-large" href="' . api_get_path(WEB_CODE_PATH) . 'course_info/download.php?archive=' . $zip_file . '&' . api_get_cidreq() . '">
76
-    ' . get_lang('Download') . '</a>';
75
+    echo '<br /><a class="btn btn-primary btn-large" href="'.api_get_path(WEB_CODE_PATH).'course_info/download.php?archive='.$zip_file.'&'.api_get_cidreq().'">
76
+    ' . get_lang('Download').'</a>';
77 77
 
78 78
 } elseif (Security::check_token('post') && (
79 79
         isset($_POST['backup_option']) &&
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
     if (!$course->has_resources()) {
96 96
         echo get_lang('NoResourcesToBackup');
97 97
     } else {
98
-        $form = new FormValidator('create_backup_form', 'post', api_get_self() . '?' . api_get_cidreq());
98
+        $form = new FormValidator('create_backup_form', 'post', api_get_self().'?'.api_get_cidreq());
99 99
         $form->addElement('header', get_lang('SelectOptionForBackup'));
100 100
         $form->addElement('radio', 'backup_option', '', get_lang('CreateFullBackup'), 'full_backup');
101 101
         $form->addElement('radio', 'backup_option', '', get_lang('LetMeSelectItems'), 'select_items');
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/classes/wiki.class.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 		$progress,
46 46
 		$version
47 47
 	) {
48
-		parent::__construct($id,RESOURCE_WIKI);
48
+		parent::__construct($id, RESOURCE_WIKI);
49 49
         $this->id = $id;
50 50
         $this->page_id = $page_id;
51 51
         $this->reflink = $reflink;
@@ -61,6 +61,6 @@  discard block
 block discarded – undo
61 61
 	public function show()
62 62
 	{
63 63
 		parent::show();
64
-		echo $this->reflink.' ('. (empty($this->group_id) ? get_lang('Everyone') : get_lang('Group') . ' ' .  $this->group_id) .') ' . '<i>(' . $this->dtime . ')</i>';
64
+		echo $this->reflink.' ('.(empty($this->group_id) ? get_lang('Everyone') : get_lang('Group').' '.$this->group_id).') '.'<i>('.$this->dtime.')</i>';
65 65
 	}
66 66
 }
Please login to merge, or discard this patch.
main/coursecopy/classes/Announcement.class.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -49,13 +49,13 @@
 block discarded – undo
49 49
 	 */
50 50
     function __construct($id, $title, $content, $date, $display_order, $email_sent, $path, $filename, $size, $comment)
51 51
     {
52
-		parent::__construct($id,RESOURCE_ANNOUNCEMENT);
52
+		parent::__construct($id, RESOURCE_ANNOUNCEMENT);
53 53
 
54
-		$this->content	= $content;
55
-		$this->title 	= $title;
56
-		$this->date 	= $date;
57
-		$this->display_order	= $display_order;
58
-		$this->email_sent	 	= $email_sent;
54
+		$this->content = $content;
55
+		$this->title = $title;
56
+		$this->date = $date;
57
+		$this->display_order = $display_order;
58
+		$this->email_sent = $email_sent;
59 59
 
60 60
 		$this->attachment_path 	= $path;
61 61
 		$this->attachment_filename = $filename;
Please login to merge, or discard this patch.