Completed
Push — 1.11.x ( ca7787...41c0f2 )
by José
31:51
created
main/auth/ldap/login.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@
 block discarded – undo
31 31
 	$loginFailed = false;
32 32
 	$uidReset = true;
33 33
 	$_user['user_id'] = $uData['user_id'];
34
-	Session::write('_uid',$_uid);
34
+	Session::write('_uid', $_uid);
35 35
     // Jand: copied from event_login in events.lib.php to enable login statistics:
36 36
 	Event::event_login($uData['user_id']);
37 37
 } else {
Please login to merge, or discard this patch.
main/auth/sso/sso.class.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
      */
35 35
     public function __construct()
36 36
     {
37
-        $this->protocol   = api_get_setting('sso_authentication_protocol');
37
+        $this->protocol = api_get_setting('sso_authentication_protocol');
38 38
         // There can be multiple domains, so make sure to take only the first
39 39
         // This might be later extended with a decision process
40 40
         $domains = explode(',', api_get_setting('sso_authentication_domain'));
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
         $this->auth_uri = api_get_setting('sso_authentication_auth_uri');
43 43
         $this->deauth_uri = api_get_setting('sso_authentication_unauth_uri');
44 44
         //cut the string to avoid recursive URL construction in case of failure
45
-        $this->referer = $this->protocol.$_SERVER['HTTP_HOST'].substr($_SERVER['REQUEST_URI'],0,strpos($_SERVER['REQUEST_URI'],'sso'));
45
+        $this->referer = $this->protocol.$_SERVER['HTTP_HOST'].substr($_SERVER['REQUEST_URI'], 0, strpos($_SERVER['REQUEST_URI'], 'sso'));
46 46
         $this->deauth_url = $this->protocol.$this->domain.$this->deauth_uri;
47 47
         $this->master_url = $this->protocol.$this->domain.$this->auth_uri;
48 48
         $this->referrer_uri = base64_encode($_SERVER['REQUEST_URI']);
@@ -130,11 +130,11 @@  discard block
 block discarded – undo
130 130
                 ) {
131 131
                     //error_log('user n password are ok');
132 132
                     //Check if the account is active (not locked)
133
-                    if ($uData['active']=='1') {
133
+                    if ($uData['active'] == '1') {
134 134
                         // check if the expiration date has not been reached
135 135
                         if (empty($uData['expiration_date'])
136 136
                             or $uData['expiration_date'] > date('Y-m-d H:i:s')
137
-                            or $uData['expiration_date']=='0000-00-00 00:00:00') {
137
+                            or $uData['expiration_date'] == '0000-00-00 00:00:00') {
138 138
 
139 139
                             //If Multiple URL is enabled
140 140
                             if (api_get_multiple_access_url()) {
@@ -168,11 +168,11 @@  discard block
 block discarded – undo
168 168
                                             // the user credentials are OK, which
169 169
                                             // should be protection enough
170 170
                                             // against evil URL spoofing...
171
-                                            $sso_target = api_get_path(WEB_PATH) . base64_decode($sso['ruri']);
171
+                                            $sso_target = api_get_path(WEB_PATH).base64_decode($sso['ruri']);
172 172
                                         } else {
173
-                                            $sso_target = isset($sso['target']) ? $sso['target'] : api_get_path(WEB_PATH) . 'index.php';
173
+                                            $sso_target = isset($sso['target']) ? $sso['target'] : api_get_path(WEB_PATH).'index.php';
174 174
                                         }
175
-                                        header('Location: '. $sso_target);
175
+                                        header('Location: '.$sso_target);
176 176
                                         exit;
177 177
                                     } else {
178 178
                                         // user does not have permission for this site
@@ -286,9 +286,9 @@  discard block
 block discarded – undo
286 286
         $userId = intval($userId);
287 287
 
288 288
         if ($asAdmin && api_is_platform_admin(true)) {
289
-            return api_get_path(WEB_CODE_PATH) . "admin/user_edit.php?user_id=$userId";
289
+            return api_get_path(WEB_CODE_PATH)."admin/user_edit.php?user_id=$userId";
290 290
         }
291 291
 
292
-        return api_get_path(WEB_CODE_PATH) . 'auth/profile.php';
292
+        return api_get_path(WEB_CODE_PATH).'auth/profile.php';
293 293
     }
294 294
 }
Please login to merge, or discard this patch.
main/auth/sso/sso.Drupal.class.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -22,11 +22,11 @@  discard block
 block discarded – undo
22 22
  */
23 23
 class ssoDrupal
24 24
 {
25
-    public $protocol;   // 'http://',
26
-    public $domain;     // 'localhost/project/drupal',
27
-    public $auth_uri;   // '/?q=user',
25
+    public $protocol; // 'http://',
26
+    public $domain; // 'localhost/project/drupal',
27
+    public $auth_uri; // '/?q=user',
28 28
     public $deauth_uri; // '/?q=logout',
29
-    public $referer;    // http://my.chamilo.com/main/auth/profile.php
29
+    public $referer; // http://my.chamilo.com/main/auth/profile.php
30 30
 
31 31
     /**
32 32
      * Instanciates the object, initializing all relevant URL strings
@@ -123,9 +123,9 @@  discard block
 block discarded – undo
123 123
                     && ($sso['username'] == $uData['username'])) {
124 124
 
125 125
                     //Check if the account is active (not locked)
126
-                    if ($uData['active']=='1') {
126
+                    if ($uData['active'] == '1') {
127 127
                         // check if the expiration date has not been reached
128
-                        if (empty($uData['expiration_date']) OR $uData['expiration_date'] > date('Y-m-d H:i:s') OR $uData['expiration_date']=='0000-00-00 00:00:00') {
128
+                        if (empty($uData['expiration_date']) OR $uData['expiration_date'] > date('Y-m-d H:i:s') OR $uData['expiration_date'] == '0000-00-00 00:00:00') {
129 129
 
130 130
                             //If Multiple URL is enabled
131 131
                             if (api_get_multiple_access_url()) {
@@ -158,11 +158,11 @@  discard block
 block discarded – undo
158 158
                                             // the user credentials are OK, which
159 159
                                             // should be protection enough
160 160
                                             // against evil URL spoofing...
161
-                                            $sso_target = api_get_path(WEB_PATH) . base64_decode($sso['ruri']);
161
+                                            $sso_target = api_get_path(WEB_PATH).base64_decode($sso['ruri']);
162 162
                                         } else {
163
-                                            $sso_target = isset($sso['target']) ? $sso['target'] : api_get_path(WEB_PATH) . 'index.php';
163
+                                            $sso_target = isset($sso['target']) ? $sso['target'] : api_get_path(WEB_PATH).'index.php';
164 164
                                         }
165
-                                        header('Location: '. $sso_target);
165
+                                        header('Location: '.$sso_target);
166 166
                                         exit;
167 167
                                     } else {
168 168
                                         // user does not have permission for this site
@@ -279,12 +279,12 @@  discard block
 block discarded – undo
279 279
         // If this is an administrator, allow him to make some changes in
280 280
         // the Chamilo profile
281 281
         if ($asAdmin && api_is_platform_admin(true)) {
282
-            return api_get_path(WEB_CODE_PATH) . "admin/user_edit.php?user_id=$userId";
282
+            return api_get_path(WEB_CODE_PATH)."admin/user_edit.php?user_id=$userId";
283 283
         }
284 284
         // If the user doesn't match a Drupal user, give the normal profile
285 285
         // link
286 286
         if ($drupalUserIdData === false) {
287
-            return api_get_path(WEB_CODE_PATH) . 'auth/profile.php';
287
+            return api_get_path(WEB_CODE_PATH).'auth/profile.php';
288 288
         }
289 289
         // In all other cases, generate a link to the Drupal profile edition
290 290
         $drupalUserId = $drupalUserIdData['value'];
Please login to merge, or discard this patch.
main/auth/sso/sso_server_test.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 
49 49
         /* 2.Get the chamilo username and password from your system or from webservices */
50 50
 
51
-        $account['username'] = 'jbrion525';             //username in Chamilo
51
+        $account['username'] = 'jbrion525'; //username in Chamilo
52 52
         $account['password'] = sha1(sha1('jbrion525')); //encrypted password with assuming that the first encrypted method is sha1 in chamilo
53 53
 
54 54
         $master_auth_uri = $my_chamilo_server.'/?q=user';
@@ -64,9 +64,9 @@  discard block
 block discarded – undo
64 64
         );
65 65
 
66 66
         $cookie = base64_encode(serialize($sso));
67
-        $url = chamilo_sso_protocol() . $master_auth_uri;
68
-        $params = 'sso_referer='. urlencode($url) .'&sso_cookie='. urlencode($cookie);
69
-        $final_url = filter_xss($_GET['sso_referer']) .'?'. $params;
67
+        $url = chamilo_sso_protocol().$master_auth_uri;
68
+        $params = 'sso_referer='.urlencode($url).'&sso_cookie='.urlencode($cookie);
69
+        $final_url = filter_xss($_GET['sso_referer']).'?'.$params;
70 70
 
71 71
         //If your user exists redirect to chamilo and set the account in a session to check it later
72 72
         $_SESSION['my_server_user_session'] = $account;
Please login to merge, or discard this patch.
main/auth/shibboleth/login.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
  * @license see /license.txt
16 16
  * @author Laurent Opprecht <[email protected]>, Nicolas Rod for the University of Geneva
17 17
  */
18
-include_once(dirname(__FILE__) . '/init.php');
18
+include_once(dirname(__FILE__).'/init.php');
19 19
 
20 20
 /*
21 21
   ==============================================================================
Please login to merge, or discard this patch.
main/auth/shibboleth/test/test.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@
 block discarded – undo
8 8
  * @license see /license.txt
9 9
  * @author Laurent Opprecht <[email protected]>, Nicolas Rod for the University of Geneva
10 10
  */
11
-include_once(dirname(__FILE__) . '/../init.php');
11
+include_once(dirname(__FILE__).'/../init.php');
12 12
 
13 13
 if (!ShibbolethTest::is_enabled())
14 14
 {
Please login to merge, or discard this patch.
main/auth/shibboleth/test/shibboleth_test_helper.class.php 1 patch
Spacing   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -59,75 +59,75 @@
 block discarded – undo
59 59
     public function setup_new_student()
60 60
     {
61 61
         $id = uniqid();
62
-        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_' . $id;
62
+        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_'.$id;
63 63
         $_SERVER['Shib-EP-Affiliation'] = 'member';
64 64
         $_SERVER['Shib-InetOrgPerson-givenName'] = 'John';
65
-        $_SERVER['Shib-Person-surname'] = 'Doe' . $id;
66
-        $_SERVER['Shib-InetOrgPerson-mail'] = 'john.' . $id . '[email protected]';
67
-        $_SERVER['persistent-id'] = 'idp!viewer!' . md5($id);
65
+        $_SERVER['Shib-Person-surname'] = 'Doe'.$id;
66
+        $_SERVER['Shib-InetOrgPerson-mail'] = 'john.'.$id.'[email protected]';
67
+        $_SERVER['persistent-id'] = 'idp!viewer!'.md5($id);
68 68
     }
69 69
 
70 70
     public function setup_new_student_no_email()
71 71
     {
72 72
         $id = uniqid();
73
-        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_' . $id;
73
+        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_'.$id;
74 74
         $_SERVER['Shib-EP-Affiliation'] = 'member';
75 75
         $_SERVER['Shib-InetOrgPerson-givenName'] = 'John';
76
-        $_SERVER['Shib-Person-surname'] = 'Doe' . $id;
76
+        $_SERVER['Shib-Person-surname'] = 'Doe'.$id;
77 77
         $_SERVER['Shib-InetOrgPerson-mail'] = '';
78
-        $_SERVER['persistent-id'] = 'idp!viewer!' . md5($id);
78
+        $_SERVER['persistent-id'] = 'idp!viewer!'.md5($id);
79 79
     }
80 80
 
81 81
     public function setup_new_student_multiple_givenname()
82 82
     {
83 83
         $id = uniqid();
84
-        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_' . $id;
84
+        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_'.$id;
85 85
         $_SERVER['Shib-EP-Affiliation'] = 'member';
86 86
         $_SERVER['Shib-InetOrgPerson-givenName'] = 'John;Alex;John Alex';
87
-        $_SERVER['Shib-Person-surname'] = 'Doe' . $id;
88
-        $_SERVER['Shib-InetOrgPerson-mail'] = 'john.' . $id . '[email protected]';
89
-        $_SERVER['persistent-id'] = 'idp!viewer!' . md5($id);
87
+        $_SERVER['Shib-Person-surname'] = 'Doe'.$id;
88
+        $_SERVER['Shib-InetOrgPerson-mail'] = 'john.'.$id.'[email protected]';
89
+        $_SERVER['persistent-id'] = 'idp!viewer!'.md5($id);
90 90
     }
91 91
 
92 92
     public function setup_new_teacher()
93 93
     {
94 94
         $id = uniqid();
95
-        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_' . $id;
95
+        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_'.$id;
96 96
         $_SERVER['Shib-EP-Affiliation'] = 'member;staff;faculty';
97 97
         $_SERVER['Shib-InetOrgPerson-givenName'] = 'John';
98
-        $_SERVER['Shib-Person-surname'] = 'Doe' . $id;
99
-        $_SERVER['Shib-InetOrgPerson-mail'] = 'john.' . $id . '[email protected]';
100
-        $_SERVER['persistent-id'] = 'idp!viewer!' . md5($id);
98
+        $_SERVER['Shib-Person-surname'] = 'Doe'.$id;
99
+        $_SERVER['Shib-InetOrgPerson-mail'] = 'john.'.$id.'[email protected]';
100
+        $_SERVER['persistent-id'] = 'idp!viewer!'.md5($id);
101 101
     }
102 102
 
103 103
     public function setup_new_staff()
104 104
     {
105 105
         $id = uniqid();
106
-        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_' . $id;
106
+        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_'.$id;
107 107
         $_SERVER['Shib-EP-Affiliation'] = 'member;staff';
108 108
         $_SERVER['Shib-InetOrgPerson-givenName'] = 'John';
109
-        $_SERVER['Shib-Person-surname'] = 'Doe' . $id;
110
-        $_SERVER['Shib-InetOrgPerson-mail'] = 'john.' . $id . '[email protected]';
111
-        $_SERVER['persistent-id'] = 'idp!viewer!' . md5($id);
109
+        $_SERVER['Shib-Person-surname'] = 'Doe'.$id;
110
+        $_SERVER['Shib-InetOrgPerson-mail'] = 'john.'.$id.'[email protected]';
111
+        $_SERVER['persistent-id'] = 'idp!viewer!'.md5($id);
112 112
     }
113 113
 
114 114
     public function setup_new_no_affiliation()
115 115
     {
116 116
         $id = uniqid();
117
-        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_' . $id;
117
+        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_'.$id;
118 118
         $_SERVER['Shib-EP-Affiliation'] = '';
119 119
         $_SERVER['Shib-InetOrgPerson-givenName'] = 'John';
120
-        $_SERVER['Shib-Person-surname'] = 'Doe' . $id;
121
-        $_SERVER['Shib-InetOrgPerson-mail'] = 'john.' . $id . '[email protected]';
122
-        $_SERVER['persistent-id'] = 'idp!viewer!' . md5($id);
120
+        $_SERVER['Shib-Person-surname'] = 'Doe'.$id;
121
+        $_SERVER['Shib-InetOrgPerson-mail'] = 'john.'.$id.'[email protected]';
122
+        $_SERVER['persistent-id'] = 'idp!viewer!'.md5($id);
123 123
     }
124 124
 
125 125
     public function setup_new_minimal_data()
126 126
     {
127 127
         $id = uniqid();
128
-        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_' . $id;       
128
+        $_SERVER['Shib-SwissEP-UniqueID'] = 'usr_'.$id;       
129 129
         $_SERVER['Shib-InetOrgPerson-givenName'] = 'John';
130
-        $_SERVER['Shib-Person-surname'] = 'Doe' . $id;
130
+        $_SERVER['Shib-Person-surname'] = 'Doe'.$id;
131 131
     }
132 132
 
133 133
 }
134 134
\ No newline at end of file
Please login to merge, or discard this patch.
main/auth/shibboleth/test/test_no_email.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -6,7 +6,7 @@  discard block
 block discarded – undo
6 6
  * @license see /license.txt
7 7
  * @author Laurent Opprecht <[email protected]>, Nicolas Rod for the University of Geneva
8 8
  */
9
-include_once(dirname(__FILE__) . '/../init.php');
9
+include_once(dirname(__FILE__).'/../init.php');
10 10
 
11 11
 if (!ShibbolethTest::is_enabled())
12 12
 {
@@ -17,4 +17,4 @@  discard block
 block discarded – undo
17 17
 Shibboleth::session()->logout();
18 18
 ShibbolethTest::helper()->setup_new_student_no_email();
19 19
 
20
-require_once dirname(__FILE__) . '/../login.php';
21 20
\ No newline at end of file
21
+require_once dirname(__FILE__).'/../login.php';
22 22
\ No newline at end of file
Please login to merge, or discard this patch.
main/auth/shibboleth/app/shibboleth.class.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -78,14 +78,14 @@
 block discarded – undo
78 78
 
79 79
     public static function sys_path()
80 80
     {
81
-        $path = dirname(__FILE__) . '/../';
81
+        $path = dirname(__FILE__).'/../';
82 82
         return $path;
83 83
     }
84 84
 
85 85
     public static function url($path = '')
86 86
     {
87 87
         $result = api_get_path('WEB_PATH');
88
-        $result .= '/main/auth/shibboleth/' . $path;
88
+        $result .= '/main/auth/shibboleth/'.$path;
89 89
         return $result;
90 90
     }
91 91
 
Please login to merge, or discard this patch.