Passed
Push — 1.10.x ( aae6b1...e5a590 )
by Yannick
131:05 queued 81:36
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/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/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/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 2 patches
Indentation   +42 added lines, -42 removed lines patch added patch discarded remove patch
@@ -9,43 +9,43 @@  discard block
 block discarded – undo
9 9
  */
10 10
 class Wiki extends Coursecopy\Resource
11 11
 {
12
-	public $id;
13
-	public $page_id;
14
-	public $reflink;
15
-	public $title;
16
-	public $content;
17
-	public $user_id;
18
-	public $group_id;
19
-	public $timestamp;
20
-	public $progress;
21
-	public $version;
12
+    public $id;
13
+    public $page_id;
14
+    public $reflink;
15
+    public $title;
16
+    public $content;
17
+    public $user_id;
18
+    public $group_id;
19
+    public $timestamp;
20
+    public $progress;
21
+    public $version;
22 22
 
23
-	/**
24
-	 * Wiki constructor.
25
-	 * @param int $id
26
-	 * @param int $page_id
27
-	 * @param $reflink
28
-	 * @param $title
29
-	 * @param $content
30
-	 * @param $user_id
31
-	 * @param $group_id
32
-	 * @param $timestamp
33
-	 * @param $progress
34
-	 * @param $version
35
-	 */
36
-	public function __construct(
37
-		$id,
38
-		$page_id,
39
-		$reflink,
40
-		$title,
41
-		$content,
42
-		$user_id,
43
-		$group_id,
44
-		$timestamp,
45
-		$progress,
46
-		$version
47
-	) {
48
-		parent::__construct($id,RESOURCE_WIKI);
23
+    /**
24
+     * Wiki constructor.
25
+     * @param int $id
26
+     * @param int $page_id
27
+     * @param $reflink
28
+     * @param $title
29
+     * @param $content
30
+     * @param $user_id
31
+     * @param $group_id
32
+     * @param $timestamp
33
+     * @param $progress
34
+     * @param $version
35
+     */
36
+    public function __construct(
37
+        $id,
38
+        $page_id,
39
+        $reflink,
40
+        $title,
41
+        $content,
42
+        $user_id,
43
+        $group_id,
44
+        $timestamp,
45
+        $progress,
46
+        $version
47
+    ) {
48
+        parent::__construct($id,RESOURCE_WIKI);
49 49
         $this->id = $id;
50 50
         $this->page_id = $page_id;
51 51
         $this->reflink = $reflink;
@@ -56,11 +56,11 @@  discard block
 block discarded – undo
56 56
         $this->dtime = $timestamp;
57 57
         $this->progress = $progress;
58 58
         $this->version = $version;
59
-	}
59
+    }
60 60
 
61
-	public function show()
62
-	{
63
-		parent::show();
64
-		echo $this->reflink.' ('. (empty($this->group_id) ? get_lang('Everyone') : get_lang('Group') . ' ' .  $this->group_id) .') ' . '<i>(' . $this->dtime . ')</i>';
65
-	}
61
+    public function show()
62
+    {
63
+        parent::show();
64
+        echo $this->reflink.' ('. (empty($this->group_id) ? get_lang('Everyone') : get_lang('Group') . ' ' .  $this->group_id) .') ' . '<i>(' . $this->dtime . ')</i>';
65
+    }
66 66
 }
Please login to merge, or discard this 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.