Passed
Push — 1.10.x ( f75118...849f97 )
by Angel Fernando Quiroz
52:16
created
plugin/courselegal/install.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,7 @@
 block discarded – undo
1 1
 <?php
2 2
 /* For license terms, see /license.txt */
3 3
 
4
-require_once dirname(__FILE__) . '/config.php';
4
+require_once dirname(__FILE__).'/config.php';
5 5
 
6 6
 if (!api_is_platform_admin()) {
7 7
     die ('You must have admin permissions to install plugins');
Please login to merge, or discard this patch.
plugin/buycourses/src/configure_course.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 
24 24
 if ($commissionsEnable == "true") {
25 25
     
26
-    $htmlHeadXtra[] = '<script type="text/javascript" src="' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/resources/js/commissions.js"></script>';
26
+    $htmlHeadXtra[] = '<script type="text/javascript" src="'.api_get_path(WEB_PLUGIN_PATH).'buycourses/resources/js/commissions.js"></script>';
27 27
     $defaultCommissions = [];
28 28
     $commissions = "";
29 29
     
@@ -170,9 +170,9 @@  discard block
 block discarded – undo
170 170
     . '<script>'
171 171
         . '$(function(){'
172 172
             . 'if ($("[name=\'commissions\']").val() === "") {'
173
-                . '$("#panelSliders").html("<button id=\"setCommissionsButton\" class=\"btn btn-warning\">' . get_plugin_lang("SetCommissions", "BuyCoursesPlugin") . '</button>");'
173
+                . '$("#panelSliders").html("<button id=\"setCommissionsButton\" class=\"btn btn-warning\">'.get_plugin_lang("SetCommissions", "BuyCoursesPlugin").'</button>");'
174 174
             . '} else {'
175
-                . 'showSliders(100, "default", "' . $commissions . '");'
175
+                . 'showSliders(100, "default", "'.$commissions.'");'
176 176
             . '}'
177 177
         . '});'
178 178
 
@@ -226,14 +226,14 @@  discard block
 block discarded – undo
226 226
     
227 227
     $platformCommission = $plugin->getPlatformCommission();
228 228
     
229
-    $form->addHtml( ''
229
+    $form->addHtml(''
230 230
             . '<div class="form-group">'
231 231
                 . '<label for="sliders" class="col-sm-2 control-label">'
232 232
                     . get_plugin_lang('Commissions', 'BuyCoursesPlugin')
233 233
                 . '</label>'
234 234
                 . '<div class="col-sm-8">'
235 235
                     . Display::return_message(
236
-                        sprintf($plugin->get_lang('TheActualPlatformCommissionIsX'), $platformCommission['commission']. '%'),
236
+                        sprintf($plugin->get_lang('TheActualPlatformCommissionIsX'), $platformCommission['commission'].'%'),
237 237
                         'info',
238 238
                         false
239 239
                     )
@@ -292,7 +292,7 @@  discard block
 block discarded – undo
292 292
         $plugin->deleteItem($productItem['id']);
293 293
     }
294 294
 
295
-    header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/configuration.php');
295
+    header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/src/configuration.php');
296 296
     exit;
297 297
 }
298 298
 
Please login to merge, or discard this patch.
plugin/buycourses/src/paymentsetup.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
             Display::return_message(get_lang('ItemRemoved'), 'success')
28 28
         );
29 29
 
30
-        header('Location: ' . api_get_self());
30
+        header('Location: '.api_get_self());
31 31
         exit;
32 32
     }
33 33
 }
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
         Display::return_message(get_lang('Saved'), 'success')
44 44
     );
45 45
 
46
-    header('Location:' . api_get_self());
46
+    header('Location:'.api_get_self());
47 47
     exit;
48 48
 }
49 49
 
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
         Display::return_message(get_lang('Saved'), 'success')
89 89
     );
90 90
 
91
-    header('Location:' . api_get_self());
91
+    header('Location:'.api_get_self());
92 92
     exit;
93 93
 }
94 94
 
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
         Display::return_message(get_lang('Saved'), 'success')
128 128
     );
129 129
 
130
-    header('Location:' . api_get_self());
130
+    header('Location:'.api_get_self());
131 131
     exit;
132 132
 }
133 133
 
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
         Display::return_message(get_lang('Saved'), 'success')
154 154
     );
155 155
 
156
-    header('Location:' . api_get_self());
156
+    header('Location:'.api_get_self());
157 157
     exit;
158 158
 }
159 159
 
Please login to merge, or discard this patch.
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,12 +1,12 @@
 block discarded – undo
1 1
 <?php
