HourglassController::maxDescendencyGenerations() - Code Metrics - Inspection of "Scrutinizer Auto-Fixes (#1526)" - fisharebest/webtrees - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (380e00)

by Greg

created 2017-12-03 12:10 UTC

HourglassController::maxDescendencyGenerations() C

↳ Parent: HourglassController

Complexity

Conditions	7
Paths	10

Size

Total Lines	24
Code Lines	15

Duplication

Lines	11
Ratio	45.83 %

Importance

Changes

Metric	Value
cc	7
eloc	15
nc	10
nop	2
dl	11
loc	24
rs	6.7272
c	0
b	0
f	0

<?php
/**
 * webtrees: online genealogy
 * Copyright (C) 2017 webtrees development team
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU General Public License for more details.
 * You should have received a copy of the GNU General Public License
 * along with this program. If not, see <http://www.gnu.org/licenses/>.
 */
namespace Fisharebest\Webtrees\Controller;

use Fisharebest\Webtrees\Filter;
use Fisharebest\Webtrees\FontAwesome;
use Fisharebest\Webtrees\Functions\FunctionsPrint;
use Fisharebest\Webtrees\I18N;
use Fisharebest\Webtrees\Individual;
use Fisharebest\Webtrees\Theme;

/**
 * Controller for the hourglass chart
 */
class HourglassController extends ChartController {
	/** @var int Whether to show spouse details. */
	public $show_spouse;

	/** @var int Number of ascendancy generations to show. */
	public $generations;

	/** @var int Number of descendancy generations that exist. */
	public $dgenerations;

	/** @var int Half height of personbox. */
	public $bhalfheight;

	const SWITCH_LINK = "<a href='hourglass.php?rootid=%s&amp;show_spouse=%s&amp;generations=%s' class='name1'>%s</a>";

	/**
	 * Create the hourglass controller.
	 *
	 * @param string $rootid
	 */
	public function __construct($rootid = '') {

		parent::__construct();

		// Extract parameters from
		$this->show_spouse = Filter::getInteger('show_spouse', 0, 1, 0);
		$this->generations = Filter::getInteger('generations', 2, $this->tree()->getPreference('MAX_DESCENDANCY_GENERATIONS'), 3);

		$this->bhalfheight = (int) ($this->getBoxDimensions()->height / 2);

		//Checks how many generations of descendency is for the person for formatting purposes
		$this->dgenerations = $this->maxDescendencyGenerations($this->root, 0);
		if ($this->dgenerations < 1) {
			$this->dgenerations = 1;
		}

		$this->setPageTitle(/* I18N: %s is an individual’s name */ I18N::translate('Hourglass chart of %s', $this->root->getFullName()));
	}

