chamilo / chamilo-lms

main/auth/cas/lib/CAS.php

Doc Comments +4 added lines, -4 removed lines

@@ -523,7 +523,7 @@ 
     /**

      * Logs a string in debug mode.

      *

-     * @param $str the string to write

+     * @param string $str the string to write

      *

      * @private

      */

@@ -545,7 +545,7 @@ 
      * This method is used by interface methods to print an error and where the function

      * was originally called from.

      *

-     * @param $msg the message to print

+     * @param string $msg the message to print

      *

      * @private

      */

@@ -678,7 +678,7 @@ 
     /**

      * This method returns the phpCAS version.

      *

-     * @return the phpCAS version.

+     * @return string phpCAS version.

      */

     function getVersion()

     {

@@ -1105,7 +1105,7 @@ 
      * @warning should not be called only after phpCAS::forceAuthentication()

      * or phpCAS::checkAuthentication().

      *

-     * @return the login name of the authenticated user

+     * @return string login name of the authenticated user

      */

     function getUser()

     {

Spacing +73 added lines, -73 removed lines

@@ -33,14 +33,14 @@ 
 // hack by Vangelis Haniotakis to handle the absence of $_SERVER['REQUEST_URI'] in IIS

 //

 if (!$_SERVER['REQUEST_URI']) {

-    $_SERVER['REQUEST_URI'] = $_SERVER['SCRIPT_NAME'] . '?' . $_SERVER['QUERY_STRING'];

+    $_SERVER['REQUEST_URI'] = $_SERVER['SCRIPT_NAME'].'?'.$_SERVER['QUERY_STRING'];

 }

 

 //

 // another one by Vangelis Haniotakis also to make phpCAS work with PHP5

 //

 if (version_compare(PHP_VERSION, '5', '>=')) {

-    require_once(dirname(__FILE__) . '/CAS/domxml-php4-to-php5.php');

+    require_once(dirname(__FILE__).'/CAS/domxml-php4-to-php5.php');

 }

 

 /**

@@ -322,7 +322,7 @@ 
 // ########################################################################

 

 // include client class

-include_once(dirname(__FILE__) . '/CAS/client.php');

+include_once(dirname(__FILE__).'/CAS/client.php');

 

 // ########################################################################

 //  INTERFACE CLASS

@@ -371,7 +371,7 @@ 
 

         phpCAS:: traceBegin();

         if (is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error($PHPCAS_INIT_CALL['method'] . '() has already been called (at ' . $PHPCAS_INIT_CALL['file'] . ':' . $PHPCAS_INIT_CALL['line'] . ')');

+            phpCAS:: error($PHPCAS_INIT_CALL['method'].'() has already been called (at '.$PHPCAS_INIT_CALL['file'].':'.$PHPCAS_INIT_CALL['line'].')');

         }

         if (gettype($server_version) != 'string') {

             phpCAS:: error('type mismatched for parameter $server_version (should be `string\')');

@@ -392,7 +392,7 @@ 
             'done' => true,

             'file' => $dbg[0]['file'],

             'line' => $dbg[0]['line'],

-            'method' => __CLASS__ . '::' . __FUNCTION__

+            'method' => __CLASS__.'::'.__FUNCTION__

         );

 

         // initialize the global object $PHPCAS_CLIENT

@@ -421,7 +421,7 @@ 
 

         phpCAS:: traceBegin();

         if (is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error($PHPCAS_INIT_CALL['method'] . '() has already been called (at ' . $PHPCAS_INIT_CALL['file'] . ':' . $PHPCAS_INIT_CALL['line'] . ')');

+            phpCAS:: error($PHPCAS_INIT_CALL['method'].'() has already been called (at '.$PHPCAS_INIT_CALL['file'].':'.$PHPCAS_INIT_CALL['line'].')');

         }

         if (gettype($server_version) != 'string') {

             phpCAS:: error('type mismatched for parameter $server_version (should be `string\')');

@@ -442,7 +442,7 @@ 
             'done' => true,

             'file' => $dbg[0]['file'],

             'line' => $dbg[0]['line'],

-            'method' => __CLASS__ . '::' . __FUNCTION__

+            'method' => __CLASS__.'::'.__FUNCTION__

         );

 

         // initialize the global object $PHPCAS_CLIENT

@@ -477,10 +477,10 @@ 
         if (empty ($filename)) {

             if (preg_match('/^Win.*/', getenv('OS'))) {

                 if (isset ($_ENV['TMP'])) {

-                    $debugDir = $_ENV['TMP'] . '/';

+                    $debugDir = $_ENV['TMP'].'/';

                 } else {

                     if (isset ($_ENV['TEMP'])) {

-                        $debugDir = $_ENV['TEMP'] . '/';

+                        $debugDir = $_ENV['TEMP'].'/';

                     } else {

                         $debugDir = '';

                     }

@@ -488,7 +488,7 @@ 
             } else {

                 $debugDir = DEFAULT_DEBUG_DIR;

             }

-            $filename = $debugDir . 'phpCAS.log';

+            $filename = $debugDir.'phpCAS.log';

         }

 

         if (empty ($PHPCAS_DEBUG['unique_id'])) {

@@ -536,7 +536,7 @@ 
             for ($i = 0; $i < $PHPCAS_DEBUG['indent']; $i++) {

                 $indent_str .= '|    ';

             }

-            error_log($PHPCAS_DEBUG['unique_id'] . ' ' . $indent_str . $str . "\n", 3, $PHPCAS_DEBUG['filename']);

+            error_log($PHPCAS_DEBUG['unique_id'].' '.$indent_str.$str."\n", 3, $PHPCAS_DEBUG['filename']);

         }

 

     }

@@ -566,7 +566,7 @@ 
                 }

             }

         }

-        echo "<br />\n<b>phpCAS error</b>: <font color=\"FF0000\"><b>" . __CLASS__ . "::" . $function . '(): ' . htmlentities($msg) . "</b></font> in <b>" . $file . "</b> on line <b>" . $line . "</b><br />\n";

+        echo "<br />\n<b>phpCAS error</b>: <font color=\"FF0000\"><b>".__CLASS__."::".$function.'(): '.htmlentities($msg)."</b></font> in <b>".$file."</b> on line <b>".$line."</b><br />\n";

         phpCAS:: trace($msg);

         phpCAS:: traceExit();

         exit ();

