Passed
Push — master ( 1f995c...3a0433 )
by Jonathan
05:20
created
src/Webtrees/Module/Hooks/Views/AdminConfigView.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -60,13 +60,13 @@
 block discarded – undo
60 60
 							?>
61 61
 							<tr>
62 62
 								<td>
63
-									<input type="hidden" name="hook-<?php echo $hook->id;?>" value="yes" >
64
-									<?php echo FunctionsEdit::twoStateCheckbox('status-'.($hook->id), ($hook->status)=='enabled'); ?>
63
+									<input type="hidden" name="hook-<?php echo $hook->id; ?>" value="yes" >
64
+									<?php echo FunctionsEdit::twoStateCheckbox('status-'.($hook->id), ($hook->status) == 'enabled'); ?>
65 65
 								</td>
66
-								<td><?php echo (($hook->status)=='enabled'); ?></td>
66
+								<td><?php echo (($hook->status) == 'enabled'); ?></td>
67 67
 								<td><?php echo $hook->hook; ?></td>
68 68
 								<td><?php echo $hook->context; ?></td>
69
-								<td><?php if($mod = Module::getModuleByName($hook->module)) echo $mod->getTitle(); ?></td>
69
+								<td><?php if ($mod = Module::getModuleByName($hook->module)) echo $mod->getTitle(); ?></td>
70 70
 								<td><input type="text" class="center" size="2" value="<?php echo $hook->priority; ?>" name="moduleorder-<?php echo $hook->id; ?>" /></td>
71 71
 								<td><?php echo $hook->priority; ?></td>
72 72
 							</tr>
Please login to merge, or discard this patch.
src/Webtrees/Module/ModuleManager.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -61,10 +61,10 @@
 block discarded – undo
61 61
      * @see \MyArtJaub\Webtrees\Module\ModuleManagerInterface::isOperational()
62 62
      */