	/**
	 * Prints pedigree of the person passed in. Which is the descendancy
	 *
	 * @param Individual $person ID of person to print the pedigree for
	 * @param int        $count  generation count, so it recursively calls itself
	 */
	public function printPersonPedigree(Individual $person, $count) {
		if ($count >= $this->generations) {
			return;
		}

		$genoffset = $this->generations; // handle pedigree n generations lines

		//
		//Prints empty table columns for children w/o parents up to the max generation
		//This allows vertical line spacing to be consistent
		//
		if (count($person->getChildFamilies()) == 0) {
			echo '<table><tr><td> ' . $this->printEmptyBox() . '</td>';
			echo '<td> ';
			// Recursively get the father’s family
			$this->printPersonPedigree($person, $count + 1);
			echo '</td></tr>';
			echo '<tr><td> ' . $this->printEmptyBox() . '</td>';
			echo '<td> ';
			// Recursively get the mother’s family
			$this->printPersonPedigree($person, $count + 1);
			echo '</td><td> </tr></table>';
		}
		foreach ($person->getChildFamilies() as $family) {
			echo '<table cellspacing="0" cellpadding="0" border="0"  class="hourglassChart">';
			echo '<tr>';
			echo '<td style="vertical-align:bottom"><img class="line3 pvline" src="' . Theme::theme()->parameter('image-vline') . '" width="3"></td>';
			echo '<td> <img class="lineh2" src="' . Theme::theme()->parameter('image-hline') . '" width="7" height="3"></td>';
			echo '<td class="myCharts"> ';
			//-- print the father box
			FunctionsPrint::printPedigreePerson($family->getHusband());
			echo '</td>';
			if ($family->getHusband()) {
				$ARID = $family->getHusband()->getXref();
				echo '<td id="td_' . $ARID . '">';
// for HTML
$sanitized = htmlentities($tainted, ENT_QUOTES);

// for URLs
$sanitized = urlencode($tainted);

				//-- print an Ajax arrow on the last generation of the adult male
				if ($count == $this->generations - 1 && $family->getHusband()->getChildFamilies()) {
					echo FontAwesome::linkIcon('arrow-end', I18N::translate('Parents'), [
						'href'           => '#',
						'data-direction' => 'asc',
						'data-xref'      => $ARID,
						'data-spouses'   => $this->show_spouse,
					]);
				}
				//-- recursively get the father’s family
				$this->printPersonPedigree($family->getHusband(), $count + 1);
				echo '</td>';
			} else {
				echo '<td> ';
				if ($count < $genoffset - 1) {
					echo '<table>';
					for ($i = $count; $i < (pow(2, ($genoffset - 1) - $count) / 2) + 2; $i++) {
						$this->printEmptyBox();
						echo '</tr>';
						$this->printEmptyBox();
						echo '</tr>';
					}
					echo '</table>';
				}
			}
			echo
			'</tr><tr>',
			"<td style='vertical-align:top'><img class='pvline' src='" . Theme::theme()->parameter('image-vline') . "' width='3' alt=''></td>",
				'<td> <img class="lineh3" src="' . Theme::theme()->parameter('image-hline') . '" width="7" height="3"></td>',
			'<td class="myCharts"> ';
			//-- print the mother box
			FunctionsPrint::printPedigreePerson($family->getWife());
			echo '</td>';
			if ($family->getWife()) {
				$ARID = $family->getWife()->getXref();
				echo '<td id="td_' . $ARID . '">';
// for HTML
$sanitized = htmlentities($tainted, ENT_QUOTES);

// for URLs
$sanitized = urlencode($tainted);

				//-- print an ajax arrow on the last generation of the adult female
				if ($count == $this->generations - 1 && $family->getWife()->getChildFamilies()) {
					echo FontAwesome::linkIcon('arrow-end', I18N::translate('Parents'), [
						'href'           => '#',
						'data-direction' => 'asc',
						'data-xref'      => $ARID,
						'data-spouses'   => $this->show_spouse,
					]);
				}
				//-- recursively print the mother’s family
				$this->printPersonPedigree($family->getWife(), $count + 1);
				echo '</td>';
			}
			echo '</tr></table>';
			break;
		}
	}

	/**
	 * Print empty box
	 *
	 * @return string
	 */

	private function printEmptyBox() {
		return Theme::theme()->individualBoxEmpty();
	}