@@ -578,7 +578,7 @@ 
     function trace($str)

     {

         $dbg = phpCAS:: backtrace();

-        phpCAS:: log($str . ' [' . basename($dbg[1]['file']) . ':' . $dbg[1]['line'] . ']');

+        phpCAS:: log($str.' ['.basename($dbg[1]['file']).':'.$dbg[1]['line'].']');

     }

 

     /**

@@ -591,9 +591,9 @@ 
         $dbg = phpCAS:: backtrace();

         $str = '=> ';

         if (!empty ($dbg[2]['class'])) {

-            $str .= $dbg[2]['class'] . '::';

+            $str .= $dbg[2]['class'].'::';

         }

-        $str .= $dbg[2]['function'] . '(';

+        $str .= $dbg[2]['function'].'(';

         if (is_array($dbg[2]['args'])) {

             foreach ($dbg[2]['args'] as $index => $arg) {

                 if ($index != 0) {

@@ -602,7 +602,7 @@ 
                 $str .= str_replace("\n", "", var_export($arg, true));

             }

         }

-        $str .= ') [' . basename($dbg[2]['file']) . ':' . $dbg[2]['line'] . ']';

+        $str .= ') ['.basename($dbg[2]['file']).':'.$dbg[2]['line'].']';

         phpCAS:: log($str);

         $PHPCAS_DEBUG['indent']++;

     }

@@ -619,7 +619,7 @@ 
         $PHPCAS_DEBUG['indent']--;

         $dbg = phpCAS:: backtrace();

         $str = '';

-        $str .= '<= ' . str_replace("\n", "", var_export($res, true));

+        $str .= '<= '.str_replace("\n", "", var_export($res, true));

         phpCAS:: log($str);

     }

 

@@ -658,7 +658,7 @@ 
     {

         global $PHPCAS_CLIENT;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

         if (gettype($lang) != 'string') {

             phpCAS:: error('type mismatched for parameter $lang (should be `string\')');

@@ -703,7 +703,7 @@ 
     {

         global $PHPCAS_CLIENT;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

         if (gettype($header) != 'string') {

             phpCAS:: error('type mismatched for parameter $header (should be `string\')');

@@ -720,7 +720,7 @@ 
     {

         global $PHPCAS_CLIENT;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

         if (gettype($footer) != 'string') {

             phpCAS:: error('type mismatched for parameter $footer (should be `string\')');

@@ -750,13 +750,13 @@ 
 

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if (!$PHPCAS_CLIENT->isProxy()) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if ($PHPCAS_AUTH_CHECK_CALL['done']) {

-            phpCAS:: error('this method should only be called before ' . $PHPCAS_AUTH_CHECK_CALL['method'] . '() (called at ' . $PHPCAS_AUTH_CHECK_CALL['file'] . ':' . $PHPCAS_AUTH_CHECK_CALL['line'] . ')');

+            phpCAS:: error('this method should only be called before '.$PHPCAS_AUTH_CHECK_CALL['method'].'() (called at '.$PHPCAS_AUTH_CHECK_CALL['file'].':'.$PHPCAS_AUTH_CHECK_CALL['line'].')');

         }

         if (gettype($format) != 'string') {

             phpCAS:: error('type mismatched for parameter $format (should be `string\')');

@@ -796,13 +796,13 @@ 
 

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if (!$PHPCAS_CLIENT->isProxy()) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if ($PHPCAS_AUTH_CHECK_CALL['done']) {

-            phpCAS:: error('this method should only be called before ' . $PHPCAS_AUTH_CHECK_CALL['method'] . '() (called at ' . $PHPCAS_AUTH_CHECK_CALL['file'] . ':' . $PHPCAS_AUTH_CHECK_CALL['line'] . ')');

+            phpCAS:: error('this method should only be called before '.$PHPCAS_AUTH_CHECK_CALL['method'].'() (called at '.$PHPCAS_AUTH_CHECK_CALL['file'].':'.$PHPCAS_AUTH_CHECK_CALL['line'].')');

         }

         if (gettype($user) != 'string') {

             phpCAS:: error('type mismatched for parameter $user (should be `string\')');

@@ -857,16 +857,16 @@ 
 

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if (!$PHPCAS_CLIENT->isProxy()) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if (!$PHPCAS_AUTH_CHECK_CALL['done']) {

-            phpCAS:: error('this method should only be called after the programmer is sure the user has been authenticated (by calling ' . __CLASS__ . '::checkAuthentication() or ' . __CLASS__ . '::forceAuthentication()');

+            phpCAS:: error('this method should only be called after the programmer is sure the user has been authenticated (by calling '.__CLASS__.'::checkAuthentication() or '.__CLASS__.'::forceAuthentication()');

         }

         if (!$PHPCAS_AUTH_CHECK_CALL['result']) {

-            phpCAS:: error('authentication was checked (by ' . $PHPCAS_AUTH_CHECK_CALL['method'] . '() at ' . $PHPCAS_AUTH_CHECK_CALL['file'] . ':' . $PHPCAS_AUTH_CHECK_CALL['line'] . ') but the method returned FALSE');

+            phpCAS:: error('authentication was checked (by '.$PHPCAS_AUTH_CHECK_CALL['method'].'() at '.$PHPCAS_AUTH_CHECK_CALL['file'].':'.$PHPCAS_AUTH_CHECK_CALL['line'].') but the method returned FALSE');

         }

         if (gettype($url) != 'string') {

             phpCAS:: error('type mismatched for parameter $url (should be `string\')');

@@ -901,16 +901,16 @@ 
 

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if (!$PHPCAS_CLIENT->isProxy()) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if (!$PHPCAS_AUTH_CHECK_CALL['done']) {

-            phpCAS:: error('this method should only be called after the programmer is sure the user has been authenticated (by calling ' . __CLASS__ . '::checkAuthentication() or ' . __CLASS__ . '::forceAuthentication()');

+            phpCAS:: error('this method should only be called after the programmer is sure the user has been authenticated (by calling '.__CLASS__.'::checkAuthentication() or '.__CLASS__.'::forceAuthentication()');

         }

         if (!$PHPCAS_AUTH_CHECK_CALL['result']) {

-            phpCAS:: error('authentication was checked (by ' . $PHPCAS_AUTH_CHECK_CALL['method'] . '() at ' . $PHPCAS_AUTH_CHECK_CALL['file'] . ':' . $PHPCAS_AUTH_CHECK_CALL['line'] . ') but the method returned FALSE');

+            phpCAS:: error('authentication was checked (by '.$PHPCAS_AUTH_CHECK_CALL['method'].'() at '.$PHPCAS_AUTH_CHECK_CALL['file'].':'.$PHPCAS_AUTH_CHECK_CALL['line'].') but the method returned FALSE');

         }

         if (gettype($url) != 'string') {

             phpCAS:: error('type mismatched for parameter $url (should be `string\')');

@@ -947,7 +947,7 @@ 
     {

         global $PHPCAS_CLIENT;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

         if (gettype($n) != 'integer') {

             phpCAS:: error('type mismatched for parameter $header (should be `string\')');

@@ -965,7 +965,7 @@ 
 

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

 

         $auth = $PHPCAS_CLIENT->checkAuthentication();

@@ -976,7 +976,7 @@ 
             'done' => true,

             'file' => $dbg[0]['file'],

             'line' => $dbg[0]['line'],

-            'method' => __CLASS__ . '::' . __FUNCTION__,

+            'method' => __CLASS__.'::'.__FUNCTION__,

             'result' => $auth

         );

         phpCAS:: traceEnd($auth);

@@ -994,7 +994,7 @@ 
 

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

 

         $auth = $PHPCAS_CLIENT->forceAuthentication();

@@ -1005,7 +1005,7 @@ 
             'done' => true,

             'file' => $dbg[0]['file'],

             'line' => $dbg[0]['line'],

-            'method' => __CLASS__ . '::' . __FUNCTION__,

+            'method' => __CLASS__.'::'.__FUNCTION__,

             'result' => $auth

         );

 

@@ -1013,7 +1013,7 @@ 
             phpCAS:: trace('user is not authenticated, redirecting to the CAS server');

             $PHPCAS_CLIENT->forceAuthentication();

         } else {

-            phpCAS:: trace('no need to authenticate (user `' . phpCAS:: getUser() . '\' is already authenticated)');

+            phpCAS:: trace('no need to authenticate (user `'.phpCAS:: getUser().'\' is already authenticated)');

         }

 

         phpCAS:: traceEnd();

@@ -1029,7 +1029,7 @@ 
 

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before'.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

 

         // store where the authentication has been checked and the result

@@ -1038,7 +1038,7 @@ 
             'done' => true,

             'file' => $dbg[0]['file'],

             'line' => $dbg[0]['line'],

-            'method' => __CLASS__ . '::' . __FUNCTION__,

+            'method' => __CLASS__.'::'.__FUNCTION__,

             'result' => $auth

         );

 

@@ -1051,7 +1051,7 @@ 
      */

     function authenticate()

     {

-        phpCAS:: error('this method is deprecated. You should use ' . __CLASS__ . '::forceAuthentication() instead');

+        phpCAS:: error('this method is deprecated. You should use '.__CLASS__.'::forceAuthentication() instead');

     }

 

     /**

@@ -1066,7 +1066,7 @@ 
 

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

 

         // call the isAuthenticated method of the global $PHPCAS_CLIENT object

@@ -1078,7 +1078,7 @@ 
             'done' => true,

             'file' => $dbg[0]['file'],

             'line' => $dbg[0]['line'],

-            'method' => __CLASS__ . '::' . __FUNCTION__,

+            'method' => __CLASS__.'::'.__FUNCTION__,

             'result' => $auth

         );

         phpCAS:: traceEnd($auth);

@@ -1095,7 +1095,7 @@ 
     {

         global $PHPCAS_CLIENT;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

         return ($PHPCAS_CLIENT->isSessionAuthenticated());

     }

@@ -1111,13 +1111,13 @@ 
     {

         global $PHPCAS_CLIENT, $PHPCAS_AUTH_CHECK_CALL;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

         if (!$PHPCAS_AUTH_CHECK_CALL['done']) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::forceAuthentication() or ' . __CLASS__ . '::isAuthenticated()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::forceAuthentication() or '.__CLASS__.'::isAuthenticated()');

         }

         if (!$PHPCAS_AUTH_CHECK_CALL['result']) {

-            phpCAS:: error('authentication was checked (by ' . $PHPCAS_AUTH_CHECK_CALL['method'] . '() at ' . $PHPCAS_AUTH_CHECK_CALL['file'] . ':' . $PHPCAS_AUTH_CHECK_CALL['line'] . ') but the method returned FALSE');

+            phpCAS:: error('authentication was checked (by '.$PHPCAS_AUTH_CHECK_CALL['method'].'() at '.$PHPCAS_AUTH_CHECK_CALL['file'].':'.$PHPCAS_AUTH_CHECK_CALL['line'].') but the method returned FALSE');

         }

         return $PHPCAS_CLIENT->getUser();

     }

@@ -1133,13 +1133,13 @@ 
     {

         global $PHPCAS_CLIENT, $PHPCAS_AUTH_CHECK_CALL;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

         if (!$PHPCAS_AUTH_CHECK_CALL['done']) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::forceAuthentication() or ' . __CLASS__ . '::isAuthenticated()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::forceAuthentication() or '.__CLASS__.'::isAuthenticated()');

         }

         if (!$PHPCAS_AUTH_CHECK_CALL['result']) {

-            phpCAS:: error('authentication was checked (by ' . $PHPCAS_AUTH_CHECK_CALL['method'] . '() at ' . $PHPCAS_AUTH_CHECK_CALL['file'] . ':' . $PHPCAS_AUTH_CHECK_CALL['line'] . ') but the method returned FALSE');

+            phpCAS:: error('authentication was checked (by '.$PHPCAS_AUTH_CHECK_CALL['method'].'() at '.$PHPCAS_AUTH_CHECK_CALL['file'].':'.$PHPCAS_AUTH_CHECK_CALL['line'].') but the method returned FALSE');

         }

         return $PHPCAS_CLIENT->getAttributes();

     }

@@ -1151,7 +1151,7 @@ 
     {

         global $PHPCAS_CLIENT;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

         return ($PHPCAS_CLIENT->handleLogoutRequests($check_client, $allowed_clients));

     }

@@ -1166,7 +1166,7 @@ 
     {

         global $PHPCAS_CLIENT;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

         return $PHPCAS_CLIENT->getServerLoginURL();

     }

@@ -1182,7 +1182,7 @@ 
         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

             phpCAS:: error('this method should only be called after

-							' . __CLASS__ . '::client()');

+							' . __CLASS__.'::client()');

         }

         if (gettype($url) != 'string') {

             phpCAS:: error('type mismatched for parameter $url (should be

@@ -1204,7 +1204,7 @@ 
         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

             phpCAS:: error('this method should only be called after

-							' . __CLASS__ . '::client()');

+							' . __CLASS__.'::client()');

         }

         if (gettype($url) != 'string') {

             phpCAS:: error('type mismatched for parameter $url (should be

@@ -1226,7 +1226,7 @@ 
         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

             phpCAS:: error('this method should only be called after

-							' . __CLASS__ . '::client()');

+							' . __CLASS__.'::client()');

         }

         if (gettype($url) != 'string') {

             phpCAS:: error('type mismatched for parameter $url (should be

@@ -1247,7 +1247,7 @@ 
         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

             phpCAS:: error('this method should only be called after

-							' . __CLASS__ . '::client()');

+							' . __CLASS__.'::client()');

         }

         if (gettype($url) != 'string') {

             phpCAS:: error('type mismatched for parameter $url (should be

@@ -1267,7 +1267,7 @@ 
     {

         global $PHPCAS_CLIENT;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should not be called before '.__CLASS__.'::client() or '.__CLASS__.'::proxy()');

         }

         return $PHPCAS_CLIENT->getServerLogoutURL();

     }

@@ -1283,7 +1283,7 @@ 
         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

             phpCAS:: error('this method should only be called after

-							' . __CLASS__ . '::client()');

+							' . __CLASS__.'::client()');

         }

         if (gettype($url) != 'string') {

             phpCAS:: error('type mismatched for parameter $url (should be

@@ -1303,7 +1303,7 @@ 
         global $PHPCAS_CLIENT;

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::client() or' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::client() or'.__CLASS__.'::proxy()');

         }

         $parsedParams = array();

         if ($params != "") {

@@ -1334,7 +1334,7 @@ 
         global $PHPCAS_CLIENT;

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::client() or' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::client() or'.__CLASS__.'::proxy()');

         }

         if (!is_string($service)) {

             phpCAS:: error('type mismatched for parameter $service (should be `string\')');

@@ -1355,7 +1355,7 @@ 
         global $PHPCAS_CLIENT;

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::client() or' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::client() or'.__CLASS__.'::proxy()');

         }

         if (!is_string($url)) {

             phpCAS:: error('type mismatched for parameter $url (should be `string\')');

@@ -1377,7 +1377,7 @@ 
         global $PHPCAS_CLIENT;

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::client() or' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::client() or'.__CLASS__.'::proxy()');

         }

         if (!is_string($service)) {

             phpCAS:: error('type mismatched for parameter $service (should be `string\')');

@@ -1404,10 +1404,10 @@ 
         global $PHPCAS_CLIENT;

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if (!$PHPCAS_CLIENT->isProxy()) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if (gettype($url) != 'string') {

             phpCAS:: error('type mismatched for parameter $url (should be `string\')');

@@ -1427,7 +1427,7 @@ 
         global $PHPCAS_CLIENT;

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if (gettype($url) != 'string') {

             phpCAS:: error('type mismatched for parameter $url (should be `string\')');

@@ -1443,7 +1443,7 @@ 
     {

         global $PHPCAS_CLIENT;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         return ($PHPCAS_CLIENT->getURL());

     }

@@ -1455,7 +1455,7 @@ 
     {

         global $PHPCAS_CLIENT;

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::proxy()');

         }

         if (gettype($target_service) != 'string') {

             phpCAS:: error('type mismatched for parameter $target_service(should be `string\')');

@@ -1473,7 +1473,7 @@ 
         global $PHPCAS_CLIENT;

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::client() or' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::client() or'.__CLASS__.'::proxy()');

         }

         if (gettype($cert) != 'string') {

             phpCAS:: error('type mismatched for parameter $cert (should be `string\')');

@@ -1492,7 +1492,7 @@ 
         global $PHPCAS_CLIENT;

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::client() or' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::client() or'.__CLASS__.'::proxy()');

         }

         if (gettype($cert) != 'string') {

             phpCAS:: error('type mismatched for parameter $cert (should be `string\')');

@@ -1509,7 +1509,7 @@ 
         global $PHPCAS_CLIENT;

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::client() or' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::client() or'.__CLASS__.'::proxy()');

         }

         $PHPCAS_CLIENT->setNoCasServerValidation();

         phpCAS:: traceEnd();

@@ -1528,7 +1528,7 @@ 
         global $PHPCAS_CLIENT;

         phpCAS:: traceBegin();

         if (!is_object($PHPCAS_CLIENT)) {

-            phpCAS:: error('this method should only be called after ' . __CLASS__ . '::client() or' . __CLASS__ . '::proxy()');

+            phpCAS:: error('this method should only be called after '.__CLASS__.'::client() or'.__CLASS__.'::proxy()');

         }

         $PHPCAS_CLIENT->setExtraCurlOption($key, $value);

         phpCAS:: traceEnd();

main/inc/ajax/sequence.ajax.php

Spacing +4 added lines, -4 removed lines

@@ -44,7 +44,7 @@ 
                     try {
                         $graphImage = $graphviz->createImageHtml($graph);
                     } catch (UnexpectedValueException $e) {
-                        error_log($e->getMessage() . ' - Graph could not be rendered in resources sequence because GraphViz command "dot" could not be executed - Make sure graphviz is installed.');
+                        error_log($e->getMessage().' - Graph could not be rendered in resources sequence because GraphViz command "dot" could not be executed - Make sure graphviz is installed.');
                         $graphImage = '<p class="text-center"><small>'.get_lang('MissingChartLibraryPleaseCheckLog').'</small></p>';
                     }
                     echo $graphImage;
@@ -90,11 +90,11 @@ 
                         );
                     }
 
-                    $link = '<div class="parent" data-id="' . $id . '">';
+                    $link = '<div class="parent" data-id="'.$id.'">';
                     $link .= '<div class="big-icon">';
                     $link .= $image;
-                    $link .= '<div class="sequence-course">' . $sessionInfo['name'] . '</div>';
-                    $link .= '<a href="#" class="sequence-id">' . $id . '</a>';
+                    $link .= '<div class="sequence-course">'.$sessionInfo['name'].'</div>';
+                    $link .= '<a href="#" class="sequence-id">'.$id.'</a>';
                     $link .= $linkDelete;
                     $link .= $linkUndo;
                     $link .= '</div></div>';

main/admin/skill.php

Spacing +3 added lines, -3 removed lines

@@ -17,7 +17,7 @@ 
 
 $listAction = api_get_self();
 
-$action =  '';
+$action = '';
 if (isset($_GET['action']) && in_array($_GET['action'], ['add', 'edit', 'delete'])) {
     $action = $_GET['action'];
 }
@@ -52,8 +52,8 @@ 
 }
 $formToDisplay = $form->returnForm();
 
-$interbreadcrumb[] = array ('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
-$interbreadcrumb[] = array ('url' => api_get_self(), 'name' => get_lang('ManageSkillsLevels'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => api_get_self(), 'name' => get_lang('ManageSkillsLevels'));
 
 $tpl = new Template($action);
 switch ($action) {

main/admin/skill_profile.php

Spacing +6 added lines, -6 removed lines

@@ -17,7 +17,7 @@ 
 
 $listAction = api_get_self();
 
-$action =  '';
+$action = '';
 if (isset($_GET['action']) && in_array($_GET['action'], ['add', 'edit', 'delete', 'move_up', 'move_down'])) {
     $action = $_GET['action'];
 }
@@ -43,9 +43,9 @@ 
 }
 $formToDisplay = $form->returnForm();
 
-$interbreadcrumb[] = array ('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
-$interbreadcrumb[] = array ('url' => 'skill.php', 'name' => get_lang('ManageSkillsLevels'));
-$interbreadcrumb[] = array ('url' =>  api_get_self(), 'name' => get_lang('SkillProfile'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => 'skill.php', 'name' => get_lang('ManageSkillsLevels'));
+$interbreadcrumb[] = array('url' =>  api_get_self(), 'name' => get_lang('SkillProfile'));
 
 $tpl = new Template($action);
 switch ($action) {
@@ -56,7 +56,7 @@ 
         $position = $item->getPosition();
 
         if (!empty($position)) {
-            $item->setPosition($position-1);
+            $item->setPosition($position - 1);
         }
         $em->persist($item);
         $em->flush();
@@ -69,7 +69,7 @@ 
 
         $position = $item->getPosition();
 
-        $item->setPosition($position+1);
+        $item->setPosition($position + 1);
 
         $em->persist($item);
         $em->flush();

main/document/create_audio.php

Spacing +50 added lines, -50 removed lines

@@ -63,35 +63,35 @@ 
 }
 
 if ($dir[0] != '/') {
-    $dir = '/' . $dir;
+    $dir = '/'.$dir;
 }
 
 if ($dir[strlen($dir) - 1] != '/') {
     $dir .= '/';
 }
 
-$filepath = api_get_path(SYS_COURSE_PATH) . $_course['path'] . '/document' . $dir;
+$filepath = api_get_path(SYS_COURSE_PATH).$_course['path'].'/document'.$dir;
 
 if (!is_dir($filepath)) {
-    $filepath = api_get_path(SYS_COURSE_PATH) . $_course['path'] . '/document/';
+    $filepath = api_get_path(SYS_COURSE_PATH).$_course['path'].'/document/';
     $dir = '/';
 }
 
 //groups //TODO: clean
 if (!empty($groupId)) {
     $interbreadcrumb[] = array(
-        "url" => "../group/group_space.php?" . api_get_cidreq(),
+        "url" => "../group/group_space.php?".api_get_cidreq(),
         "name" => get_lang('GroupSpace')
     );
     $group = GroupManager:: get_group_properties($groupId);
     $path = explode('/', $dir);
-    if ('/' . $path[1] != $group['directory']) {
+    if ('/'.$path[1] != $group['directory']) {
         api_not_allowed(true);
     }
 }
 
 $interbreadcrumb[] = array(
-    "url" => "./document.php?curdirpath=" . urlencode($dir) . "&" . api_get_cidreq(),
+    "url" => "./document.php?curdirpath=".urlencode($dir)."&".api_get_cidreq(),
     "name" => get_lang('Documents')
 );
 
@@ -128,11 +128,11 @@ 
 
 $dir_acum = '';
 for ($i = 0; $i < $array_len; $i++) {
-    $url_dir = 'document.php?&curdirpath=' . $dir_acum . $dir_array[$i];
+    $url_dir = 'document.php?&curdirpath='.$dir_acum.$dir_array[$i];
     //Max char 80
     $url_to_who = cut($dir_array[$i], 80);
     $interbreadcrumb[] = array('url' => $url_dir, 'name' => $url_to_who);
-    $dir_acum .= $dir_array[$i] . '/';
+    $dir_acum .= $dir_array[$i].'/';
 }
 
 $service = isset($_GET['service']) ? $_GET['service'] : 'pediaphon';
@@ -140,15 +140,15 @@ 
 Display:: display_header($nameTools, 'Doc');
 
 echo '<div class="actions">';
-echo '<a href="document.php?id=' . $document_id . '">' .
-    Display::return_icon('back.png', get_lang('BackTo') . ' ' . get_lang('DocumentsOverview'), '',
-        ICON_SIZE_MEDIUM) . '</a>';
+echo '<a href="document.php?id='.$document_id.'">'.
+    Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('DocumentsOverview'), '',
+        ICON_SIZE_MEDIUM).'</a>';
 
-echo '<a href="create_audio.php?' . api_get_cidreq() . '&id=' . $document_id . '&service=pediaphon">' .
-    Display::return_icon('pediaphon.png', get_lang('Pediaphon'), '', ICON_SIZE_MEDIUM) . '</a>';
+echo '<a href="create_audio.php?'.api_get_cidreq().'&id='.$document_id.'&service=pediaphon">'.
+    Display::return_icon('pediaphon.png', get_lang('Pediaphon'), '', ICON_SIZE_MEDIUM).'</a>';
 
-echo '<a href="create_audio.php?' . api_get_cidreq() . '&id=' . $document_id . '&service=google">' .
-    Display::return_icon('google.png', get_lang('GoogleAudio'), '', ICON_SIZE_MEDIUM) . '</a>';
+echo '<a href="create_audio.php?'.api_get_cidreq().'&id='.$document_id.'&service=google">'.
+    Display::return_icon('google.png', get_lang('GoogleAudio'), '', ICON_SIZE_MEDIUM).'</a>';
 echo '</div>';
 ?>
     <!-- javascript and styles for textareaCounter-->
@@ -213,14 +213,14 @@ 
 $selected_language = null;
 
 while ($row = Database::fetch_array($result_select)) {
-    $options[$row['isocode']] = $row['original_name'] . ' (' . $row['english_name'] . ')';
+    $options[$row['isocode']] = $row['original_name'].' ('.$row['english_name'].')';
     if (in_array($row['isocode'], array('de', 'en', 'es', 'fr'))) {
-        $options_pedia[$row['isocode']] = $row['original_name'] . ' (' . $row['english_name'] . ')';
+        $options_pedia[$row['isocode']] = $row['original_name'].' ('.$row['english_name'].')';
     }
 }
 
 if ($service == 'google') {
-    $selected_language = api_get_language_isocode();//lang default is the course language
+    $selected_language = api_get_language_isocode(); //lang default is the course language
 
     $form = new FormValidator('form1', 'post', api_get_self().'?'.api_get_cidreq(), '', array('id' => 'form1'));
     $form->addHeader(get_lang('HelpText2Audio'));
@@ -272,16 +272,16 @@ 
         var voices = new Array()
 
         <!--German -->
-        voices['de'] = ["<?php echo get_lang('Female') . ' (de1)'; ?>|de1", "<?php echo get_lang('Male') . ' (de2)'; ?>|de2", "<?php echo get_lang('Female') . ' (de3)'; ?>|de3", "<?php echo get_lang('Male') . ' (de4)'; ?>|de4", "<?php echo get_lang('Female') . ' (de5)'; ?>|de5", "<?php echo get_lang('Male') . ' (de6)'; ?>|de6", "<?php echo get_lang('Female') . ' (de7)'; ?>|de7", "<?php echo get_lang('Female') . ' (de8 HQ)'; ?>|de8"]
+        voices['de'] = ["<?php echo get_lang('Female').' (de1)'; ?>|de1", "<?php echo get_lang('Male').' (de2)'; ?>|de2", "<?php echo get_lang('Female').' (de3)'; ?>|de3", "<?php echo get_lang('Male').' (de4)'; ?>|de4", "<?php echo get_lang('Female').' (de5)'; ?>|de5", "<?php echo get_lang('Male').' (de6)'; ?>|de6", "<?php echo get_lang('Female').' (de7)'; ?>|de7", "<?php echo get_lang('Female').' (de8 HQ)'; ?>|de8"]
 
         <!--English -->
-        voices['en'] = ["<?php echo get_lang('Male') . ' (en1)'; ?>|en1", "<?php echo get_lang('Male') . ' (en2 HQ)'; ?>|en2", "<?php echo get_lang('Female') . ' (us1)'; ?>|us1", "<?php echo get_lang('Male') . ' (us2)'; ?>|us2", "<?php echo get_lang('Male') . ' (us3)'; ?>|us3", "<?php echo get_lang('Female') . '(us4 HQ)'; ?>|us4"]
+        voices['en'] = ["<?php echo get_lang('Male').' (en1)'; ?>|en1", "<?php echo get_lang('Male').' (en2 HQ)'; ?>|en2", "<?php echo get_lang('Female').' (us1)'; ?>|us1", "<?php echo get_lang('Male').' (us2)'; ?>|us2", "<?php echo get_lang('Male').' (us3)'; ?>|us3", "<?php echo get_lang('Female').'(us4 HQ)'; ?>|us4"]
 
         <!--Spanish -->
-        voices['es'] = ["<?php echo get_lang('Male') . ' (es5 HQ)'; ?>|es5"]
+        voices['es'] = ["<?php echo get_lang('Male').' (es5 HQ)'; ?>|es5"]
 
         <!--French -->
-        voices['fr'] = ["<?php echo get_lang('Female') . ' (fr8 HQ)'; ?>|fr8"]
+        voices['fr'] = ["<?php echo get_lang('Female').' (fr8 HQ)'; ?>|fr8"]
 
         $(document).ready(function () {
             $('.lang').on('change', function () {
@@ -312,11 +312,11 @@ 
  */
 function downloadMP3_google($filepath, $dir)
 {
-    $location = 'create_audio.php?' . api_get_cidreq() . '&id=' . intval($_POST['id']) . '&service=google';
+    $location = 'create_audio.php?'.api_get_cidreq().'&id='.intval($_POST['id']).'&service=google';
 
     //security
     if (!isset($_POST['lang']) && !isset($_POST['text']) && !isset($_POST['title']) && !isset($filepath) && !isset($dir)) {
-        echo '<script>window.location.href="' . $location . '"</script>';
+        echo '<script>window.location.href="'.$location.'"</script>';
         return;
     }
 
@@ -326,33 +326,33 @@ 
     $clean_title = trim($_POST['title']);
     $clean_text = trim($_POST['text']);
     if (empty($clean_title) || empty($clean_text)) {
-        echo '<script>window.location.href="' . $location . '"</script>';
+        echo '<script>window.location.href="'.$location.'"</script>';
         
         return;
     }
     $clean_title = Security::remove_XSS($clean_title);
     $clean_title = Database::escape_string($clean_title);
-    $clean_title = str_replace(' ', '_', $clean_title);//compound file names
+    $clean_title = str_replace(' ', '_', $clean_title); //compound file names
 
     $clean_text = Security::remove_XSS($clean_text);
     $clean_lang = Security::remove_XSS($_POST['lang']);
 
     $extension = 'mp3';
-    $audio_filename = $clean_title . '.' . $extension;
+    $audio_filename = $clean_title.'.'.$extension;
     $audio_title = str_replace('_', ' ', $clean_title);
 
     //prevent duplicates
-    if (file_exists($filepath . '/' . $clean_title . '.' . $extension)) {
+    if (file_exists($filepath.'/'.$clean_title.'.'.$extension)) {
         $i = 1;
-        while (file_exists($filepath . '/' . $clean_title . '_' . $i . '.' . $extension)) {
+        while (file_exists($filepath.'/'.$clean_title.'_'.$i.'.'.$extension)) {
             $i++;
         }
-        $audio_filename = $clean_title . '_' . $i . '.' . $extension;
-        $audio_title = $clean_title . '_' . $i . '.' . $extension;
+        $audio_filename = $clean_title.'_'.$i.'.'.$extension;
+        $audio_title = $clean_title.'_'.$i.'.'.$extension;
         $audio_title = str_replace('_', ' ', $audio_title);
     }
 
-    $documentPath = $filepath . '/' . $audio_filename;
+    $documentPath = $filepath.'/'.$audio_filename;
 
     $clean_text = api_replace_dangerous_char($clean_text);
 
@@ -361,7 +361,7 @@ 
 
     $proxySettings = api_get_configuration_value('proxy_settings');
     $key = api_get_configuration_value('translate_app_google_key');
-    $url = "https://www.googleapis.com/language/translate/v2?key=$key&" . $clean_lang . "&target=$clean_lang&q=" . urlencode($clean_text) . "";
+    $url = "https://www.googleapis.com/language/translate/v2?key=$key&".$clean_lang."&target=$clean_lang&q=".urlencode($clean_text)."";
 
     if (empty($proxySettings)) {
         $content = file_get_contents($url);
@@ -381,7 +381,7 @@ 
     $relativeUrlPath = $dir;
     $doc_id = add_document(
         $_course,
-        $relativeUrlPath . $audio_filename,
+        $relativeUrlPath.$audio_filename,
         'file',
         filesize($documentPath),
         $audio_title
@@ -400,7 +400,7 @@ 
     );
     Display::display_confirmation_message(get_lang('DocumentCreated'));
     //return to location
-    echo '<script>window.location.href="' . $location . '"</script>';
+    echo '<script>window.location.href="'.$location.'"</script>';
 }
 
 /**
@@ -413,10 +413,10 @@ 
  */
 function downloadMP3_pediaphon($filepath, $dir)
 {
-    $location = 'create_audio.php?' . api_get_cidreq() . '&id=' . intval($_POST['id']) . '&service=pediaphon';
+    $location = 'create_audio.php?'.api_get_cidreq().'&id='.intval($_POST['id']).'&service=pediaphon';
     //security
     if (!isset($_POST['lang']) && !isset($_POST['text']) && !isset($_POST['title']) && !isset($filepath) && !isset($dir)) {
-        echo '<script>window.location.href="' . $location . '"</script>';
+        echo '<script>window.location.href="'.$location.'"</script>';
         return;
     }
     $_course = api_get_course_info();
@@ -426,33 +426,33 @@ 
     $clean_text = trim($_POST['text']);
     $clean_voices = Security::remove_XSS($_POST['voices']);
     if (empty($clean_title) || empty($clean_text) || empty($clean_voices)) {
-        echo '<script>window.location.href="' . $location . '"</script>';
+        echo '<script>window.location.href="'.$location.'"</script>';
 
         return;
     }
     $clean_title = Security::remove_XSS($clean_title);
     $clean_title = Database::escape_string($clean_title);
-    $clean_title = str_replace(' ', '_', $clean_title);//compound file names
+    $clean_title = str_replace(' ', '_', $clean_title); //compound file names
     $clean_text = Security::remove_XSS($clean_text);
     $clean_lang = Security::remove_XSS($_POST['lang']);
     $clean_speed = Security::remove_XSS($_POST['speed']);
 
     $extension = 'mp3';
-    $audio_filename = $clean_title . '.' . $extension;
+    $audio_filename = $clean_title.'.'.$extension;
     $audio_title = str_replace('_', ' ', $clean_title);
 
     //prevent duplicates
-    if (file_exists($filepath . '/' . $clean_title . '.' . $extension)) {
+    if (file_exists($filepath.'/'.$clean_title.'.'.$extension)) {
         $i = 1;
-        while (file_exists($filepath . '/' . $clean_title . '_' . $i . '.' . $extension)) {
+        while (file_exists($filepath.'/'.$clean_title.'_'.$i.'.'.$extension)) {
             $i++;
         }
-        $audio_filename = $clean_title . '_' . $i . '.' . $extension;
-        $audio_title = $clean_title . '_' . $i . '.' . $extension;
+        $audio_filename = $clean_title.'_'.$i.'.'.$extension;
+        $audio_title = $clean_title.'_'.$i.'.'.$extension;
         $audio_title = str_replace('_', ' ', $audio_title);
     }
 
-    $documentPath = $filepath . '/' . $audio_filename;
+    $documentPath = $filepath.'/'.$audio_filename;
     $clean_text = api_replace_dangerous_char($clean_text);
 
     //adding the file
@@ -461,17 +461,17 @@ 
         $url_pediaphon = 'http://www.pediaphon.org/~bischoff/radiopedia/sprich_multivoice.cgi';
         $find_t2v = '/http\:\/\/www\.pediaphon\.org\/\~bischoff\/radiopedia\/mp3\/(.*)\.mp3\"/';
     } else {
-        $url_pediaphon = 'http://www.pediaphon.org/~bischoff/radiopedia/sprich_multivoice_' . $clean_lang . '.cgi';//en, es, fr
-        $find_t2v = '/http\:\/\/www\.pediaphon\.org\/\~bischoff\/radiopedia\/mp3\/' . $clean_lang . '\/(.*)\.mp3\"/';
+        $url_pediaphon = 'http://www.pediaphon.org/~bischoff/radiopedia/sprich_multivoice_'.$clean_lang.'.cgi'; //en, es, fr
+        $find_t2v = '/http\:\/\/www\.pediaphon\.org\/\~bischoff\/radiopedia\/mp3\/'.$clean_lang.'\/(.*)\.mp3\"/';
     }
 
-    $data = "stimme=" . $clean_voices . "&inputtext=" . $clean_text . "&speed=" . $clean_speed . "&go=speak";
+    $data = "stimme=".$clean_voices."&inputtext=".$clean_text."&speed=".$clean_speed."&go=speak";
     $opts = array(
         'http' =>
             array(
                 'method' => 'POST',
                 'header' => "Content-Type: application/x-www-form-urlencoded\r\n",
-                "Content-Length: " . strlen($data) . "\r\n",
+                "Content-Length: ".strlen($data)."\r\n",
                 'content' => $data
             )
     );
@@ -515,5 +515,5 @@ 
     );
     Display::display_confirmation_message(get_lang('DocumentCreated'));
     //return to location
-    echo '<script>window.location.href="' . $location . '"</script>';
+    echo '<script>window.location.href="'.$location.'"</script>';
 }

main/badge/issued_all.php

Spacing +10 added lines, -10 removed lines

@@ -13,7 +13,7 @@ 
 $skillId = isset($_GET['skill']) ? intval($_GET['skill']) : 0;
 
 if (!$userId || !$skillId) {
-    header('Location: ' . api_get_path(WEB_PATH));
+    header('Location: '.api_get_path(WEB_PATH));
     exit;
 }
 
@@ -31,7 +31,7 @@ 
         Display::return_message(get_lang('NoResults'), 'error')
     );
 
-    header('Location: ' . api_get_path(WEB_PATH));
+    header('Location: '.api_get_path(WEB_PATH));
     exit;
 }
 
@@ -167,7 +167,7 @@ 
         $entityManager->persist($skillIssue);
         $entityManager->flush();
 
-        header("Location: " . $skillIssue->getIssueUrlAll());
+        header("Location: ".$skillIssue->getIssueUrlAll());
         exit;
     }
 
