Completed
Push — 1.11.x ( 683301...61da54 )
by José
78:34 queued 46:29
created
main/webservices/user_info.soap.php 2 patches
Indentation   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -26,11 +26,11 @@  discard block
 block discarded – undo
26 26
         'all',
27 27
         '',
28 28
         array(
29
-          'name'=>'code'  , 'type'=>'xsd:string',
30
-          'name'=>'title'  , 'type'=>'xsd:string',
31
-          'name'=>'url'    , 'type'=>'xsd:string',
32
-          'name'=>'teacher', 'type'=>'xsd:string',
33
-          'name'=>'language','type'=>'xsd:string',
29
+            'name'=>'code'  , 'type'=>'xsd:string',
30
+            'name'=>'title'  , 'type'=>'xsd:string',
31
+            'name'=>'url'    , 'type'=>'xsd:string',
32
+            'name'=>'teacher', 'type'=>'xsd:string',
33
+            'name'=>'language','type'=>'xsd:string',
34 34
         )
35 35
 );
36 36
 
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
 // Register the method to expose
52 52
 $server->register('WSCourseListOfUser',   // method name
53 53
     array('username' => 'xsd:string',
54
-          'signature' => 'xsd:string'),         // input parameters
54
+            'signature' => 'xsd:string'),         // input parameters
55 55
     array('return' => 'xsd:Array'),             // output parameters
56 56
     'urn:WSUserInfo',                           // namespace
57 57
     'urn:WSUserInfo#WSUserInfo',          // soapaction
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
     $list = UserManager::get_api_keys($user_id, 'dokeos');
78 78
     $key = '';
79 79
     foreach ($list as $key) {
80
-    	break;
80
+        break;
81 81
     }
82 82
 
83 83
     $local_key = $username.$key;
@@ -129,9 +129,9 @@  discard block
 block discarded – undo
129 129
 // Register the method to expose
130 130
 $server->register('WSEventsList',       // method name
131 131
     array('username' => 'xsd:string',
132
-          'signature' => 'xsd:string',
133
-          'datestart' => 'xsd:int',
134
-          'dateend'   => 'xsd:int'),          // input parameters
132
+            'signature' => 'xsd:string',
133
+            'datestart' => 'xsd:int',
134
+            'dateend'   => 'xsd:int'),          // input parameters
135 135
     array('return' => 'xsd:Array'),           // output parameters
136 136
     'urn:WSUserInfo',                         // namespace
137 137
     'urn:WSUserInfo#WSEventsList',      // soapaction
@@ -159,7 +159,7 @@  discard block
 block discarded – undo
159 159
     $list = UserManager::get_api_keys($user_id, 'dokeos');
160 160
     $key = '';
161 161
     foreach ($list as $key) {
162
-    	break;
162
+        break;
163 163
     }
164 164
 
165 165
     $local_key = $username.$key;
Please login to merge, or discard this patch.
Spacing   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -26,11 +26,11 @@  discard block
 block discarded – undo
26 26
         'all',
27 27
         '',
28 28
         array(
29
-          'name'=>'code'  , 'type'=>'xsd:string',
30
-          'name'=>'title'  , 'type'=>'xsd:string',
31
-          'name'=>'url'    , 'type'=>'xsd:string',
29
+          'name'=>'code', 'type'=>'xsd:string',
30
+          'name'=>'title', 'type'=>'xsd:string',
31
+          'name'=>'url', 'type'=>'xsd:string',
32 32
           'name'=>'teacher', 'type'=>'xsd:string',
33
-          'name'=>'language','type'=>'xsd:string',
33
+          'name'=>'language', 'type'=>'xsd:string',
34 34
         )
35 35
 );
36 36
 
@@ -49,14 +49,14 @@  discard block
 block discarded – undo
49 49
 );
50 50
 
51 51
 // Register the method to expose
52
-$server->register('WSCourseListOfUser',   // method name
52
+$server->register('WSCourseListOfUser', // method name
53 53
     array('username' => 'xsd:string',
54
-          'signature' => 'xsd:string'),         // input parameters
55
-    array('return' => 'xsd:Array'),             // output parameters
56
-    'urn:WSUserInfo',                           // namespace
57
-    'urn:WSUserInfo#WSUserInfo',          // soapaction
58
-    'rpc',                                      // style
59
-    'encoded',                                  // use
54
+          'signature' => 'xsd:string'), // input parameters
55
+    array('return' => 'xsd:Array'), // output parameters
56
+    'urn:WSUserInfo', // namespace
57
+    'urn:WSUserInfo#WSUserInfo', // soapaction
58
+    'rpc', // style
59
+    'encoded', // use
60 60
     'This service returns a list of courses'    // documentation
61 61
 );
62 62
 
@@ -104,11 +104,11 @@  discard block
 block discarded – undo
104 104
     'all',
105 105
     '',
106 106
     array(
107
-        'name'=>'datestart','type'=>'xsd:string',
108
-        'name'=>'dateend','type'=>'xsd:string',
109
-        'name'=>'title','type'=>'xsd:string',
110
-        'name'=>'link','type'=>'xsd:string',
111
-        'name'=>'coursetitle','type'=>'xsd:string',
107
+        'name'=>'datestart', 'type'=>'xsd:string',
108
+        'name'=>'dateend', 'type'=>'xsd:string',
109
+        'name'=>'title', 'type'=>'xsd:string',
110
+        'name'=>'link', 'type'=>'xsd:string',
111
+        'name'=>'coursetitle', 'type'=>'xsd:string',
112 112
     )
113 113
 );
114 114
 
@@ -127,16 +127,16 @@  discard block
 block discarded – undo
127 127
 );
128 128
 
129 129
 // Register the method to expose
130
-$server->register('WSEventsList',       // method name
130
+$server->register('WSEventsList', // method name
131 131
     array('username' => 'xsd:string',
132 132
           'signature' => 'xsd:string',
133 133
           'datestart' => 'xsd:int',
134
-          'dateend'   => 'xsd:int'),          // input parameters
135
-    array('return' => 'xsd:Array'),           // output parameters
136
-    'urn:WSUserInfo',                         // namespace
137
-    'urn:WSUserInfo#WSEventsList',      // soapaction
138
-    'rpc',                                    // style
139
-    'encoded',                                // use
134
+          'dateend'   => 'xsd:int'), // input parameters
135
+    array('return' => 'xsd:Array'), // output parameters
136
+    'urn:WSUserInfo', // namespace
137
+    'urn:WSUserInfo#WSEventsList', // soapaction
138
+    'rpc', // style
139
+    'encoded', // use
140 140
     'This service returns a list of events of the courses the given user is subscribed to'      // documentation
141 141
 );