	/**
	 * Prints descendency of passed in person
	 *
	 * @param Individual $person  person to print descendency for
	 * @param int        $count   count of generations to print
	 * @param bool       $showNav
	 *
	 * @return int
	 */
	public function printDescendency($person, $count, $showNav = true) {
		global $lastGenSecondFam;
function myFunction($a, $b) {
    // Do something
}

		if ($count > $this->dgenerations) {
			return 0;
		}
		$pid         = $person->getXref();
		$tablealign  = 'right';
		$otablealign = 'left';
		if (I18N::direction() === 'rtl') {
			$tablealign  = 'left';
			$otablealign = 'right';
		}

		//-- put a space between families on the last generation
		if ($count == $this->dgenerations - 1) {
			if (isset($lastGenSecondFam)) {
				echo '<br>';
			}
			$lastGenSecondFam = true;
		}
		echo "<table cellspacing=\"0\" cellpadding=\"0\" border=\"0\" id='table_$pid' class='hourglassChart' style='float:$tablealign'>";
		echo '<tr>';
		echo "<td style='text-align:$tablealign'>";
		$numkids  = 0;
		$families = $person->getSpouseFamilies();
		$famNum   = 0;
		$children = [];
		if ($count < $this->dgenerations) {
			// Put all of the children in a common array
			foreach ($families as $family) {
				$famNum++;
				foreach ($family->getChildren() as $child) {
					$children[] = $child;
				}
			}

			$ct = count($children);
			if ($ct > 0) {
				echo "<table cellspacing=\"0\" cellpadding=\"0\" border=\"0\" style='position: relative; top: auto; float: $tablealign;'>";
				for ($i = 0; $i < $ct; $i++) {
					$person2 = $children[$i];
					$chil    = $person2->getXref();
					echo '<tr>';
					echo '<td id="td_', $chil, '" class="', I18N::direction(), '" style="text-align:', $otablealign, '">';
// for HTML
$sanitized = htmlentities($tainted, ENT_QUOTES);

// for URLs
$sanitized = urlencode($tainted);
					$kids = $this->printDescendency($person2, $count + 1, $showNav);
					$numkids += $kids;
					echo '</td>';

					// Print the lines
					if ($ct > 1) {
						if ($i == 0) {
							// First child
							echo "<td style='vertical-align:bottom'><img alt='' class='line1 tvertline' id='vline_$chil' src='" . Theme::theme()->parameter('image-vline') . "' width='3'></td>";
						} elseif ($i == $ct - 1) {
							// Last child
							echo "<td style='vertical-align:top'><img alt='' class='bvertline' id='vline_$chil' src='" . Theme::theme()->parameter('image-vline') . "' width='3'></td>";
						} else {
							// Middle child
							echo '<td style="background: url(\'' . Theme::theme()->parameter('image-vline') . '\');"><img src=\'' . Theme::theme()->parameter('image-spacer') . '\' width="3"></td>';
						}
					}
					echo '</tr>';
				}
				echo '</table>';
			}
			echo '</td>';
			echo '<td class="myCharts" width="', $this->getBoxDimensions()->width, '">';
		}

		// Print the descendency expansion arrow
		if ($count == $this->dgenerations) {
			$numkids = 1;
			$tbwidth = $this->getBoxDimensions()->width + 16;
			for ($j = $count; $j < $this->dgenerations; $j++) {
				echo "<div style='width: ", $tbwidth, "px;'><br></div></td><td style='width:", $this->getBoxDimensions()->width, "px'>";
			}
			$kcount = 0;
			foreach ($families as $family) {
				$kcount += $family->getNumberOfChildren();
			}
			if ($kcount == 0) {
				echo "</td><td style='width:", $this->getBoxDimensions()->width, "px'>";
			} else {
				echo FontAwesome::linkIcon('arrow-start', I18N::translate('Children'), [
					'href'           => '#',
					'data-direction' => 'desc',
					'data-xref'      => $pid,
					'data-spouses'   => $this->show_spouse,
				]);

				//-- move the arrow up to line up with the correct box
				if ($this->show_spouse) {
					echo str_repeat('<br><br><br>', count($families));
				}
				echo "</td><td style='width:", $this->getBoxDimensions()->width, "px'>";
			}
		}

		echo '<table cellspacing="0" cellpadding="0" border="0" id="table2_' . $pid . '"><tr><td> ';
// for HTML
$sanitized = htmlentities($tainted, ENT_QUOTES);

// for URLs
$sanitized = urlencode($tainted);
		FunctionsPrint::printPedigreePerson($person);
		echo '</td><td> <img class="lineh1" src="' . Theme::theme()->parameter('image-hline') . '" width="7" height="3">';

		//----- Print the spouse
		if ($this->show_spouse) {
			foreach ($families as $family) {
				echo "</td></tr><tr><td style='text-align:$otablealign'>";
				FunctionsPrint::printPedigreePerson($family->getSpouse($person));
				$numkids++;
				echo '</td><td> </td>';
			}
			//-- add offset divs to make things line up better
			if ($count == $this->dgenerations) {
				echo "<tr><td colspan '2'><div style='height:", ($this->bhalfheight / 2), 'px; width:', $this->getBoxDimensions()->width, "px;'><br></div>";
			}
		}
		echo '</td></tr></table>';

		// For the root person, print a down arrow that allows changing the root of tree
		if ($showNav && $count == 1) {
			if ($person->canShowName()) {
				// -- print left arrow for decendants so that we can move down the tree
				$famids = $person->getSpouseFamilies();
				//-- make sure there is more than 1 child in the family with parents
				$cfamids = $person->getChildFamilies();
				$num     = 0;
				foreach ($cfamids as $family) {
					$num += $family->getNumberOfChildren();
				}
				if ($num > 0) {
					echo '<div class="center" id="childarrow" style="position:absolute; width:', $this->getBoxDimensions()->width, 'px;">';
					echo FontAwesome::linkIcon('arrow-down', I18N::translate('Family'), ['href' => '#', 'id' => 'spouse-child-links']);
					echo '<div id="childbox">';
					echo '<table cellspacing="0" cellpadding="0" border="0" class="person_box"><tr><td> ';

					foreach ($famids as $family) {
						echo "<span class='name1'>" . I18N::translate('Family') . '</span>';
						$spouse = $family->getSpouse($person);
						if ($spouse) {
							printf(self::SWITCH_LINK, $spouse->getXref(), $this->show_spouse, $this->generations, $spouse->getFullName());
// for HTML
$sanitized = htmlentities($tainted, ENT_QUOTES);

// for URLs
$sanitized = urlencode($tainted);
						}
						foreach ($family->getChildren() as $child) {
							printf(self::SWITCH_LINK, $child->getXref(), $this->show_spouse, $this->generations, $child->getFullName());
// for HTML
$sanitized = htmlentities($tainted, ENT_QUOTES);

// for URLs
$sanitized = urlencode($tainted);
						}
					}

					//-- print the siblings
					foreach ($cfamids as $family) {
						if ($family->getHusband() || $family->getWife()) {
							echo "<span class='name1'>" . I18N::translate('Parents') . '</span>';
							$husb = $family->getHusband();
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
							if ($husb) {
								printf(self::SWITCH_LINK, $husb->getXref(), $this->show_spouse, $this->generations, $husb->getFullName());
// for HTML
$sanitized = htmlentities($tainted, ENT_QUOTES);

// for URLs
$sanitized = urlencode($tainted);
							}
							$wife = $family->getWife();
							if ($wife) {
								printf(self::SWITCH_LINK, $wife->getXref(), $this->show_spouse, $this->generations, $wife->getFullName());
// for HTML
$sanitized = htmlentities($tainted, ENT_QUOTES);

// for URLs
$sanitized = urlencode($tainted);
							}
						}

						// filter out root person from children array so only siblings remain
						$siblings = array_filter($family->getChildren(), function (Individual $item) use ($pid) {
							return $item->getXref() != $pid;
						});
						$num = count($siblings);
						if ($num) {
							echo "<span class='name1'>";
							echo $num > 1 ? I18N::translate('Siblings') : I18N::translate('Sibling');
							echo '</span>';
							foreach ($siblings as $child) {
								printf(self::SWITCH_LINK, $child->getXref(), $this->show_spouse, $this->generations, $child->getFullName());
// for HTML
$sanitized = htmlentities($tainted, ENT_QUOTES);

// for URLs
$sanitized = urlencode($tainted);
							}
						}
					}
					echo '</td></tr></table>';
					echo '</div>';
					echo '</div>';
				}
			}
		}
		echo '</td></tr></table>';

		return $numkids;
	}