@@ -198,7 +198,7 @@ 
         $entityManager->persist($skillUserComment);
         $entityManager->flush();
 
-        header("Location: " . $skillIssue->getIssueUrlAll());
+        header("Location: ".$skillIssue->getIssueUrlAll());
         exit;
     }
 
@@ -214,11 +214,11 @@ 
             $backpack = $configBackpack;
         }
 
-        $htmlHeadXtra[] = '<script src="' . $backpack . 'issuer.js"></script>';
+        $htmlHeadXtra[] = '<script src="'.$backpack.'issuer.js"></script>';
         $objSkill = new Skill();
         $assertionUrl = $skillIssueInfo['badge_assertion'];
         $skills = $objSkill->get($skillId);
-        $unbakedBadge = api_get_path(SYS_UPLOAD_PATH) . "badges/".$skills['icon'];
+        $unbakedBadge = api_get_path(SYS_UPLOAD_PATH)."badges/".$skills['icon'];
         if (!is_file($unbakedBadge)) {
             $unbakedBadge = api_get_path(WEB_CODE_PATH).'img/icons/128/badges-default.png';
         }
@@ -236,12 +236,12 @@ 
                 mkdir($bakedBadge, api_get_permissions_for_new_directories(), true);
             }
             $skillRelUserId = $skillIssueInfo['id'];
