QueryManager

Complexity

Total Complexity

6

Size/Duplication

Total Lines	94
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

41 Methods

Rating	Name	Duplication	Size	Complexity
B	current()	0	27	5
A	__construct()	0	1	1
	getEcapedString()	0	1	?
	testTotalPokemon()	0	1	?
	testTotalGyms()	0	1	?
	testTotalPokestops()	0	1	?
	getTotalPokemon()	0	1	?
	getTotalLures()	0	1	?
	getTotalGyms()	0	1	?
	getTotalRaids()	0	1	?
	getTotalGymsForTeam()	0	1	?
	getRecentAll()	0	1	?
	getRecentMythic()	0	1	?
	getGymsProtectedByPokemon()	0	1	?
	getPokemonLastSeen()	0	1	?
	getTop50Pokemon()	0	1	?
	getTop50Trainers()	0	1	?
	getPokemonHeatmap()	0	1	?
	getPokemonGraph()	0	1	?
	getPokemonLive()	0	1	?
	getMapsCoords()	0	1	?
	getPokemonSliderMinMax()	0	1	?
	getCaptchaCount()	0	1	?
	getNestData()	0	1	?
	getPokemonCount()	0	1	?
	getRaidCount()	0	1	?
	getTotalPokestops()	0	1	?
	getAllPokestops()	0	1	?
	getTeamGuardians()	0	1	?
	getOwnedAndPoints()	0	1	?
	getAllGyms()	0	1	?
	getGymData()	0	1	?
	getGymDefenders()	0	1	?
	getGymHistories()	0	1	?
	getGymHistoriesPokemon()	0	1	?
	getHistoryForGym()	0	1	?
	getAllRaids()	0	1	?
	getTrainers()	0	1	?
	getTrainerLevelCount()	0	1	?
	getPokemonCountsActive()	0	1	?
	getPokemonCountsLastDay()	0	1	?

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 7 and the first side effect is on line 3.

include_once __DIR__ . '/../../../config.php';
include_once __DIR__ . '/QueryManagerMysql.php';
include_once __DIR__ . '/QueryManagerPostgresql.php';

abstract class QueryManager {

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

	protected static $time_offset;
	protected static $config;

	private static $current;
	public static function current() {
		global $time_offset;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

		if (self::$current == null) {

Blank line found at start of control structure

			$variables = realpath(dirname(__FILE__)) . '/../../json/variables.json';
			self::$config = json_decode(file_get_contents($variables));

			include_once(SYS_PATH.'/core/process/timezone.loader.php');
			self::$time_offset = $time_offset;


			switch (strtolower(SYS_DB_TYPE)) {
				case "monocle-alt":
				case "monocle-alt-mysql":
					self::$current = new QueryManagerMysqlMonocleAlternate();
					break;
				case "monocle-alt-pgsql":
					self::$current = new QueryManagerPostgresqlMonocleAlternate();
					break;
				default: //rocketmap
					self::$current = new QueryManagerMysqlRocketmap();
					break;
			}
		}
		return self::$current;
	}
	private function __construct(){}

Something seems to be off here. Are you sure you want to declare the constructor as private, and the class as abstract?

	// Misc
	abstract public function getEcapedString($string);

	// Tester
	abstract public function testTotalPokemon();
	abstract public function testTotalGyms();
	abstract public function testTotalPokestops();

	// Homepage
	abstract public function getTotalPokemon();
	abstract public function getTotalLures();
	abstract public function getTotalGyms();
	abstract public function getTotalRaids();
	abstract public function getTotalGymsForTeam($team_id);
	abstract public function getRecentAll();
	abstract public function getRecentMythic($mythic_pokemon);

	// Single Pokemon
	abstract public function getGymsProtectedByPokemon($pokemon_id);
	abstract public function getPokemonLastSeen($pokemon_id);
	abstract public function getTop50Pokemon($pokemon_id, $top_order_by, $top_direction);
	abstract public function getTop50Trainers($pokemon_id, $best_order_by, $best_direction);
	abstract public function getPokemonHeatmap($pokemon_id, $start, $end);
	abstract public function getPokemonGraph($pokemon_id);
	abstract public function getPokemonLive($pokemon_id, $ivMin, $ivMax, $inmap_pokemons);
	abstract public function getPokemonSliderMinMax();
	abstract public function getMapsCoords();
	abstract public function getPokemonCount($pokemon_id);
	abstract public function getRaidCount($pokemon_id);

	// Pokestops
	abstract public function getTotalPokestops();
	abstract public function getAllPokestops();

	// Gyms
	abstract public function getTeamGuardians($team_id);
	abstract public function getOwnedAndPoints($team_id);
	abstract public function getAllGyms();
	abstract public function getGymData($gym_id);
	abstract public function getGymDefenders($gym_id);

	// Gym History
	abstract public function getGymHistories($gym_name, $team, $page, $ranking);
	abstract public function getGymHistoriesPokemon($gym_id);
	abstract public function getHistoryForGym($page, $gym_id);

	// Raids
	abstract public function getAllRaids($page);

	// Trainers
	abstract public function getTrainers($trainer_name, $team, $page, $ranking);
	abstract public function getTrainerLevelCount($team_id);

	// Cron
	abstract public function getPokemonCountsActive();
	abstract public function getPokemonCountsLastDay();
	abstract public function getCaptchaCount();
	abstract public function getNestData();
}