	/**
	 * Calculates number of generations a person has
	 *
	 * @param Individual $individual Start individual
	 * @param int        $depth      Pass in 0 and it calculates how far down descendency goes
	 *
	 * @return int Number of generations the descendency actually goes
	 */
	private function maxDescendencyGenerations(Individual $individual, $depth) {
		if ($depth > $this->generations) {
			return $depth;
		}
		$maxdc = $depth;
		foreach ($individual->getSpouseFamilies() as $family) {
			foreach ($family->getChildren() as $child) {
				$dc = $this->maxDescendencyGenerations($child, $depth + 1);
				if ($dc >= $this->generations) {
					return $dc;
				}
				if ($dc > $maxdc) {
					$maxdc = $dc;
				}
			}
		}

		$maxdc++;
		if ($maxdc == 1) {
			$maxdc++;
		}

		return $maxdc;
	}

	/**
	 * setup all of the javascript that is needed for the hourglass chart
	 *
	 * @return string
	 */
	public function setupJavascript() {
		return "
			(function() {
				function sizeLines() {
					$('.tvertline').each(function(i,e) {
						var pid = e.id.split('_').pop();
						e.style.height = Math.abs($('#table_' + pid)[0].offsetHeight - ($('#table2_' + pid)[0].offsetTop + {$this->bhalfheight})) + 'px';
					});

					$('.bvertline').each(function(i,e) {
						var pid = e.id.split('_').pop();
						e.style.height = $('#table_' + pid)[0].offsetTop + $('#table2_' + pid)[0].offsetTop + {$this->bhalfheight} + 'px';
					});

					$('.pvline').each(function(i,e) {
						var el = $(e);
						el.height(Math.floor(el.parent().height()/2));
					});
				}

				$('#spouse-child-links').on('click', function(e) {
					e.preventDefault();
					$('#childbox').slideToggle('fast');
				})
				$('.hourglassChart').on('click', '.wt-icon-arrow-start, .wt-icon-arrow-end', function (e) {
					e.preventDefault();
					e.stopPropagation();

					var direction = this.parentNode.dataset.direction;
					var xref      = this.parentNode.dataset.xref;
					var spouses   = this.parentNode.dataset.spouses;
					
					$('#td_' + xref).load('hourglass_ajax.php?rootid='+ xref +'&generations=1&type=' + direction + '&show_spouse=' + spouses, function() {
						sizeLines();
					});
				});

				sizeLines();
			})();
		";
	}
}


