chamilo / chamilo-lms

src/Chamilo/CoreBundle/Entity/ClassItem.php

Doc Comments +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
      * Set code
      *
      * @param string $code
-     * @return Class
+     * @return ClassItem
      */
     public function setCode($code)
     {
@@ -62,7 +62,7 @@ 
      * Set name
      *
      * @param string $name
-     * @return Class
+     * @return ClassItem
      */
     public function setName($name)
     {

src/Chamilo/CoreBundle/Entity/UsergroupRelUser.php

Doc Comments +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
      * Set relationType
      *
      * @param integer $relationType
-     * @return GroupRelUser
+     * @return UsergroupRelUser
      */
     public function setRelationType($relationType)
     {

src/Chamilo/FaqBundle/Entity/QuestionTranslation.php

Doc Comments +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
 
     /**
      * @param string $headline
-     * @return CategoryTranslation
+     * @return QuestionTranslation
      */
     public function setHeadline($headline)
     {
@@ -77,7 +77,7 @@ 
 
     /**
      * @param string $body
-     * @return CategoryTranslation
+     * @return QuestionTranslation
      */
     public function setBody($body)
     {

src/Chamilo/ThemeBundle/Model/MenuItemInterface.php

Doc Comments +12 added lines, -12 removed lines

@@ -20,29 +20,29 @@ 
     public function getIdentifier();
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getLabel();
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getRoute();
 
     /**
-     * @return mixed
+     * @return boolean
      */
     public function isActive();
 
     /**
-     * @param $isActive
+     * @param boolean $isActive
      *
-     * @return mixed
+     * @return MenuItemModel
      */
     public function setIsActive($isActive);
 
     /**
-     * @return mixed
+     * @return boolean
      */
     public function hasChildren();
 
@@ -54,14 +54,14 @@ 
     /**
      * @param MenuItemInterface $child
      *
-     * @return mixed
+     * @return MenuItemModel
      */
     public function addChild(MenuItemInterface $child);
 
     /**
      * @param MenuItemInterface $child
      *
-     * @return mixed
+     * @return MenuItemModel
      */
     public function removeChild(MenuItemInterface $child);
 
@@ -76,24 +76,24 @@ 
     public function getBadge();
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getBadgeColor();
 
     /**
-     * @return mixed
+     * @return MenuItemInterface
      */
     public function getParent();
 
     /**
-     * @return mixed
+     * @return boolean
      */
     public function hasParent();
 
     /**
      * @param MenuItemInterface $parent
      *
-     * @return mixed
+     * @return MenuItemModel
      */
     public function setParent(MenuItemInterface $parent = null);
 

main/auth/cas/authcas.php

Doc Comments +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 require_once api_get_path(SYS_PATH) . 'main/auth/external_login/functions.inc.php';
 
 /**
- * @return true if cas is configured
+ * @return boolean if cas is configured
  *
  **/
 function cas_configured()

Spacing +5 added lines, -5 removed lines

@@ -5,9 +5,9 @@ 
 * Gets all the info via the ldap module (ldap has to work)
 
 */
-require_once api_get_path(SYS_PATH) . 'main/auth/cas/cas_var.inc.php';
-require_once api_get_path(SYS_PATH) . 'main/auth/external_login/ldap.inc.php';
-require_once api_get_path(SYS_PATH) . 'main/auth/external_login/functions.inc.php';
+require_once api_get_path(SYS_PATH).'main/auth/cas/cas_var.inc.php';
+require_once api_get_path(SYS_PATH).'main/auth/external_login/ldap.inc.php';
+require_once api_get_path(SYS_PATH).'main/auth/external_login/functions.inc.php';
 
 /**
  * @return true if cas is configured
@@ -187,14 +187,14 @@ 
  */
 function get_cas_direct_URL($in_course_code)
 {
-    return api_get_path(WEB_PATH) . 'main/auth/cas/logincas.php?firstpage=' . $in_course_code;
+    return api_get_path(WEB_PATH).'main/auth/cas/logincas.php?firstpage='.$in_course_code;
 }
 
 function getCASLogoHTML()
 {
     $out_res = "";
     if (api_get_setting("casLogoURL") != "") {
-        $out_res = "<img src='" . api_get_setting("casLogoURL") . "' alt='CAS Logo' />";
+        $out_res = "<img src='".api_get_setting("casLogoURL")."' alt='CAS Logo' />";
     }
     return $out_res;
 }

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();