142 142
 
@@ -171,8 +171,8 @@  discard block
 block discarded – undo
171 171
 
172 172
     $user_id = UserManager::get_user_id_from_username($username);
173 173
     if ($user_id === false) { return $events_list; } // Error in user id recovery.
174
-    $ds = substr($datestart,0,4).'-'.substr($datestart,4,2).'-'.substr($datestart,6,2).' 00:00:00';
175
-    $de = substr($dateend,0,4).'-'.substr($dateend,4,2).'-'.substr($dateend,6,2).' 00:00:00';
174
+    $ds = substr($datestart, 0, 4).'-'.substr($datestart, 4, 2).'-'.substr($datestart, 6, 2).' 00:00:00';
175
+    $de = substr($dateend, 0, 4).'-'.substr($dateend, 4, 2).'-'.substr($dateend, 6, 2).' 00:00:00';
176 176
     $events_list = Agenda::get_personal_agenda_items_between_dates($user_id, $ds, $de);
177 177
     return $events_list;
178 178
 }
Please login to merge, or discard this patch.
main/webservices/soap.php 2 patches
Indentation   +61 added lines, -61 removed lines patch added patch discarded remove patch
@@ -11,84 +11,84 @@  discard block
 block discarded – undo
11 11
  * SOAP error handler. Handles an error sending a SOAP fault
12 12
  */
13 13
 class WSSoapErrorHandler implements WSErrorHandler {
14
-	/**
15
-	 * Handles the error by sending a SOAP fault through the server
16
-	 *
17
-	 * @param WSError Error to handle
18
-	 */
19
-	public function handle($error) {
20
-		$server = WSSoapServer::singleton();
21
-		$server->fault(strval($error->code), $error->message);
22
-	}
14
+    /**
15
+     * Handles the error by sending a SOAP fault through the server
16
+     *
17
+     * @param WSError Error to handle
18
+     */
19
+    public function handle($error) {
20
+        $server = WSSoapServer::singleton();
21
+        $server->fault(strval($error->code), $error->message);
22
+    }
23 23
 }
24 24
 
25 25
 /**
26 26
  * SOAP server wrapper implementing a Singleton
27 27
  */
28 28
 class WSSoapServer {
29
-	/**
30
-	 * SOAP server instance
31
-	 *
32
-	 * @var soap_server
33
-	 */
34
-	private static $_instance;
29
+    /**
30
+     * SOAP server instance
31
+     *
32
+     * @var soap_server
33
+     */
34
+    private static $_instance;
35 35
 
36
-	/**
37
-	 * Private constructor
38
-	 */
39
-	private function __construct() {
40
-	}
36
+    /**
37
+     * Private constructor
38
+     */
39
+    private function __construct() {
40
+    }
41 41
 
42
-	/**
43
-	 * Singleton method
44
-	 */
45
-	public static function singleton() {
46
-		if(!isset(self::$_instance)) {
47
-			self::$_instance = new soap_server();
48
-			// Set the error handler
49
-			WSError::setErrorHandler(new WSSoapErrorHandler());
50
-			// Configure the service
51
-			self::$_instance->configureWSDL('WSService', 'urn:WSService');
52
-		}
42
+    /**
43
+     * Singleton method
44
+     */
45
+    public static function singleton() {
46
+        if(!isset(self::$_instance)) {
47
+            self::$_instance = new soap_server();
48
+            // Set the error handler
49
+            WSError::setErrorHandler(new WSSoapErrorHandler());
50
+            // Configure the service
51
+            self::$_instance->configureWSDL('WSService', 'urn:WSService');
52
+        }
53 53
 
54
-		return self::$_instance;
55
-	}
54
+        return self::$_instance;
55
+    }
56 56
 }
57 57
 
58 58
 $s = WSSoapServer::singleton();
59 59
 
60 60
 $s->wsdl->addComplexType(
61
-	'result',
62
-	'complexType',
63
-	'struct',
64
-	'all',
65
-	'',
66
-	array(
67
-		'code' => array('name' => 'code', 'type' => 'xsd:int'),
68
-		'message' => array('name' => 'message', 'type' => 'xsd:string')
69
-	)
61
+    'result',
62
+    'complexType',
63
+    'struct',
64
+    'all',
65
+    '',
66
+    array(
67
+        'code' => array('name' => 'code', 'type' => 'xsd:int'),
68
+        'message' => array('name' => 'message', 'type' => 'xsd:string')
69
+    )
70 70
 );
71 71
 
72 72
 $s->wsdl->addComplexType(
73
-  'extras',
74
-  'complexType',
75
-  'struct',
76
-  'all',
77
-  '',
78
-  array(
79
-          'field_name' => array('name' => 'field_name', 'type' => 'xsd:string'),
80
-          'field_value' => array('name' => 'field_value', 'type' => 'xsd:string')
81
-       )
73
+    'extras',
74
+    'complexType',
75
+    'struct',
76
+    'all',
77
+    '',
78
+    array(
79
+            'field_name' => array('name' => 'field_name', 'type' => 'xsd:string'),
80
+            'field_value' => array('name' => 'field_value', 'type' => 'xsd:string')
81
+        )
82 82
 );
83 83
 
84 84
 $s->wsdl->addComplexType(
85
-  'extra_field',
86
-  'complexType',
87
-  'array',
88
-  '',
89
-  'SOAP-ENC:Array',
90
-  array(),
91
-  array(array('ref'=>'SOAP-ENC:arrayType','wsdl:arrayType' => 'tns:extras[]')),'tns:extras'
85
+    'extra_field',
86
+    'complexType',
87
+    'array',
88
+    '',
89
+    'SOAP-ENC:Array',
90
+    array(),
91
+    array(array('ref'=>'SOAP-ENC:arrayType','wsdl:arrayType' => 'tns:extras[]')),'tns:extras'
92 92
 );
93 93
 
94 94
 /*
@@ -106,9 +106,9 @@  discard block
 block discarded – undo
106 106
 */