Branch — master (380e00)

HourglassController::maxDescendencyGenerations() C

Complexity

Size

Duplication

Importance

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

1. Pass all data via parameters

2. Create a class that maintains your state

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

1			<?php
2			/**
3			* webtrees: online genealogy
4			* Copyright (C) 2017 webtrees development team
5			* This program is free software: you can redistribute it and/or modify
6			* it under the terms of the GNU General Public License as published by
7			* the Free Software Foundation, either version 3 of the License, or
8			* (at your option) any later version.
9			* This program is distributed in the hope that it will be useful,
10			* but WITHOUT ANY WARRANTY; without even the implied warranty of
11			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12			* GNU General Public License for more details.
13			* You should have received a copy of the GNU General Public License
14			* along with this program. If not, see <http://www.gnu.org/licenses/>.
15			*/
16			namespace Fisharebest\Webtrees\Controller;
17
18			use Fisharebest\Webtrees\Filter;
19			use Fisharebest\Webtrees\FontAwesome;
20			use Fisharebest\Webtrees\Functions\FunctionsPrint;
21			use Fisharebest\Webtrees\I18N;
22			use Fisharebest\Webtrees\Individual;
23			use Fisharebest\Webtrees\Theme;
24
25			/**
26			* Controller for the hourglass chart
27			*/
28			class HourglassController extends ChartController {
29			/** @var int Whether to show spouse details. */
30			public $show_spouse;
31
32			/** @var int Number of ascendancy generations to show. */
33			public $generations;
34
35			/** @var int Number of descendancy generations that exist. */
36			public $dgenerations;
37
38			/** @var int Half height of personbox. */
39			public $bhalfheight;
40

fisharebest / webtrees

Branch — master (380e00)

HourglassController::maxDescendencyGenerations() C

Complexity

Size

Duplication

Importance

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

1. Pass all data via parameters

2. Create a class that maintains your state

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

11 paths for user data to reach this point

Preventing Cross-Site-Scripting Attacks

General Strategies to prevent injection

Duplication Side-by-Side

Filter issues like