2 2
 /* For license terms, see /license.txt */
3 3
 /**
4
- * Configuration page for payment methods for the Buy Courses plugin
5
- * @package chamilo.plugin.buycourses
6
- */
4
+     * Configuration page for payment methods for the Buy Courses plugin
5
+     * @package chamilo.plugin.buycourses
6
+     */
7 7
 /**
8
- * Initialization
9
- */
8
+     * Initialization
9
+     */
10 10
 $cidReset = true;
11 11
 
12 12
 require_once '../../../main/inc/global.inc.php';
Please login to merge, or discard this patch.
plugin/buycourses/src/panel.ajax.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -37,20 +37,20 @@  discard block
 block discarded – undo
37 37
         
38 38
         $userInfo = api_get_user_info($sale['user_id']);
39 39
         
40
-        $html = '<h2>' . $sale['product_name'] .'</h2>';
40
+        $html = '<h2>'.$sale['product_name'].'</h2>';
41 41
         $html .= '<div class="row">';
42 42
         $html .= '<div class="col-sm-6 col-md-6">';
43 43
         $html .= '<ul>';
44
-        $html .= '<li><b>'. $plugin->get_lang('OrderPrice') . ':</b> '. $sale['price'] . '</li>';
45
-        $html .= '<li><b>'. $plugin->get_lang('CurrencyType') . ':</b> '. $currency['iso_code'] . '</li>';
46
-        $html .= '<li><b>'. $plugin->get_lang('ProductType') . ':</b> '. $productType . '</li>';
47
-        $html .= '<li><b>'. $plugin->get_lang('OrderDate') . ':</b> '. api_format_date($sale['date'], DATE_TIME_FORMAT_LONG_24H) . '</li>';
48
-        $html .= '<li><b>'. $plugin->get_lang('Buyer') . ':</b> '. $userInfo['complete_name'] . '</li>';
49
-        $html .= '<li><b>'. $plugin->get_lang('PaymentMethods') . ':</b> '. $paymentType . '</li>';
44
+        $html .= '<li><b>'.$plugin->get_lang('OrderPrice').':</b> '.$sale['price'].'</li>';
45
+        $html .= '<li><b>'.$plugin->get_lang('CurrencyType').':</b> '.$currency['iso_code'].'</li>';
46
+        $html .= '<li><b>'.$plugin->get_lang('ProductType').':</b> '.$productType.'</li>';
47
+        $html .= '<li><b>'.$plugin->get_lang('OrderDate').':</b> '.api_format_date($sale['date'], DATE_TIME_FORMAT_LONG_24H).'</li>';
48
+        $html .= '<li><b>'.$plugin->get_lang('Buyer').':</b> '.$userInfo['complete_name'].'</li>';
49
+        $html .= '<li><b>'.$plugin->get_lang('PaymentMethods').':</b> '.$paymentType.'</li>';
50 50
         $html .= '</ul>';
51 51
         $html .= '</div>';
52 52
         $html .= '<div class="col-sm-6 col-md-6">';
53
-        $html .= '<img class="thumbnail" src="'. $productImage .'" >';
53
+        $html .= '<img class="thumbnail" src="'.$productImage.'" >';
54 54
         $html .= '</div>';
55 55
         $html .= '</div>';
56 56
 
@@ -95,9 +95,9 @@  discard block
 block discarded – undo
95 95
         $html = '<div class="row">'
96 96
         . '<p>'
97 97
             . '<ul>'
98
-                . '<li>'. get_plugin_lang("PayoutsTotalCompleted", "BuyCoursesPlugin") .' <b>'. $stats['completed_count'] .'</b> - '. get_plugin_lang("TotalAmount", "BuyCoursesPlugin") .' <b>'. $stats['completed_total_amount'] .' '. $currency['iso_code'] . '</b></li>'
99
-                . '<li>'. get_plugin_lang("PayoutsTotalPending", "BuyCoursesPlugin") .' <b>'. $stats['pending_count'] .'</b> - '. get_plugin_lang("TotalAmount", "BuyCoursesPlugin") .' <b>'. $stats['pending_total_amount'] .' '. $currency['iso_code'] . '</b></li>'
100
-                . '<li>'. get_plugin_lang("PayoutsTotalCanceled", "BuyCoursesPlugin") .' <b>'. $stats['canceled_count'] .'</b> - '. get_plugin_lang("TotalAmount", "BuyCoursesPlugin") .' <b>'. $stats['canceled_total_amount'] .' '. $currency['iso_code'] . '</b></li>'
98
+                . '<li>'.get_plugin_lang("PayoutsTotalCompleted", "BuyCoursesPlugin").' <b>'.$stats['completed_count'].'</b> - '.get_plugin_lang("TotalAmount", "BuyCoursesPlugin").' <b>'.$stats['completed_total_amount'].' '.$currency['iso_code'].'</b></li>'
99
+                . '<li>'.get_plugin_lang("PayoutsTotalPending", "BuyCoursesPlugin").' <b>'.$stats['pending_count'].'</b> - '.get_plugin_lang("TotalAmount", "BuyCoursesPlugin").' <b>'.$stats['pending_total_amount'].' '.$currency['iso_code'].'</b></li>'
100
+                . '<li>'.get_plugin_lang("PayoutsTotalCanceled", "BuyCoursesPlugin").' <b>'.$stats['canceled_count'].'</b> - '.get_plugin_lang("TotalAmount", "BuyCoursesPlugin").' <b>'.$stats['canceled_total_amount'].' '.$currency['iso_code'].'</b></li>'
101 101
             . '</ul>'