-            if (!file_exists($bakedBadge . "/badge_" . $skillRelUserId)) {
-                file_put_contents($bakedBadge . "/badge_" . $skillRelUserId . ".png", $bakedInfo);
+            if (!file_exists($bakedBadge."/badge_".$skillRelUserId)) {
+                file_put_contents($bakedBadge."/badge_".$skillRelUserId.".png", $bakedInfo);
             }
 
             //Process to validate a baked badge
-            $badgeContent = file_get_contents($bakedBadge . "/badge_" . $skillRelUserId . ".png");
+            $badgeContent = file_get_contents($bakedBadge."/badge_".$skillRelUserId.".png");
             $verifyBakedBadge = $png->extractBadgeInfo($badgeContent);
             if (!is_array($verifyBakedBadge)) {
                 $badgeInfoError = true;
@@ -249,7 +249,7 @@ 
 
             if (!$badgeInfoError) {
                 $personalBadge = UserManager::getUserPathById($userId, "web");
-                $personalBadge = $personalBadge."badges/badge_" . $skillRelUserId . ".png";
+                $personalBadge = $personalBadge."badges/badge_".$skillRelUserId.".png";
             }
         }
     }

src/Chamilo/CoreBundle/Entity/SkillRelUser.php

Spacing +3 added lines, -3 removed lines

