chamilo / chamilo-lms

plugin/clockworksms/lib/clockworksms_plugin.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     /**
      * create (a singleton function that ensures ClockworksmsPlugin instance is
      * created only once. If it is already created, it returns the instance)
-     * @return  object  ClockworksmsPlugin instance
+     * @return  ClockworksmsPlugin  ClockworksmsPlugin instance
      */
     public static function create()
     {

plugin/kannelsms/lib/kannelsms_plugin.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     /**
      * create (a singleton function that ensures KannelsmsPlugin instance is
      * created only once. If it is already created, it returns the instance)
-     * @return  object  KannelsmsPlugin instance
+     * @return  KannelsmsPlugin  KannelsmsPlugin instance
      */
     public static function create()
     {

plugin/openmeetings/lib/openmeetings.class.php

Doc Comments +2 added lines, -1 removed lines

@@ -213,7 +213,8 @@
     /**
      * Returns a meeting "join" URL
      * @param string The name of the meeting (usually the course code)
-     * @return mixed The URL to join the meeting, or false on error
+     * @param false|string $meetingId
+     * @return false|null The URL to join the meeting, or false on error
      * @todo implement moderator pass
      * @assert ('') === false
      * @assert ('abcdefghijklmnopqrstuvwxyzabcdefghijklmno') === false

Indentation +6 added lines, -6 removed lines

@@ -126,7 +126,7 @@ 
             return false;
         }
     }
-    */
+     */
 
     /*
      * Creating a Room for the meeting
@@ -256,10 +256,10 @@ 
         return true;
     }
 
-     /**
-     * Gets the password for a specific meeting for the current user
-     * @return string A moderator password if user is teacher, or the course code otherwise
-     */
+        /**
+         * Gets the password for a specific meeting for the current user
+         * @return string A moderator password if user is teacher, or the course code otherwise
+         */
     public function getMeetingUserPassword()
     {
         if ($this->isTeacher()) {
@@ -538,7 +538,7 @@ 
     //
     //            }
     //
-                 //$item['created_at'] = api_convert_and_format_date($meetingDb['created_at']);
+                    //$item['created_at'] = api_convert_and_format_date($meetingDb['created_at']);
     //            //created_at
     //
     //            $item['publish_url'] = api_get_self().'?action=publish&id='.$meetingDb['id'];

plugin/openmeetings/lib/openmeetings_gateway.php

Doc Comments +5 added lines

@@ -36,6 +36,11 @@
     private $_pass;
     private $_url;
 
+    /**
+     * @param string $host
+     * @param string $user
+     * @param string $pass
+     */
     public function __construct($host, $user, $pass)
     {
         $this->_user = urlencode($user);

Indentation +3 added lines, -3 removed lines

@@ -351,9 +351,9 @@
                 . '&externalRoomType='.$room->externalRoomType;
         if ($room->allowRecording) {
             $url .= '&allowUserQuestions='.$this->var_to_str($room->allowUserQuestions)
-                 . '&isAudioOnly='.$this->var_to_str($room->isAudioOnly)
-                 . '&waitForRecording='.$this->var_to_str($room->waitForRecording)
-                 . '&allowRecording='.$this->var_to_str($room->allowRecording);
+                    . '&isAudioOnly='.$this->var_to_str($room->isAudioOnly)
+                    . '&waitForRecording='.$this->var_to_str($room->waitForRecording)
+                    . '&allowRecording='.$this->var_to_str($room->allowRecording);
         } elseif ($room->isAudioOnly) {
             $url .= '&isAudioOnly='.$this->var_to_str($room->isAudioOnly);
         }

plugin/openmeetings/lib/openmeetings_rest_service.php

Doc Comments +6 added lines

@@ -27,6 +27,9 @@ 
  */
 class OpenMeetingsRestService
 {
+    /**
+     * @param string $request
+     */
     function call($request, $returnAttribute = "return")
     {
         // This will allow you to view errors in the browser
@@ -121,6 +124,9 @@ 
             
     }
 
+    /**
+     * @param DOMDocument $node
+     */
     function getArray($node)
     {
         if (is_null($node) || !is_object($node)) {

Spacing +13 added lines, -13 removed lines

@@ -40,31 +40,31 @@ 
         // $request
         
         // Initialize the session by passing the request as a parameter
-        $session = curl_init ( $request );
+        $session = curl_init($request);
         
         // Set curl options by passing session and flags
         // CURLOPT_HEADER allows us to receive the HTTP header
-        curl_setopt ( $session, CURLOPT_HEADER, true );
+        curl_setopt($session, CURLOPT_HEADER, true);
         
         // CURLOPT_RETURNTRANSFER will return the response
-        curl_setopt ( $session, CURLOPT_RETURNTRANSFER, true );
+        curl_setopt($session, CURLOPT_RETURNTRANSFER, true);
         
         // Make the request
-        $response = curl_exec ( $session );
+        $response = curl_exec($session);
         
         // Close the curl session
-        curl_close ( $session );
+        curl_close($session);
         
         // Confirm that the request was transmitted to the OpenMeetings! Image Search Service
-        if (! $response) {
-            die ( "Request OpenMeetings! OpenMeetings Service failed and no response was returned in ".__CLASS__.'::'.__FUNCTION__.'()' );
+        if (!$response) {
+            die ("Request OpenMeetings! OpenMeetings Service failed and no response was returned in ".__CLASS__.'::'.__FUNCTION__.'()');
         }
         
         // Create an array to store the HTTP response codes
-        $status_code = array ();
+        $status_code = array();
         
         // Use regular expressions to extract the code from the header
-        preg_match ( '/\d\d\d/', $response, $status_code );
+        preg_match('/\d\d\d/', $response, $status_code);
         $bt = debug_backtrace();
         $caller = array_shift($bt);
         $extra = ' (from '.$caller['file'].' at line '.$caller['line'].') ';
@@ -74,24 +74,24 @@ 
                 // Success
                 break;
             case 503 :
-                error_log( 'Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 503.
+                error_log('Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 503.
                                  That means: Service unavailable. An internal problem prevented us from returning data to you.' );
                 return false;
                 break;
             case 403 :
-                error_log( 'Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 403.
+                error_log('Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 403.
                                  That means: Forbidden. You do not have permission to access this resource, or are over your rate limit.' );
                 return false;
                 break;
             case 400 :
                 // You may want to fall through here and read the specific XML error
-                error_log( 'Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 400.
+                error_log('Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 400.
                                  That means:  Bad request. The parameters passed to the service did not match as expected.   
                                  The exact error is returned in the XML response.' );
                 return false;
                 break;
             default:
-                error_log( 'Your call to OpenMeetings Web Services '.$extra.' returned an unexpected HTTP status of: ' . $status_code [0] . " Request " . $request );
+                error_log('Your call to OpenMeetings Web Services '.$extra.' returned an unexpected HTTP status of: '.$status_code [0]." Request ".$request);
                 return false;
         }
         

plugin/openmeetings/lib/room.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
     /**
      * Gets a string from a boolean attribute
      * @param string $attribute Name of the attribute
-     * @param mixed  $voidReturn What to return if the value is not defined
+     * @param string  $voidReturn What to return if the value is not defined
      * @return string The boolean value expressed as string ('true' or 'false')
      */
     public function getString($attribute, $voidReturn = false)

plugin/search_course/lib/search_course_widget.class.php

Doc Comments +8 added lines

@@ -40,11 +40,19 @@
         return isset($_GET[$key]) ? $_GET[$key] : $default;
     }
 
+    /**
+     * @param string $key
+     *
+     * @return string
+     */
     public static function server($key, $default = '')
     {
         return isset($_SERVER[$key]) ? $_SERVER[$key] : $default;
     }
 
+    /**
+     * @param string $name
+     */
     public static function get_lang($name)
     {
         return SearchCoursePlugin::create()->get_lang($name);

Braces +15 added lines, -33 removed lines

@@ -108,8 +108,7 @@ 
      */
     function run()
     {
-        if (!$this->accept())
-        {
+        if (!$this->accept()) {
             return;
         }
         $this->display_header();
@@ -120,21 +119,15 @@ 
         $action = self::get('action');
 
         $has_content = !empty($search_term) || !empty($action);
-        if ($has_content)
-        {
+        if ($has_content) {
             echo '<div class="list">';
-        }
-        else
-        {
+        } else {
             echo '<div>';
         }
 
-        if (RegisterCourseWidget::factory()->run())
-        {
+        if (RegisterCourseWidget::factory()->run()) {
             $result = true;
-        }
-        else
-        {
+        } else {
             $result = $this->action_display();
         }
 
@@ -148,8 +141,7 @@ 
     {
         $self = $_SERVER['PHP_SELF'];
         $parameters = array();
-        if ($action)
-        {
+        if ($action) {
             $parameters[self::PARAM_ACTION] = $action;
         }
         $parameters = implode('&', $parameters);
@@ -165,8 +157,7 @@ 
         global $charset;
 
         $search_term = self::post('search_term');
-        if ($search_term)
-        {
+        if ($search_term) {
             $search_result_for_label = self::get_lang('SearchResultsFor');
             $search_term_html = htmlentities($search_term, ENT_QUOTES, $charset);
             echo "<h5>$search_result_for_label $search_term_html</h5>";
@@ -275,8 +266,7 @@ 
 
         //Already subscribed
         $code = $current_course['code'];
-        if (isset($user_courses[$code]))
-        {
+        if (isset($user_courses[$code])) {
             echo self::get_lang('AlreadySubscribed');
             return false;
         }
@@ -296,8 +286,7 @@ 
 EOT;
 
         $search_term = $this->post('search_term');
-        if ($search_term)
-        {
+        if ($search_term) {
             $search_term = Security::remove_XSS($search_term);
             echo <<<EOT
                     <input type="hidden" name="search_course" value="1" />
@@ -324,19 +313,15 @@ 
      */
     function retrieve_courses($search_term)
     {
-        if (empty($search_term))
-        {
+        if (empty($search_term)) {
             return array();
         }
         $search_term = Database::escape_string($search_term);
         $course_table = Database::get_main_table(TABLE_MAIN_COURSE);
 
-        if (api_is_anonymous())
-        {
+        if (api_is_anonymous()) {
             $course_fiter = 'visibility = ' . COURSE_VISIBILITY_OPEN_WORLD;
-        }
-        else
-        {
+        } else {
             $course_fiter = 'visibility = ' . COURSE_VISIBILITY_OPEN_WORLD . ' OR ';
             $course_fiter .= 'visibility = ' . COURSE_VISIBILITY_OPEN_PLATFORM . ' OR ';
             $course_fiter .= '(visibility = ' . COURSE_VISIBILITY_REGISTERED . ' AND subscribe = 1)';
@@ -374,8 +359,7 @@ 
      */
     function retrieve_user_courses($user_id = null)
     {
-        if (is_null($user_id))
-        {
+        if (is_null($user_id)) {
             global $_user;
             $user_id = $_user['user_id'];
         }
@@ -423,8 +407,7 @@ 
      */
     function filter_out_user_courses($courses)
     {
-        if (empty($courses))
-        {
+        if (empty($courses)) {
             return $courses;
         }
 
@@ -432,8 +415,7 @@ 
         $user_id = $_user['user_id'];
 
         $user_courses = $this->retrieve_user_courses($user_id);
-        foreach ($user_courses as $key => $value)
-        {
+        foreach ($user_courses as $key => $value) {
             unset($courses[$key]);
         }
         return $courses;

Upper-Lower-Casing +5 added lines, -5 removed lines

@@ -184,7 +184,7 @@ 
         <div class="well course_search">
         <div class="menusection">
             <h4>$search_course_label</h4>
-EOT;
+eot;
     }
 
     function display_footer()
@@ -209,7 +209,7 @@ 
             <input type="text" name="search_term" class="span2" value="$search_term" />
             &nbsp;<input class="btn btn-default" type="submit" value="$search_label" />
         </form>
-EOT;
+eot;
         echo $form;
     }
 
@@ -293,7 +293,7 @@ 
                 <form action="$self?action=subscribe" method="post">
                     <input type="hidden" name="sec_token" value="$stok" />
                     <input type="hidden" name="subscribe" value="$code" />
-EOT;
+eot;
 
         $search_term = $this->post('search_term');
         if ($search_term)
@@ -302,7 +302,7 @@ 
             echo <<<EOT
                     <input type="hidden" name="search_course" value="1" />
                     <input type="hidden" name="search_term" value="$search_term" />
-EOT;
+eot;
         }
         echo '<input type="image" name="unsub" src="'.Display::returnIconPath('enroll.gif').'" alt="'.get_lang('Subscribe').'" />
                 '.get_lang('Subscribe').'
@@ -346,7 +346,7 @@ 
                 SELECT * FROM $course_table
                 WHERE ($course_fiter) AND (code LIKE '%$search_term%' OR visual_code LIKE '%$search_term%' OR title LIKE '%$search_term%' OR tutor_name LIKE '%$search_term%')
                 ORDER BY title, visual_code ASC
-EOT;
+eot;
 
         $result = array();
         $resultset = Database::query($sql);

Spacing +12 added lines, -12 removed lines

@@ -60,12 +60,12 @@ 
         $url = self::server('REQUEST_URI');
         $url = explode('?', $url);
         $url = reset($url);
-        $url = self::server('SERVER_NAME') . $url;
+        $url = self::server('SERVER_NAME').$url;
 
         $root = api_get_path('WEB_PATH');
         $root = str_replace('https://', '', $root);
         $root = str_replace('http://', '', $root);
-        $index_url = $root . 'index.php';
+        $index_url = $root.'index.php';
 
         return $url == $index_url || $url == $root;
     }
@@ -80,12 +80,12 @@ 
         $url = self::server('REQUEST_URI');
         $url = explode('?', $url);
         $url = reset($url);
-        $url = self::server('SERVER_NAME') . $url;
+        $url = self::server('SERVER_NAME').$url;
 
         $root = api_get_path('WEB_PATH');
         $root = str_replace('https://', '', $root);
         $root = str_replace('http://', '', $root);
-        $index_url = $root . 'user_portal.php';
+        $index_url = $root.'user_portal.php';
 
         return $url == $index_url || $url == $root;
     }
@@ -153,8 +153,8 @@ 
             $parameters[self::PARAM_ACTION] = $action;
         }
         $parameters = implode('&', $parameters);
-        $parameters = $parameters ? '?' . $parameters : '';
-        return $self . $parameters;
+        $parameters = $parameters ? '?'.$parameters : '';
+        return $self.$parameters;
     }
 
     /**
@@ -241,8 +241,8 @@ 
             $details = implode(' - ', $details);
             $title = $course['title'];
 
-            $href = api_get_path(WEB_COURSE_PATH).$course['code'] .'/index.php';
-            echo '<tr><td><b><a href="' . $href . '">' . "$title</a></b><br/>$details</td><td>";
+            $href = api_get_path(WEB_COURSE_PATH).$course['code'].'/index.php';
+            echo '<tr><td><b><a href="'.$href.'">'."$title</a></b><br/>$details</td><td>";
             if (!api_is_anonymous()) {
                 if ($course['registration_code']) {
                     Display::display_icon(
@@ -333,13 +333,13 @@ 
 
         if (api_is_anonymous())
         {
-            $course_fiter = 'visibility = ' . COURSE_VISIBILITY_OPEN_WORLD;
+            $course_fiter = 'visibility = '.COURSE_VISIBILITY_OPEN_WORLD;
         }
         else
         {
-            $course_fiter = 'visibility = ' . COURSE_VISIBILITY_OPEN_WORLD . ' OR ';
-            $course_fiter .= 'visibility = ' . COURSE_VISIBILITY_OPEN_PLATFORM . ' OR ';
-            $course_fiter .= '(visibility = ' . COURSE_VISIBILITY_REGISTERED . ' AND subscribe = 1)';
+            $course_fiter = 'visibility = '.COURSE_VISIBILITY_OPEN_WORLD.' OR ';
+            $course_fiter .= 'visibility = '.COURSE_VISIBILITY_OPEN_PLATFORM.' OR ';
+            $course_fiter .= '(visibility = '.COURSE_VISIBILITY_REGISTERED.' AND subscribe = 1)';
         }
 
         $sql = <<<EOT

src/Chamilo/CoreBundle/Component/Editor/Finder.php

Spacing +5 added lines, -5 removed lines

@@ -31,7 +31,7 @@ 
         $this->time  = $this->utime();
         $this->debug = (isset($opts['debug']) && $opts['debug'] ? true : false);
         $this->timeout = (isset($opts['timeout']) ? $opts['timeout'] : 0);
-        $this->netVolumesSessionKey = !empty($opts['netVolumesSessionKey'])? $opts['netVolumesSessionKey'] : 'elFinderNetVolumes';
+        $this->netVolumesSessionKey = !empty($opts['netVolumesSessionKey']) ? $opts['netVolumesSessionKey'] : 'elFinderNetVolumes';
         $this->callbackWindowURL = (isset($opts['callbackWindowURL']) ? $opts['callbackWindowURL'] : '');
 
         // setlocale and global locale regists to elFinder::locale
@@ -46,20 +46,20 @@ 
             $_reqCmd = isset($_req['cmd']) ? $_req['cmd'] : '';
             foreach ($opts['bind'] as $cmd => $handlers) {
                 $doRegist = (strpos($cmd, '*') !== false);
-                if (! $doRegist) {
+                if (!$doRegist) {
                     $_getcmd = create_function('$cmd', 'list($ret) = explode(\'.\', $cmd);return trim($ret);');
                     $doRegist = ($_reqCmd && in_array($_reqCmd, array_map($_getcmd, explode(' ', $cmd))));
                 }
                 if ($doRegist) {
-                    if (! is_array($handlers) || is_object($handlers[0])) {
+                    if (!is_array($handlers) || is_object($handlers[0])) {
                         $handlers = array($handlers);
                     }
-                    foreach($handlers as $handler) {
+                    foreach ($handlers as $handler) {
                         if ($handler) {
                             if (is_string($handler) && strpos($handler, '.')) {
                                 list($_domain, $_name, $_method) = array_pad(explode('.', $handler), 3, '');
                                 if (strcasecmp($_domain, 'plugin') === 0) {
-                                    if ($plugin = $this->getPluginInstance($_name, isset($opts['plugin'][$_name])? $opts['plugin'][$_name] : array())
+                                    if ($plugin = $this->getPluginInstance($_name, isset($opts['plugin'][$_name]) ? $opts['plugin'][$_name] : array())
                                         and method_exists($plugin, $_method)) {
                                         $this->bind($cmd, array($plugin, $_method));
                                     }

plugin/resubscription/src/HookResubscription.php

Indentation +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 
             $resubscriptionLimit = Resubscription::create()->get('resubscription_limit');
 
-             // Initialize variables as a calendar year by default
+                // Initialize variables as a calendar year by default
             $limitDateFormat = 'Y-01-01';
             $limitDate = gmdate($limitDateFormat);
             $resubscriptionOffset = "1 year";

Doc Comments +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 
     /**
      * Limit session resubscription when a Chamilo user is resubscribed to a session
-     * @param HookCreateUserEventInterface $hook The hook
+     * @param HookResubscribeEventInterface $hook The hook
      */
     public function hookResubscribe(HookResubscribeEventInterface $hook)
     {