102 102
         . '</p>';
103 103
         $html .= '</div>';
@@ -137,15 +137,15 @@  discard block
 block discarded – undo
137 137
         
138 138
         $isoCode = $currentCurrency['iso_code'];
139 139
         
140
-        $html .= '<p>'. get_plugin_lang("VerifyTotalAmountToProceedPayout", "BuyCoursesPlugin") .'</p>';
140
+        $html .= '<p>'.get_plugin_lang("VerifyTotalAmountToProceedPayout", "BuyCoursesPlugin").'</p>';
141 141
         $html .= ''
142 142
         . '<p>'
143 143
             . '<ul>'
144
-                . '<li>'. get_plugin_lang("TotalAcounts", "BuyCoursesPlugin") .' <b>'. $totalAccounts .'</b></li>'
145
-                . '<li>'. get_plugin_lang("TotalPayout", "BuyCoursesPlugin") .' <b>'. $isoCode .' '. $totalPayout .'</b></li>'
144
+                . '<li>'.get_plugin_lang("TotalAcounts", "BuyCoursesPlugin").' <b>'.$totalAccounts.'</b></li>'
145
+                . '<li>'.get_plugin_lang("TotalPayout", "BuyCoursesPlugin").' <b>'.$isoCode.' '.$totalPayout.'</b></li>'
146 146
             . '</ul>'
147 147
         . '</p>';
148
-        $html .= '<p>'. get_plugin_lang("CautionThisProcessCantBeCanceled", "BuyCoursesPlugin") .'</p>';
148
+        $html .= '<p>'.get_plugin_lang("CautionThisProcessCantBeCanceled", "BuyCoursesPlugin").'</p>';
149 149
         $html .= '</br></br>';
150 150
         $html .= '<div id="spinner" class="text-center"></div>';
151 151
                 
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
             
200 200
         } else {
201 201
 
202
-            echo Display::return_message('<b>'.$result['L_SEVERITYCODE0'].' '.$result['L_ERRORCODE0'].'</b> - '.$result['L_SHORTMESSAGE0'].'</br><ul><li>'. $result['L_LONGMESSAGE0'].'</li></ul>', 'error', false);
202
+            echo Display::return_message('<b>'.$result['L_SEVERITYCODE0'].' '.$result['L_ERRORCODE0'].'</b> - '.$result['L_SHORTMESSAGE0'].'</br><ul><li>'.$result['L_LONGMESSAGE0'].'</li></ul>', 'error', false);
203 203
             
204 204
         }
205 205
         
Please login to merge, or discard this patch.
plugin/buycourses/src/error.php 1 patch
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,12 +1,12 @@
 block discarded – undo
1 1
 <?php
2 2
 /* For license terms, see /license.txt */
3 3
 /**
4
- * Errors management for the Buy Courses plugin - Redirects to course_catalog.php
5
- * @package chamilo.plugin.buycourses
6
- */
4
+         * Errors management for the Buy Courses plugin - Redirects to course_catalog.php
5
+         * @package chamilo.plugin.buycourses
6
+         */
7 7
 /**
8
- * Config
9
- */
8
+         * Config
9
+         */
10 10
 unset($_SESSION['bc_sale_id']);
11 11
 
12 12
 header('Location: course_catalog.php');
Please login to merge, or discard this patch.
plugin/buycourses/src/expresscheckout.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -40,8 +40,8 @@
 block discarded – undo
40 40
     $ErrorSeverityCode = urldecode($resArray["L_SEVERITYCODE0"]);
41 41
 
42 42
     echo "SetExpressCheckout API call failed. ";