@@ -455,7 +455,7 @@ 
      */
     public function getIssueUrl()
     {
-        return api_get_path(WEB_PATH) . "badge/{$this->id}";
+        return api_get_path(WEB_PATH)."badge/{$this->id}";
     }
 
     /**
@@ -464,7 +464,7 @@ 
      */
     public function getIssueUrlAll()
     {
-        return api_get_path(WEB_PATH) . "skill/{$this->skill->getId()}/user/{$this->user->getId()}";
+        return api_get_path(WEB_PATH)."skill/{$this->skill->getId()}/user/{$this->user->getId()}";
     }
 
     /**
@@ -473,7 +473,7 @@ 
      */
     public function getAssertionUrl()
     {
-        $url = api_get_path(WEB_CODE_PATH) . "badge/assertion.php?";
+        $url = api_get_path(WEB_CODE_PATH)."badge/assertion.php?";
 
         $url .= http_build_query(array(
             'user' => $this->user->getId(),

Doc Comments +2 added lines, -2 removed lines

@@ -197,7 +197,7 @@ 
 
     /**
      * Get user
-     * @return \User
+     * @return User
      */
     public function getUser()
     {
@@ -512,7 +512,7 @@ 
 
     /**
      * Calculate the average value from the feedback comments
-     * @return int
+     * @return string
      */
     public function getAverage()
     {

main/dropbox/dropbox_class.inc.php

Doc Comments +1 added lines, -1 removed lines

@@ -267,7 +267,7 @@
 	 * @param string $author
 	 * @param string $filename
 	 * @param int $filesize
-	 * @param array $recipient_ids
+	 * @param unknown_type|null $recipient_ids
 	 */
 	public function _createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
     {

Braces +9 added lines, -3 removed lines

@@ -496,13 +496,19 @@
 		$id = intval($id);
 		$sql = "DELETE FROM ".$dropbox_cnf['tbl_file']."
 		        WHERE c_id = $course_id AND cat_id = '".$id."' ";
-		if (!Database::query($sql)) return false;
+		if (!Database::query($sql)) {
+		    return false;
+		}
 		$sql = "DELETE FROM ".$dropbox_cnf['tbl_category']."
 		        WHERE c_id = $course_id AND cat_id = '".$id."' ";
-		if (!Database::query($sql)) return false;
+		if (!Database::query($sql)) {
+		    return false;
+		}
 		$sql = "DELETE FROM ".$dropbox_cnf['tbl_post']."
 		        WHERE c_id = $course_id AND cat_id = '".$id."' ";
-		if (!Database::query($sql)) return false;
+		if (!Database::query($sql)) {
+		    return false;
+		}
 		return true;
 	}
 

Spacing +28 added lines, -28 removed lines

@@ -104,7 +104,7 @@ 
         // Check if object exists already. If it does, the old object is used
         // with updated information (authors, description, upload_date)
         $this->isOldWork = false;
-		$sql = "SELECT id, upload_date FROM ". Database::get_course_table(TABLE_DROPBOX_FILE) ."
+		$sql = "SELECT id, upload_date FROM ".Database::get_course_table(TABLE_DROPBOX_FILE)."
 				WHERE c_id = $course_id AND filename = '".Database::escape_string($this->filename)."'";
         $result = Database::query($sql);
 		$res = Database::fetch_array($result);
@@ -148,20 +148,20 @@ 
 
 			$this->id = Database::insert(Database::get_course_table(TABLE_DROPBOX_FILE), $params);
 			if ($this->id) {
-				$sql = "UPDATE ". Database::get_course_table(TABLE_DROPBOX_FILE) ." SET id = iid WHERE iid = {$this->id}";
+				$sql = "UPDATE ".Database::get_course_table(TABLE_DROPBOX_FILE)." SET id = iid WHERE iid = {$this->id}";
 				Database::query($sql);
 			}
 		}
 
         $sql = "SELECT count(file_id) as count
-        		FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
+        		FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON)."
 				WHERE c_id = $course_id AND file_id = ".intval($this->id)." AND user_id = ".$this->uploader_id;
         $result = Database::query($sql);
         $row = Database::fetch_array($result);
         if ($row['count'] == 0) {
 
             // Insert entries into person table
-            $sql = "INSERT INTO ". Database::get_course_table(TABLE_DROPBOX_PERSON) ." (c_id, file_id, user_id)
+            $sql = "INSERT INTO ".Database::get_course_table(TABLE_DROPBOX_PERSON)." (c_id, file_id, user_id)
                     VALUES ($course_id, ".intval($this->id)." , ".intval($this->uploader_id).")";
             Database::query($sql);
         }
@@ -179,11 +179,11 @@ 
         $action = isset($_GET['action']) ? $_GET['action'] : null;
 
         // Do some sanity checks
-        $id	= intval($id);
+        $id = intval($id);
 
         // Get the data from DB
         $sql = "SELECT uploader_id, filename, filesize, title, description, author, upload_date, last_upload_date, cat_id
-                FROM ". Database::get_course_table(TABLE_DROPBOX_FILE) ."
+                FROM ". Database::get_course_table(TABLE_DROPBOX_FILE)."
                 WHERE c_id = $course_id AND id = ".$id."";
         $result = Database::query($sql);
         $res = Database::fetch_array($result, 'ASSOC');
@@ -212,7 +212,7 @@ 
         // Getting the feedback on the work.
         if ($action == 'viewfeedback' AND $this->id == $_GET['id']) {
             $feedback2 = array();
-            $sql = "SELECT * FROM ". Database::get_course_table(TABLE_DROPBOX_FEEDBACK) ."
+            $sql = "SELECT * FROM ".Database::get_course_table(TABLE_DROPBOX_FEEDBACK)."
                     WHERE c_id = $course_id AND file_id='".$id."' 
                     ORDER BY feedback_id ASC";
             $result = Database::query($sql);
@@ -220,14 +220,14 @@ 
                 $row_feedback['feedback'] = Security::remove_XSS($row_feedback['feedback']);
                 $feedback2[] = $row_feedback;
             }
-            $this->feedback2= $feedback2;
+            $this->feedback2 = $feedback2;
         }
 	}
 }
 
 class Dropbox_SentWork extends Dropbox_Work
 {
-	public $recipients;	//array of ['id']['name'] arrays
+	public $recipients; //array of ['id']['name'] arrays
 
 	/**
 	 * Constructor calls private functions to create a new work or retreive an existing work from DB
@@ -292,7 +292,7 @@ 
 			$recipient_ids = array($uploader_id);
 		}
 
-		if (! is_array($recipient_ids) || count($recipient_ids) == 0) {
+		if (!is_array($recipient_ids) || count($recipient_ids) == 0) {
 			die(get_lang('GeneralError').' (code 209)');
 		}
 
@@ -308,13 +308,13 @@ 
         $table_post = Database::get_course_table(TABLE_DROPBOX_POST);
         $table_person = Database::get_course_table(TABLE_DROPBOX_PERSON);
         $session_id = api_get_session_id();
-        $user  = api_get_user_id();
+        $user = api_get_user_id();
         $now = api_get_utc_datetime();
 
 		// Insert data in dropbox_post and dropbox_person table for each recipient
 		foreach ($this->recipients as $rec) {
-            $file_id = (int)$this->id;
-            $user_id = (int)$rec['id'];
+            $file_id = (int) $this->id;
+            $user_id = (int) $rec['id'];
 			$sql = "INSERT INTO $table_post (c_id, file_id, dest_user_id, session_id, feedback_date, cat_id)
                     VALUES ($course_id, $file_id, $user_id, $session_id, '$now', 0)";
 	        Database::query($sql);
@@ -330,7 +330,7 @@ 
 
                 // Do not add recipient in person table if mailing zip or just upload.
                 if (!$justSubmit) {
-                    Database::query($sql);	// If work already exists no error is generated
+                    Database::query($sql); // If work already exists no error is generated
                 }
             }
 
@@ -339,7 +339,7 @@ 
 			    $ownerid = getUserOwningThisMailing($ownerid);
 			}
 			if (($recipid = $rec["id"]) > $dropbox_cnf['mailingIdBase']) {
-			    $recipid = $ownerid;  // mailing file recipient = mailing id, not a person
+			    $recipid = $ownerid; // mailing file recipient = mailing id, not a person
 			}
             api_item_property_update(
                 $_course,
@@ -396,13 +396,13 @@ 
 class Dropbox_Person
 {
 	// The receivedWork and the sentWork arrays are sorted.
-	public $receivedWork;	// an array of Dropbox_Work objects
-	public $sentWork;		// an array of Dropbox_SentWork objects
+	public $receivedWork; // an array of Dropbox_Work objects
+	public $sentWork; // an array of Dropbox_SentWork objects
 
 	public $userId = 0;
 	public $isCourseAdmin = false;
 	public $isCourseTutor = false;
-	public $_orderBy = '';	// private property that determines by which field
+	public $_orderBy = ''; // private property that determines by which field
 
 	/**
 	 * Constructor for recreating the Dropbox_Person object
@@ -471,7 +471,7 @@ 
 	    $course_id = api_get_course_int_id();
 		// Delete entries in person table concerning received works
 		foreach ($this->receivedWork as $w) {
-            $sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
+            $sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_PERSON)."
 			        WHERE c_id = $course_id AND user_id='".$this->userId."' AND file_id='".$w->id."'";
 			Database::query($sql);
 		}
@@ -488,10 +488,10 @@ 
         $course_id = api_get_course_int_id();
 
 		$id = intval($id);
-		$sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_FILE) ."
+		$sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_FILE)."
 		        WHERE c_id = $course_id AND cat_id = '".$id."' ";
 		if (!Database::query($sql)) return false;
-		$sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_CATEGORY) ."
+		$sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_CATEGORY)."
 		        WHERE c_id = $course_id AND cat_id = '".$id."' ";
 		if (!Database::query($sql)) return false;
 		$sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_POST)."
@@ -525,10 +525,10 @@ 
 			}
 		}
 		// Delete entries in person table concerning received works
-        $sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
+        $sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_PERSON)."
                 WHERE c_id = $course_id AND user_id = '".$this->userId."' AND file_id ='".$id."'";
 		Database::query($sql);
-		removeUnusedFiles();	// Check for unused files
+		removeUnusedFiles(); // Check for unused files
 	}
 
 	/**
@@ -539,12 +539,12 @@ 
 	    $course_id = api_get_course_int_id();
 		//delete entries in person table concerning sent works
 		foreach ($this->sentWork as $w) {
-            $sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
+            $sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_PERSON)."
                     WHERE c_id = $course_id AND user_id='".$this->userId."' AND file_id='".$w->id."'";
 			Database::query($sql);
 			removeMoreIfMailing($w->id);
 		}
-		removeUnusedFiles();	// Check for unused files
+		removeUnusedFiles(); // Check for unused files
 	}
 
 	/**
@@ -573,11 +573,11 @@ 
 		}
 		//$file_id = $this->sentWork[$index]->id;
 		// Delete entries in person table concerning sent works
-        $sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
+        $sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_PERSON)."
                 WHERE c_id = $course_id AND user_id='".$this->userId."' AND file_id='".$id."'";
 		Database::query($sql);
 		removeMoreIfMailing($id);
-		removeUnusedFiles();	// Check for unused files
+		removeUnusedFiles(); // Check for unused files
 	}
 
 	/**
@@ -599,7 +599,7 @@ 
 		$wi = -1;
 		foreach ($this->receivedWork as $w) {
 			$wi++;
-			if ($w->id == $id){
+			if ($w->id == $id) {
 			   $found = true;
 			   break;
 			}  // foreach (... as $wi -> $w) gives error 221! (no idea why...)

Indentation +324 added lines, -324 removed lines

@@ -76,20 +76,20 @@ 
         }
     }
 
-	/**
-	 * private function creating a new work object
-	 *
-	 * @param int $uploader_id
-	 * @param string $title
-	 * @param string $description
-	 * @param string $author
-	 * @param string $filename
-	 * @param int $filesize
-	 *
-	 * @todo 	$author was originally a field but this has now been replaced by the first and lastname of the uploader (to prevent anonymous uploads)
-	 * 			As a consequence this parameter can be removed
-	 */
-	public function _createNewWork($uploader_id, $title, $description, $author, $filename, $filesize)
+    /**
+     * private function creating a new work object
+     *
+     * @param int $uploader_id
+     * @param string $title
+     * @param string $description
+     * @param string $author
+     * @param string $filename
+     * @param int $filesize
+     *
+     * @todo 	$author was originally a field but this has now been replaced by the first and lastname of the uploader (to prevent anonymous uploads)
+     * 			As a consequence this parameter can be removed
+     */
+    public function _createNewWork($uploader_id, $title, $description, $author, $filename, $filesize)
     {
         // Fill in the properties
         $this->uploader_id = intval($uploader_id);
@@ -104,17 +104,17 @@ 
         // Check if object exists already. If it does, the old object is used
         // with updated information (authors, description, upload_date)
         $this->isOldWork = false;
-		$sql = "SELECT id, upload_date FROM ". Database::get_course_table(TABLE_DROPBOX_FILE) ."
+        $sql = "SELECT id, upload_date FROM ". Database::get_course_table(TABLE_DROPBOX_FILE) ."
 				WHERE c_id = $course_id AND filename = '".Database::escape_string($this->filename)."'";
         $result = Database::query($sql);
-		$res = Database::fetch_array($result);
-		if ($res) {
-			$this->isOldWork = true;
-		}
-		// Insert or update the dropbox_file table and set the id property
-		if ($this->isOldWork) {
-			$this->id = $res['id'];
-			$this->upload_date = $res['upload_date'];
+        $res = Database::fetch_array($result);
+        if ($res) {
+            $this->isOldWork = true;
+        }
+        // Insert or update the dropbox_file table and set the id property
+        if ($this->isOldWork) {
+            $this->id = $res['id'];
+            $this->upload_date = $res['upload_date'];
 
             $params = [
                 'filesize' => $this->filesize,
@@ -130,9 +130,9 @@ 
                 $params,
                 ['c_id = ? AND id = ?' => [$course_id, $this->id]]
             );
-		} else {
-			$this->upload_date = $this->last_upload_date;
-			$params = [
+        } else {
+            $this->upload_date = $this->last_upload_date;
+            $params = [
                 'c_id' => $course_id,
                 'uploader_id' => $this->uploader_id,
                 'filename' => $this->filename,
@@ -144,14 +144,14 @@ 
                 'last_upload_date' => $this->last_upload_date,
                 'session_id' => api_get_session_id(),
                 'cat_id' => 0
-			];
+            ];
 
-			$this->id = Database::insert(Database::get_course_table(TABLE_DROPBOX_FILE), $params);
-			if ($this->id) {
-				$sql = "UPDATE ". Database::get_course_table(TABLE_DROPBOX_FILE) ." SET id = iid WHERE iid = {$this->id}";
-				Database::query($sql);
-			}
-		}
+            $this->id = Database::insert(Database::get_course_table(TABLE_DROPBOX_FILE), $params);
+            if ($this->id) {
+                $sql = "UPDATE ". Database::get_course_table(TABLE_DROPBOX_FILE) ." SET id = iid WHERE iid = {$this->id}";
+                Database::query($sql);
+            }
+        }
 
         $sql = "SELECT count(file_id) as count
         		FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
@@ -165,16 +165,16 @@ 
                     VALUES ($course_id, ".intval($this->id)." , ".intval($this->uploader_id).")";
             Database::query($sql);
         }
-	}
-
-	/**
-	 * private function creating existing object by retreiving info from db
-	 *
-	 * @param int $id
-	 */
-	public function _createExistingWork($id)
+    }
+
+    /**
+     * private function creating existing object by retreiving info from db
+     *
+     * @param int $id
+     */
+    public function _createExistingWork($id)
     {
-	    $course_id = api_get_course_int_id();
+        $course_id = api_get_course_int_id();
 
         $action = isset($_GET['action']) ? $_GET['action'] : null;
 
@@ -222,52 +222,52 @@ 
             }
             $this->feedback2= $feedback2;
         }
