chamilo / chamilo-lms

main/inc/lib/template.lib.php

Spacing +30 added lines, -30 removed lines

@@ -264,7 +264,7 @@ 
                 $content = '<li class="help">';
                 $content .= Display::url(
                     Display::return_icon('help.large.png', get_lang('Help')),
-                    api_get_path(WEB_CODE_PATH) . 'help/help.php?open=' . $help,
+                    api_get_path(WEB_CODE_PATH).'help/help.php?open='.$help,
                     [
                         'class' => 'ajax',
                         'data-title' => get_lang('Help')
@@ -478,7 +478,7 @@ 
             'web_course' => api_get_path(WEB_COURSE_PATH),
             'web_main' => api_get_path(WEB_CODE_PATH),
             'web_css' => api_get_path(WEB_CSS_PATH),
-            'web_css_theme' => api_get_path(WEB_CSS_PATH) . 'themes/' . $this->theme . '/',
+            'web_css_theme' => api_get_path(WEB_CSS_PATH).'themes/'.$this->theme.'/',
             'web_ajax' => api_get_path(WEB_AJAX_PATH),
             'web_img' => api_get_path(WEB_IMG_PATH),
             'web_plugin' => api_get_path(WEB_PLUGIN_PATH),
@@ -536,12 +536,12 @@ 
         foreach ($bowerCSSFiles as $file) {
             $css[] = api_get_path(WEB_PATH).'web/assets/'.$file;
         }
-        $css[] = api_get_path(WEB_LIBRARY_PATH) . 'javascript/bootstrap-select/css/bootstrap-select.min.css';
-        $css[] = api_get_path(WEB_LIBRARY_PATH) . 'javascript/chosen/chosen.css';
-        $css[] = api_get_path(WEB_LIBRARY_PATH) . 'javascript/tag/style.css';
+        $css[] = api_get_path(WEB_LIBRARY_PATH).'javascript/bootstrap-select/css/bootstrap-select.min.css';
+        $css[] = api_get_path(WEB_LIBRARY_PATH).'javascript/chosen/chosen.css';
+        $css[] = api_get_path(WEB_LIBRARY_PATH).'javascript/tag/style.css';
 
         if (api_is_global_chat_enabled()) {
-            $css[] = api_get_path(WEB_LIBRARY_PATH) . 'javascript/chat/css/chat.css';
+            $css[] = api_get_path(WEB_LIBRARY_PATH).'javascript/chat/css/chat.css';
         }
 
         //THEME CSS STYLE
@@ -608,7 +608,7 @@ 
 
             $style_print = '';
             if (is_readable(api_get_path(SYS_CSS_PATH).$this->theme.'/print.css')) {
-                $style_print = api_get_css(api_get_cdn_path(api_get_path(WEB_CSS_PATH) . $this->theme . '/print.css'),
+                $style_print = api_get_css(api_get_cdn_path(api_get_path(WEB_CSS_PATH).$this->theme.'/print.css'),
                     'print');
             }
             $this->assign('css_style_print', $style_print);
@@ -631,10 +631,10 @@ 
 
         $isoCode = api_get_language_isocode();
 
-        $selectLink = 'bootstrap-select/js/i18n/defaults-' . $isoCode . '_' . strtoupper($isoCode) . '.min.js';
+        $selectLink = 'bootstrap-select/js/i18n/defaults-'.$isoCode.'_'.strtoupper($isoCode).'.min.js';
 
         if ($isoCode == 'en') {
-            $selectLink = 'bootstrap-select/js/i18n/defaults-' . $isoCode . '_US.min.js';
+            $selectLink = 'bootstrap-select/js/i18n/defaults-'.$isoCode.'_US.min.js';
         }
         // JS files
         $js_files = array(
@@ -684,8 +684,8 @@ 
         }
 
         if ($isoCode != 'en') {
-            $bowerJsFiles[] = 'jqueryui-timepicker-addon/dist/i18n/jquery-ui-timepicker-' . $isoCode . '.js';
-            $bowerJsFiles[] = 'jquery-ui/ui/minified/i18n/datepicker-' . $isoCode . '.min.js';
+            $bowerJsFiles[] = 'jqueryui-timepicker-addon/dist/i18n/jquery-ui-timepicker-'.$isoCode.'.js';
+            $bowerJsFiles[] = 'jquery-ui/ui/minified/i18n/datepicker-'.$isoCode.'.min.js';
         }
 
         foreach ($bowerJsFiles as $file) {
@@ -766,7 +766,7 @@ 
         }
 
         $this->assign('online_button', Display::return_icon('statusonline.png', null, null, ICON_SIZE_ATOM));
-        $this->assign('offline_button',Display::return_icon('statusoffline.png', null, null, ICON_SIZE_ATOM));
+        $this->assign('offline_button', Display::return_icon('statusoffline.png', null, null, ICON_SIZE_ATOM));
 
         // Get language iso-code for this page - ignore errors
         $this->assign('document_language', api_get_language_isocode());
@@ -825,15 +825,15 @@ 
         $this->assign('section_name', 'section-'.$this_section);
 
         //Defaul root chamilo favicon
-        $favico = '<link rel="shortcut icon" href="' . api_get_path(WEB_PATH) . 'favicon.ico" type="image/x-icon" />';
+        $favico = '<link rel="shortcut icon" href="'.api_get_path(WEB_PATH).'favicon.ico" type="image/x-icon" />';
 
         //Added to verify if in the current Chamilo Theme exist a favicon
-        $favicoThemeUrl = api_get_path(SYS_CSS_PATH) . 'themes/' . $this->theme . '/images/';
+        $favicoThemeUrl = api_get_path(SYS_CSS_PATH).'themes/'.$this->theme.'/images/';
 
         //If exist pick the current chamilo theme favicon
-        if (is_file($favicoThemeUrl . 'favicon.ico')) {
-            $favico = '<link rel="shortcut icon" href="' . api_get_path(WEB_CSS_PATH)
-                . 'themes/' . $this->theme . '/images/favicon.ico" type="image/x-icon" />';
+        if (is_file($favicoThemeUrl.'favicon.ico')) {
+            $favico = '<link rel="shortcut icon" href="'.api_get_path(WEB_CSS_PATH)
+                . 'themes/'.$this->theme.'/images/favicon.ico" type="image/x-icon" />';
         }
 
         if (api_is_multiple_url_enabled()) {
@@ -863,7 +863,7 @@ 
         if (api_get_setting('show_link_bug_notification') == 'true' && $this->user_is_logged_in) {
             $bug_notification_link = '<li class="report">
 		<a href="http://support.chamilo.org/projects/chamilo-18/wiki/How_to_report_bugs" target="_blank">
-                    '. $iconBug . '
+                    '. $iconBug.'
                 </a>
 		</li>';
         }
@@ -975,26 +975,26 @@ 
         $socialMeta = '';
         $metaTitle = api_get_setting('meta_title');
         if (!empty($metaTitle)) {
-            $socialMeta .= '<meta name="twitter:card" content="summary" />' . "\n";
-            $socialMeta .= '<meta property="og:title" content="' . $metaTitle . '" />' . "\n";
-            $socialMeta .= '<meta property="og:url" content="' . api_get_path(WEB_PATH) . '" />' . "\n";
+            $socialMeta .= '<meta name="twitter:card" content="summary" />'."\n";
+            $socialMeta .= '<meta property="og:title" content="'.$metaTitle.'" />'."\n";
+            $socialMeta .= '<meta property="og:url" content="'.api_get_path(WEB_PATH).'" />'."\n";
             $metaDescription = api_get_setting('meta_description');
             if (!empty($metaDescription)) {
-                $socialMeta .= '<meta property="og:description" content="' . $metaDescription . '" />' . "\n";
+                $socialMeta .= '<meta property="og:description" content="'.$metaDescription.'" />'."\n";
             }
             $metaSite = api_get_setting('meta_twitter_site');
             if (!empty($metaSite)) {
-                $socialMeta .= '<meta name="twitter:site" content="' . $metaSite . '" />' . "\n";
+                $socialMeta .= '<meta name="twitter:site" content="'.$metaSite.'" />'."\n";
                 $metaCreator = api_get_setting('meta_twitter_creator');
                 if (!empty($metaCreator)) {
-                    $socialMeta .= '<meta name="twitter:creator" content="' . $metaCreator . '" />' . "\n";
+                    $socialMeta .= '<meta name="twitter:creator" content="'.$metaCreator.'" />'."\n";
                 }
             }
             $metaImage = api_get_setting('meta_image_path');
             if (!empty($metaImage)) {
-                if (is_file(api_get_path(SYS_PATH) . $metaImage)) {
-                    $path = api_get_path(WEB_PATH) . $metaImage;
-                    $socialMeta .= '<meta property="og:image" content="' . $path . '" />' . "\n";
+                if (is_file(api_get_path(SYS_PATH).$metaImage)) {
+                    $path = api_get_path(WEB_PATH).$metaImage;
+                    $socialMeta .= '<meta property="og:image" content="'.$path.'" />'."\n";
                 }
             }
         }
@@ -1027,7 +1027,7 @@ 
         // Tutor name
         if (api_get_setting('show_tutor_data') == 'true') {
             // Course manager
-            $courseId  = api_get_course_int_id();
+            $courseId = api_get_course_int_id();
             $id_session = api_get_session_id();
             if (!empty($courseId)) {
                 $tutor_data = '';
@@ -1059,7 +1059,7 @@ 
             $courseId = api_get_course_int_id();
             if (!empty($courseId)) {
                 $teacher_data = '';
-                $mail= CourseManager::get_emails_of_tutors_to_course($courseId);
+                $mail = CourseManager::get_emails_of_tutors_to_course($courseId);
                 if (!empty($mail)) {
                     $teachers_parsed = array();
                     foreach ($mail as $value) {
@@ -1293,7 +1293,7 @@ 
                     'sessionVar'   => basename(__FILE__, '.php'),
                     'imageOptions' => array(
                         'font_size' => 20,
-                        'font_path' => api_get_path(SYS_FONTS_PATH) . 'opensans/',
+                        'font_path' => api_get_path(SYS_FONTS_PATH).'opensans/',
                         'font_file' => 'OpenSans-Regular.ttf',
                         //'output' => 'gif'
                     )

main/inc/lib/diagnoser.lib.php

Spacing +28 added lines, -28 removed lines

@@ -15,7 +15,7 @@ 
 {
     const STATUS_OK = 1;
     const STATUS_WARNING = 2;
-    const STATUS_ERROR 	= 3;
+    const STATUS_ERROR = 3;
     const STATUS_INFORMATION = 4;
 
     /**
@@ -44,21 +44,21 @@ 
                 $html .= '<li>';
             }
             $params['section'] = $section;
-            $html .='<a href="system_status.php?section='.$section.'">'.get_lang($section).'</a></li>';
+            $html .= '<a href="system_status.php?section='.$section.'">'.get_lang($section).'</a></li>';
         }
 
         $html .= '</ul><div class="tab-pane">';
 
-        $data = call_user_func(array($this, 'get_' . $current_section . '_data'));
+        $data = call_user_func(array($this, 'get_'.$current_section.'_data'));
         echo $html;
         $table = new SortableTableFromArray($data, 1, 100);
 
-        $table->set_header(0,'', false);
-        $table->set_header(1,get_lang('Section'), false);
-        $table->set_header(2,get_lang('Setting'), false);
-        $table->set_header(3,get_lang('Current'), false);
-        $table->set_header(4,get_lang('Expected'), false);
-        $table->set_header(5,get_lang('Comment'), false);
+        $table->set_header(0, '', false);
+        $table->set_header(1, get_lang('Section'), false);
+        $table->set_header(2, get_lang('Setting'), false);
+        $table->set_header(3, get_lang('Current'), false);
+        $table->set_header(4, get_lang('Expected'), false);
+        $table->set_header(5, get_lang('Comment'), false);
 
         $table->display();
         echo '</div></div>';
@@ -72,11 +72,11 @@ 
     {
         $array = array();
         $writable_folders = array(
-            api_get_path(SYS_APP_PATH) .'cache',
+            api_get_path(SYS_APP_PATH).'cache',
             api_get_path(SYS_COURSE_PATH),
-            api_get_path(SYS_APP_PATH) .'home',
-            api_get_path(SYS_APP_PATH) .'upload/users/',
-            api_get_path(SYS_PATH) .'main/default_course_document/images/',
+            api_get_path(SYS_APP_PATH).'home',
+            api_get_path(SYS_APP_PATH).'upload/users/',
+            api_get_path(SYS_PATH).'main/default_course_document/images/',
         );
         foreach ($writable_folders as $index => $folder) {
             $writable = is_writable($folder);
@@ -95,10 +95,10 @@ 
 
         $exists = file_exists(api_get_path(SYS_CODE_PATH).'install');
         $status = $exists ? self :: STATUS_WARNING : self :: STATUS_OK;
-        $array[] = $this->build_setting($status, '[FILES]', get_lang('DirectoryExists') . ': /install', 'http://be2.php.net/file_exists', $exists, 0, 'yes_no', get_lang('DirectoryShouldBeRemoved'));
+        $array[] = $this->build_setting($status, '[FILES]', get_lang('DirectoryExists').': /install', 'http://be2.php.net/file_exists', $exists, 0, 'yes_no', get_lang('DirectoryShouldBeRemoved'));
 
         $app_version = api_get_setting('chamilo_database_version');
-        $array[] = $this->build_setting(self :: STATUS_INFORMATION, '[DB]', 'chamilo_database_version', '#', $app_version, 0, null,  'Chamilo DB version');
+        $array[] = $this->build_setting(self :: STATUS_INFORMATION, '[DB]', 'chamilo_database_version', '#', $app_version, 0, null, 'Chamilo DB version');
 
         return $array;
     }
@@ -165,33 +165,33 @@ 
         $array[] = $this->build_setting($status, '[INI]', 'default_charset', 'http://www.php.net/manual/en/ini.core.php#ini.default-charset', $setting, $req_setting, null, get_lang('DefaultCharsetInfo'));
 
         $setting = ini_get('max_execution_time');
-        $req_setting = '300 (' . get_lang('Minimum') . ')';
+        $req_setting = '300 ('.get_lang('Minimum').')';
         $status = $setting >= 300 ? self :: STATUS_OK : self :: STATUS_WARNING;
         $array[] = $this->build_setting($status, '[INI]', 'max_execution_time', 'http://www.php.net/manual/en/ini.core.php#ini.max-execution-time', $setting, $req_setting, null, get_lang('MaxExecutionTimeInfo'));
 
         $setting = ini_get('max_input_time');
-        $req_setting = '300 (' . get_lang('Minimum') . ')';
+        $req_setting = '300 ('.get_lang('Minimum').')';
         $status = $setting >= 300 ? self :: STATUS_OK : self :: STATUS_WARNING;
         $array[] = $this->build_setting($status, '[INI]', 'max_input_time', 'http://www.php.net/manual/en/ini.core.php#ini.max-input-time', $setting, $req_setting, null, get_lang('MaxInputTimeInfo'));
 
         $setting = ini_get('memory_limit');
         $req_setting = '>= '.REQUIRED_MIN_MEMORY_LIMIT.'M';
         $status = self :: STATUS_ERROR;
-        if ((float)$setting >= REQUIRED_MIN_MEMORY_LIMIT)
+        if ((float) $setting >= REQUIRED_MIN_MEMORY_LIMIT)
             $status = self :: STATUS_OK;
         $array[] = $this->build_setting($status, '[INI]', 'memory_limit', 'http://www.php.net/manual/en/ini.core.php#ini.memory-limit', $setting, $req_setting, null, get_lang('MemoryLimitInfo'));
 
         $setting = ini_get('post_max_size');
         $req_setting = '>= '.REQUIRED_MIN_POST_MAX_SIZE.'M';
         $status = self :: STATUS_ERROR;
-        if ((float)$setting >= REQUIRED_MIN_POST_MAX_SIZE)
+        if ((float) $setting >= REQUIRED_MIN_POST_MAX_SIZE)
             $status = self :: STATUS_OK;
         $array[] = $this->build_setting($status, '[INI]', 'post_max_size', 'http://www.php.net/manual/en/ini.core.php#ini.post-max-size', $setting, $req_setting, null, get_lang('PostMaxSizeInfo'));
 
         $setting = ini_get('upload_max_filesize');
         $req_setting = '>= '.REQUIRED_MIN_UPLOAD_MAX_FILESIZE.'M';
         $status = self :: STATUS_ERROR;
-        if ((float)$setting >= REQUIRED_MIN_UPLOAD_MAX_FILESIZE)
+        if ((float) $setting >= REQUIRED_MIN_UPLOAD_MAX_FILESIZE)
             $status = self :: STATUS_OK;
         $array[] = $this->build_setting($status, '[INI]', 'upload_max_filesize', 'http://www.php.net/manual/en/ini.core.php#ini.upload_max_filesize', $setting, $req_setting, null, get_lang('UploadMaxFilesizeInfo'));
 
@@ -205,7 +205,7 @@ 
         $status = $setting == $req_setting ? self :: STATUS_OK : self :: STATUS_WARNING;
         $array[] = $this->build_setting($status, '[SESSION]', 'session.gc_maxlifetime', 'http://www.php.net/manual/en/ini.core.php#session.gc-maxlifetime', $setting, $req_setting, null, get_lang('SessionGCMaxLifetimeInfo'));
 
-        if (api_check_browscap()){$setting = true;}else{$setting=false;}
+        if (api_check_browscap()) {$setting = true; } else {$setting = false; }
         $req_setting = true;
         $status = $setting == $req_setting ? self :: STATUS_OK : self :: STATUS_WARNING;
         $array[] = $this->build_setting($status, '[INI]', 'browscap', 'http://www.php.net/manual/en/misc.configuration.php#ini.browscap', $setting, $req_setting, 'on_off', get_lang('BrowscapInfo'));
@@ -261,7 +261,7 @@ 
 
             $loaded = extension_loaded($extension);
             $status = $loaded ? self :: STATUS_OK : self :: STATUS_ERROR;
-            $array[] = $this->build_setting($status, '[EXTENSION]', get_lang('LoadedExtension') . ': ' . $extension, $url, $loaded, $expected_value, 'yes_no_optional', $comment);
+            $array[] = $this->build_setting($status, '[EXTENSION]', get_lang('LoadedExtension').': '.$extension, $url, $loaded, $expected_value, 'yes_no_optional', $comment);
         }
 
         return $array;
@@ -368,7 +368,7 @@ 
                 break;
         }
 
-        if (! $img_path) {
+        if (!$img_path) {
             $img_path = api_get_path(WEB_IMG_PATH);
         }
         $image = Display::return_icon($img, $status);
@@ -379,9 +379,9 @@ 
         $formatted_expected_value = $expected_value;
 
         if ($formatter) {
-            if (method_exists($this, 'format_' . $formatter)) {
-                $formatted_current_value = call_user_func(array($this, 'format_' . $formatter), $current_value);
-                $formatted_expected_value = call_user_func(array($this, 'format_' . $formatter), $expected_value);
+            if (method_exists($this, 'format_'.$formatter)) {
+                $formatted_current_value = call_user_func(array($this, 'format_'.$formatter), $current_value);
+                $formatted_expected_value = call_user_func(array($this, 'format_'.$formatter), $expected_value);
             }
         }
 
@@ -396,13 +396,13 @@ 
      */
     public function get_link($title, $url)
     {
-        return '<a href="' . $url . '" target="about:bank">' . $title . '</a>';
+        return '<a href="'.$url.'" target="about:bank">'.$title.'</a>';
     }
 
     public function format_yes_no_optional($value)
     {
     	$return = '';
-    	switch($value) {
+    	switch ($value) {
      		case 0:
      			$return = get_lang('No');
      			break;

main/inc/lib/skill.visualizer.lib.php

Spacing +13 added lines, -13 removed lines

@@ -27,7 +27,7 @@ 
     {
         $this->skills   = $skills;
         $this->type     = $type;
-        $this->center_x = intval($offset_x + $this->canvas_x/2 - $this->block_size/2);
+        $this->center_x = intval($offset_x + $this->canvas_x / 2 - $this->block_size / 2);
     }
 
     function prepare_skill_box($skill, $position, $class)
@@ -39,9 +39,9 @@ 
             $extra_class = 'second_window';
         }
 
-        $this->html .= '<div id="block_'.$block_id.'" class = "open_block window '.$extra_class.'  '.$class.'" style = "top:' . $position['y'] . 'px; left:' . $position['x'] . 'px;">';
+        $this->html .= '<div id="block_'.$block_id.'" class = "open_block window '.$extra_class.'  '.$class.'" style = "top:'.$position['y'].'px; left:'.$position['x'].'px;">';
 
-        $content =  $skill['name'];
+        $content = $skill['name'];
         $content .= '<div class="btn-group">';
         $content .= Display::url(get_lang('Edit'), '#', array('id'=>'edit_block_'.$block_id, 'class'=>'edit_block btn'));
         $content .= Display::url('+', '#', array('id'=>'edit_block_'.$block_id, 'class'=>'edit_block btn'));
@@ -87,9 +87,9 @@ 
         }
         //default_arrow_color
 
-        $this->js .= 'var e'.$block_id.' = prepare("block_' . $block_id.'",  '.$end_point.');'."\n";
-        $this->js .= 'var e'.$skill['parent_id'].' = prepare("block_' . $skill['parent_id'].'",  '.$end_point.');'."\n";
-        $this->js .= 'jsPlumb.connect({source: e'.$block_id.', target:e'.$skill['parent_id'].'});'."\n";;
+        $this->js .= 'var e'.$block_id.' = prepare("block_'.$block_id.'",  '.$end_point.');'."\n";
+        $this->js .= 'var e'.$skill['parent_id'].' = prepare("block_'.$skill['parent_id'].'",  '.$end_point.');'."\n";
+        $this->js .= 'jsPlumb.connect({source: e'.$block_id.', target:e'.$skill['parent_id'].'});'."\n"; ;
     }
 
     /**
@@ -125,10 +125,10 @@ 
         $brothers = array();
 
         foreach ($this->skills as &$skill) {
-            if (!in_array($skill['parent_id'], array(0,1))) {
+            if (!in_array($skill['parent_id'], array(0, 1))) {
                 continue;
             }
-            $childs = isset($skill['children']) ? count($skill['children']) : 0 ;
+            $childs = isset($skill['children']) ? count($skill['children']) : 0;
 
             //$x = round($this->offsetX * sin(deg2rad($corner * $count)));
             //$y = round($this->offsetY * cos(deg2rad($corner * $count)));
@@ -144,10 +144,10 @@ 
             if ($skill['parent_id'] == 0) {
                 //$x = 130*$childs/2;
                 //$x = $this->space_between_blocks_x*$childs/2;
-                $x = $this->canvas_x/2  - $this->block_size/2;
+                $x = $this->canvas_x / 2 - $this->block_size / 2;
             } else {
                 $max = isset($this->skills[$skill['parent_id']]['children']) ? count($this->skills[$skill['parent_id']]['children']) : 0;
-                foreach($this->skills[$skill['parent_id']]['children'] as  $id => $sk) {
+                foreach ($this->skills[$skill['parent_id']]['children'] as  $id => $sk) {
                     if ($skill['id'] == $sk['id']) {
                         break;
                     }
@@ -155,17 +155,17 @@ 
                 }
                 $parent_x = isset($this->skills[$skill['parent_id']]['x']) ? $this->skills[$skill['parent_id']]['x'] : 0;
                 //$x = $my_count*$this->space_between_blocks_x + $parent_x  + $this->block_size - ($this->space_between_blocks_x*$max/2) ;
-                $x = $my_count*$this->space_between_blocks_x + $parent_x  + $this->block_size - ($this->canvas_x/2 ) ;
+                $x = $my_count * $this->space_between_blocks_x + $parent_x + $this->block_size - ($this->canvas_x / 2);
             }
 
-            $y = $skill['level']*$this->space_between_blocks_y;
+            $y = $skill['level'] * $this->space_between_blocks_y;
 
             $skill['x'] = $x;
             $skill['y'] = $y;
 
             //$skill['description']  = "{$brothers[$skill['parent_id']]} $x - $y";
             //$skill['name']  =  $skill['name']."  |  $x = $my_count * 150  +  $parent_x - (150* $max/2) - 10*$childs ";
-            $this->add_item($skill, array('x' => $this->offset_x + $x, 'y' => $this->offset_y +$y));
+            $this->add_item($skill, array('x' => $this->offset_x + $x, 'y' => $this->offset_y + $y));
         }
         return $this->get_html();
     }

main/inc/lib/extra_field_value.lib.php

Spacing +8 added lines, -8 removed lines

@@ -209,7 +209,7 @@ 
                             break;
                     }
 
-                    $fileName = ExtraField::FIELD_TYPE_FILE_IMAGE . "_{$params['item_id']}.png";
+                    $fileName = ExtraField::FIELD_TYPE_FILE_IMAGE."_{$params['item_id']}.png";
 
                     if (!file_exists($fileDir)) {
                         mkdir($fileDir, $dirPermissions, true);
@@ -217,11 +217,11 @@ 
 
                     if ($value['error'] == 0) {
                         $imageExtraField = new Image($value['tmp_name']);
-                        $imageExtraField->send_image($fileDir . $fileName, -1, 'png');
+                        $imageExtraField->send_image($fileDir.$fileName, -1, 'png');
                         $newParams = array(
                             'item_id' => $params['item_id'],
                             'field_id' => $extraFieldInfo['id'],
-                            'value' => $fileDirStored . $fileName,
+                            'value' => $fileDirStored.$fileName,
                             'comment' => $comment
                         );
 
@@ -247,18 +247,18 @@ 
                     }
 
                     $cleanedName = api_replace_dangerous_char($value['name']);
-                    $fileName = ExtraField::FIELD_TYPE_FILE . "_{$params['item_id']}_$cleanedName";
+                    $fileName = ExtraField::FIELD_TYPE_FILE."_{$params['item_id']}_$cleanedName";
                     if (!file_exists($fileDir)) {
                         mkdir($fileDir, $dirPermissions, true);
                     }
 
                     if ($value['error'] == 0) {
-                        moveUploadedFile($value, $fileDir . $fileName);
+                        moveUploadedFile($value, $fileDir.$fileName);
 
                         $new_params = array(
                             'item_id' => $params['item_id'],
                             'field_id' => $extraFieldInfo['id'],
-                            'value' => $fileDirStored . $fileName
+                            'value' => $fileDirStored.$fileName
                         );
 
                         if ($this->type !== 'session' && $this->type !== 'course') {
@@ -380,7 +380,7 @@ 
                 */
                 if (false) {
                     global $app;
-                    switch($this->type) {
+                    switch ($this->type) {
                         case 'question':
                             $extraFieldValue = new ChamiloLMS\Entity\QuestionFieldValues();
                             $extraFieldValue->setUserId(api_get_user_id());
@@ -442,7 +442,7 @@ 
                 */
                 if (false) {
                     global $app;
-                    switch($this->type) {
+                    switch ($this->type) {
                         case 'question':
                             $extraFieldValue = $app['orm.ems']['db_write']->getRepository('ChamiloLMS\Entity\QuestionFieldValues')->find($field_values['id']);
                             $extraFieldValue->setUserId(api_get_user_id());

main/inc/lib/export.lib.inc.php

Spacing +5 added lines, -5 removed lines

@@ -99,7 +99,7 @@ 
         }
         foreach ($data as $row) {
             $string = implode("</td><td>", $row);
-            $string = '<tr><td>' . $string . '</td></tr>';
+            $string = '<tr><td>'.$string.'</td></tr>';
             if ($encoding != 'utf-8') {
                 $string = api_convert_encoding($string, $encoding, $systemEncoding);
             }
@@ -173,7 +173,7 @@ 
             fwrite($handle, '<'.$wrapper_tagname.'>');
         }
         $s = self::_export_complex_table_xml_helper($data);
-        fwrite($handle,$s);
+        fwrite($handle, $s);
         if (!is_null($wrapper_tagname)) {
             fwrite($handle, '</'.$wrapper_tagname.'>'."\n");
         }
@@ -195,10 +195,10 @@ 
         }
         $string = '';
         foreach ($data as $row) {
-            $string .= "\n".str_repeat("\t",$level).'<'.$row['name'].'>';
+            $string .= "\n".str_repeat("\t", $level).'<'.$row['name'].'>';
             if (is_array($row['value'])) {
-            	$string .= self::_export_complex_table_xml_helper($row['value'],$level+1)."\n";
-                $string .= str_repeat("\t",$level).'</'.$row['name'].'>';
+            	$string .= self::_export_complex_table_xml_helper($row['value'], $level + 1)."\n";
+                $string .= str_repeat("\t", $level).'</'.$row['name'].'>';
             } else {
                 $string .= $row['value'];
                 $string .= '</'.$row['name'].'>';

main/inc/lib/browser/Browser.php

Spacing +16 added lines, -16 removed lines

@@ -345,9 +345,9 @@ 
      */
     public function __toString()
     {
-        return "<strong>Browser Name:</strong> {$this->getBrowser()}<br/>\n" .
-        "<strong>Browser Version:</strong> {$this->getVersion()}<br/>\n" .
-        "<strong>Browser User Agent String:</strong> {$this->getUserAgent()}<br/>\n" .
+        return "<strong>Browser Name:</strong> {$this->getBrowser()}<br/>\n".
+        "<strong>Browser Version:</strong> {$this->getVersion()}<br/>\n".
+        "<strong>Browser User Agent String:</strong> {$this->getUserAgent()}<br/>\n".
         "<strong>Platform:</strong> {$this->getPlatform()}<br/>";
     }
 
@@ -585,13 +585,13 @@ 
             $aresult = explode(' ', stristr(str_replace(';', '; ', $this->_agent), 'msie'));
             $this->setBrowser(self::BROWSER_IE);
             $this->setVersion(str_replace(array('(', ')', ';'), '', $aresult[1]));
-            if(stripos($this->_agent, 'IEMobile') !== false) {
+            if (stripos($this->_agent, 'IEMobile') !== false) {
                 $this->setBrowser(self::BROWSER_POCKET_IE);
                 $this->setMobile(true);
             }
             return true;
         } // Test for versions > IE 10
-		else if(stripos($this->_agent, 'trident') !== false) {
+		else if (stripos($this->_agent, 'trident') !== false) {
 			$this->setBrowser(self::BROWSER_IE);
 			$result = explode('rv:', $this->_agent);
 			$this->setVersion(preg_replace('/[^0-9.]+/', '', $result[1]));
@@ -1012,7 +1012,7 @@ 
      */
     protected function checkFacebookExternalHit()
     {
-        if(stristr($this->_agent,'FacebookExternalHit'))
+        if (stristr($this->_agent, 'FacebookExternalHit'))
         {
             $this->setRobot(true);
             $this->setFacebook(true);
@@ -1027,7 +1027,7 @@ 
      */
     protected function checkForFacebookIos()
     {
-        if(stristr($this->_agent,'FBIOS'))
+        if (stristr($this->_agent, 'FBIOS'))
         {
             $this->setFacebook(true);
             return true;
@@ -1041,10 +1041,10 @@ 
      */
     protected function getSafariVersionOnIos() 
     {
-        $aresult = explode('/',stristr($this->_agent,'Version'));
-        if( isset($aresult[1]) ) 
+        $aresult = explode('/', stristr($this->_agent, 'Version'));
+        if (isset($aresult[1])) 
         {
-            $aversion = explode(' ',$aresult[1]);
+            $aversion = explode(' ', $aresult[1]);
             $this->setVersion($aversion[0]);
             return true;
         }
@@ -1057,10 +1057,10 @@ 
      */
     protected function getChromeVersionOnIos() 
     {
-        $aresult = explode('/',stristr($this->_agent,'CriOS'));
-        if( isset($aresult[1]) ) 
+        $aresult = explode('/', stristr($this->_agent, 'CriOS'));
+        if (isset($aresult[1])) 
         {
-            $aversion = explode(' ',$aresult[1]);
+            $aversion = explode(' ', $aresult[1]);
             $this->setVersion($aversion[0]);
             $this->setBrowser(self::BROWSER_CHROME);
             return true;
@@ -1073,7 +1073,7 @@ 
      * @return boolean True if the browser is iPhone otherwise false
      */
     protected function checkBrowseriPhone() {
-        if( stripos($this->_agent,'iPhone') !== false ) {
+        if (stripos($this->_agent, 'iPhone') !== false) {
             $this->setVersion(self::VERSION_UNKNOWN);
             $this->setBrowser(self::BROWSER_IPHONE);
             $this->getSafariVersionOnIos();
@@ -1090,7 +1090,7 @@ 
      * @return boolean True if the browser is iPad otherwise false
      */
     protected function checkBrowseriPad() {
-        if( stripos($this->_agent,'iPad') !== false ) {
+        if (stripos($this->_agent, 'iPad') !== false) {
             $this->setVersion(self::VERSION_UNKNOWN);
             $this->setBrowser(self::BROWSER_IPAD);
             $this->getSafariVersionOnIos();
@@ -1107,7 +1107,7 @@ 
      * @return boolean True if the browser is iPod otherwise false
      */
     protected function checkBrowseriPod() {
-        if( stripos($this->_agent,'iPod') !== false ) {
+        if (stripos($this->_agent, 'iPod') !== false) {
             $this->setVersion(self::VERSION_UNKNOWN);
             $this->setBrowser(self::BROWSER_IPOD);
             $this->getSafariVersionOnIos();

main/inc/lib/certificate.lib.php

Spacing +13 added lines, -13 removed lines

@@ -166,7 +166,7 @@ 
 
             $scoredisplay = ScoreDisplay :: instance();
             $scorecourse = $my_category[0]->calc_score($this->user_id);
-            $scorecourse_display = isset($scorecourse) ? $scoredisplay->display_score($scorecourse,SCORE_AVERAGE) : get_lang('NoResultsAvailable');
+            $scorecourse_display = isset($scorecourse) ? $scoredisplay->display_score($scorecourse, SCORE_AVERAGE) : get_lang('NoResultsAvailable');
 
             // Prepare all necessary variables:
             $organization_name = api_get_setting('Institution');
@@ -180,7 +180,7 @@ 
                 $organization_name, $stud_fn.' '.$stud_ln, $my_category[0]->get_name(),
                 $scorecourse_display
             );
-            $certif_text = str_replace("\\n","\n", $certif_text);
+            $certif_text = str_replace("\\n", "\n", $certif_text);
 
             //If the gradebook is related to skills we added the skills to the user
 
@@ -219,7 +219,7 @@ 
                             // Creating new name
                             $name = md5($this->user_id.$this->certificate_data['cat_id']).'.html';
                             $my_path_certificate = $this->certification_user_path.$name;
-                            $path_certificate    ='/'.$name;
+                            $path_certificate    = '/'.$name;
 
                             //Getting QR filename
                             $file_info = pathinfo($path_certificate);
@@ -230,7 +230,7 @@ 
                                 Display::img($this->certification_web_user_path.$file_info['filename'].'_qr.png', 'QR'),
                                 $new_content_html['content']
                             );
-                            $my_new_content_html = mb_convert_encoding($my_new_content_html,'UTF-8', api_get_system_encoding());
+                            $my_new_content_html = mb_convert_encoding($my_new_content_html, 'UTF-8', api_get_system_encoding());
 
                             $result = @file_put_contents($my_path_certificate, $my_new_content_html);
                             if ($result) {
@@ -273,8 +273,8 @@ 
         $path_certificate
     ) {
         $table_certificate = Database::get_main_table(TABLE_MAIN_GRADEBOOK_CERTIFICATE);
-        if (!UserManager::is_user_certified($cat_id,$user_id)) {
-            $sql='UPDATE '.$table_certificate.' SET path_certificate="'.Database::escape_string($path_certificate).'"
+        if (!UserManager::is_user_certified($cat_id, $user_id)) {
+            $sql = 'UPDATE '.$table_certificate.' SET path_certificate="'.Database::escape_string($path_certificate).'"
                  WHERE cat_id="'.intval($cat_id).'" AND user_id="'.intval($user_id).'" ';
             Database::query($sql);
         }
@@ -330,7 +330,7 @@ 
 
         if (!empty($content)) {
             foreach ($content as $key => $value) {
-                $my_header = str_replace(array('((', '))') , '', $headers[$key]);
+                $my_header = str_replace(array('((', '))'), '', $headers[$key]);
                 $final_content[$my_header] = $value;
             }
         }
@@ -358,11 +358,11 @@ 
             $final_content['gradebook_institution'].' - '.
             $final_content['gradebook_sitename'].' - '.
             get_lang('Certification').$break_space.
-            get_lang('Student'). ': '.$final_content['user_firstname'].' '.$final_content['user_lastname'].$break_space.
-            get_lang('Teacher'). ': '.$final_content['teacher_firstname'].' '.$final_content['teacher_lastname'].$break_space.
-            get_lang('Date'). ': '.$final_content['date_certificate'].$break_space.
-            get_lang('Score'). ': '.$final_content['gradebook_grade'].$break_space.
-            'URL'. ': '.$final_content['certificate_link'];
+            get_lang('Student').': '.$final_content['user_firstname'].' '.$final_content['user_lastname'].$break_space.
+            get_lang('Teacher').': '.$final_content['teacher_firstname'].' '.$final_content['teacher_lastname'].$break_space.
+            get_lang('Date').': '.$final_content['date_certificate'].$break_space.
+            get_lang('Score').': '.$final_content['gradebook_grade'].$break_space.
+            'URL'.': '.$final_content['certificate_link'];
 
         return $text;
     }
@@ -415,7 +415,7 @@ 
         if (!empty($this->certificate_data['path_certificate'])) {
             $user_certificate = $this->certification_user_path.basename($this->certificate_data['path_certificate']);
             if (file_exists($user_certificate)) {
-                header('Content-Type: text/html; charset='. api_get_system_encoding());
+                header('Content-Type: text/html; charset='.api_get_system_encoding());
                 echo @file_get_contents($user_certificate);
             }
         } else {

main/inc/lib/phpmailer/class.smtp.php

Spacing +154 added lines, -154 removed lines

@@ -63,7 +63,7 @@ 
    *  Sets whether debugging is turned on

    *  @var bool

    */

-  public $do_debug;       // the level of debug to perform

+  public $do_debug; // the level of debug to perform

 

   /**

    *  Sets VERP use on/off (default is off)

@@ -76,7 +76,7 @@ 
   /////////////////////////////////////////////////

 

   private $smtp_conn; // the socket to the server

-  private $error;     // error if any on the last call

+  private $error; // error if any on the last call

   private $helo_rply; // the reply the server sent to us for HELO

 

   /**

@@ -114,43 +114,43 @@ 
     $this->error = null;

 

     // make sure we are __not__ connected

-    if($this->connected()) {

+    if ($this->connected()) {

       // already connected, generate error

       $this->error = array("error" => "Already connected to a server");

       return false;

     }

 

-    if(empty($port)) {

+    if (empty($port)) {

       $port = $this->SMTP_PORT;

     }

 

     // connect to the smtp server

-    $this->smtp_conn = @fsockopen($host,    // the host of the server

-                                 $port,    // the port to use

-                                 $errno,   // error number if any

-                                 $errstr,  // error message if any

-                                 $tval);   // give up after ? secs

+    $this->smtp_conn = @fsockopen($host, // the host of the server

+                                 $port, // the port to use

+                                 $errno, // error number if any

+                                 $errstr, // error message if any

+                                 $tval); // give up after ? secs

     // verify we connected properly

-    if(empty($this->smtp_conn)) {

+    if (empty($this->smtp_conn)) {

       $this->error = array("error" => "Failed to connect to server",

                            "errno" => $errno,

                            "errstr" => $errstr);

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": $errstr ($errno)" . $this->CRLF . '<br />';

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": $errstr ($errno)".$this->CRLF.'<br />';

       }

       return false;

     }

 

     // SMTP server can take longer to respond, give longer timeout for first read

     // Windows does not have support for this timeout function

-    if(substr(PHP_OS, 0, 3) != "WIN")

+    if (substr(PHP_OS, 0, 3) != "WIN")

      socket_set_timeout($this->smtp_conn, $tval, 0);

 

     // get any announcement

     $announce = $this->get_lines();

 

-    if($this->do_debug >= 2) {

-      echo "SMTP -> FROM SERVER:" . $announce . $this->CRLF . '<br />';

+    if ($this->do_debug >= 2) {

+      echo "SMTP -> FROM SERVER:".$announce.$this->CRLF.'<br />';

     }

 

     return true;

@@ -168,33 +168,33 @@ 
   public function StartTLS() {

     $this->error = null; # to avoid confusion

 

-    if(!$this->connected()) {

+    if (!$this->connected()) {

       $this->error = array("error" => "Called StartTLS() without being connected");

       return false;

     }

 

-    fputs($this->smtp_conn,"STARTTLS" . $this->CRLF);

+    fputs($this->smtp_conn, "STARTTLS".$this->CRLF);

 

     $rply = $this->get_lines();

-    $code = substr($rply,0,3);

+    $code = substr($rply, 0, 3);

 

-    if($this->do_debug >= 2) {

-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';

+    if ($this->do_debug >= 2) {

+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';

     }

 

-    if($code != 220) {

+    if ($code != 220) {

       $this->error =

          array("error"     => "STARTTLS not accepted from server",

                "smtp_code" => $code,

-               "smtp_msg"  => substr($rply,4));

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';

+               "smtp_msg"  => substr($rply, 4));

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';

       }

       return false;

     }

 

     // Begin encrypted connection

-    if(!stream_socket_enable_crypto($this->smtp_conn, true, STREAM_CRYPTO_METHOD_TLS_CLIENT)) {

+    if (!stream_socket_enable_crypto($this->smtp_conn, true, STREAM_CRYPTO_METHOD_TLS_CLIENT)) {

       return false;

     }

 

@@ -209,52 +209,52 @@ 
    */

   public function Authenticate($username, $password) {

     // Start authentication

-    fputs($this->smtp_conn,"AUTH LOGIN" . $this->CRLF);

+    fputs($this->smtp_conn, "AUTH LOGIN".$this->CRLF);

 

     $rply = $this->get_lines();

-    $code = substr($rply,0,3);

+    $code = substr($rply, 0, 3);

 

-    if($code != 334) {

+    if ($code != 334) {

       $this->error =

         array("error" => "AUTH not accepted from server",

               "smtp_code" => $code,

-              "smtp_msg" => substr($rply,4));

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';

+              "smtp_msg" => substr($rply, 4));

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';

       }

       return false;

     }

 

     // Send encoded username

-    fputs($this->smtp_conn, base64_encode($username) . $this->CRLF);

+    fputs($this->smtp_conn, base64_encode($username).$this->CRLF);

 

     $rply = $this->get_lines();

-    $code = substr($rply,0,3);

+    $code = substr($rply, 0, 3);

 

-    if($code != 334) {

+    if ($code != 334) {

       $this->error =

         array("error" => "Username not accepted from server",

               "smtp_code" => $code,

-              "smtp_msg" => substr($rply,4));

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';

+              "smtp_msg" => substr($rply, 4));

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';

       }

       return false;

     }

 

     // Send encoded password

-    fputs($this->smtp_conn, base64_encode($password) . $this->CRLF);

+    fputs($this->smtp_conn, base64_encode($password).$this->CRLF);

 

     $rply = $this->get_lines();

-    $code = substr($rply,0,3);

+    $code = substr($rply, 0, 3);

 

-    if($code != 235) {

+    if ($code != 235) {

       $this->error =

         array("error" => "Password not accepted from server",

               "smtp_code" => $code,

-              "smtp_msg" => substr($rply,4));

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';

+              "smtp_msg" => substr($rply, 4));

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';

       }

       return false;

     }

@@ -268,12 +268,12 @@ 
    * @return bool

    */

   public function Connected() {

-    if(!empty($this->smtp_conn)) {

+    if (!empty($this->smtp_conn)) {

       $sock_status = socket_get_status($this->smtp_conn);

-      if($sock_status["eof"]) {

+      if ($sock_status["eof"]) {

         // the socket is valid but we are not connected

-        if($this->do_debug >= 1) {

-            echo "SMTP -> NOTICE:" . $this->CRLF . "EOF caught while checking if connected";

+        if ($this->do_debug >= 1) {

+            echo "SMTP -> NOTICE:".$this->CRLF."EOF caught while checking if connected";

         }

         $this->Close();

         return false;

@@ -293,7 +293,7 @@ 
   public function Close() {

     $this->error = null; // so there is no confusion

     $this->helo_rply = null;

-    if(!empty($this->smtp_conn)) {

+    if (!empty($this->smtp_conn)) {

       // close the connection and cleanup

       fclose($this->smtp_conn);

       $this->smtp_conn = 0;

@@ -326,28 +326,28 @@ 
   public function Data($msg_data) {

     $this->error = null; // so no confusion is caused

 

-    if(!$this->connected()) {

+    if (!$this->connected()) {

       $this->error = array(

               "error" => "Called Data() without being connected");

       return false;

     }

 

-    fputs($this->smtp_conn,"DATA" . $this->CRLF);

+    fputs($this->smtp_conn, "DATA".$this->CRLF);

 

     $rply = $this->get_lines();

-    $code = substr($rply,0,3);

+    $code = substr($rply, 0, 3);

 

-    if($this->do_debug >= 2) {

-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';

+    if ($this->do_debug >= 2) {

+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';

     }

 

-    if($code != 354) {

+    if ($code != 354) {

       $this->error =

         array("error" => "DATA command not accepted from server",

               "smtp_code" => $code,

-              "smtp_msg" => substr($rply,4));

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';

+              "smtp_msg" => substr($rply, 4));

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';

       }

       return false;

     }

@@ -364,9 +364,9 @@ 
      */

 

     // normalize the line breaks so we know the explode works

-    $msg_data = str_replace("\r\n","\n",$msg_data);

-    $msg_data = str_replace("\r","\n",$msg_data);

-    $lines = explode("\n",$msg_data);

+    $msg_data = str_replace("\r\n", "\n", $msg_data);

+    $msg_data = str_replace("\r", "\n", $msg_data);

+    $lines = explode("\n", $msg_data);

 

     /* we need to find a good way to determine is headers are

      * in the msg_data or if it is a straight msg body

@@ -377,71 +377,71 @@ 
      * headers.

      */

 

-    $field = substr($lines[0],0,strpos($lines[0],":"));

+    $field = substr($lines[0], 0, strpos($lines[0], ":"));

     $in_headers = false;

-    if(!empty($field) && !strstr($field," ")) {

+    if (!empty($field) && !strstr($field, " ")) {

       $in_headers = true;

     }

 

     $max_line_length = 998; // used below; set here for ease in change

 

-    while(list(,$line) = @each($lines)) {

+    while (list(,$line) = @each($lines)) {

       $lines_out = null;

-      if($line == "" && $in_headers) {

+      if ($line == "" && $in_headers) {

         $in_headers = false;

       }

       // ok we need to break this line up into several smaller lines

-      while(strlen($line) > $max_line_length) {

-        $pos = strrpos(substr($line,0,$max_line_length)," ");

+      while (strlen($line) > $max_line_length) {

+        $pos = strrpos(substr($line, 0, $max_line_length), " ");

 

         // Patch to fix DOS attack

-        if(!$pos) {

+        if (!$pos) {

           $pos = $max_line_length - 1;

-          $lines_out[] = substr($line,0,$pos);

-          $line = substr($line,$pos);

+          $lines_out[] = substr($line, 0, $pos);

+          $line = substr($line, $pos);

         } else {

-          $lines_out[] = substr($line,0,$pos);

-          $line = substr($line,$pos + 1);

+          $lines_out[] = substr($line, 0, $pos);

+          $line = substr($line, $pos + 1);

         }

 

         /* if processing headers add a LWSP-char to the front of new line

          * rfc 822 on long msg headers

          */

-        if($in_headers) {

-          $line = "\t" . $line;

+        if ($in_headers) {

+          $line = "\t".$line;

         }

       }

       $lines_out[] = $line;

 

       // send the lines to the server

-      while(list(,$line_out) = @each($lines_out)) {

-        if(strlen($line_out) > 0)

+      while (list(,$line_out) = @each($lines_out)) {

+        if (strlen($line_out) > 0)

         {

-          if(substr($line_out, 0, 1) == ".") {

-            $line_out = "." . $line_out;

+          if (substr($line_out, 0, 1) == ".") {

+            $line_out = ".".$line_out;

           }

         }

-        fputs($this->smtp_conn,$line_out . $this->CRLF);

+        fputs($this->smtp_conn, $line_out.$this->CRLF);

       }

     }

 

     // message data has been sent

-    fputs($this->smtp_conn, $this->CRLF . "." . $this->CRLF);

+    fputs($this->smtp_conn, $this->CRLF.".".$this->CRLF);

 

     $rply = $this->get_lines();

-    $code = substr($rply,0,3);

+    $code = substr($rply, 0, 3);

 

-    if($this->do_debug >= 2) {

-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';

+    if ($this->do_debug >= 2) {

+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';

     }

 

-    if($code != 250) {

+    if ($code != 250) {

       $this->error =

         array("error" => "DATA not accepted from server",

               "smtp_code" => $code,

-              "smtp_msg" => substr($rply,4));

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';

+              "smtp_msg" => substr($rply, 4));

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';

       }

       return false;

     }

@@ -463,21 +463,21 @@ 
   public function Hello($host = '') {

     $this->error = null; // so no confusion is caused

 

-    if(!$this->connected()) {

+    if (!$this->connected()) {

       $this->error = array(

             "error" => "Called Hello() without being connected");

       return false;

     }

 

     // if hostname for HELO was not specified send default

-    if(empty($host)) {

+    if (empty($host)) {

       // determine appropriate default to send to server

       $host = "localhost";

     }

 

     // Send extended hello first (RFC 2821)

-    if(!$this->SendHello("EHLO", $host)) {

-      if(!$this->SendHello("HELO", $host)) {

+    if (!$this->SendHello("EHLO", $host)) {

+      if (!$this->SendHello("HELO", $host)) {

         return false;

       }

     }

@@ -491,22 +491,22 @@ 
    * @return bool

    */

   private function SendHello($hello, $host) {

-    fputs($this->smtp_conn, $hello . " " . $host . $this->CRLF);

+    fputs($this->smtp_conn, $hello." ".$host.$this->CRLF);

 

     $rply = $this->get_lines();

-    $code = substr($rply,0,3);

+    $code = substr($rply, 0, 3);

 

-    if($this->do_debug >= 2) {

-      echo "SMTP -> FROM SERVER: " . $rply . $this->CRLF . '<br />';

+    if ($this->do_debug >= 2) {

+      echo "SMTP -> FROM SERVER: ".$rply.$this->CRLF.'<br />';

     }

 

-    if($code != 250) {

+    if ($code != 250) {

       $this->error =

-        array("error" => $hello . " not accepted from server",

+        array("error" => $hello." not accepted from server",

               "smtp_code" => $code,

-              "smtp_msg" => substr($rply,4));

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';

+              "smtp_msg" => substr($rply, 4));

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';

       }

       return false;

     }

@@ -533,29 +533,29 @@ 
   public function Mail($from) {

     $this->error = null; // so no confusion is caused

 

-    if(!$this->connected()) {

+    if (!$this->connected()) {

       $this->error = array(

               "error" => "Called Mail() without being connected");

       return false;

     }

 

     $useVerp = ($this->do_verp ? "XVERP" : "");

-    fputs($this->smtp_conn,"MAIL FROM:<" . $from . ">" . $useVerp . $this->CRLF);

+    fputs($this->smtp_conn, "MAIL FROM:<".$from.">".$useVerp.$this->CRLF);

 

     $rply = $this->get_lines();

-    $code = substr($rply,0,3);

+    $code = substr($rply, 0, 3);

 

-    if($this->do_debug >= 2) {

-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';

+    if ($this->do_debug >= 2) {

+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';

     }

 

-    if($code != 250) {

+    if ($code != 250) {

       $this->error =

         array("error" => "MAIL not accepted from server",

               "smtp_code" => $code,

-              "smtp_msg" => substr($rply,4));

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';

+              "smtp_msg" => substr($rply, 4));

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';

       }

       return false;

     }

@@ -576,38 +576,38 @@ 
   public function Quit($close_on_error = true) {

     $this->error = null; // so there is no confusion

 

-    if(!$this->connected()) {

+    if (!$this->connected()) {

       $this->error = array(

               "error" => "Called Quit() without being connected");

       return false;

     }

 

     // send the quit command to the server

-    fputs($this->smtp_conn,"quit" . $this->CRLF);

+    fputs($this->smtp_conn, "quit".$this->CRLF);

 

     // get any good-bye messages

     $byemsg = $this->get_lines();

 

-    if($this->do_debug >= 2) {

-      echo "SMTP -> FROM SERVER:" . $byemsg . $this->CRLF . '<br />';

+    if ($this->do_debug >= 2) {

+      echo "SMTP -> FROM SERVER:".$byemsg.$this->CRLF.'<br />';

     }

 

     $rval = true;

     $e = null;

 

-    $code = substr($byemsg,0,3);

-    if($code != 221) {

+    $code = substr($byemsg, 0, 3);

+    if ($code != 221) {

       // use e as a tmp var cause Close will overwrite $this->error

       $e = array("error" => "SMTP server rejected quit command",

                  "smtp_code" => $code,

-                 "smtp_rply" => substr($byemsg,4));

+                 "smtp_rply" => substr($byemsg, 4));

       $rval = false;

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $e["error"] . ": " . $byemsg . $this->CRLF . '<br />';

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$e["error"].": ".$byemsg.$this->CRLF.'<br />';

       }

     }

 

-    if(empty($e) || $close_on_error) {

+    if (empty($e) || $close_on_error) {

       $this->Close();

     }

 

@@ -629,28 +629,28 @@ 
   public function Recipient($to) {

     $this->error = null; // so no confusion is caused

 

-    if(!$this->connected()) {

+    if (!$this->connected()) {

       $this->error = array(

               "error" => "Called Recipient() without being connected");

       return false;

     }

 

-    fputs($this->smtp_conn,"RCPT TO:<" . $to . ">" . $this->CRLF);

+    fputs($this->smtp_conn, "RCPT TO:<".$to.">".$this->CRLF);

 

     $rply = $this->get_lines();

-    $code = substr($rply,0,3);

+    $code = substr($rply, 0, 3);

 

-    if($this->do_debug >= 2) {

-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';

+    if ($this->do_debug >= 2) {

+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';

     }

 

-    if($code != 250 && $code != 251) {

+    if ($code != 250 && $code != 251) {

       $this->error =

         array("error" => "RCPT not accepted from server",

               "smtp_code" => $code,

-              "smtp_msg" => substr($rply,4));

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';

+              "smtp_msg" => substr($rply, 4));

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';

       }

       return false;

     }

@@ -672,28 +672,28 @@ 
   public function Reset() {

     $this->error = null; // so no confusion is caused

 

-    if(!$this->connected()) {

+    if (!$this->connected()) {

       $this->error = array(

               "error" => "Called Reset() without being connected");

       return false;

     }

 

-    fputs($this->smtp_conn,"RSET" . $this->CRLF);

+    fputs($this->smtp_conn, "RSET".$this->CRLF);

 

     $rply = $this->get_lines();

-    $code = substr($rply,0,3);

+    $code = substr($rply, 0, 3);

 

-    if($this->do_debug >= 2) {

-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';

+    if ($this->do_debug >= 2) {

+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';

     }

 

-    if($code != 250) {

+    if ($code != 250) {

       $this->error =

         array("error" => "RSET failed",

               "smtp_code" => $code,

-              "smtp_msg" => substr($rply,4));

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';

+              "smtp_msg" => substr($rply, 4));

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';

       }

       return false;

     }

@@ -720,28 +720,28 @@ 
   public function SendAndMail($from) {

     $this->error = null; // so no confusion is caused

 

-    if(!$this->connected()) {

+    if (!$this->connected()) {

       $this->error = array(

           "error" => "Called SendAndMail() without being connected");

       return false;

     }

 

-    fputs($this->smtp_conn,"SAML FROM:" . $from . $this->CRLF);

+    fputs($this->smtp_conn, "SAML FROM:".$from.$this->CRLF);

 

     $rply = $this->get_lines();

-    $code = substr($rply,0,3);

+    $code = substr($rply, 0, 3);

 

-    if($this->do_debug >= 2) {

-      echo "SMTP -> FROM SERVER:" . $rply . $this->CRLF . '<br />';

+    if ($this->do_debug >= 2) {

+      echo "SMTP -> FROM SERVER:".$rply.$this->CRLF.'<br />';

     }

 

-    if($code != 250) {

+    if ($code != 250) {

       $this->error =

         array("error" => "SAML not accepted from server",

               "smtp_code" => $code,

-              "smtp_msg" => substr($rply,4));

-      if($this->do_debug >= 1) {

-        echo "SMTP -> ERROR: " . $this->error["error"] . ": " . $rply . $this->CRLF . '<br />';

+              "smtp_msg" => substr($rply, 4));

+      if ($this->do_debug >= 1) {

+        echo "SMTP -> ERROR: ".$this->error["error"].": ".$rply.$this->CRLF.'<br />';

       }

       return false;

     }

@@ -764,8 +764,8 @@ 
   public function Turn() {

     $this->error = array("error" => "This method, TURN, of the SMTP ".

                                     "is not implemented");

-    if($this->do_debug >= 1) {

-      echo "SMTP -> NOTICE: " . $this->error["error"] . $this->CRLF . '<br />';

+    if ($this->do_debug >= 1) {

+      echo "SMTP -> NOTICE: ".$this->error["error"].$this->CRLF.'<br />';

     }

     return false;

   }

@@ -794,17 +794,17 @@ 
    */

   private function get_lines() {

     $data = "";

-    while($str = @fgets($this->smtp_conn,515)) {

-      if($this->do_debug >= 4) {

-        echo "SMTP -> get_lines(): \$data was \"$data\"" . $this->CRLF . '<br />';

-        echo "SMTP -> get_lines(): \$str is \"$str\"" . $this->CRLF . '<br />';

+    while ($str = @fgets($this->smtp_conn, 515)) {

+      if ($this->do_debug >= 4) {

+        echo "SMTP -> get_lines(): \$data was \"$data\"".$this->CRLF.'<br />';

+        echo "SMTP -> get_lines(): \$str is \"$str\"".$this->CRLF.'<br />';

       }

       $data .= $str;

-      if($this->do_debug >= 4) {

-        echo "SMTP -> get_lines(): \$data is \"$data\"" . $this->CRLF . '<br />';

+      if ($this->do_debug >= 4) {

+        echo "SMTP -> get_lines(): \$data is \"$data\"".$this->CRLF.'<br />';

       }

       // if 4th character is a space, we are done reading, break the loop

-      if(substr($str,3,1) == " ") { break; }

+      if (substr($str, 3, 1) == " ") { break; }

     }

     return $data;

   }

main/inc/lib/phpmailer/class.pop3.php

Spacing +17 added lines, -17 removed lines

@@ -120,7 +120,7 @@ 
 
   private $pop_conn;
   private $connected;
-  private $error;     //  Error log array
+  private $error; //  Error log array
 
   /**
    * Constructor, sets the initial values
@@ -142,7 +142,7 @@ 
    * @param string $username
    * @param string $password
    */
-  public function Authorise ($host, $port = false, $tval = false, $username, $password, $debug_level = 0) {
+  public function Authorise($host, $port = false, $tval = false, $username, $password, $debug_level = 0) {
     $this->host = $host;
 
     //  If no port value is passed, retrieve it
@@ -194,7 +194,7 @@ 
    * @param integer $tval
    * @return boolean
    */
-  public function Connect ($host, $port = false, $tval = 30) {
+  public function Connect($host, $port = false, $tval = 30) {
     //  Are we already connected?
     if ($this->connected) {
       return true;
@@ -208,11 +208,11 @@ 
     set_error_handler(array(&$this, 'catchWarning'));
 
     //  Connect to the POP3 server
-    $this->pop_conn = fsockopen($host,    //  POP3 Host
-                  $port,    //  Port #
-                  $errno,   //  Error Number
-                  $errstr,  //  Error Message
-                  $tval);   //  Timeout (seconds)
+    $this->pop_conn = fsockopen($host, //  POP3 Host
+                  $port, //  Port #
+                  $errno, //  Error Number
+                  $errstr, //  Error Message
+                  $tval); //  Timeout (seconds)
 
     //  Restore the error handler
     restore_error_handler();
@@ -269,7 +269,7 @@ 
    * @param string $password
    * @return boolean
    */
-  public function Login ($username = '', $password = '') {
+  public function Login($username = '', $password = '') {
     if ($this->connected == false) {
       $this->error = 'Not connected to POP3 server';
 
@@ -286,8 +286,8 @@ 
       $password = $this->password;
     }
 
-    $pop_username = "USER $username" . $this->CRLF;
-    $pop_password = "PASS $password" . $this->CRLF;
+    $pop_username = "USER $username".$this->CRLF;
+    $pop_password = "PASS $password".$this->CRLF;
 
     //  Send the Username
     $this->sendString($pop_username);
@@ -312,7 +312,7 @@ 
    * Disconnect from the POP3 server
    * @access public
    */
-  public function Disconnect () {
+  public function Disconnect() {
     $this->sendString('QUIT');
 
     fclose($this->pop_conn);
@@ -329,7 +329,7 @@ 
    * @param integer $size
    * @return string
    */
-  private function getResponse ($size = 128) {
+  private function getResponse($size = 128) {
     $pop3_response = fgets($this->pop_conn, $size);
 
     return $pop3_response;
@@ -341,7 +341,7 @@ 
    * @param string $string
    * @return integer
    */
-  private function sendString ($string) {
+  private function sendString($string) {
     $bytes_sent = fwrite($this->pop_conn, $string, strlen($string));
 
     return $bytes_sent;
@@ -353,7 +353,7 @@ 
    * @param string $string
    * @return boolean
    */
-  private function checkResponse ($string) {
+  private function checkResponse($string) {
     if (substr($string, 0, 3) !== '+OK') {
       $this->error = array(
         'error' => "Server reported an error: $string",
@@ -376,7 +376,7 @@ 
    * If debug is enabled, display the error message array
    * @access private
    */
-  private function displayErrors () {
+  private function displayErrors() {
     echo '<pre>';
 
     foreach ($this->error as $single_error) {
@@ -394,7 +394,7 @@ 
    * @param string $errfile
    * @param integer $errline
    */
-  private function catchWarning ($errno, $errstr, $errfile, $errline) {
+  private function catchWarning($errno, $errstr, $errfile, $errline) {
     $this->error[] = array(
       'error' => "Connecting to the POP3 server raised a PHP warning: ",
       'errno' => $errno,