43
-    echo "Detailed Error Message: " . $ErrorLongMsg;
44
-    echo "Short Error Message: " . $ErrorShortMsg;
45
-    echo "Error Code: " . $ErrorCode;
46
-    echo "Error Severity Code: " . $ErrorSeverityCode;
43
+    echo "Detailed Error Message: ".$ErrorLongMsg;
44
+    echo "Short Error Message: ".$ErrorShortMsg;
45
+    echo "Error Code: ".$ErrorCode;
46
+    echo "Error Severity Code: ".$ErrorSeverityCode;
47 47
 }
Please login to merge, or discard this patch.
plugin/buycourses/src/success.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
 
58 58
         unset($_SESSION['bc_sale_id']);
59 59
 
60
-        header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/index.php');
60
+        header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/index.php');
61 61
         exit;
62 62
     }
63 63
 
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
     }
167 167
 
168 168
     unset($_SESSION['bc_sale_id']);
169
-    header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/course_catalog.php');
169
+    header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/src/course_catalog.php');
170 170
     exit;
171 171
 }
172 172
 
Please login to merge, or discard this patch.
plugin/buycourses/src/process.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
 $currentUserId = api_get_user_id();
13 13
 
14 14
 if (empty($currentUserId)) {
15
-    header('Location: ' . api_get_path(WEB_CODE_PATH) . 'auth/inscription.php');
15
+    header('Location: '.api_get_path(WEB_CODE_PATH).'auth/inscription.php');
16 16
     exit;
17 17
 }
18 18
 
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 
28 28
 $buyingCourse = intval($_REQUEST['t']) === BuyCoursesPlugin::PRODUCT_TYPE_COURSE;
29 29
 $buyingSession = intval($_REQUEST['t']) === BuyCoursesPlugin::PRODUCT_TYPE_SESSION;
30
-$queryString = 'i=' . intval($_REQUEST['i']) . '&t=' . intval($_REQUEST['t']);
30
+$queryString = 'i='.intval($_REQUEST['i']).'&t='.intval($_REQUEST['t']);
31 31
 
32 32
 if ($buyingCourse) {
33 33
     $courseInfo = $plugin->getCourseInfo($_REQUEST['i']);
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
         Display::addFlash(
49 49
             Display::return_message($plugin->get_lang('NeedToSelectPaymentType'), 'error', false)
50 50
         );
51
-        header('Location:' . api_get_self() . '?' . $queryString);
51
+        header('Location:'.api_get_self().'?'.$queryString);
52 52
         exit;
53 53
     }
54 54
     
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
 
57 57
     if ($saleId !== false) {
58 58
         $_SESSION['bc_sale_id'] = $saleId;
59
-        header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/process_confirm.php');  
59
+        header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/src/process_confirm.php');  
60 60
     }
61 61
 
62 62
     exit;
Please login to merge, or discard this patch.
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,12 +1,12 @@
 block discarded – undo
1 1
 <?php
2 2
 /* For license terms, see /license.txt */
3 3
 /**
4
- * Process payments for the Buy Courses plugin
5
- * @package chamilo.plugin.buycourses
6
- */
4
+     * Process payments for the Buy Courses plugin
5
+     * @package chamilo.plugin.buycourses
6
+     */
7 7
 /**
8
- * Initialization
9
- */
8
+     * Initialization
9
+     */
10 10
 require_once '../config.php';
11 11
 
12 12
 $currentUserId = api_get_user_id();
Please login to merge, or discard this patch.
plugin/buycourses/src/process_confirm.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -45,8 +45,8 @@  discard block
 block discarded – undo
45 45
             $sale['price'],
46 46
             $currency['iso_code'],
47 47
             'paypal',
48
-            api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/success.php',
49
-            api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/error.php',
48
+            api_get_path(WEB_PLUGIN_PATH).'buycourses/src/success.php',
49
+            api_get_path(WEB_PLUGIN_PATH).'buycourses/src/error.php',
50 50
             $extra
51 51
         );
52 52
 
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 
93 93
                 unset($_SESSION['bc_sale_id']);
94 94
 
95
-                header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/index.php');
95
+                header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/index.php');
96 96
                 exit;
97 97
             }
98 98
 
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
             );
130 130
 
131 131
             unset($_SESSION['bc_sale_id']);
132
-            header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'buycourses/src/course_catalog.php');
132
+            header('Location: '.api_get_path(WEB_PLUGIN_PATH).'buycourses/src/course_catalog.php');
133 133
             exit;
134 134
         }
135 135
 
Please login to merge, or discard this patch.