-	}
+    }
 }
 
 class Dropbox_SentWork extends Dropbox_Work
 {
-	public $recipients;	//array of ['id']['name'] arrays
-
-	/**
-	 * Constructor calls private functions to create a new work or retreive an existing work from DB
-	 * depending on the number of parameters
-	 *
-	 * @param unknown_type $arg1
-	 * @param unknown_type $arg2
-	 * @param unknown_type $arg3
-	 * @param unknown_type $arg4
-	 * @param unknown_type $arg5
-	 * @param unknown_type $arg6
-	 * @param unknown_type $arg7
-	 * @return Dropbox_SentWork
-	 */
-	public function __construct($arg1, $arg2 = null, $arg3 = null, $arg4 = null, $arg5 = null, $arg6 = null, $arg7 = null)
+    public $recipients;	//array of ['id']['name'] arrays
+
+    /**
+     * Constructor calls private functions to create a new work or retreive an existing work from DB
+     * depending on the number of parameters
+     *
+     * @param unknown_type $arg1
+     * @param unknown_type $arg2
+     * @param unknown_type $arg3
+     * @param unknown_type $arg4
+     * @param unknown_type $arg5
+     * @param unknown_type $arg6
+     * @param unknown_type $arg7
+     * @return Dropbox_SentWork
+     */
+    public function __construct($arg1, $arg2 = null, $arg3 = null, $arg4 = null, $arg5 = null, $arg6 = null, $arg7 = null)
     {
-		if (func_num_args() > 1) {
-		    $this->_createNewSentWork($arg1, $arg2, $arg3, $arg4, $arg5, $arg6, $arg7);
-		} else {
-			$this->_createExistingSentWork($arg1);
-		}
-	}
-
-	/**
-	 * private function creating a new SentWork object
-	 *
-	 * @param int $uploader_id
-	 * @param string $title
-	 * @param string $description
-	 * @param string $author
-	 * @param string $filename
-	 * @param int $filesize
-	 * @param array $recipient_ids
-	 */
-	public function _createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
+        if (func_num_args() > 1) {
+            $this->_createNewSentWork($arg1, $arg2, $arg3, $arg4, $arg5, $arg6, $arg7);
+        } else {
+            $this->_createExistingSentWork($arg1);
+        }
+    }
+
+    /**
+     * private function creating a new SentWork object
+     *
+     * @param int $uploader_id
+     * @param string $title
+     * @param string $description
+     * @param string $author
+     * @param string $filename
+     * @param int $filesize
+     * @param array $recipient_ids
+     */
+    public function _createNewSentWork($uploader_id, $title, $description, $author, $filename, $filesize, $recipient_ids)
     {
         $dropbox_cnf = getDropboxConf();
         $_course = api_get_course_info();
 
-		// Call constructor of Dropbox_Work object
+        // Call constructor of Dropbox_Work object
         parent::__construct(
             $uploader_id,
             $title,
@@ -277,33 +277,33 @@ 
             $filesize
         );
 
-		$course_id = api_get_course_int_id();
-
-		// Do sanity checks on recipient_ids array & property fillin
-		// The sanity check for ex-coursemembers is already done in base constructor
-		$uploader_id = (int) $uploader_id;
+        $course_id = api_get_course_int_id();
 
-		$justSubmit = false;
-		if (is_int($recipient_ids)) {
-			$justSubmit = true;
-			$recipient_ids = array($recipient_ids + $this->id);
-		} elseif (count($recipient_ids) == 0) {
-			$justSubmit = true;
-			$recipient_ids = array($uploader_id);
-		}
+        // Do sanity checks on recipient_ids array & property fillin
+        // The sanity check for ex-coursemembers is already done in base constructor
+        $uploader_id = (int) $uploader_id;
+
+        $justSubmit = false;
+        if (is_int($recipient_ids)) {
+            $justSubmit = true;
+            $recipient_ids = array($recipient_ids + $this->id);
+        } elseif (count($recipient_ids) == 0) {
+            $justSubmit = true;
+            $recipient_ids = array($uploader_id);
+        }
 
-		if (! is_array($recipient_ids) || count($recipient_ids) == 0) {
-			die(get_lang('GeneralError').' (code 209)');
-		}
+        if (! is_array($recipient_ids) || count($recipient_ids) == 0) {
+            die(get_lang('GeneralError').' (code 209)');
+        }
 
-		foreach ($recipient_ids as $rec) {
-			if (empty($rec)) {
-			    continue;
+        foreach ($recipient_ids as $rec) {
+            if (empty($rec)) {
+                continue;
             }
 
             //this check is done when validating submitted data
-			$this->recipients[] = array('id' => $rec);
-		}
+            $this->recipients[] = array('id' => $rec);
+        }
 
         $table_post = Database::get_course_table(TABLE_DROPBOX_POST);
         $table_person = Database::get_course_table(TABLE_DROPBOX_PERSON);
@@ -313,12 +313,12 @@ 
         $mailId = get_mail_id_base();
 
         // Insert data in dropbox_post and dropbox_person table for each recipient
-		foreach ($this->recipients as $rec) {
+        foreach ($this->recipients as $rec) {
             $file_id = (int)$this->id;
             $user_id = (int)$rec['id'];
-			$sql = "INSERT INTO $table_post (c_id, file_id, dest_user_id, session_id, feedback_date, cat_id)
+            $sql = "INSERT INTO $table_post (c_id, file_id, dest_user_id, session_id, feedback_date, cat_id)
                     VALUES ($course_id, $file_id, $user_id, $session_id, '$now', 0)";
-	        Database::query($sql);
+            Database::query($sql);
             // If work already exists no error is generated
 
             /**
@@ -335,13 +335,13 @@ 
                 }
             }
 
-			// Update item_property table for each recipient
-			if (($ownerid = $this->uploader_id) > $mailId) {
-			    $ownerid = getUserOwningThisMailing($ownerid);
-			}
-			if (($recipid = $rec["id"]) > $mailId) {
-			    $recipid = $ownerid;  // mailing file recipient = mailing id, not a person
-			}
+            // Update item_property table for each recipient
+            if (($ownerid = $this->uploader_id) > $mailId) {
+                $ownerid = getUserOwningThisMailing($ownerid);
+            }
+            if (($recipid = $rec["id"]) > $mailId) {
+                $recipid = $ownerid;  // mailing file recipient = mailing id, not a person
+            }
             api_item_property_update(
                 $_course,
                 TOOL_DROPBOX,
@@ -351,90 +351,90 @@ 
                 null,
                 $recipid
             );
-		}
-	}
-
-	/**
-	 * private function creating existing object by retreiving info from db
-	 *
-	 * @param unknown_type $id
-	 */
-	public function _createExistingSentWork($id)
+        }
+    }
+
+    /**
+     * private function creating existing object by retreiving info from db
+     *
+     * @param unknown_type $id
+     */
+    public function _createExistingSentWork($id)
     {
         $id = intval($id);
 
-		$course_id = api_get_course_int_id();
+        $course_id = api_get_course_int_id();
 
-		// Call constructor of Dropbox_Work object
-		parent::__construct($id);
+        // Call constructor of Dropbox_Work object
+        parent::__construct($id);
 
-		// Fill in recipients array
-		$this->recipients = array();
-		$sql = "SELECT dest_user_id, feedback_date, feedback
+        // Fill in recipients array
+        $this->recipients = array();
+        $sql = "SELECT dest_user_id, feedback_date, feedback
 				FROM ".Database::get_course_table(TABLE_DROPBOX_POST)."
 				WHERE c_id = $course_id AND file_id = ".intval($id)."";
         $result = Database::query($sql);
-		while ($res = Database::fetch_array($result, 'ASSOC')) {
-			// Check for deleted users
-			$dest_user_id = $res['dest_user_id'];
-			$user_info = api_get_user_info($dest_user_id);
-			//$this->category = $res['cat_id'];
-			if (!$user_info) {
-				$this->recipients[] = array('id' => -1, 'name' => get_lang('Unknown', ''));
-			} else {
-				$this->recipients[] = array(
+        while ($res = Database::fetch_array($result, 'ASSOC')) {
+            // Check for deleted users
+            $dest_user_id = $res['dest_user_id'];
+            $user_info = api_get_user_info($dest_user_id);
+            //$this->category = $res['cat_id'];
+            if (!$user_info) {
+                $this->recipients[] = array('id' => -1, 'name' => get_lang('Unknown', ''));
+            } else {
+                $this->recipients[] = array(
                     'id' => $dest_user_id,
                     'name' => $user_info['complete_name'],
                     'user_id' => $dest_user_id,
-				    'feedback_date' => $res['feedback_date'],
+                    'feedback_date' => $res['feedback_date'],
                     'feedback' => $res['feedback']
                 );
-			}
-		}
-	}
+            }
+        }
+    }
 }
 
 class Dropbox_Person
 {
-	// The receivedWork and the sentWork arrays are sorted.
-	public $receivedWork;	// an array of Dropbox_Work objects
-	public $sentWork;		// an array of Dropbox_SentWork objects
-
-	public $userId = 0;
-	public $isCourseAdmin = false;
-	public $isCourseTutor = false;
-	public $_orderBy = '';	// private property that determines by which field
-
-	/**
-	 * Constructor for recreating the Dropbox_Person object
-	 *
-	 * @param int $userId
-	 * @param bool $isCourseAdmin
-	 * @param bool $isCourseTutor
-	 * @return Dropbox_Person
-	 */
-	public function __construct($userId, $isCourseAdmin, $isCourseTutor)
+    // The receivedWork and the sentWork arrays are sorted.
+    public $receivedWork;	// an array of Dropbox_Work objects
+    public $sentWork;		// an array of Dropbox_SentWork objects
+
+    public $userId = 0;
+    public $isCourseAdmin = false;
+    public $isCourseTutor = false;
+    public $_orderBy = '';	// private property that determines by which field
+
+    /**
+     * Constructor for recreating the Dropbox_Person object
+     *
+     * @param int $userId
+     * @param bool $isCourseAdmin
+     * @param bool $isCourseTutor
+     * @return Dropbox_Person
+     */
+    public function __construct($userId, $isCourseAdmin, $isCourseTutor)
     {
-	    $course_id = api_get_course_int_id();
+        $course_id = api_get_course_int_id();
 
-		// Fill in properties
+        // Fill in properties
         $this->userId = $userId;
         $this->isCourseAdmin = $isCourseAdmin;
         $this->isCourseTutor = $isCourseTutor;
         $this->receivedWork = array();
         $this->sentWork = array();
 
-		// Note: perhaps include an ex coursemember check to delete old files
+        // Note: perhaps include an ex coursemember check to delete old files
 
-		$session_id = api_get_session_id();
-		$condition_session = api_get_session_condition($session_id);
+        $session_id = api_get_session_id();
+        $condition_session = api_get_session_condition($session_id);
 
-		$post_tbl = Database::get_course_table(TABLE_DROPBOX_POST);
-		$person_tbl = Database::get_course_table(TABLE_DROPBOX_PERSON);
-		$file_tbl = Database::get_course_table(TABLE_DROPBOX_FILE);
+        $post_tbl = Database::get_course_table(TABLE_DROPBOX_POST);
+        $person_tbl = Database::get_course_table(TABLE_DROPBOX_PERSON);
+        $file_tbl = Database::get_course_table(TABLE_DROPBOX_FILE);
 
         // Find all entries where this person is the recipient
-		$sql = "SELECT DISTINCT r.file_id, r.cat_id
+        $sql = "SELECT DISTINCT r.file_id, r.cat_id
                 FROM $post_tbl r
                 INNER JOIN $person_tbl p
                     ON (r.file_id = p.file_id AND r.c_id = $course_id AND p.c_id = $course_id )
@@ -443,12 +443,12 @@ 
                      r.dest_user_id = ".intval($this->userId)." $condition_session ";
 
         $result = Database::query($sql);
-		while ($res = Database::fetch_array($result)) {
-			$temp = new Dropbox_Work($res['file_id']);
-			$temp->category = $res['cat_id'];
-			$this->receivedWork[] = $temp;
-		}
-		// Find all entries where this person is the sender/uploader
+        while ($res = Database::fetch_array($result)) {
+            $temp = new Dropbox_Work($res['file_id']);
+            $temp->category = $res['cat_id'];
+            $this->receivedWork[] = $temp;
+        }
+        // Find all entries where this person is the sender/uploader
         $sql = "SELECT DISTINCT f.id
 				FROM $file_tbl f
 				INNER JOIN $person_tbl p
@@ -459,160 +459,160 @@ 
                     $condition_session
                 ";
         $result = Database::query($sql);
-		while ($res = Database::fetch_array($result)) {
-			$this->sentWork[] = new Dropbox_SentWork($res['id']);
-		}
-	}
-
-	/**
-	 * Deletes all the received work of this person
-	 */
-	public function deleteAllReceivedWork()
+        while ($res = Database::fetch_array($result)) {
+            $this->sentWork[] = new Dropbox_SentWork($res['id']);
+        }
+    }
+
+    /**
+     * Deletes all the received work of this person
+     */
+    public function deleteAllReceivedWork()
     {
-	    $course_id = api_get_course_int_id();
-		// Delete entries in person table concerning received works
-		foreach ($this->receivedWork as $w) {
+        $course_id = api_get_course_int_id();
+        // Delete entries in person table concerning received works
+        foreach ($this->receivedWork as $w) {
             $sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
 			        WHERE c_id = $course_id AND user_id='".$this->userId."' AND file_id='".$w->id."'";
-			Database::query($sql);
-		}
+            Database::query($sql);
+        }
         // Check for unused files
-		removeUnusedFiles();
-	}
-
-	/**
-	 * Deletes all the received categories and work of this person
-	 * @param integer $id
-	 */
-	public function deleteReceivedWorkFolder($id)
+        removeUnusedFiles();
+    }
+
+    /**
+     * Deletes all the received categories and work of this person
+     * @param integer $id
+     */
+    public function deleteReceivedWorkFolder($id)
     {
         $course_id = api_get_course_int_id();
 
-		$id = intval($id);
-		$sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_FILE) ."
+        $id = intval($id);
+        $sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_FILE) ."
 		        WHERE c_id = $course_id AND cat_id = '".$id."' ";