63 63
     public function isOperational($moduleName) {
64
-    	if(!array_key_exists($moduleName, $this->modules_list)) {
65
-    		if($module = fw\Module::getModuleByName($moduleName)) {
66
-    			if($module instanceof DependentInterface) {
67
-    				if($module->validatePrerequisites()) {
64
+    	if (!array_key_exists($moduleName, $this->modules_list)) {
65
+    		if ($module = fw\Module::getModuleByName($moduleName)) {
66
+    			if ($module instanceof DependentInterface) {
67
+    				if ($module->validatePrerequisites()) {
68 68
     					$this->modules_list[$moduleName] = TRUE;
69 69
     					return true;
70 70
     				} else {
Please login to merge, or discard this patch.
src/Webtrees/Module/GeoDispersion/Model/GeoDisplayOptions.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 	 *
45 45
      * @return (OutlineMap|null)
46 46
      */
47
-    public function getMap(){
47
+    public function getMap() {
48 48
         return $this->map;
49 49
     }
50 50
     
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 	 *
65 65
 	 * @return int
66 66
 	 */
67
-    public function getMapLevel(){
67
+    public function getMapLevel() {
68 68
         return $this->map_level;
69 69
     }
70 70
     
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 	 *
85 85
 	 * @return bool
86 86
 	 */
87
-    public function isUsingFlags(){
87
+    public function isUsingFlags() {
88 88
         return $this->use_flags;
89 89
     }
90 90
     
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
 	 *
105 105
 	 * @return int
106 106
 	 */
107
-    public function getMaxDetailsInGen(){
107
+    public function getMaxDetailsInGen() {
108 108
         return $this->max_details_in_gen;
109 109
     }
110 110
     
Please login to merge, or discard this patch.
src/Webtrees/Module/GeoDispersion/Views/GeoAnalysisView.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
 			<h2><?php echo $this->data->get('title'); ?></h2>
31 31
 			
32 32
 			<?php
33
-			if($this->data->get('has_analysis', false)) { 
33
+			if ($this->data->get('has_analysis', false)) { 
34 34
 			    /** @var GeoAnalysis $ga  */
35 35
     		    $ga = $this->data->get('geoanalysis');
36 36
     		?>
Please login to merge, or discard this patch.
src/Webtrees/Module/GeoDispersion/Views/GeoAnalysisTabGenerationsView.php 1 patch
Spacing   +30 added lines, -31 removed lines patch added patch discarded remove patch
@@ -38,39 +38,38 @@  discard block
 block discarded – undo
38 38
         '<div id="geodispersion_gen">
39 39
         	<table id="geodispersion_gentable" class="center">';
40 40
             
41
-        foreach($results_by_gen as $gen => $genData){
41
+        foreach ($results_by_gen as $gen => $genData) {
42 42
             $html .= 
43 43
             '<tr>
44 44
                 <td class="descriptionbox">' .
45 45
                     I18N::translate("Generation %s", I18N::number($gen)).
46 46
                     ($display_all_places ? '<br />' : ' ').
47
-                    I18N::translate('(%s)', I18N::percentage(Functions::safeDivision($genData['sum'] + $genData['other'], $genData['sum'] + $genData['other'] + $genData['unknown']),1)) . 
47
+                    I18N::translate('(%s)', I18N::percentage(Functions::safeDivision($genData['sum'] + $genData['other'], $genData['sum'] + $genData['other'] + $genData['unknown']), 1)). 
48 48
                 '</td>
49 49
                  <td class="optionbox left">'.
50 50
                     ($display_all_places ? 
51
-                        $this->htmlGenerationAllPlacesRow($genData, $analysis_level) :
52
-                        $this->htmlGenerationTopPlacesRow($genData, $analysis_level)
53
-                     ) .
51
+                        $this->htmlGenerationAllPlacesRow($genData, $analysis_level) : $this->htmlGenerationTopPlacesRow($genData, $analysis_level)
52
+                     ).
54 53
                 '</ditdv>
55 54
             </tr>';
56 55
         }       
57 56
         
58
-        $html.= 
57
+        $html .= 
59 58
             '</table>
60 59
             <div class="left">
61
-                <strong>' . I18N::translate('Interpretation help:') . '</strong>
60
+                <strong>' . I18N::translate('Interpretation help:').'</strong>
62 61
                 <br />'.
63
-                I18N::translate('<strong>Generation X (yy %%)</strong>: The percentage indicates the number of found places compared to the total number of ancestors in this generation.') . 
62
+                I18N::translate('<strong>Generation X (yy %%)</strong>: The percentage indicates the number of found places compared to the total number of ancestors in this generation.'). 
64 63
                 '<br />';
65
-        if(!is_null($max_details_gen) && $max_details_gen == 0){
64
+        if (!is_null($max_details_gen) && $max_details_gen == 0) {
66 65
             $html .= I18N::translate('<strong><em>Place</em> or <em>Flag</em> aa (bb %%)</strong>: The first number indicates the total number of ancestors born in this place, the percentage relates this count to the total number of found places. No percentage means it is less than 10%%.').'<br />';
67 66
             $html .= I18N::translate('If any, the darker area indicates the number of unknown places within the generation or places outside the analysed area, and its percentage compared to the number of ancestors. No percentage means it is less than 10%%.');
68 67
         }
69
-        else{
68
+        else {
70 69
             $html .= I18N::translate('<strong><em>Place</em> [aa - bb %%]</strong>: The first number indicates the total number of ancestors born in this place, the percentage compares this count to the total number of found places.').'<br />';
71 70
             $html .= I18N::translate('Only the %d more frequent places for each generation are displayed.', $max_details_gen);
72 71
         }
73
-        $html.= 
72
+        $html .= 
74 73
             '</div>
75 74
         </div>';
76 75
         
@@ -92,25 +91,25 @@  discard block
 block discarded – undo
92 91
         
93 92
         $sum_gen = $data['sum'];
94 93
         $unknownother = $data['unknown'] + $data['other'];
95
-        foreach($data['places'] as $placename=> $dataplace){
96
-            $levels = array_map('trim',explode(',', $placename));
94
+        foreach ($data['places'] as $placename=> $dataplace) {
95
+            $levels = array_map('trim', explode(',', $placename));
97 96
             $content = '';
98
-            if(isset($dataplace['flag'])){
99
-                $content .= '<td class="geodispersion_flag">'. FunctionsPrint::htmlPlaceIcon($dataplace['place'], $dataplace['flag']) .'</td><td>';
97
+            if (isset($dataplace['flag'])) {
98
+                $content .= '<td class="geodispersion_flag">'.FunctionsPrint::htmlPlaceIcon($dataplace['place'], $dataplace['flag']).'</td><td>';
100 99
             }
101
-            else{
102
-                $content .= '<td><span title="'.implode(I18N::$list_separator, array_reverse($levels)).'">'.$levels[$analysis_level-1].'</span><br/>';
100
+            else {
101
+                $content .= '<td><span title="'.implode(I18N::$list_separator, array_reverse($levels)).'">'.$levels[$analysis_level - 1].'</span><br/>';
103 102
             }
104 103
             $count = $dataplace['count'];
105 104
             $content .= I18N::number($count);
106 105
             $perc = Functions::safeDivision($count, $sum_gen + $unknownother);
107
-            $perc2= Functions::safeDivision($count, $sum_gen);
108
-            if($perc2>=0.1) 
109
-                $content.= '<br/><span class="small">('.I18N::percentage($perc2, 1).')</span>';
106
+            $perc2 = Functions::safeDivision($count, $sum_gen);
107
+            if ($perc2 >= 0.1) 
108
+                $content .= '<br/><span class="small">('.I18N::percentage($perc2, 1).')</span>';
110 109
             $content .= '</td>';
111 110
                
112 111
             $html .= '
113
-                <td class="geodispersion_rowitem" width="'.max(round(100*$perc, 0),1).'%">
112
+                <td class="geodispersion_rowitem" width="'.max(round(100 * $perc, 0), 1).'%">
114 113
                     <table>
115 114
                         <tr>
116 115
                             <td>
@@ -123,11 +122,11 @@  discard block
 block discarded – undo
123 122
                 </td>';
124 123
         }
125 124
         
126
-        if($unknownother>0){
127
-            $perc= Functions::safeDivision($unknownother, $sum_gen + $unknownother);
128
-            $html .='<td class="geodispersion_unknownitem left" >'.I18N::number($unknownother);
129
-            if($perc>=0.1) $html.= '<br/><span class="small">('.I18N::percentage($perc, 1).')</span>';
130
-            $html .='</td>';
125
+        if ($unknownother > 0) {
126
+            $perc = Functions::safeDivision($unknownother, $sum_gen + $unknownother);
127
+            $html .= '<td class="geodispersion_unknownitem left" >'.I18N::number($unknownother);
128
+            if ($perc >= 0.1) $html .= '<br/><span class="small">('.I18N::percentage($perc, 1).')</span>';
129
+            $html .= '</td>';
131 130
         }
132 131
         
133 132
         $html .= 
@@ -148,12 +147,12 @@  discard block
 block discarded – undo
148 147
         $sum_gen = $data['sum'];
149 148
         $other = $data['other'];
150 149
         
151
-        foreach($data['places'] as $placename => $count) {
152
-            if($placename != 'other'){
153
-                $levels = array_map('trim',explode(',', $placename));
154
-                $placename = '<span title="'.implode(I18N::$list_separator, array_reverse($levels)).'">'.$levels[$analysis_level-1].'</span>';
150
+        foreach ($data['places'] as $placename => $count) {
151
+            if ($placename != 'other') {
152
+                $levels = array_map('trim', explode(',', $placename));
153
+                $placename = '<span title="'.implode(I18N::$list_separator, array_reverse($levels)).'">'.$levels[$analysis_level - 1].'</span>';
155 154
             }
156
-            else{
155
+            else {
157 156
                 $placename = I18N::translate('Other places');
158 157
             }
159 158
             $tmp_places[] = I18N::translate('<strong>%s</strong> [%d - %s]', $placename, $count, I18N::percentage(Functions::safeDivision($count, $sum_gen + $other), 1));         	
Please login to merge, or discard this patch.
src/Webtrees/Module/GeoDispersion/Views/GeoAnalysisListView.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -29,17 +29,17 @@
 block discarded – undo
29 29
 			<h2><?php echo $this->data->get('title'); ?></h2>
30 30
 			
31 31
 			<?php
32
-			if($this->data->get('has_list', false)) { 
32
+			if ($this->data->get('has_list', false)) { 
33 33
     		    $galist = $this->data->get('geoanalysislist');
34 34
     		?>
35 35
     		
36 36
     		<p class="center"><?php echo I18N::translate('Choose a geographical dispersion analysis:'); ?><p>
37 37
 			
38 38
 			<div class="maj-table">
39
-				<?php foreach($galist as $ga) {?>
39
+				<?php foreach ($galist as $ga) {?>
40 40
 				<div class="maj-row">
41 41
 					<div class="label">	
42
-						<?php if($ga->hasMap()) { ?>
42
+						<?php if ($ga->hasMap()) { ?>
43 43
 						<i class="icon-maj-map" title="<?php echo I18N::translate('Map'); ?>"></i>
44 44
 						<?php } else { ?>
45 45
 						<i class="icon-maj-table" title="<?php echo I18N::translate('Table'); ?>"></i>
Please login to merge, or discard this patch.
Webtrees/Module/GeoDispersion/Views/AbstractGeoAnalysisTabGeneralView.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -34,26 +34,26 @@
 block discarded – undo
34 34
         $html = '<div id="geodispersion_summary">
35 35
         	<div class="maj-table center">
36 36
         		<div class="maj-row">
37
-        			<div class="label">' . I18N::translate('Places found'). '</div>
38
-        			<div class="value">' . I18N::translate('%1$d (%2$s)',$nb_found - $nb_other, I18N::percentage($perc_known)). '</div>
37
+        			<div class="label">' . I18N::translate('Places found').'</div>
38
+        			<div class="value">' . I18N::translate('%1$d (%2$s)', $nb_found - $nb_other, I18N::percentage($perc_known)).'</div>
39 39
         		</div>';
40
-        if($nb_other > 0){
40
+        if ($nb_other > 0) {
41 41
             $perc_other = Functions::safeDivision($nb_other, $nb_found + $nb_unknown);
42 42
             $html .=
43 43
         		'<div class="maj-row">
44
-        			<div class="label">' . I18N::translate('Other places'). '</div>
45
-        			<div class="value">' . I18N::translate('%1$d (%2$s)',$nb_other, I18N::percentage($perc_other)). '</div>
44
+        			<div class="label">' . I18N::translate('Other places').'</div>
45
+        			<div class="value">' . I18N::translate('%1$d (%2$s)', $nb_other, I18N::percentage($perc_other)).'</div>
46 46
         		</div>';
47 47
         }
48 48
         $html .= '<div class="maj-row">
49
-        			<div class="label">' . I18N::translate('Places not found'). '</div>
50
-        			<div class="value">' . I18N::translate('%1$d (%2$s)',$nb_unknown, I18N::percentage(1 - $perc_known)). '</div>
49
+        			<div class="label">' . I18N::translate('Places not found').'</div>
50
+        			<div class="value">' . I18N::translate('%1$d (%2$s)', $nb_unknown, I18N::percentage(1 - $perc_known)).'</div>
51 51
         		</div>
52 52
         	</div>
53 53
         </div>
54 54
         <br/>
55 55
 		<div id="geodispersion_data">
56
-		' . $this->htmlAnalysisData(). '
56
+		' . $this->htmlAnalysisData().'
57 57
 		</div>';
58 58
         
59 59
         return $html;
Please login to merge, or discard this patch.
src/Webtrees/Module/GeoDispersion/Views/GeoAnalysisTabGeneralTableView.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -29,34 +29,34 @@
 block discarded – undo
29 29
         $nb_found = $this->data->get('stats_gen_nb_found');
30 30
         $nb_other = $this->data->get('stats_gen_nb_other');
31 31
         
32
-        $i=1;
33
-        $previous_nb=0;        
32
+        $i = 1;
33
+        $previous_nb = 0;        
34 34
         
35
-        $html='<div class="maj-table center">';        
36
-        foreach($results as $place => $nb){
35
+        $html = '<div class="maj-table center">';        
36
+        foreach ($results as $place => $nb) {
37 37
             $perc = Functions::safeDivision($nb, $nb_found - $nb_other);
38
-            if($nb!=$previous_nb){
39
-                $j= I18N::number($i);
38
+            if ($nb != $previous_nb) {
39
+                $j = I18N::number($i);
40 40
             }
41
-            else{
42
-                $j='&nbsp;';
41
+            else {
42
+                $j = '&nbsp;';
43 43
             }
44 44
             	
45
-            $levels = array_map('trim',explode(',', $place));
46
-            $placename = $levels[$analysis_level-1];
47
-            if($placename == '' && $analysis_level > 1) $placename = I18N::translate('Unknown (%s)', $levels[$analysis_level-2]);
48
-            $html.=
45
+            $levels = array_map('trim', explode(',', $place));
46
+            $placename = $levels[$analysis_level - 1];
47
+            if ($placename == '' && $analysis_level > 1) $placename = I18N::translate('Unknown (%s)', $levels[$analysis_level - 2]);
48
+            $html .=
49 49
             '<div class="maj-row">
50 50
                 <div class="label"><strong>'.$j.'</strong></div>
51 51
                 <div class="label">'.$placename.'</div>
52
-                <div class="value">'.I18N::translate('%d',$nb).'</div>
53
-                <div class="value">'.I18N::percentage($perc,1).'</div>
52
+                <div class="value">'.I18N::translate('%d', $nb).'</div>
53
+                <div class="value">'.I18N::percentage($perc, 1).'</div>
54 54
              </div>';
55 55
             $i++;
56
-            $previous_nb=$nb;
56
+            $previous_nb = $nb;
57 57
         }
58 58
         
59
-        $html.='</div>';
59
+        $html .= '</div>';
60 60
         
61 61
         return $html;
62 62
     }
Please login to merge, or discard this patch.
src/Webtrees/Module/Certificates/Model/CertificateProviderInterface.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
 /**
16 16
  * Interface for providers of certificates
17 17
  */
18
-interface CertificateProviderInterface  {
18
+interface CertificateProviderInterface {
19 19
     
20 20
     /**
21 21
      * Returns the certificates directory path as it is really (within the firewall directory).
Please login to merge, or discard this patch.