Passed
Push — 1.10.x ( f08406...b77fe8 )
by
unknown
575:03 queued 528:43
created
main/admin/configure_homepage.php 1 patch
Spacing   +49 added lines, -49 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
 {
20 20
 	$post = strpos($file_logged_in, "_logged_in");
21 21
 	if ($post !== false) {
22
-		$file_logged_out = str_replace('_logged_in','', $file_logged_in);
22
+		$file_logged_out = str_replace('_logged_in', '', $file_logged_in);
23 23
 		//variables initialization
24 24
 		$data_logged_out = array();
25 25
 		$data_logged_in  = array();
@@ -27,9 +27,9 @@  discard block
 block discarded – undo
27 27
 		//we read the file with all links
28 28
 		$file = file($file_logged_in);
29 29
 		foreach ($file as $line) {
30
-			$line = str_replace("\n", '',$line);
30
+			$line = str_replace("\n", '', $line);
31 31
 			//not logged user only sees public links
32
-			if (!preg_match('/::private/',$line)) {
32
+			if (!preg_match('/::private/', $line)) {
33 33
 				$data_logged_out[] = $line;
34 34
 			}
35 35
 			//logged user only sees all links
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
 $tbl_category = Database::get_main_table(TABLE_MAIN_CATEGORY);
75 75
 $tool_name = get_lang('ConfigureHomePage');
76 76
 $_languages = api_get_languages();
77
-$selfUrl =  api_get_self();
77
+$selfUrl = api_get_self();
78 78
 $interbreadcrumb[] = array(
79 79
 	'url' => 'index.php',
80 80
 	'name' => get_lang('PlatformAdmin')
@@ -262,7 +262,7 @@  discard block
 block discarded – undo
262 262
 				}
263 263
 
264 264
 				if (EventsMail::check_if_using_class('portal_homepage_edited')) {
265
-					EventsDispatcher::events('portal_homepage_edited',array('about_user' => api_get_user_id()));
265
+					EventsDispatcher::events('portal_homepage_edited', array('about_user' => api_get_user_id()));
266 266
 				}
267 267
 				Event::addEvent(
268 268
 					LOG_HOMEPAGE_CHANGED,
@@ -394,9 +394,9 @@  discard block
 block discarded – undo
394 394
 				if ($link_url == 'http://' || $link_url == 'https://') {
395 395
 					$link_url = '';
396 396
 				} elseif (!empty($link_url) && !strstr($link_url, '://')) {
397
-					$link_url='http://'.$link_url;
397
+					$link_url = 'http://'.$link_url;
398 398
 				}
399
-				$menuf = ($action == 'insert_tabs' || $action == 'edit_tabs')? $mtloggedin : $menuf;
399
+				$menuf = ($action == 'insert_tabs' || $action == 'edit_tabs') ? $mtloggedin : $menuf;
400 400
 
401 401
 				if (!is_writable($homep.$menuf.'_'.$lang.$ext)) {
402 402
 					$errorMsg = get_lang('HomePageFilesNotWritable');
@@ -414,7 +414,7 @@  discard block
 block discarded – undo
414 414
 					}
415 415
 					// If the typical language suffix is not found in the file name,
416 416
 					// replace the ".html" suffix by "_en.html" or the active menu language
417
-					if (!strstr($filename,'_'.$lang.$ext)) {
417
+					if (!strstr($filename, '_'.$lang.$ext)) {
418 418
 						$filename = str_replace($ext, '_'.$lang.$ext, $filename);
419 419
 					}
420 420
 					// Get the contents of home_menu_en.html (or active menu language
@@ -436,7 +436,7 @@  discard block
 block discarded – undo
436 436
 					//
437 437
 					// If the given link url is empty, then replace the link url by a link to the link file created
438 438
 
439
-					if (empty($link_url) || $link_url == 'http://' ||  $link_url == 'https://') {
439
+					if (empty($link_url) || $link_url == 'http://' || $link_url == 'https://') {
440 440
 						$link_url = api_get_path(WEB_PATH).'index.php?include='.urlencode($filename);
441 441
 						// If the file doesn't exist, then create it and
442 442
 						// fill it with default text
@@ -455,7 +455,7 @@  discard block
 block discarded – undo
455 455
 					}
456 456
 					// If the requested action is to edit a link, open the file and
457 457
 					// write to it (if the file doesn't exist, create it)
458
-					if (in_array($action, array('edit_link'))  && !empty($link_html)) {
458
+					if (in_array($action, array('edit_link')) && !empty($link_html)) {
459 459
 						$fp = @fopen($homep.$filename, 'w');
460 460
 						if ($fp) {
461 461
 							fputs($fp, $link_html);
@@ -481,10 +481,10 @@  discard block
 block discarded – undo
481 481
 								break;
482 482
 							}
483 483
 						}
484
-						$home_menu[$insert_where + 1] = '<li><a href="'.$link_url.'" target="'.($target_blank ? '_blank' : '_self').'">'. $link_name .'</a></li>';
484
+						$home_menu[$insert_where + 1] = '<li><a href="'.$link_url.'" target="'.($target_blank ? '_blank' : '_self').'">'.$link_name.'</a></li>';
485 485
 					} else {
486 486
 						// If the request is about a link edition, change the link
487
-						$home_menu[$link_index]='<li><a href="'.$link_url.'" target="'.($target_blank?'_blank':'_self').'">'. $link_name .'</a></li>';
487
+						$home_menu[$link_index] = '<li><a href="'.$link_url.'" target="'.($target_blank ? '_blank' : '_self').'">'.$link_name.'</a></li>';
488 488
 					}
489 489
 					// Re-build the file from the home_menu array
490 490
 					$home_menu = implode("\n", $home_menu);
@@ -538,7 +538,7 @@  discard block
 block discarded – undo
538 538
 				Event::addEvent(
539 539
 					LOG_HOMEPAGE_CHANGED,
540 540
 					$action,
541
-					cut($link_name . ':' . $link_url, 254),
541
+					cut($link_name.':'.$link_url, 254),
542 542
 					api_get_utc_datetime(),
543 543
 					api_get_user_id()
544 544
 				);
@@ -561,7 +561,7 @@  discard block
 block discarded – undo
561 561
 				// A link is deleted by getting the file into an array, removing the
562 562
 				// link and re-writing the array to the file
563 563
 				$link_index = intval($_GET['link_index']);
564
-				$menuf = ($action == 'delete_tabs')? $mtloggedin : $menuf;
564
+				$menuf = ($action == 'delete_tabs') ? $mtloggedin : $menuf;
565 565
 				$home_menu = @file($homep.$menuf.'_'.$lang.$ext);
566 566
 				if (empty($home_menu)) {
567 567
 					$home_menu = array();
@@ -582,7 +582,7 @@  discard block
 block discarded – undo
582 582
 				fclose($fp);
583 583
 				if (file_exists($homep.$menuf.$ext)) {
584 584
 					if (is_writable($homep.$menuf.$ext)) {
585
-						$fpo = fopen($homep.$menuf.$ext,'w');
585
+						$fpo = fopen($homep.$menuf.$ext, 'w');
586 586
 						fputs($fpo, $home_menu);
587 587
 						home_tabs($homep.$menuf.$ext);
588 588
 						fclose($fpo);
@@ -595,9 +595,9 @@  discard block
 block discarded – undo
595 595
 				// This request is only the preparation for the update of the home_top
596 596
 				$home_top = '';
597 597
 				if (is_file($homep.$topf.'_'.$lang.$ext) && is_readable($homep.$topf.'_'.$lang.$ext)) {
598
-					$home_top = @(string)file_get_contents($homep.$topf.'_'.$lang.$ext);
598
+					$home_top = @(string) file_get_contents($homep.$topf.'_'.$lang.$ext);
599 599
 				} elseif (is_file($homep.$topf.$lang.$ext) && is_readable($homep.$topf.$lang.$ext)) {
600
-					$home_top = @(string)file_get_contents($homep.$topf.$lang.$ext);
600
+					$home_top = @(string) file_get_contents($homep.$topf.$lang.$ext);
601 601
 				} else {
602 602
 					$errorMsg = get_lang('HomePageFilesNotReadable');
603 603
 				}
@@ -625,9 +625,9 @@  discard block
 block discarded – undo
625 625
 				// This request is the preparation for the update of the home_news page
626 626
 				$home_news = '';
627 627
 				if (is_file($homep.$newsf.'_'.$lang.$ext) && is_readable($homep.$newsf.'_'.$lang.$ext)) {
628
-					$home_news = @(string)file_get_contents($homep.$newsf.'_'.$lang.$ext);
628
+					$home_news = @(string) file_get_contents($homep.$newsf.'_'.$lang.$ext);
629 629
 				} elseif (is_file($homep.$newsf.$lang.$ext) && is_readable($homep.$newsf.$lang.$ext)) {
630
-					$home_news = @(string)file_get_contents($homep.$newsf.$lang.$ext);
630
+					$home_news = @(string) file_get_contents($homep.$newsf.$lang.$ext);
631 631
 				} else {
632 632
 					$errorMsg = get_lang('HomePageFilesNotReadable');
633 633
 				}
@@ -636,10 +636,10 @@  discard block
 block discarded – undo
636 636
 			case 'insert_link':
637 637
 				// This request is the preparation for the addition of an item in home_menu
638 638
 				$home_menu = '';
639
-				$menuf = ($action == 'edit_tabs')? $mtloggedin : $menuf;
639
+				$menuf = ($action == 'edit_tabs') ? $mtloggedin : $menuf;
640 640
 				if (is_file($homep.$menuf.'_'.$lang.$ext) && is_readable($homep.$menuf.'_'.$lang.$ext)) {
641 641
 					$home_menu = @file($homep.$menuf.'_'.$lang.$ext);
642
-				} elseif(is_file($homep.$menuf.$lang.$ext) && is_readable($homep.$menuf.$lang.$ext)) {
642
+				} elseif (is_file($homep.$menuf.$lang.$ext) && is_readable($homep.$menuf.$lang.$ext)) {
643 643
 					$home_menu = @file($homep.$menuf.$lang.$ext);
644 644
 				} else {
645 645
 					$errorMsg = get_lang('HomePageFilesNotReadable');
@@ -680,10 +680,10 @@  discard block
 block discarded – undo
680 680
 			case 'edit_link':
681 681
 				// This request is the preparation for the edition of the links array
682 682
 				$home_menu = '';
683
-				$menuf = ($action == 'edit_tabs')? $mtloggedin : $menuf;
683
+				$menuf = ($action == 'edit_tabs') ? $mtloggedin : $menuf;
684 684
 				if (is_file($homep.$menuf.'_'.$lang.$ext) && is_readable($homep.$menuf.'_'.$lang.$ext)) {
685 685
 					$home_menu = @file($homep.$menuf.'_'.$lang.$ext);
686
-				} elseif(is_file($homep.$menuf.$lang.$ext) && is_readable($homep.$menuf.$lang.$ext)) {
686
+				} elseif (is_file($homep.$menuf.$lang.$ext) && is_readable($homep.$menuf.$lang.$ext)) {
687 687
 					$home_menu = @file($homep.$menuf.$lang.$ext);
688 688
 				} else {
689 689
 					$errorMsg = get_lang('HomePageFilesNotReadable');
@@ -746,7 +746,7 @@  discard block
 block discarded – undo
746 746
 						$link_name = strip_tags($enreg);
747 747
 
748 748
 						// Get the contents of "href" attribute in $link_url
749
-						$enreg = explode('href="',$enreg);
749
+						$enreg = explode('href="', $enreg);
750 750
 						list($link_url) = explode('"', $enreg[sizeof($enreg) - 1]);
751 751
 
752 752
 						// If the link contains the web root of this portal, then strip
@@ -784,7 +784,7 @@  discard block
 block discarded – undo
784 784
 	case 'open_link':
785 785
 		if (!empty($link)) {
786 786
 			// $link is only set in case of action=open_link and is filtered
787
-			$open = @(string)file_get_contents($homep.$link);
787
+			$open = @(string) file_get_contents($homep.$link);
788 788
 			$open = api_to_system_encoding($open, api_detect_encoding(strip_tags($open)));
789 789
 			echo $open;
790 790
 		}
@@ -848,7 +848,7 @@  discard block
 block discarded – undo
848 848
 		}
849 849
 		$default = array();
850 850
 		$form = new FormValidator('configure_homepage_'.$action, 'post', $selfUrl.'?action='.$action, '', array('style' => 'margin: 0px;'));
851
-		$renderer =& $form->defaultRenderer();
851
+		$renderer = & $form->defaultRenderer();
852 852
 
853 853
 		$form->addElement('header', '', $tool_name);
854 854
 		$form->addElement('hidden', 'formSent', '1');
@@ -869,7 +869,7 @@  discard block
 block discarded – undo
869 869
 
870 870
 		if ($action == 'insert_link' || $action == 'insert_tabs') {
871 871
 			$add_in_tab = 1;
872
-			if (is_array($home_menu)){
872
+			if (is_array($home_menu)) {
873 873
 				foreach ($home_menu as $key => $enreg) {
874 874
 					if (strlen($enreg = trim(strip_tags($enreg))) > 0) {
875 875
 						$options[$key] = get_lang('After').' &quot;'.$enreg.'&quot;';
@@ -879,7 +879,7 @@  discard block
 block discarded – undo
879 879
 				}
880 880
 			}
881 881
 			$default['insert_link'] = $selected;
882
-			$form->addElement('select', 'insert_where', get_lang('InsertThisLink') , $options);
882
+			$form->addElement('select', 'insert_where', get_lang('InsertThisLink'), $options);
883 883
 		}
884 884
 
885 885
 		$target_blank_checkbox = $form->addElement('checkbox', 'target_blank', null, get_lang('OpenInNewWindow'), 1);
@@ -894,15 +894,15 @@  discard block
 block discarded – undo
894 894
 		if ($action == 'edit_link' && (empty($link_url) || $link_url == 'http://' || $link_url == 'https://')) {
895 895
 			$default['link_html'] = isset($_POST['link_html']) ? $_POST['link_html'] : $link_html;
896 896
 			$form->addHtmlEditor('link_html', get_lang('Content'), false, false, array('ToolbarSet' => 'PortalHomePage', 'Width' => '100%', 'Height' => '400'));
897
-            $form->addButtonSave(get_lang('Save'),'submit');
897
+            $form->addButtonSave(get_lang('Save'), 'submit');
898 898
 
899 899
 		} else {
900
-			if (in_array($action, array('edit_tabs','insert_tabs'))) {
900
+			if (in_array($action, array('edit_tabs', 'insert_tabs'))) {
901 901
 				$default['link_html'] = isset($_POST['link_html']) ? $_POST['link_html'] : (!empty($link_html) ? $link_html : '');
902 902
 				$form->addHtmlEditor('link_html', get_lang('Content'), false, false, array('ToolbarSet' => 'PortalHomePage', 'Width' => '100%', 'Height' => '400'));
903 903
 			}
904 904
 			$form->addElement('checkbox', 'all_langs', null, get_lang('ApplyAllLanguages'), array('id' => 'all_langs'));
905
-			$form->addElement('html','<table id="table_langs" style="margin-left:159px;"><tr>');
905
+			$form->addElement('html', '<table id="table_langs" style="margin-left:159px;"><tr>');
906 906
 			$i = 0;
907 907
 			foreach ($_languages['name'] as $key => $value) {
908 908
 				$i++;
@@ -915,8 +915,8 @@  discard block
 block discarded – undo
915 915
 				}
916 916
 				$form->addElement('html', $html_langs);
917 917
 			}
918
-			$form->addElement('html','</tr></table><br/>');
919
-            $form->addButtonSave(get_lang('Save'),'submit');
918
+			$form->addElement('html', '</tr></table><br/>');
919
+            $form->addButtonSave(get_lang('Save'), 'submit');
920 920
 		}
921 921
 
922 922
 		$form->setDefaults($default);
@@ -930,7 +930,7 @@  discard block
 block discarded – undo
930 930
 			$open = $home_top;
931 931
 		} else {
932 932
 			$name = $newsf;
933
-			$open = @(string)file_get_contents($homep.$newsf.'_'.$lang.$ext);
933
+			$open = @(string) file_get_contents($homep.$newsf.'_'.$lang.$ext);
934 934
 		}
935 935
 		$open = api_to_system_encoding($open, api_detect_encoding(strip_tags($open)));
936 936
 
@@ -946,7 +946,7 @@  discard block
 block discarded – undo
946 946
 			'',
947 947
 			array('style' => 'margin: 0px;')
948 948
 		);
949
-		$renderer =& $form->defaultRenderer();
949
+		$renderer = & $form->defaultRenderer();
950 950
 		$renderer->setHeaderTemplate('');
951 951
 		$renderer->setFormTemplate('<form{attributes}><table border="0" cellpadding="5" cellspacing="0" width="100%">{content}</table></form>');
952 952
 		$renderer->setCustomElementTemplate('<tr><td>{element}</td></tr>');
@@ -972,8 +972,8 @@  discard block
 block discarded – undo
972 972
 
973 973
 		$default[$name] = str_replace('{rel_path}', api_get_path(REL_PATH), $open);
974 974
 		$form->addHtmlEditor($name, '', true, false, array('ToolbarSet' => 'PortalHomePage', 'Width' => '100%', 'Height' => '400'));
975
-		$form->addElement('checkbox', 'all_langs', null, get_lang('ApplyAllLanguages'),array('id' => 'all_langs'));
976
-		$form->addElement('html','<table id="table_langs" style="margin-left:5px;"><tr>');
975
+		$form->addElement('checkbox', 'all_langs', null, get_lang('ApplyAllLanguages'), array('id' => 'all_langs'));
976
+		$form->addElement('html', '<table id="table_langs" style="margin-left:5px;"><tr>');
977 977
 
978 978
 		$currentLanguage = api_get_interface_language();
979 979
 		$i = 0;
@@ -982,7 +982,7 @@  discard block
 block discarded – undo
982 982
 			$i++;
983 983
 
984 984
 			$checked = null;
985
-			if ($languageGet == $lang_name)  {
985
+			if ($languageGet == $lang_name) {
986 986
 				$checked = "checked";
987 987
 			}
988 988
 			$html_langs = '<td width="300">';
@@ -993,7 +993,7 @@  discard block
 block discarded – undo
993 993
 			}
994 994
 			$form->addElement('html', $html_langs);
995 995
 		}
996
-		$form->addElement('html','</tr></table><br/>');
996
+		$form->addElement('html', '</tr></table><br/>');
997 997
 		$form->addButtonSave(get_lang('Save'));
998 998
 		$form->setDefaults($default);
999 999
 		$form->display();
@@ -1046,9 +1046,9 @@  discard block
 block discarded – undo
1046 1046
                                 <?php
1047 1047
                                     $home_notice = '';
1048 1048
                                     if (file_exists($homep.$noticef.'_'.$lang.$ext)) {
1049
-                                        $home_notice = @(string)file_get_contents($homep.$noticef.'_'.$lang.$ext);
1049
+                                        $home_notice = @(string) file_get_contents($homep.$noticef.'_'.$lang.$ext);
1050 1050
                                     } else {
1051
-                                        $home_notice = @(string)file_get_contents($homep.$noticef.$ext);
1051
+                                        $home_notice = @(string) file_get_contents($homep.$noticef.$ext);
1052 1052
                                     }
1053 1053
                                     $home_notice = api_to_system_encoding($home_notice, api_detect_encoding(strip_tags($home_notice)));
1054 1054
                                     echo '<div class="homepage_notice">';
@@ -1073,7 +1073,7 @@  discard block
 block discarded – undo
1073 1073
                     </div>
1074 1074
                     <div id="links-list" class="panel-collapse collapse in" role="tabpanel" aria-labelledby="headingOne">
1075 1075
                         <div class="panel-body">
1076
-                            <a href="<?php echo $selfUrl; ?>?action=insert_link"><?php echo Display::return_icon('add.png', get_lang('InsertLink')).'&nbsp;'. get_lang('InsertLink'); ?>
1076
+                            <a href="<?php echo $selfUrl; ?>?action=insert_link"><?php echo Display::return_icon('add.png', get_lang('InsertLink')).'&nbsp;'.get_lang('InsertLink'); ?>
1077 1077
                             </a>
1078 1078
                             <ul class="menulist">
1079 1079
                                 <?php
@@ -1112,16 +1112,16 @@  discard block
 block discarded – undo
1112 1112
         <div class="col-md-9">
1113 1113
             <div class="actions">
1114 1114
 		<a href="<?php echo $selfUrl; ?>?action=edit_top&language=<?php echo $languageGet; ?>">
1115
-                    <?php echo Display::return_icon('edit.png', get_lang('EditHomePage'),null,ICON_SIZE_SMALL).'&nbsp;'. get_lang('EditHomePage'); ?>
1115
+                    <?php echo Display::return_icon('edit.png', get_lang('EditHomePage'), null, ICON_SIZE_SMALL).'&nbsp;'.get_lang('EditHomePage'); ?>
1116 1116
 		</a>
1117 1117
             </div>
1118 1118
             <section id="homepage-home">
1119 1119
                 <?php
1120 1120
                 //print home_top contents
1121 1121
                 if (file_exists($homep.$topf.'_'.$lang.$ext)) {
1122
-                    $home_top_temp = @(string)file_get_contents($homep.$topf.'_'.$lang.$ext);
1122
+                    $home_top_temp = @(string) file_get_contents($homep.$topf.'_'.$lang.$ext);
1123 1123
                 } else {
1124
-                    $home_top_temp = @(string)file_get_contents($homep.$topf.$ext);
1124
+                    $home_top_temp = @(string) file_get_contents($homep.$topf.$ext);
1125 1125
                 }
1126 1126
                 $open = str_replace('{rel_path}', api_get_path(REL_PATH), $home_top_temp);
1127 1127
                 $open = api_to_system_encoding($open, api_detect_encoding(strip_tags($open)));
@@ -1157,11 +1157,11 @@  discard block
 block discarded – undo
1157 1157
 
1158 1158
             <?php
1159 1159
 		if (file_exists($homep.$newsf.'_'.$lang.$ext)) {
1160
-                    $open = @(string)file_get_contents($homep.$newsf.'_'.$lang.$ext);
1160
+                    $open = @(string) file_get_contents($homep.$newsf.'_'.$lang.$ext);
1161 1161
                     $open = api_to_system_encoding($open, api_detect_encoding(strip_tags($open)));
1162 1162
                     echo $open;
1163 1163
 		} else {
1164
-                    $open = @(string)file_get_contents($homep.$newsf.$ext);
1164
+                    $open = @(string) file_get_contents($homep.$newsf.$ext);
1165 1165
                     $open = api_to_system_encoding($open, api_detect_encoding(strip_tags($open)));
1166 1166
                     echo $open;
1167 1167
 		}
@@ -1198,7 +1198,7 @@  discard block
 block discarded – undo
1198 1198
                             $tab_string = str_replace(array('href="'.api_get_path(WEB_PATH).'index.php?include=', '</li>'),
1199 1199
                                 array('href="'.api_get_path(WEB_CODE_PATH).'admin/'.basename($selfUrl).'?action=open_link&link=', $edit_link.$delete_link.'</li>'),
1200 1200
 				$enreg);
1201
-                            $tab_string = str_replace(array('<li>', '</li>','class="hide_menu"', 'hide_menu'), '', $tab_string);
1201
+                            $tab_string = str_replace(array('<li>', '</li>', 'class="hide_menu"', 'hide_menu'), '', $tab_string);
1202 1202
 				$link_list .= Display::tag('li', $tab_string, array('class' => 'list-group-item'));
1203 1203
 				$tab_counter++;
1204 1204
 			}
@@ -1206,7 +1206,7 @@  discard block
 block discarded – undo
1206 1206
             ?>
1207 1207
             <div class="actions">
1208 1208
 		<a href="<?php echo $selfUrl; ?>?action=insert_tabs">
1209
-                    <?php echo Display::return_icon('add.png', get_lang('InsertLink')) .'&nbsp;'. get_lang('InsertLink'); ?>
1209
+                    <?php echo Display::return_icon('add.png', get_lang('InsertLink')).'&nbsp;'.get_lang('InsertLink'); ?>
1210 1210
                 </a>
1211 1211
             </div>
1212 1212
             <?php
Please login to merge, or discard this patch.