-		if (!Database::query($sql)) return false;
-		$sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_CATEGORY) ."
+        if (!Database::query($sql)) return false;
+        $sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_CATEGORY) ."
 		        WHERE c_id = $course_id AND cat_id = '".$id."' ";
-		if (!Database::query($sql)) return false;
-		$sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_POST)."
+        if (!Database::query($sql)) return false;
+        $sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_POST)."
 		        WHERE c_id = $course_id AND cat_id = '".$id."' ";
-		if (!Database::query($sql)) return false;
-		return true;
-	}
-
-	/**
-	 * Deletes a received dropbox file of this person with id=$id
-	 *
-	 * @param integer $id
-	 */
-	public function deleteReceivedWork($id)
+        if (!Database::query($sql)) return false;
+        return true;
+    }
+
+    /**
+     * Deletes a received dropbox file of this person with id=$id
+     *
+     * @param integer $id
+     */
+    public function deleteReceivedWork($id)
     {
-	    $course_id = api_get_course_int_id();
-		$id = intval($id);
-
-		// index check
-		$found = false;
-		foreach ($this->receivedWork as $w) {
-			if ($w->id == $id) {
-			   $found = true;
-			   break;
-			}
-		}
-
-		if (!$found) {
-			if (!$this->deleteReceivedWorkFolder($id)) {
-				die(get_lang('GeneralError').' (code 216)');
-			}
-		}
-		// Delete entries in person table concerning received works
+        $course_id = api_get_course_int_id();
+        $id = intval($id);
+
+        // index check
+        $found = false;
+        foreach ($this->receivedWork as $w) {
+            if ($w->id == $id) {
+                $found = true;
+                break;
+            }
+        }
+
+        if (!$found) {
+            if (!$this->deleteReceivedWorkFolder($id)) {
+                die(get_lang('GeneralError').' (code 216)');
+            }
+        }
+        // Delete entries in person table concerning received works
         $sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
                 WHERE c_id = $course_id AND user_id = '".$this->userId."' AND file_id ='".$id."'";
-		Database::query($sql);
-		removeUnusedFiles();	// Check for unused files
-	}
-
-	/**
-	 * Deletes all the sent dropbox files of this person
-	 */
-	public function deleteAllSentWork()
+        Database::query($sql);
+        removeUnusedFiles();	// Check for unused files
+    }
+
+    /**
+     * Deletes all the sent dropbox files of this person
+     */
+    public function deleteAllSentWork()
     {
-	    $course_id = api_get_course_int_id();
-		//delete entries in person table concerning sent works
-		foreach ($this->sentWork as $w) {
+        $course_id = api_get_course_int_id();
+        //delete entries in person table concerning sent works
+        foreach ($this->sentWork as $w) {
             $sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
                     WHERE c_id = $course_id AND user_id='".$this->userId."' AND file_id='".$w->id."'";
-			Database::query($sql);
-			removeMoreIfMailing($w->id);
-		}
-		removeUnusedFiles();	// Check for unused files
-	}
-
-	/**
-	 * Deletes a sent dropbox file of this person with id=$id
-	 *
-	 * @param unknown_type $id
-	 */
-	public function deleteSentWork($id)
+            Database::query($sql);
+            removeMoreIfMailing($w->id);
+        }
+        removeUnusedFiles();	// Check for unused files
+    }
+
+    /**
+     * Deletes a sent dropbox file of this person with id=$id
+     *
+     * @param unknown_type $id
+     */
+    public function deleteSentWork($id)
     {
-	    $course_id = api_get_course_int_id();
-
-		$id = intval($id);
-
-		// index check
-		$found = false;
-		foreach ($this->sentWork as $w) {
-			if ($w->id == $id) {
-			   $found = true;
-			   break;
-			}
-		}
-		if (!$found) {
-			if (!$this->deleteReceivedWorkFolder($id)) {
-				die(get_lang('GeneralError').' (code 219)');
-			}
-		}
-		//$file_id = $this->sentWork[$index]->id;
-		// Delete entries in person table concerning sent works
+        $course_id = api_get_course_int_id();
+
+        $id = intval($id);
+
+        // index check
+        $found = false;
+        foreach ($this->sentWork as $w) {
+            if ($w->id == $id) {
+                $found = true;
+                break;
+            }
+        }
+        if (!$found) {
+            if (!$this->deleteReceivedWorkFolder($id)) {
+                die(get_lang('GeneralError').' (code 219)');
+            }
+        }
+        //$file_id = $this->sentWork[$index]->id;
+        // Delete entries in person table concerning sent works
         $sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
                 WHERE c_id = $course_id AND user_id='".$this->userId."' AND file_id='".$id."'";
-		Database::query($sql);
-		removeMoreIfMailing($id);
-		removeUnusedFiles();	// Check for unused files
-	}
-
-	/**
-	 * Updates feedback for received work of this person with id=$id
-	 *
-	 * @param string $id
-	 * @param string $text
-	 */
-	public function updateFeedback($id, $text)
+        Database::query($sql);
+        removeMoreIfMailing($id);
+        removeUnusedFiles();	// Check for unused files
+    }
+
+    /**
+     * Updates feedback for received work of this person with id=$id
+     *
+     * @param string $id
+     * @param string $text
+     */
+    public function updateFeedback($id, $text)
     {
-	    $course_id = api_get_course_int_id();
+        $course_id = api_get_course_int_id();
         $_course = api_get_course_info();
         $dropbox_cnf = getDropboxConf();
 
-		$id = intval($id);
+        $id = intval($id);
 
-		// index check
-		$found = false;
-		$wi = -1;
-		foreach ($this->receivedWork as $w) {
-			$wi++;
-			if ($w->id == $id){
-			   $found = true;
-			   break;
-			}  // foreach (... as $wi -> $w) gives error 221! (no idea why...)
-		}
+        // index check
+        $found = false;
+        $wi = -1;
+        foreach ($this->receivedWork as $w) {
+            $wi++;
+            if ($w->id == $id){
+                $found = true;
+                break;
+            }  // foreach (... as $wi -> $w) gives error 221! (no idea why...)
+        }
 
-		if (!$found) {
-			return false;
-		}
+        if (!$found) {
+            return false;
+        }
 
-		$feedback_date = api_get_utc_datetime();
-		$this->receivedWork[$wi]->feedback_date = $feedback_date;
-		$this->receivedWork[$wi]->feedback = $text;
+        $feedback_date = api_get_utc_datetime();
+        $this->receivedWork[$wi]->feedback_date = $feedback_date;
+        $this->receivedWork[$wi]->feedback = $text;
 
         $params = [
             'feedback_date' => $feedback_date,
@@ -630,11 +630,11 @@ 
             ]
         );
 
