Completed
Push — feature/code-analysis ( 820b23...92a398 )
by Jonathan
04:40 queued 01:48
created
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.
src/Webtrees/Module/Certificates/Views/CertificateView.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
         <div id="maj-cert-page" class="center">
31 31
 			<h2><?php echo $this->data->get('title'); ?></h2>
32 32
 			
33
-    		<?php if($this->data->get('has_certif', false)) { 
33
+    		<?php if ($this->data->get('has_certif', false)) { 
34 34
     		    /** @var Certificate $certificate   */
35 35
     		  $certificate = $this->data->get('certificate');
36 36
     		  $has_linked_indis = $this->data->get('has_linked_indis', false);
@@ -49,16 +49,16 @@  discard block
 block discarded – undo
49 49
     				<div id="certificate-edit">
50 50
     					<?php echo $certificate->displayImage(); ?>
51 51
     				</div>
52
-    				<?php if($has_linked_indis || $has_linked_fams) { ?>
52
+    				<?php if ($has_linked_indis || $has_linked_fams) { ?>
53 53
     				<ul>
54
-    					<?php if($has_linked_indis) { ?>
54
+    					<?php if ($has_linked_indis) { ?>
55 55
     					<li>
56 56
     						<a href="#indi-certificate">
57 57
     							<span id="indisource"><?php echo I18N::translate('Individuals'); ?></span>
58 58
     						</a>
59 59
     					</li>
60 60
     					<?php } ?>
61
-    					<?php if($has_linked_fams) { ?>
61
+    					<?php if ($has_linked_fams) { ?>
62 62
     					<li>
63 63
     						<a href="#fam-certificate">
64 64
     							<span id="famsource"><?php echo I18N::translate('Families'); ?></span>
@@ -67,13 +67,13 @@  discard block
 block discarded – undo
67 67
     					<?php } ?>
68 68
     				</ul>
69 69
     				
70
-    				<?php if($has_linked_indis) { ?>
70
+    				<?php if ($has_linked_indis) { ?>
71 71
     				<div id="indi-certificate">
72 72
     					<?php echo FunctionsPrintLists::individualTable($this->data->get('linked_indis')); ?>
73 73
     				</div>
74 74
 					<?php } ?>
75 75
 					
76
-					<?php if($has_linked_fams) { ?>
76
+					<?php if ($has_linked_fams) { ?>
77 77
     				<div id="fam-certificate">
78 78
     					<?php echo FunctionsPrintLists::familyTable($this->data->get('linked_fams')); ?>
79 79
     				</div>
Please login to merge, or discard this patch.
src/Webtrees/Module/PatronymicLineageModule.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
      public function getMenu(fw\Tree $tree, $reference) {
49 49
          $tree_url = $tree ? $tree->getNameUrl() : '';
50 50
          $surname = $reference && is_string($reference) ? $reference : '';
51
-         return new fw\Menu($this->getTitle(), 'module.php?mod=' . $this->getName() . '&mod_action=Lineage&ged=' . $tree_url . '&surname=' . $surname , 'menu-maj-list-lineage', array('rel' => 'nofollow'));
51
+         return new fw\Menu($this->getTitle(), 'module.php?mod='.$this->getName().'&mod_action=Lineage&ged='.$tree_url.'&surname='.$surname, 'menu-maj-list-lineage', array('rel' => 'nofollow'));
52 52
      }
53 53
 
54 54
 }
Please login to merge, or discard this patch.
src/Webtrees/Module/WelcomeBlock/Views/PiwikStatsView.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -23,11 +23,11 @@
 block discarded – undo
23 23
 	 * @see \MyArtJaub\Webtrees\Mvc\View\AbstractView::renderContent()
24 24
 	 */
25 25
     protected function renderContent() {    
26
-        if($this->data->get('has_stats', false)) {
27
-            $html = I18N::translate('%1$s visits since the beginning of %2$s<br>(%3$s today)' ,
28
-                '<span class="odometer">' . I18N::number($this->data->get('visits_year')) . '</span>',
26
+        if ($this->data->get('has_stats', false)) {
27
+            $html = I18N::translate('%1$s visits since the beginning of %2$s<br>(%3$s today)',
28
+                '<span class="odometer">'.I18N::number($this->data->get('visits_year')).'</span>',
29 29
                 date('Y'),
30
-                '<span class="odometer">' . I18N::number($this->data->get('visits_today')) . '</span>'
30
+                '<span class="odometer">'.I18N::number($this->data->get('visits_today')).'</span>'
31 31
                 );
32 32
         }
33 33
         else {
Please login to merge, or discard this patch.
src/Webtrees/Module/WelcomeBlock/Views/WelcomeBlockConfigView.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -28,42 +28,42 @@
 block discarded – undo
28 28
         return '
29 29
             <tr>
30 30
                 <td class="descriptionbox wrap width33">' .
31
-                I18N::translate('Enable Piwik Statistics') .
31
+                I18N::translate('Enable Piwik Statistics').
32 32
                 // Ideally, would like to add helpLink, but this does not work for modules...
33 33
                 // FunctionsPrint::helpLink('piwik_enabled', $this->getName());
34 34
                 '</td>
35 35
                 <td class="optionbox">' .
36
-                FunctionsEdit::editFieldYesNo('piwik_enabled', $this->data->get('piwik_enabled', '0')) .
36
+                FunctionsEdit::editFieldYesNo('piwik_enabled', $this->data->get('piwik_enabled', '0')).
37 37
                 '</td>
38 38
             </tr>' .
39 39
             
40 40
             '<tr>
41 41
                 <td class="descriptionbox wrap width33">' .
42
-                I18N::translate('Piwik URL') .
42
+                I18N::translate('Piwik URL').
43 43
                 '</td>
44 44
                 <td class="optionbox">
45 45
                     <input type="text" name="piwik_url" size="45" value="' . 
46
-                    $this->data->get('piwik_url', '') . '" />
46
+                    $this->data->get('piwik_url', '').'" />
47 47
                 </td>
48 48
             </tr>' .
49 49
             
50 50
             '<tr>
51 51
                 <td class="descriptionbox wrap width33">' .
52
-                    I18N::translate('Piwik Token') .
52
+                    I18N::translate('Piwik Token').
53 53
                 '</td>
54 54
                 <td class="optionbox">
55 55
                     <input type="text" name="piwik_token" size="45" value="' . 
56
-                    $this->data->get('piwik_token', '') . '" />
56
+                    $this->data->get('piwik_token', '').'" />
57 57
                 </td>
58 58
             </tr>' .
59 59
             
60 60
             '<tr>
61 61
                 <td class="descriptionbox wrap width33">' .
62
-                    I18N::translate('Piwik Site ID') .
62
+                    I18N::translate('Piwik Site ID').
63 63
                 '</td>
64 64
                 <td class="optionbox">
65 65
                     <input type="text" name="piwik_siteid" size="4" value="' . 
66
-                    $this->data->get('piwik_siteid', '') . '" />
66
+                    $this->data->get('piwik_siteid', '').'" />
67 67
                 </td>
68 68
             </tr>' ;
69 69
         
Please login to merge, or discard this patch.
src/Webtrees/Mvc/View/ViewFactory.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -53,13 +53,13 @@
 block discarded – undo
53 53
      */
54 54
     public function makeView($view_name, MvcController $mvc_ctrl, BaseController $ctrl, ViewBag $data) 
55 55
     {
56
-        if(!$mvc_ctrl) throw new \Exception('Mvc Controller not defined');
57
-        if(!$ctrl) throw new \Exception('Base Controller not defined');
58
-        if(!$view_name) throw new \Exception('View not defined');
56
+        if (!$mvc_ctrl) throw new \Exception('Mvc Controller not defined');
57
+        if (!$ctrl) throw new \Exception('Base Controller not defined');
58
+        if (!$view_name) throw new \Exception('View not defined');
59 59
         
60 60
         $mvc_ctrl_refl = new \ReflectionObject($mvc_ctrl);
61
-        $view_class = $mvc_ctrl_refl->getNamespaceName() . '\\Views\\' . $view_name . 'View';       
62
-        if(!class_exists($view_class)) throw new \Exception('View does not exist');
61
+        $view_class = $mvc_ctrl_refl->getNamespaceName().'\\Views\\'.$view_name.'View';       
62
+        if (!class_exists($view_class)) throw new \Exception('View does not exist');
63 63
         
64 64
         return new $view_class($ctrl, $data);
65 65
     }
Please login to merge, or discard this patch.