107 107
 
108 108
 $s->register(
109
-	'WS.test',
110
-	array(),
111
-	array('return' => 'xsd:string')
109
+    'WS.test',
110
+    array(),
111
+    array('return' => 'xsd:string')
112 112
 );
113 113
 
114 114
 require_once(dirname(__FILE__).'/soap_user.php');
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
 	 * Singleton method
44 44
 	 */
45 45
 	public static function singleton() {
46
-		if(!isset(self::$_instance)) {
46
+		if (!isset(self::$_instance)) {
47 47
 			self::$_instance = new soap_server();
48 48
 			// Set the error handler
49 49
 			WSError::setErrorHandler(new WSSoapErrorHandler());
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
   '',
89 89
   'SOAP-ENC:Array',
90 90
   array(),
91
-  array(array('ref'=>'SOAP-ENC:arrayType','wsdl:arrayType' => 'tns:extras[]')),'tns:extras'
91
+  array(array('ref'=>'SOAP-ENC:arrayType', 'wsdl:arrayType' => 'tns:extras[]')), 'tns:extras'
92 92
 );
93 93
 
94 94
 /*
Please login to merge, or discard this patch.
main/webservices/cm_soap_forum.php 1 patch
Indentation   +47 added lines, -47 removed lines patch added patch discarded remove patch
@@ -12,12 +12,12 @@  discard block
 block discarded – undo
12 12
 
13 13
 $s->register(
14 14
         'WSCMForum.get_foruns_id',
15
-	array(
16
-		'username' => 'xsd:string',
17
-		'password' => 'xsd:string',
15
+    array(
16
+        'username' => 'xsd:string',
17
+        'password' => 'xsd:string',
18 18
                 'course_code' => 'xsd:string'
19
-	),
20
-	array('return' => 'xsd:string'),
19
+    ),
20
+    array('return' => 'xsd:string'),
21 21
         'urn:WSCMService',
22 22
         '',
23 23
         '',
@@ -26,14 +26,14 @@  discard block
 block discarded – undo
26 26
 );
27 27
 
28 28
 $s->register(
29
-	'WSCMForum.get_forum_title',
30
-	array(
31
-		'username' => 'xsd:string',
32
-		'password' => 'xsd:string',
29
+    'WSCMForum.get_forum_title',
30
+    array(
31
+        'username' => 'xsd:string',
32
+        'password' => 'xsd:string',
33 33
                 'course_code' => 'xsd:string',
34 34
                 'forum_id' => 'xsd:string'
35
-	),
36
-	array('return' => 'xsd:string'),
35
+    ),
36
+    array('return' => 'xsd:string'),
37 37
         'urn:WSCMService',
38 38
         '',
39 39
         '',
@@ -42,14 +42,14 @@  discard block
 block discarded – undo
42 42
 );
43 43
 
44 44
 $s->register(
45
-	'WSCMForum.get_forum_threads_id',
46
-	array(
47
-		'username' => 'xsd:string',
48
-		'password' => 'xsd:string',
45
+    'WSCMForum.get_forum_threads_id',
46
+    array(
47
+        'username' => 'xsd:string',
48
+        'password' => 'xsd:string',
49 49
                 'course_code' => 'xsd:string',
50 50
                 'forum_id' => 'xsd:string'
51
-	),
52
-	array('return' => 'xsd:string'),
51
+    ),
52
+    array('return' => 'xsd:string'),
53 53
         'urn:WSCMService',
54 54
         '',
55 55
         '',
@@ -58,15 +58,15 @@  discard block
 block discarded – undo
58 58
 );
59 59
 
60 60
 $s->register(
61
-	'WSCMForum.get_forum_thread_data',
62
-	array(
63
-		'username' => 'xsd:string',
64
-		'password' => 'xsd:string',
61
+    'WSCMForum.get_forum_thread_data',
62
+    array(
63
+        'username' => 'xsd:string',
64
+        'password' => 'xsd:string',
65 65
                 'course_code' => 'xsd:string',
66 66
                 'thread_id' => 'xsd:string',
67 67
                 'field' => 'xsd:string'
68
-	),
69
-	array('return' => 'xsd:string'),
68
+    ),
69
+    array('return' => 'xsd:string'),
70 70
         'urn:WSCMService',
71 71
         '',
72 72
         '',
@@ -75,14 +75,14 @@  discard block
 block discarded – undo
75 75
 );
76 76
 
77 77
 $s->register(
78
-	'WSCMForum.get_forum_thread_title',
79
-	array(
80
-		'username' => 'xsd:string',
81
-		'password' => 'xsd:string',
78
+    'WSCMForum.get_forum_thread_title',
79
+    array(
80
+        'username' => 'xsd:string',
81
+        'password' => 'xsd:string',
82 82
                 'course_code' => 'xsd:string',
83 83
                 'thread_id' => 'xsd:string'
84
-	),
85
-	array('return' => 'xsd:string'),
84
+    ),
85
+    array('return' => 'xsd:string'),
86 86
         'urn:WSCMService',
87 87
         '',
88 88
         '',
@@ -92,14 +92,14 @@  discard block
 block discarded – undo
92 92
 
93 93
 
94 94
 $s->register(
95
-	'WSCMForum.get_posts_id',
96
-	array(
97
-		'username' => 'xsd:string',
98
-		'password' => 'xsd:string',
95
+    'WSCMForum.get_posts_id',
96
+    array(
97
+        'username' => 'xsd:string',
98
+        'password' => 'xsd:string',
99 99
                 'course_code' => 'xsd:string',
100 100
                 'thread_id' => 'xsd:string'
101
-	),
102
-	array('return' => 'xsd:string'),
101
+    ),
102
+    array('return' => 'xsd:string'),
103 103
         'urn:WSCMService',
104 104
         '',
105 105
         '',
@@ -108,15 +108,15 @@  discard block
 block discarded – undo
108 108
 );
109 109
 
110 110
 $s->register(
111
-	'WSCMForum.get_post_data',
112
-	array(
113
-		'username' => 'xsd:string',
114
-		'password' => 'xsd:string',
111
+    'WSCMForum.get_post_data',
112
+    array(
113
+        'username' => 'xsd:string',
114
+        'password' => 'xsd:string',
115 115
                 'course_code' => 'xsd:string',
116 116
                 'post_id' => 'xsd:string',
117 117
                 'field' => 'xsd:string'
118
-	),
119
-	array('return' => 'xsd:string'),
118
+    ),
119
+    array('return' => 'xsd:string'),
120 120
         'urn:WSCMService',
121 121
         '',
122 122
         '',
@@ -126,17 +126,17 @@  discard block
 block discarded – undo
126 126
 
127 127
 
128 128
 $s->register(
129
-	'WSCMForum.send_post',
130
-	array(
131
-		'username' => 'xsd:string',
132
-		'password' => 'xsd:string',
129
+    'WSCMForum.send_post',
130
+    array(
131
+        'username' => 'xsd:string',
132
+        'password' => 'xsd:string',
133 133
                 'course_code' => 'xsd:string',
134 134
                 'forum_id' => 'xsd:string',
135 135
                 'thread_id' => 'xsd:string',
136 136
                 'title' => 'xsd:string',
137 137
                 'content' => 'xsd:string'
138
-	),
139
-	array('return' => 'xsd:string'),
138
+    ),
139
+    array('return' => 'xsd:string'),
140 140
         'urn:WSCMService',
141 141
         '',
142 142
         '',
Please login to merge, or discard this patch.
main/webservices/test.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -15,13 +15,13 @@  discard block
 block discarded – undo
15 15
 $script = isset($_POST['script']) ? $_POST['script'] : false;
16 16
 $function = isset($_POST['function']) ? $_POST['function'] : false;
17 17
 
18
-$contact= $server.$script.'?wsdl';
18
+$contact = $server.$script.'?wsdl';
19 19
 
20 20
 $client = new nusoap_client($contact);
21 21
 $err = $client->getError();
22 22
 if ($err) {
23 23
     // Display the error
24
-    echo '<h2>Constructor error</h2><pre>' . $err . '</pre>';
24
+    echo '<h2>Constructor error</h2><pre>'.$err.'</pre>';
25 25
     // At this point, you know the call that follows will fail
26 26
 }
27 27
 $response = array();
@@ -37,8 +37,8 @@  discard block
 block discarded – undo
37 37
 $list = scandir($serversys);
38 38
 $scripts = array();
39 39
 foreach ($list as $item) {
40
-    if (substr($item,0,1) == '.') { continue; }
41
-    if (substr($item,-8)=='soap.php') {
40
+    if (substr($item, 0, 1) == '.') { continue; }
41
+    if (substr($item, -8) == 'soap.php') {
42 42
         $scripts[] = $item;
43 43
     }
44 44
 }
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 ?>
56 56
 </select><br />
57 57
 <label for="function">Function</label>
58
-<input type="text" name="function" value="<?php echo $function;?>"></input><br />
58
+<input type="text" name="function" value="<?php echo $function; ?>"></input><br />
59 59
 <label for="param[0]">Param 0</label>
60 60
 <input type="text" name="param[0]" ></input><br />
61 61
 <input type="submit" name="submit" value="Send"/>
Please login to merge, or discard this patch.
main/webservices/courses_list.soap.php 2 patches
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -28,11 +28,11 @@  discard block
 block discarded – undo
28 28
         'all',
29 29
         '',
30 30
         array(
31
-          'name'=>'code'  , 'type'=>'xsd:string',
32
-          'name'=>'title'  , 'type'=>'xsd:string',
33
-          'name'=>'url'    , 'type'=>'xsd:string',
31
+          'name'=>'code', 'type'=>'xsd:string',
32
+          'name'=>'title', 'type'=>'xsd:string',
33
+          'name'=>'url', 'type'=>'xsd:string',
34 34
           'name'=>'teacher', 'type'=>'xsd:string',
35
-          'name'=>'language','type'=>'xsd:string',
35
+          'name'=>'language', 'type'=>'xsd:string',
36 36
         )
37 37
 );
38 38
 
@@ -51,15 +51,15 @@  discard block
 block discarded – undo
51 51
 );
52 52
 
53 53
 // Register the method to expose
54
-$server->register('WSCourseList',         // method name
54
+$server->register('WSCourseList', // method name
55 55
     array('username' => 'xsd:string',
56 56
           'signature' => 'xsd:string',
57
-          'visibilities' => 'xsd:string'),      // input parameters
58
-    array('return' => 'xsd:Array'),             // output parameters
59
-    'urn:WSCourseList',                         // namespace
60
-    'urn:WSCourseList#WSCourseList',      // soapaction
61
-    'rpc',                                      // style
62
-    'encoded',                                  // use
57
+          'visibilities' => 'xsd:string'), // input parameters
58
+    array('return' => 'xsd:Array'), // output parameters
59
+    'urn:WSCourseList', // namespace
60
+    'urn:WSCourseList#WSCourseList', // soapaction
61
+    'rpc', // style
62
+    'encoded', // use
63 63
     'This service returns a list of courses'    // documentation
64 64
 );
65 65
 
Please login to merge, or discard this patch.
Indentation   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -28,11 +28,11 @@  discard block
 block discarded – undo
28 28
         'all',
29 29
         '',
30 30
         array(
31
-          'name'=>'code'  , 'type'=>'xsd:string',
32
-          'name'=>'title'  , 'type'=>'xsd:string',
33
-          'name'=>'url'    , 'type'=>'xsd:string',
34
-          'name'=>'teacher', 'type'=>'xsd:string',
35
-          'name'=>'language','type'=>'xsd:string',
31
+            'name'=>'code'  , 'type'=>'xsd:string',
32
+            'name'=>'title'  , 'type'=>'xsd:string',
33
+            'name'=>'url'    , 'type'=>'xsd:string',
34
+            'name'=>'teacher', 'type'=>'xsd:string',
35
+            'name'=>'language','type'=>'xsd:string',
36 36
         )
37 37
 );
38 38
 
@@ -53,8 +53,8 @@  discard block
 block discarded – undo
53 53
 // Register the method to expose
54 54
 $server->register('WSCourseList',         // method name
55 55
     array('username' => 'xsd:string',
56
-          'signature' => 'xsd:string',
57
-          'visibilities' => 'xsd:string'),      // input parameters
56
+            'signature' => 'xsd:string',
57
+            'visibilities' => 'xsd:string'),      // input parameters
58 58
     array('return' => 'xsd:Array'),             // output parameters
59 59
     'urn:WSCourseList',                         // namespace
60 60
     'urn:WSCourseList#WSCourseList',      // soapaction
@@ -97,19 +97,19 @@  discard block
 block discarded – undo
97 97
 
98 98
     $courses_list = array();
99 99
 
100
-	if (!is_array($visibilities)) {
101
-		$visibilities = split(',', $visibilities);
102
-	}
103
-	foreach ($visibilities as $visibility) {
104
-		if (!in_array($visibility, array_keys($vis))) {
105
-   			return array('error_msg' => 'Security check failed');
106
-		}
107
-		$courses_list_tmp = CourseManager::get_courses_list(null, null, null, null, $vis[$visibility]);
108
-		foreach ($courses_list_tmp as $index => $course) {
109
-			$course_info = CourseManager::get_course_information($course['code']);
110
-			$courses_list[] = array('code' => $course['code'], 'title' => api_utf8_encode($course_info['title']), 'url' => api_get_path(WEB_COURSE_PATH).$course_info['directory'].'/', 'teacher' => api_utf8_encode($course_info['tutor_name']), 'language' => $course_info['course_language']);
111
-		}
112
-	}
100
+    if (!is_array($visibilities)) {
101
+        $visibilities = split(',', $visibilities);
102
+    }
103
+    foreach ($visibilities as $visibility) {
104
+        if (!in_array($visibility, array_keys($vis))) {
105
+                return array('error_msg' => 'Security check failed');
106
+        }
107
+        $courses_list_tmp = CourseManager::get_courses_list(null, null, null, null, $vis[$visibility]);
108
+        foreach ($courses_list_tmp as $index => $course) {
109
+            $course_info = CourseManager::get_course_information($course['code']);
110
+            $courses_list[] = array('code' => $course['code'], 'title' => api_utf8_encode($course_info['title']), 'url' => api_get_path(WEB_COURSE_PATH).$course_info['directory'].'/', 'teacher' => api_utf8_encode($course_info['tutor_name']), 'language' => $course_info['course_language']);
111
+        }
112
+    }
113 113
     return $courses_list;
114 114
 }
115 115
 
Please login to merge, or discard this patch.
main/webservices/user_import/service.php 2 patches
Indentation   +39 added lines, -39 removed lines patch added patch discarded remove patch
@@ -14,45 +14,45 @@
 block discarded – undo
14 14
  */
15 15
 function import_users_from_file($filepath, $security_key) {
16 16
 
17
-	global $_configuration;
18
-
19
-   	$errors_returned = array(
20
-	   	0 => 'success',
21
-	   	1 => 'file import does not exist',
22
-	   	2 => 'no users to import',
23
-	   	3 => 'wrong datas in file',
24
-	   	4 => 'security error'
25
-   	);
26
-
27
-   	// Check whether this script is launch by server and security key is ok.
28
-   	if (empty($_SERVER['REMOTE_ADDR']) || $_SERVER['REMOTE_ADDR'] != $_SERVER['SERVER_ADDR'] || $security_key != $_configuration['security_key']) {
29
-   		return $errors_returned[4];
30
-   	}
31
-
32
-   	// Libraries
33
-   	require_once 'import.lib.php';
34
-
35
-	// Check is users file exists.
36
-	if (!is_file($filepath)) {
37
-		return $errors_returned[1];
38
-	}
39
-
40
-	// Get list of users
41
-	$users = parse_csv_data($filepath);
42
-	if (count($users) == 0) {
43
-		return $errors_returned[2];
44
-	}
45
-
46
-	// Check the datas for each user
47
-	$errors = validate_data($users);
48
-	if (count($errors) > 0) {
49
-		return $errors_returned[3];
50
-	}
51
-
52
-	// Apply modifications in database
53
-	save_data($users);
54
-
55
-   	return $errors_returned[0]; // Import successfull
17
+    global $_configuration;
18
+
19
+        $errors_returned = array(
20
+            0 => 'success',
21
+            1 => 'file import does not exist',
22
+            2 => 'no users to import',
23
+            3 => 'wrong datas in file',
24
+            4 => 'security error'
25
+        );
26
+
27
+        // Check whether this script is launch by server and security key is ok.
28
+        if (empty($_SERVER['REMOTE_ADDR']) || $_SERVER['REMOTE_ADDR'] != $_SERVER['SERVER_ADDR'] || $security_key != $_configuration['security_key']) {
29
+            return $errors_returned[4];
30
+        }
31
+
32
+        // Libraries
33
+        require_once 'import.lib.php';
34
+
35
+    // Check is users file exists.
36
+    if (!is_file($filepath)) {
37
+        return $errors_returned[1];
38
+    }
39
+
40
+    // Get list of users
41
+    $users = parse_csv_data($filepath);
42
+    if (count($users) == 0) {
43
+        return $errors_returned[2];
44
+    }
45
+
46
+    // Check the datas for each user
47
+    $errors = validate_data($users);
48
+    if (count($errors) > 0) {
49
+        return $errors_returned[3];
50
+    }
51
+
52
+    // Apply modifications in database
53
+    save_data($users);
54
+
55
+        return $errors_returned[0]; // Import successfull
56 56
 }
57 57
 
58 58
 $server = new soap_server();
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -59,5 +59,5 @@
 block discarded – undo
59 59
 
60 60
 $server->register('import_users_from_file');
61 61
 
62
-$http_request = (isset($HTTP_RAW_POST_DATA)?$HTTP_RAW_POST_DATA:'');
62
+$http_request = (isset($HTTP_RAW_POST_DATA) ? $HTTP_RAW_POST_DATA : '');
63 63
 $server->service($http_request);
Please login to merge, or discard this patch.
main/webservices/user_import/import.lib.php 3 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -4,12 +4,12 @@  discard block
 block discarded – undo
4 4
  */
5 5
 function validate_data($users) {
6 6
 	global $defined_auth_sources;
7
-	$errors = array ();
8
-	$usernames = array ();
9
-	if(is_array($users)) {
7
+	$errors = array();
8
+	$usernames = array();
9
+	if (is_array($users)) {
10 10
 		foreach ($users as $index => $user) {
11 11
 			// 1. Check whether mandatory fields have been set.
12
-			$mandatory_fields = array ('LastName', 'FirstName');
12
+			$mandatory_fields = array('LastName', 'FirstName');
13 13
 			if (api_get_setting('registration', 'email') == 'true') {
14 14
 				$mandatory_fields[] = 'Email';
15 15
 			}
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
  */
91 91
 function save_data($users) {
92 92
 	$user_table = Database :: get_main_table(TABLE_MAIN_USER);
93
-	if(is_array($users)) {
93
+	if (is_array($users)) {
94 94
 		foreach ($users as $index => $user) {
95 95
 			$user = complete_missing_data($user);
96 96
 
@@ -98,8 +98,8 @@  discard block
 block discarded – undo
98 98
 
99 99
 			$user_id = UserManager :: create_user($user['FirstName'], $user['LastName'], $user['Status'], $user['Email'], $user['UserName'], $user['Password'], $user['OfficialCode'], api_get_setting('PlatformLanguage'), $user['PhoneNumber'], '', $user['AuthSource']);
100 100
 			foreach ($user['Courses'] as $index => $course) {
101
-				if(CourseManager :: course_exists($course))
102
-					CourseManager :: subscribe_user($user_id, $course,$user['Status']);
101
+				if (CourseManager :: course_exists($course))
102
+					CourseManager :: subscribe_user($user_id, $course, $user['Status']);
103 103
 			}
104 104
 			if (strlen($user['ClassName']) > 0) {
105 105
 				$class_id = ClassManager :: get_class_id($user['ClassName']);
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -98,8 +98,9 @@
 block discarded – undo
98 98
 
99 99
 			$user_id = UserManager :: create_user($user['FirstName'], $user['LastName'], $user['Status'], $user['Email'], $user['UserName'], $user['Password'], $user['OfficialCode'], api_get_setting('PlatformLanguage'), $user['PhoneNumber'], '', $user['AuthSource']);
100 100
 			foreach ($user['Courses'] as $index => $course) {
101
-				if(CourseManager :: course_exists($course))
102
-					CourseManager :: subscribe_user($user_id, $course,$user['Status']);
101
+				if(CourseManager :: course_exists($course)) {
102
+									CourseManager :: subscribe_user($user_id, $course,$user['Status']);
103
+				}
103 104
 			}
104 105
 			if (strlen($user['ClassName']) > 0) {
105 106
 				$class_id = ClassManager :: get_class_id($user['ClassName']);
Please login to merge, or discard this patch.
Indentation   +104 added lines, -104 removed lines patch added patch discarded remove patch
@@ -3,121 +3,121 @@  discard block
 block discarded – undo
3 3
  * Validates imported data.
4 4
  */
5 5
 function validate_data($users) {
6
-	global $defined_auth_sources;
7
-	$errors = array ();
8
-	$usernames = array ();
9
-	if(is_array($users)) {
10
-		foreach ($users as $index => $user) {
11
-			// 1. Check whether mandatory fields have been set.
12
-			$mandatory_fields = array ('LastName', 'FirstName');
13
-			if (api_get_setting('registration', 'email') == 'true') {
14
-				$mandatory_fields[] = 'Email';
15
-			}
16
-			foreach ($mandatory_fields as $key => $field) {
17
-				if (!isset ($user[$field]) || strlen($user[$field]) == 0) {
18
-					$user['error'] = get_lang($field.'Mandatory');
19
-					$errors[] = $user;
20
-				}
21
-			}
22
-			// 2. Check username.
23
-			if (!UserManager::is_username_empty($user['UserName'])) {
24
-				// 2.1. Check whether username was used twice in the import file.
25
-				if (isset($usernames[$user['UserName']])) {
26
-					$user['error'] = get_lang('UserNameUsedTwice');
27
-					$errors[] = $user;
28
-				}
29
-				$usernames[$user['UserName']] = 1;
30
-				// 2.2. Check whether username is allready in use in database.
31
-				if (!UserManager::is_username_available($user['UserName'])) {
32
-					$user['error'] = get_lang('UserNameNotAvailable');
33
-					$errors[] = $user;
34
-				}
35
-				// 2.3. Check whether username is too long.
36
-				if (UserManager::is_username_too_long($user['UserName'])) {
37
-					$user['error'] = get_lang('UserNameTooLong');
38
-					$errors[] = $user;
39
-				}
40
-			}
41
-			// 3. Check status.
42
-			if (isset ($user['Status']) && !api_status_exists($user['Status'])) {
43
-				$user['error'] = get_lang('WrongStatus');
44
-				$errors[] = $user;
45
-			}
46
-			// 5. Check authentication source.
47
-			if (isset ($user['AuthSource']) && strlen($user['AuthSource']) != 0) {
48
-				if (!in_array($user['AuthSource'], $defined_auth_sources)) {
49
-					$user['error'] = get_lang('AuthSourceNotAvailable');
50
-					$errors[] = $user;
51
-				}
52
-			}
53
-		}
54
-	}
55
-	return $errors;
6
+    global $defined_auth_sources;
7
+    $errors = array ();
8
+    $usernames = array ();
9
+    if(is_array($users)) {
10
+        foreach ($users as $index => $user) {
11
+            // 1. Check whether mandatory fields have been set.
12
+            $mandatory_fields = array ('LastName', 'FirstName');
13
+            if (api_get_setting('registration', 'email') == 'true') {
14
+                $mandatory_fields[] = 'Email';
15
+            }
16
+            foreach ($mandatory_fields as $key => $field) {
17
+                if (!isset ($user[$field]) || strlen($user[$field]) == 0) {
18
+                    $user['error'] = get_lang($field.'Mandatory');
19
+                    $errors[] = $user;
20
+                }
21
+            }
22
+            // 2. Check username.
23
+            if (!UserManager::is_username_empty($user['UserName'])) {
24
+                // 2.1. Check whether username was used twice in the import file.
25
+                if (isset($usernames[$user['UserName']])) {
26
+                    $user['error'] = get_lang('UserNameUsedTwice');
27
+                    $errors[] = $user;
28
+                }
29
+                $usernames[$user['UserName']] = 1;
30
+                // 2.2. Check whether username is allready in use in database.
31
+                if (!UserManager::is_username_available($user['UserName'])) {
32
+                    $user['error'] = get_lang('UserNameNotAvailable');
33
+                    $errors[] = $user;
34
+                }
35
+                // 2.3. Check whether username is too long.
36
+                if (UserManager::is_username_too_long($user['UserName'])) {
37
+                    $user['error'] = get_lang('UserNameTooLong');
38
+                    $errors[] = $user;
39
+                }
40
+            }
41
+            // 3. Check status.
42
+            if (isset ($user['Status']) && !api_status_exists($user['Status'])) {
43
+                $user['error'] = get_lang('WrongStatus');
44
+                $errors[] = $user;
45
+            }
46
+            // 5. Check authentication source.
47
+            if (isset ($user['AuthSource']) && strlen($user['AuthSource']) != 0) {
48
+                if (!in_array($user['AuthSource'], $defined_auth_sources)) {
49
+                    $user['error'] = get_lang('AuthSourceNotAvailable');
50
+                    $errors[] = $user;
51
+                }
52
+            }
53
+        }
54
+    }
55
+    return $errors;
56 56
 }
57 57
 
58 58
 /**
59 59
  * Adds missing user-information (which isn't required, like password, username, etc).
60 60
  */
61 61
 function complete_missing_data($user) {
62
-	// 1. Create a username if necessary.
63
-	if (UserManager::is_username_empty($user['UserName'])) {
64
-		$user['UserName'] = UserManager::create_unique_username($user['FirstName'], $user['LastName']);
65
-	}
66
-	// 2. Generate a password if necessary.
67
-	if (!isset ($user['Password']) || strlen($user['Password']) == 0) {
68
-		$user['Password'] = api_generate_password();
69
-	}
70
-	// 3. set status if not allready set.
71
-	if (!isset ($user['Status']) || strlen($user['Status']) == 0) {
72
-		$user['Status'] = 'user';
73
-	}
74
-	// 4. Set authsource if not allready set.
75
-	if (!isset ($user['AuthSource']) || strlen($user['AuthSource']) == 0) {
76
-		$user['AuthSource'] = PLATFORM_AUTH_SOURCE;
77
-	}
78
-	return $user;
62
+    // 1. Create a username if necessary.
63
+    if (UserManager::is_username_empty($user['UserName'])) {
64
+        $user['UserName'] = UserManager::create_unique_username($user['FirstName'], $user['LastName']);
65
+    }
66
+    // 2. Generate a password if necessary.
67
+    if (!isset ($user['Password']) || strlen($user['Password']) == 0) {
68
+        $user['Password'] = api_generate_password();
69
+    }
70
+    // 3. set status if not allready set.
71
+    if (!isset ($user['Status']) || strlen($user['Status']) == 0) {
72
+        $user['Status'] = 'user';
73
+    }
74
+    // 4. Set authsource if not allready set.
75
+    if (!isset ($user['AuthSource']) || strlen($user['AuthSource']) == 0) {
76
+        $user['AuthSource'] = PLATFORM_AUTH_SOURCE;
77
+    }
78
+    return $user;
79 79
 }
80 80
 
81 81
 /**
82 82
  * Save the imported data
83 83
  */
84 84
 function save_data($users) {
85
-	$user_table = Database :: get_main_table(TABLE_MAIN_USER);
86
-	if(is_array($users)) {
87
-		foreach ($users as $index => $user) {
88
-			$user = complete_missing_data($user);
85
+    $user_table = Database :: get_main_table(TABLE_MAIN_USER);
86
+    if(is_array($users)) {
87
+        foreach ($users as $index => $user) {
88
+            $user = complete_missing_data($user);
89 89
 
90
-			$user['Status'] = api_status_key($user['Status']);
90
+            $user['Status'] = api_status_key($user['Status']);
91 91
 
92
-			$user_id = UserManager :: create_user($user['FirstName'], $user['LastName'], $user['Status'], $user['Email'], $user['UserName'], $user['Password'], $user['OfficialCode'], api_get_setting('PlatformLanguage'), $user['PhoneNumber'], '', $user['AuthSource']);
93
-			foreach ($user['Courses'] as $index => $course) {
94
-				if(CourseManager :: course_exists($course))
95
-					CourseManager :: subscribe_user($user_id, $course,$user['Status']);
96
-			}
92
+            $user_id = UserManager :: create_user($user['FirstName'], $user['LastName'], $user['Status'], $user['Email'], $user['UserName'], $user['Password'], $user['OfficialCode'], api_get_setting('PlatformLanguage'), $user['PhoneNumber'], '', $user['AuthSource']);
93
+            foreach ($user['Courses'] as $index => $course) {
94
+                if(CourseManager :: course_exists($course))
95
+                    CourseManager :: subscribe_user($user_id, $course,$user['Status']);
96
+            }
97 97
 
98
-			// TODO: Hard-coded French texts.
98
+            // TODO: Hard-coded French texts.
99 99
 
100
-			// Qualite
101
-			if (!empty($user['Qualite'])) {
102
-				UserManager::update_extra_field_value($user_id, 'qualite', $user['Qualite']);
103
-			}
100
+            // Qualite
101
+            if (!empty($user['Qualite'])) {
102
+                UserManager::update_extra_field_value($user_id, 'qualite', $user['Qualite']);
103
+            }
104 104
 
105
-			// Categorie
106
-			if (!empty($user['Categorie'])) {
107
-				UserManager::update_extra_field_value($user_id, 'categorie', $user['Categorie']);
108
-			}
105
+            // Categorie
106
+            if (!empty($user['Categorie'])) {
107
+                UserManager::update_extra_field_value($user_id, 'categorie', $user['Categorie']);
108
+            }
109 109
 
110
-			// Etat
111
-			if (!empty($user['Etat'])) {
112
-				UserManager::update_extra_field_value($user_id, 'etat', $user['Etat']);
113
-			}
110
+            // Etat
111
+            if (!empty($user['Etat'])) {
112
+                UserManager::update_extra_field_value($user_id, 'etat', $user['Etat']);
113
+            }
114 114
 
115
-			// Niveau
116
-			if (!empty($user['Niveau'])) {
117
-				UserManager::update_extra_field_value($user_id, 'niveau', $user['Niveau']);
118
-			}
119
-		}
120
-	}
115
+            // Niveau
116
+            if (!empty($user['Niveau'])) {
117
+                UserManager::update_extra_field_value($user_id, 'niveau', $user['Niveau']);
118
+            }
119
+        }
120
+    }
121 121
 }
122 122
 
123 123
 /**
@@ -126,12 +126,12 @@  discard block
 block discarded – undo
126 126
  * @return array All userinformation read from the file
127 127
  */
128 128
 function parse_csv_data($file) {
129
-	$users = Import :: csvToArray($file);
130
-	foreach ($users as $index => $user) {
131
-		if (isset ($user['Courses'])) {
132
-			$user['Courses'] = explode('|', trim($user['Courses']));
133
-		}
134
-		$users[$index] = $user;
135
-	}
136
-	return $users;
129
+    $users = Import :: csvToArray($file);
130
+    foreach ($users as $index => $user) {
131
+        if (isset ($user['Courses'])) {
132
+            $user['Courses'] = explode('|', trim($user['Courses']));
133
+        }
134
+        $users[$index] = $user;
135
+    }
136
+    return $users;
137 137
 }
Please login to merge, or discard this patch.
main/webservices/cm_soap_courses.php 1 patch
Indentation   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -11,12 +11,12 @@  discard block
 block discarded – undo
11 11
 
12 12
 
13 13
 $s->register(
14
-	'WSCMCourses.get_courses_code',
15
-	array(
16
-		'username' => 'xsd:string',
17
-		'password' => 'xsd:string'
18
-	),
19
-	array('return' => 'xsd:string'),
14
+    'WSCMCourses.get_courses_code',
15
+    array(
16
+        'username' => 'xsd:string',
17
+        'password' => 'xsd:string'
18
+    ),
19
+    array('return' => 'xsd:string'),
20 20
         'urn:WSCMService',
21 21
         '',
22 22
         '',
@@ -26,13 +26,13 @@  discard block
 block discarded – undo
26 26
 );
27 27
 
28 28
 $s->register(
29
-	'WSCMCourses.get_course_title',
30
-	array(
31
-		'username' => 'xsd:string',
32
-		'password' => 'xsd:string',
29
+    'WSCMCourses.get_course_title',
30
+    array(
31
+        'username' => 'xsd:string',
32
+        'password' => 'xsd:string',
33 33
                 'course_code' => 'xsd:string',
34
-	),
35
-	array('return' => 'xsd:string'),
34
+    ),
35
+    array('return' => 'xsd:string'),
36 36
         'urn:WSCMService',
37 37
         '',
38 38
         '',
Please login to merge, or discard this patch.
main/webservices/cm_webservice_announcements.php 2 patches
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
 
16 16
     public function get_announcements_id($username, $password, $course_code)
17 17
     {
18
-        if($this->verifyUserPass($username, $password) == "valid")
18
+        if ($this->verifyUserPass($username, $password) == "valid")
19 19
         {
20 20
 
21 21
             $result = self::get_announcements($username, $course_code);
@@ -33,13 +33,13 @@  discard block
 block discarded – undo
33 33
 
34 34
     public function get_announcement_data($username, $password, $course_code, $announcement_id, $field)
35 35
     {
36
-        if($this->verifyUserPass($username, $password) == "valid")
36
+        if ($this->verifyUserPass($username, $password) == "valid")
37 37
         {
38 38
             $htmlcode = false;
39 39
             $user_id = UserManager::get_user_id_from_username($username);
40 40
 
41 41
             $result = self::get_announcements($username, $course_code, $announcement_id);
42
-            while($announcement = Database::fetch_array($result))
42
+            while ($announcement = Database::fetch_array($result))
43 43
             {
44 44
                 $announcements[] = $announcement;
45 45
             }
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 
70 70
             return (htmlcode) ? html_entity_decode($announcements[0][$field_table]) : $announcements[0][$field_table];
71 71
 
72
-        }else
72
+        } else
73 73
             return get_lang('InvalidId');
74 74
     }
75 75
 
@@ -82,11 +82,11 @@  discard block
 block discarded – undo
82 82
         $announcement_id = ($announcement_id == 0) ? "" : "AND announcement.id=".$announcement_id;
83 83
         $user_id = UserManager::get_user_id_from_username($username);
84 84
         $course_info = CourseManager::get_course_information($course_code);
85
-        $tbl_item_property  	= Database::get_course_table(TABLE_ITEM_PROPERTY);
86
-        $tbl_announcement		= Database::get_course_table(TABLE_ANNOUNCEMENT);
87
-        $maximum 	= '12';
85
+        $tbl_item_property = Database::get_course_table(TABLE_ITEM_PROPERTY);
86
+        $tbl_announcement = Database::get_course_table(TABLE_ANNOUNCEMENT);
87
+        $maximum = '12';
88 88
 
89
-        $group_memberships=GroupManager::get_group_ids($course_info['real_id'], $user_id);
89
+        $group_memberships = GroupManager::get_group_ids($course_info['real_id'], $user_id);
90 90
 
91 91
         if (api_get_group_id() == 0) {
92 92
             $cond_user_id = " AND (
@@ -104,8 +104,8 @@  discard block
 block discarded – undo
104 104
 
105 105
         // the user is member of several groups => display personal
106 106
         // announcements AND his group announcements AND the general announcements
107
-        if (is_array($group_memberships) && count($group_memberships)>0) {
108
-            $sql="SELECT
107
+        if (is_array($group_memberships) && count($group_memberships) > 0) {
108
+            $sql = "SELECT
109 109
                     announcement.*, ip.visibility, ip.to_group_id, ip.insert_user_id
110 110
                 FROM $tbl_announcement announcement, $tbl_item_property ip
111 111
                 WHERE
Please login to merge, or discard this patch.
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -26,8 +26,9 @@  discard block
 block discarded – undo
26 26
             }
27 27
             return $announcements;
28 28
 
29
-        } else
30
-            return get_lang('InvalidId');
29
+        } else {
30
+                    return get_lang('InvalidId');
31
+        }
31 32
 
32 33
     }
33 34
 
@@ -69,8 +70,9 @@  discard block
 block discarded – undo
69 70
 
70 71
             return (htmlcode) ? html_entity_decode($announcements[0][$field_table]) : $announcements[0][$field_table];
71 72
 
72
-        }else
73
-            return get_lang('InvalidId');
73
+        } else {
74
+                    return get_lang('InvalidId');
75
+        }
74 76
     }
75 77
 
76 78
 
Please login to merge, or discard this patch.