-		// Update item_property table
+        // Update item_property table
         $mailId = get_mail_id_base();
-		if (($ownerid = $this->receivedWork[$wi]->uploader_id) > $mailId) {
-		    $ownerid = getUserOwningThisMailing($ownerid);
-		}
+        if (($ownerid = $this->receivedWork[$wi]->uploader_id) > $mailId) {
+            $ownerid = getUserOwningThisMailing($ownerid);
+        }
 
         api_item_property_update(
             $_course,
@@ -646,33 +646,33 @@ 
             $ownerid
         );
 
-	}
+    }
 
-	/**
-	 * Filter the received work
-	 * @param string $type
-	 * @param string $value
-	 */
-	public function filter_received_work($type, $value)
+    /**
+     * Filter the received work
+     * @param string $type
+     * @param string $value
+     */
+    public function filter_received_work($type, $value)
     {
         $dropbox_cnf = getDropboxConf();
-    	$new_received_work = array();
+        $new_received_work = array();
         $mailId = get_mail_id_base();
         foreach ($this->receivedWork as $work) {
-			switch ($type) {
-				case 'uploader_id':
-					if ($work->uploader_id == $value ||
-						($work->uploader_id > $mailId &&
+            switch ($type) {
+                case 'uploader_id':
+                    if ($work->uploader_id == $value ||
+                        ($work->uploader_id > $mailId &&
                         getUserOwningThisMailing($work->uploader_id) == $value)
                     ) {
-						$new_received_work[] = $work;
-					}
-					break;
-				default:
-					$new_received_work[] = $work;
+                        $new_received_work[] = $work;
+                    }
+                    break;
+                default:
+                    $new_received_work[] = $work;
                     break;
-			}
-		}
-		$this->receivedWork = $new_received_work;
-	}
+            }
+        }
+        $this->receivedWork = $new_received_work;
+    }
 }

main/inc/lib/chat.lib.php

Indentation +10 added lines, -10 removed lines

@@ -39,12 +39,12 @@ 
     }
 
     /**
-    * Set user chat status
-    * @param int 0 if disconnected, 1 if connected
-    * @param integer $status
+     * Set user chat status
+     * @param int 0 if disconnected, 1 if connected
+     * @param integer $status
      *
-    * @return void
-    */
+     * @return void
+     */
     public function setUserStatus($status)
     {
         UserManager::update_extra_field_value(api_get_user_id(), 'user_chat_status', $status);
@@ -148,10 +148,10 @@ 
     }
 
     /**
-    * Returns an array of messages inside a chat session with a specific user
-    * @param int The ID of the user with whom the current user is chatting
-    * @return array Messages list
-    */
+     * Returns an array of messages inside a chat session with a specific user
+     * @param int The ID of the user with whom the current user is chatting
+     * @return array Messages list
+     */
     public function box_session($user_id)
     {
         $items = array();
@@ -299,7 +299,7 @@ 
             }
         }
          
-         return false;
+            return false;
     }
     
     public function is_chat_blocked_by_exercises()

Spacing +3 added lines, -3 removed lines

@@ -188,7 +188,7 @@ 
         $to_user_id,
         $message,
         $printResult = true,
-        $sanitize =  true
+        $sanitize = true
     ) {
         $user_friend_relation = SocialManager::get_relation_between_contacts(
             $from_user_id,
@@ -289,9 +289,9 @@ 
      */
     public static function disableChat()
     {
-        if (!empty($_SESSION['disable_chat'])){
+        if (!empty($_SESSION['disable_chat'])) {
             $status = $_SESSION['disable_chat'];
-            if ($status == true){
+            if ($status == true) {
                 $_SESSION['disable_chat'] = null;
                 return true;
             }