StatsTopCreators::execute()

Complexity

Conditions	1
Paths	1

Size

Total Lines	134
Code Lines	101

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	2
Bugs	0	Features	0

<?php
	protected function execute()
	{
		global $smarty;

		$qb = new QueryBrowser();
		$qb->numberedList = true;
		$qb->numberedListTitle = "Position";

		$qb->tableCallbackFunction = "statsTopCreatorsRowCallback";

It seems like 'statsTopCreatorsRowCallback' of type string is incompatible with the declared type boolean|callable of property $tableCallbackFunction.

		$qb->overrideTableTitles = array("# Created", "Username");

		// Retrieve all-time stats
		$top5aout = $qb->executeQueryToTable(<<<SQL
SELECT
	/* StatsTopCreators::execute()/top5aout */
    COUNT(*),
    log.user user_id,
    user.username log_user,
    user.status user_level
FROM log
LEFT JOIN emailtemplate ON concat('Closed ', emailtemplate.id) = log.action
INNER JOIN user ON user.id = log.user
WHERE emailtemplate.oncreated = '1'
   OR log.action = 'Closed custom-y'

GROUP BY log.user, user.username, user.status
ORDER BY COUNT(*) DESC;
SQL
		);

		// Retrieve all-time stats for active users only
		$top5activeout = $qb->executeQueryToTable(<<<SQL
SELECT
	/* StatsTopCreators::execute()/top5activeout */
    COUNT(*),
    log.user user_id,
    user.username log_user,
    user.status user_level
FROM log
LEFT JOIN emailtemplate ON concat('Closed ', emailtemplate.id) = log.action
INNER JOIN user ON user.id = log.user
WHERE
	(emailtemplate.oncreated = 1 OR log.action = 'Closed custom-y')
    AND user.status != 'Suspended'
GROUP BY user.username, user.id
ORDER BY COUNT(*) DESC;
SQL
		);

		// Retrieve today's stats (so far)
		$now = date("Y-m-d", mktime(0, 0, 0, date("m"), date("d")));

date('m') of type string is incompatible with the type integer expected by parameter $month of mktime().

date('d') of type string is incompatible with the type integer expected by parameter $day of mktime().

		$top5out = $qb->executeQueryToTable(<<<SQL
SELECT
	/* StatsTopCreators::execute()/top5out */
    COUNT(*),
    log.user user_id,
    user.username log_user,
    user.status user_level
FROM log
INNER JOIN user ON user.id = log.user
LEFT JOIN emailtemplate ON CONCAT('Closed ', emailtemplate.id) = log.action
WHERE (emailtemplate.oncreated = '1' OR log.action = 'Closed custom-y')
  AND log.timestamp LIKE '{$now}%'
GROUP BY log.user, user.username
ORDER BY COUNT(*) DESC;
SQL
		);

		// Retrieve Yesterday's stats
		$yesterday = date("Y-m-d", mktime(0, 0, 0, date("m"), date("d") - 1));
		$top5yout = $qb->executeQueryToTable(<<<SQL
SELECT
	/* StatsTopCreators::execute()/top5yout */
    COUNT(*),
    log.user user_id,
    user.username log_user,
    user.status user_level
FROM log
INNER JOIN user ON user.id = log.user
LEFT JOIN emailtemplate ON CONCAT('Closed ', emailtemplate.id) = log.action
WHERE (emailtemplate.oncreated = '1' OR log.action = 'Closed custom-y')
  AND log.timestamp LIKE '{$yesterday}%'
GROUP BY log.user, user.username
ORDER BY COUNT(*) DESC;
SQL
		);

		// Retrieve last 7 days
		$lastweek = date("Y-m-d", mktime(0, 0, 0, date("m"), date("d") - 7));
		$top5wout = $qb->executeQueryToTable(<<<SQL
SELECT
	/* StatsTopCreators::execute()/top5wout */
    COUNT(*),
    log.user user_id,
    user.username log_user,
    user.status user_level
FROM log
INNER JOIN user ON user.id = log.user
LEFT JOIN emailtemplate ON CONCAT('Closed ', emailtemplate.id) = log.action
WHERE (emailtemplate.oncreated = '1' OR log.action = 'Closed custom-y')
  AND log.timestamp > '{$lastweek}%'
GROUP BY log.user, user.username
ORDER BY COUNT(*) DESC;
SQL
		);

		// Retrieve last month's stats
		$lastmonth = date("Y-m-d", mktime(0, 0, 0, date("m"), date("d") - 28));
		$top5mout = $qb->executeQueryToTable(<<<SQL
SELECT
	/* StatsTopCreators::execute()/top5mout */
    COUNT(*),
    log.user user_id,
    user.username log_user,
    user.status user_level
FROM log
INNER JOIN user ON user.id = log.user
LEFT JOIN emailtemplate ON CONCAT('Closed ', emailtemplate.id) = log.action
WHERE (emailtemplate.oncreated = '1' OR log.action = 'Closed custom-y')
  AND log.timestamp > '{$lastmonth}%'
GROUP BY log.user, user.username
ORDER BY COUNT(*) DESC;
SQL
		);

		// Put it all together
		$smarty->assign("top5aout", $top5aout);
		$smarty->assign("top5activeout", $top5activeout);
		$smarty->assign("top5out", $top5out);
		$smarty->assign("top5yout", $top5yout);
		$smarty->assign("top5wout", $top5wout);
		$smarty->assign("top5mout", $top5mout);

		return $smarty->fetch("statistics/topcreators.tpl");
	}