Sources/Subs-Calendar.php ➔ getBirthdayRange()

Complexity

Conditions	12
Paths	52

Size

Total Lines	87
Code Lines	49

Duplication

Lines	2
Ratio	2.3 %

Importance

Changes

0

<?php
function getBirthdayRange($low_date, $high_date)
{
	global $smcFunc;

	// We need to search for any birthday in this range, and whatever year that birthday is on.
	$year_low = (int) substr($low_date, 0, 4);
	$year_high = (int) substr($high_date, 0, 4);

	if($smcFunc['db_title'] != "PostgreSQL") {
		// Collect all of the birthdays for this month.  I know, it's a painful query.
		$result = $smcFunc['db_query']('birthday_array', '
			SELECT id_member, real_name, YEAR(birthdate) AS birth_year, birthdate
			FROM {db_prefix}members
			WHERE YEAR(birthdate) != {string:year_one}
				AND MONTH(birthdate) != {int:no_month}
				AND DAYOFMONTH(birthdate) != {int:no_day}
				AND YEAR(birthdate) <= {int:max_year}
				AND (
					DATE_FORMAT(birthdate, {string:year_low}) BETWEEN {date:low_date} AND {date:high_date}' . ($year_low == $year_high ? '' : '
					OR DATE_FORMAT(birthdate, {string:year_high}) BETWEEN {date:low_date} AND {date:high_date}') . '
				)
				AND is_activated = {int:is_activated}',
			array(
				'is_activated' => 1,
				'no_month' => 0,
				'no_day' => 0,
				'year_one' => '0001',
				'year_low' => $year_low . '-%m-%d',
				'year_high' => $year_high . '-%m-%d',
				'low_date' => $low_date,
				'high_date' => $high_date,
				'max_year' => $year_high,
			)
		);
	}
	else
	{
		$result = $smcFunc['db_query']('birthday_array', '
			SELECT id_member, real_name, YEAR(birthdate) AS birth_year, birthdate
			FROM {db_prefix}members
			WHERE YEAR(birthdate) != {string:year_one}
				AND MONTH(birthdate) != {int:no_month}
				AND DAYOFMONTH(birthdate) != {int:no_day}
				AND (
					indexable_month_day(birthdate) BETWEEN indexable_month_day({date:year_low_low_date}) AND indexable_month_day({date:year_low_high_date})' . ($year_low == $year_high ? '' : '
					OR  indexable_month_day(birthdate) BETWEEN indexable_month_day({date:year_high_low_date}) AND indexable_month_day({date:year_high_high_date})') . '
				)
				AND is_activated = {int:is_activated}',
			array(
				'is_activated' => 1,
				'no_month' => 0,
				'no_day' => 0,
				'year_one' => '0001',
				'year_low' => $year_low . '-%m-%d',
				'year_high' => $year_high . '-%m-%d',
				'year_low_low_date' => $low_date,
				'year_low_high_date' => ($year_low == $year_high ? $high_date : $year_low .'-12-31'),
				'year_high_low_date' => ($year_low == $year_high ? $low_date : $year_high .'-01-01'),
				'year_high_high_date' => $high_date,
			)
		);
	}
	$bday = array();
	while ($row = $smcFunc['db_fetch_assoc']($result))
	{
		if ($year_low != $year_high)
			$age_year = substr($row['birthdate'], 5) < substr($high_date, 5) ? $year_high : $year_low;
		else
			$age_year = $year_low;

		$bday[$age_year . substr($row['birthdate'], 4)][] = array(
			'id' => $row['id_member'],
			'name' => $row['real_name'],
			'age' => $row['birth_year'] > 4 && $row['birth_year'] <= $age_year ? $age_year - $row['birth_year'] : null,
			'is_last' => false
		);
	}
	$smcFunc['db_free_result']($result);

	ksort($bday);

	// Set is_last, so the themes know when to stop placing separators.
	foreach ($bday as $mday => $array)

This code seems to be duplicated across your project.

		$bday[$mday][count($array) - 1]['is_last'] = true;

	return $bday;
}

?>

It is not recommended to use PHP's closing tag ?> in files other than templates.