NNTmux / newznab-tmux

nntmux/db/DB.php

Doc Comments +1 added lines, -1 removed lines

@@ -1150,7 +1150,7 @@
 	 *
 	 * @param int $attribute
 	 *
-	 * @return bool|mixed
+	 * @return string
 	 */
 	public function getAttribute($attribute)
 	{

Spacing +16 added lines, -16 removed lines

@@ -180,7 +180,7 @@ 
 		$this->pdo = null;
 	}
 
-	public function checkDbExists ($name = null)
+	public function checkDbExists($name = null)
 	{
 		if (empty($name)) {
 			$name = $this->opts['dbname'];
@@ -253,7 +253,7 @@ 
 		}
 	}
 
-	public function getTableList ()
+	public function getTableList()
 	{
 		$query  = ($this->opts['dbtype'] === 'mysql' ? 'SHOW DATABASES' : 'SELECT datname AS Database FROM pg_database');
 		$result = $this->pdo->query($query);
@@ -412,12 +412,12 @@ 
 	 *
 	 * @return string
 	 */
-	public function likeString($str, $left=true, $right=true)
+	public function likeString($str, $left = true, $right = true)
 	{
 		return (
 			'LIKE ' .
 			$this->escapeString(
-				($left  ? '%' : '') .
+				($left ? '%' : '') .
 				$str .
 				($right ? '%' : '')
 			)
@@ -453,13 +453,13 @@ 
 
 		$i = 2;
 		$error = '';
-		while($i < 11) {
+		while ($i < 11) {
 			$result = $this->queryExecHelper($query, true);
 			if (is_array($result) && isset($result['deadlock'])) {
 				$error = $result['message'];
 				if ($result['deadlock'] === true) {
-					$this->echoError("A Deadlock or lock wait timeout has occurred, sleeping.(" . ($i-1) . ")", 'queryInsert', 4);
-					$this->ct->showsleep($i * ($i/2));
+					$this->echoError("A Deadlock or lock wait timeout has occurred, sleeping.(" . ($i - 1) . ")", 'queryInsert', 4);
+					$this->ct->showsleep($i * ($i / 2));
 					$i++;
 				} else {
 					break;
@@ -498,13 +498,13 @@ 
 
 		$i = 2;
 		$error = '';
-		while($i < 11) {
+		while ($i < 11) {
 			$result = $this->queryExecHelper($query);
 			if (is_array($result) && isset($result['deadlock'])) {
 				$error = $result['message'];
 				if ($result['deadlock'] === true) {
-					$this->echoError("A Deadlock or lock wait timeout has occurred, sleeping.(" . ($i-1) . ")", 'queryExec', 4);
-					$this->ct->showsleep($i * ($i/2));
+					$this->echoError("A Deadlock or lock wait timeout has occurred, sleeping.(" . ($i - 1) . ")", 'queryExec', 4);
+					$this->ct->showsleep($i * ($i / 2));
 					$i++;
 				} else {
 					break;
@@ -534,7 +534,7 @@ 
 	protected function queryExecHelper($query, $insert = false)
 	{
 		try {
-			if ($insert === false ) {
+			if ($insert === false) {
 				$run = $this->pdo->prepare($query);
 				$run->execute();
 				return $run;
@@ -687,7 +687,7 @@ 
 		// Remove LIMIT and OFFSET from query to allow queryCalc usage with browse
 		$query = preg_replace('#(\s+LIMIT\s+\d+)?\s+OFFSET\s+\d+\s*$#i', '', $query);
 
-		if ($cache === true && $this->cacheEnabled === true ) {
+		if ($cache === true && $this->cacheEnabled === true) {
 			try {
 				$count = $this->cacheServer->get($this->cacheServer->createKey($query . 'count'));
 				if ($count !== false) {
@@ -1109,7 +1109,7 @@ 
 	public function ping($restart = false)
 	{
 		try {
-			return (bool) $this->pdo->query('SELECT 1+1');
+			return (bool)$this->pdo->query('SELECT 1+1');
 		} catch (\PDOException $e) {
 			if ($restart == true) {
 				$this->initialiseDatabase();
@@ -1175,7 +1175,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function getDbVersion ()
+	public function getDbVersion()
 	{
 		return $this->dbVersion;
 	}
@@ -1186,7 +1186,7 @@ 
 	 * @return bool|null       TRUE if Db version is greater than or eaqual to $requiredVersion,
 	 * false if not, and null if the version isn't available to check against.
 	 */
-	public function isDbVersionAtLeast ($requiredVersion)
+	public function isDbVersionAtLeast($requiredVersion)
 	{
 		if (empty($this->dbVersion)) {
 			return null;
@@ -1197,7 +1197,7 @@ 
 	/**
 	 * Performs the fetch from the Db server and stores the resulting Major.Minor.Version number.
 	 */
-	private function fetchDbVersion ()
+	private function fetchDbVersion()
 	{
 		$result = $this->queryOneRow("SELECT VERSION() AS version");
 		if (!empty($result)) {

Braces +2 added lines, -1 removed lines

@@ -880,7 +880,8 @@
 		if (preg_match('#\s+LIMIT\s+(?P<lower>\d+)(,\s+(?P<upper>\d+))?(;)?$#i', $query, $matches)) {
 			If (!isset($matches['upper']) && isset($matches['lower']) && $matches['lower'] == 1) {
 				// good it's already correctly set.
-			} else { // We have a limit, but it's not for a single row
+			} else {
+// We have a limit, but it's not for a single row
 				return false;
 			}
 

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -338,7 +338,7 @@ 
 					}
 				}
 			}
-			$this->pdo->query("USE {$this->opts['dbname']}");
+			$this->pdo->query("use {$this->opts['dbname']}");
 		}
 
 		// In case PDO is not set to produce exceptions (PHP's default behaviour).
@@ -878,7 +878,7 @@ 
 		// Force the query to only return 1 row, so queryArray doesn't potentially run out of memory on a large data set.
 		// First check if query already contains a LIMIT clause.
 		if (preg_match('#\s+LIMIT\s+(?P<lower>\d+)(,\s+(?P<upper>\d+))?(;)?$#i', $query, $matches)) {
-			If (!isset($matches['upper']) && isset($matches['lower']) && $matches['lower'] == 1) {
+			if (!isset($matches['upper']) && isset($matches['lower']) && $matches['lower'] == 1) {
 				// good it's already correctly set.
 			} else { // We have a limit, but it's not for a single row
 				return false;

nntmux/DnzbFailures.php

Doc Comments +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
 	 *
 	 * @param $relid
 	 * @param $gid
-	 * @param $uid
+	 * @param string $uid
 	 */
 	public function postComment($relid, $gid, $uid)
 	{

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -175,7 +175,7 @@
 		);
 
 		if ($check instanceof \Traversable) {
-			foreach ($check AS $dbl) {
+			foreach ($check as $dbl) {
 				if ($dbl['text'] == $text) {
 					$dupe = 1;
 					break;

nntmux/Groups.php

Doc Comments +2 added lines, -2 removed lines

@@ -144,7 +144,7 @@ 
 	/**
 	 * Get all group columns by Name
 	 *
-	 * @param $grp
+	 * @param string $grp
 	 *
 	 * @return array|bool
 	 */
@@ -314,7 +314,7 @@ 
 	 *
 	 * @param string $groupName The full name of the usenet group being evaluated
 	 *
-	 * @return string|bool The name of the group replacing shorthand prefix or false if groupname was malformed
+	 * @return string|false The name of the group replacing shorthand prefix or false if groupname was malformed
 	 */
 	public function isValidGroup($groupName)
 	{

Upper-Lower-Casing +6 added lines, -6 removed lines

@@ -423,7 +423,7 @@ 
 			WHERE mp.group_id = {$id}"
 		);
 
-		foreach ($this->cbpm AS $tablePrefix) {
+		foreach ($this->cbpm as $tablePrefix) {
 			$this->pdo->queryExec(
 				"DROP TABLE IF EXISTS {$tablePrefix}_{$id}"
 			);
@@ -445,15 +445,15 @@ 
 	 */
 	public function resetall()
 	{
-		foreach ($this->cbpm AS $tablePrefix) {
+		foreach ($this->cbpm as $tablePrefix) {
 			$this->pdo->queryExec("TRUNCATE TABLE {$tablePrefix}");
 		}
 
 		$groups = $this->pdo->queryDirect("SELECT id FROM groups");
 
 		if ($groups instanceof \Traversable) {
-			foreach ($groups AS $group) {
-				foreach ($this->cbpm AS $tablePrefix) {
+			foreach ($groups as $group) {
+				foreach ($this->cbpm as $tablePrefix) {
 					$this->pdo->queryExec("DROP TABLE IF EXISTS {$tablePrefix}_{$group['id']}");
 				}
 			}
@@ -492,7 +492,7 @@ 
 			$releases     = new Releases(['Settings' => $this->pdo, 'Groups' => $this]);
 			$nzb          = new NZB($this->pdo);
 			$releaseImage = new ReleaseImage($this->pdo);
-			foreach ($res AS $row) {
+			foreach ($res as $row) {
 				$releases->deleteSingle(
 					[
 						'g' => $row['guid'],
@@ -576,7 +576,7 @@ 
 			WHERE id = {$id}"
 		);
 
-		return "Group {$id}: {$column} has been " . (($status == 0) ? 'deactivated' : 'activated') . '.';
+		return "group {$id}: {$column} has been " . (($status == 0) ? 'deactivated' : 'activated') . '.';
 	}
 
 	/**

nntmux/http/XML_Response.php

Doc Comments +2 added lines, -2 removed lines

@@ -112,7 +112,7 @@ 
 	}
 
 	/**
-	 * @return bool|void
+	 * @return string|false
 	 */
 	public function returnXML()
 	{
@@ -668,7 +668,7 @@ 
 	/**
 	 * Accepts an array of values to loop through to build cData from the release info
 	 *
-	 * @param array $columns The columns in the release we need to insert
+	 * @param string[] $columns The columns in the release we need to insert
 	 *
 	 * @return string The HTML format cData
 	 */

Spacing +13 added lines, -13 removed lines

@@ -206,7 +206,7 @@ 
 	protected function addNode($element)
 	{
 		$this->xml->startElement($element['name']);
-		foreach($element['data'] AS $attr => $val) {
+		foreach ($element['data'] AS $attr => $val) {
 			$this->xml->writeAttribute($attr, $val);
 		}
 		$this->xml->endElement();
@@ -219,7 +219,7 @@ 
 	protected function addNodes($element)
 	{
 		$this->xml->startElement($element['name']);
-		foreach($element['data'] AS $elem => $value) {
+		foreach ($element['data'] AS $elem => $value) {
 			$subelement['name'] = $elem;
 			$subelement['data'] = $value;
 			$this->addNode($subelement);
@@ -240,7 +240,7 @@ 
 			if ($p['description'] != '') {
 				$this->xml->writeAttribute('description', html_entity_decode($p['description']));
 			}
-			foreach($p['subcatlist'] AS $c) {
+			foreach ($p['subcatlist'] AS $c) {
 				$this->xml->startElement('subcat');
 				$this->xml->writeAttribute('id', $c['id']);
 				$this->xml->writeAttribute('name', html_entity_decode($c['title']));
@@ -259,7 +259,7 @@ 
 	 */
 	protected function includeRssAtom()
 	{
-		switch($this->namespace) {
+		switch ($this->namespace) {
 			case 'newznab':
 				$url = 'http://www.newznab.com/DTD/2010/feeds/attributes/';
 				break;
@@ -300,7 +300,7 @@ 
 	{
 		$server = $this->server['server'];
 
-		switch($this->namespace) {
+		switch ($this->namespace) {
 			case 'newznab':
 				$path = 'apihelp/';
 				$tag = 'API';
@@ -340,7 +340,7 @@ 
 
 	public function includeTotalRows()
 	{
-		$this->xml->startElement($this->namespace.":response");
+		$this->xml->startElement($this->namespace . ":response");
 		$this->xml->writeAttribute('offset', $this->offset);
 		$this->xml->writeAttribute('total', isset($this->releases[0]['_totalrows']) ? $this->releases[0]['_totalrows'] : 0);
 		$this->xml->endElement();
@@ -351,7 +351,7 @@ 
 	 */
 	public function includeReleases()
 	{
-		if(is_array($this->releases) && !empty($this->releases)) {
+		if (is_array($this->releases) && !empty($this->releases)) {
 			foreach ($this->releases AS $this->release) {
 				$this->xml->startElement('item');
 				$this->includeReleaseMain();
@@ -385,7 +385,7 @@ 
 		} else {
 			$this->writeRssCdata();
 		}
-		if((isset($this->parameters['dl']) && $this->parameters['dl'] == 1) || !isset($this->parameters['dl'])) {
+		if ((isset($this->parameters['dl']) && $this->parameters['dl'] == 1) || !isset($this->parameters['dl'])) {
 			$this->xml->startElement('enclosure');
 			$this->xml->writeAttribute(
 				'url',
@@ -416,7 +416,7 @@ 
 		if ($this->parameters['extended'] == 1) {
 			$this->writeZedAttr('files', $this->release['totalpart']);
 			$this->writeZedAttr('poster', $this->release['fromname']);
-			if(($this->release['videos_id'] > 0 || $this->release['tv_episodes_id'] > 0) && $this->namespace === 'newznab') {
+			if (($this->release['videos_id'] > 0 || $this->release['tv_episodes_id'] > 0) && $this->namespace === 'newznab') {
 				$this->setTvAttr();
 			}
 
@@ -452,7 +452,7 @@ 
 	 */
 	protected function setTvAttr()
 	{
-		switch(true) {
+		switch (true) {
 			case !empty($this->release['title']):
 				$this->writeZedAttr('title', $this->release['title']);
 				break;
@@ -514,7 +514,7 @@ 
 		$p = $this->parameters;
 
 		$this->cdata = "\n\t<div>\n";
-		switch(1) {
+		switch (1) {
 			case !empty($r['cover']):
 				$dir = 'movies';
 				$column = 'imdbid';
@@ -618,7 +618,7 @@ 
 
 		$cData = $this->buildCdata($musicCol);
 
-		if ($r['mu_url'] !== '' ) {
+		if ($r['mu_url'] !== '') {
 			$cDataUrl = "<li>Amazon: <a href=\"{$r['mu_url']}\">{$r['mu_title']}</a></li>";
 		}
 
@@ -679,7 +679,7 @@ 
 		$cData = '';
 
 		foreach ($columns AS $info) {
-			if (!empty($r[$info]))  {
+			if (!empty($r[$info])) {
 				if ($info == 'mu_releasedate') {
 					$ucInfo = 'Released';
 					$rDate = date('Y-m-d', strtotime($r[$info]));

Braces +3 added lines, -2 removed lines

@@ -90,7 +90,8 @@ 
 	 *
 	 * @param array $options
 	 */
-	public function __construct($options = []) {
+	public function __construct($options = [])
+	{
 		$defaults = [
 			'Parameters' => null,
 			'Data'       => null,
@@ -679,7 +680,7 @@ 
 		$cData = '';
 
 		foreach ($columns AS $info) {
-			if (!empty($r[$info]))  {
+			if (!empty($r[$info])) {
 				if ($info == 'mu_releasedate') {
 					$ucInfo = 'Released';
 					$rDate = date('Y-m-d', strtotime($r[$info]));

Upper-Lower-Casing +7 added lines, -7 removed lines

@@ -206,7 +206,7 @@ 
 	protected function addNode($element)
 	{
 		$this->xml->startElement($element['name']);
-		foreach($element['data'] AS $attr => $val) {
+		foreach($element['data'] as $attr => $val) {
 			$this->xml->writeAttribute($attr, $val);
 		}
 		$this->xml->endElement();
@@ -219,7 +219,7 @@ 
 	protected function addNodes($element)
 	{
 		$this->xml->startElement($element['name']);
-		foreach($element['data'] AS $elem => $value) {
+		foreach($element['data'] as $elem => $value) {
 			$subelement['name'] = $elem;
 			$subelement['data'] = $value;
 			$this->addNode($subelement);
@@ -233,14 +233,14 @@ 
 	protected function writeCategoryListing()
 	{
 		$this->xml->startElement('categories');
-		foreach ($this->server['categories'] AS $p) {
+		foreach ($this->server['categories'] as $p) {
 			$this->xml->startElement('category');
 			$this->xml->writeAttribute('id', $p['id']);
 			$this->xml->writeAttribute('name', html_entity_decode($p['title']));
 			if ($p['description'] != '') {
 				$this->xml->writeAttribute('description', html_entity_decode($p['description']));
 			}
-			foreach($p['subcatlist'] AS $c) {
+			foreach($p['subcatlist'] as $c) {
 				$this->xml->startElement('subcat');
 				$this->xml->writeAttribute('id', $c['id']);
 				$this->xml->writeAttribute('name', html_entity_decode($c['title']));
@@ -352,7 +352,7 @@ 
 	public function includeReleases()
 	{
 		if(is_array($this->releases) && !empty($this->releases)) {
-			foreach ($this->releases AS $this->release) {
+			foreach ($this->releases as $this->release) {
 				$this->xml->startElement('item');
 				$this->includeReleaseMain();
 				$this->setZedAttributes();
@@ -632,7 +632,7 @@ 
 		if ($r['mu_tracks'] != '') {
 			$tracks = explode('|', $r['mu_tracks']);
 			if (count($tracks) > 0) {
-				foreach ($tracks AS $track) {
+				foreach ($tracks as $track) {
 					$track = trim($track);
 					$tData .= "<li>{$track}</li>";
 				}
@@ -678,7 +678,7 @@ 
 
 		$cData = '';
 
-		foreach ($columns AS $info) {
+		foreach ($columns as $info) {
 			if (!empty($r[$info]))  {
 				if ($info == 'mu_releasedate') {
 					$ucInfo = 'Released';

nntmux/IRCClient.php

Doc Comments +2 added lines, -1 removed lines

@@ -329,7 +329,7 @@ 
 	 * @param string $nickName The nick name - visible in the channel.
 	 * @param string $userName The user name - visible in the host name.
 	 * @param string $realName The real name - visible in the WhoIs.
-	 * @param null   $password The password  - some servers require a password.
+	 * @param string|false   $password The password  - some servers require a password.
 	 *
 	 * @return bool
 	 *
@@ -641,6 +641,7 @@ 
 	 * Write a single character to the socket.
 	 *
 	 * @param string (char) $character A single character.
+	 * @param string $character
 	 *
 	 * @return int|bool Number of bytes written or false.
 	 */

nntmux/libraries/Forking.php

Doc Comments +3 added lines, -3 removed lines

@@ -278,7 +278,7 @@ 
 	////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 
 	/**
-	 * @return int
+	 * @return string|null
 	 */
 	private function backfillMainMethod()
 	{
@@ -304,7 +304,7 @@ 
 	}
 
 	/**
-	 * @return int
+	 * @return string|null
 	 */
 	private function safeBackfillMainMethod()
 	{
@@ -422,7 +422,7 @@ 
 	}
 
 	/**
-	 * @return int
+	 * @return string|null
 	 */
 	private function safeBinariesMainMethod()
 	{

Spacing +1 added lines, -1 removed lines

@@ -504,7 +504,7 @@
 			$threads = 1;
 		}
 
-		$leftguids = ['0','1','2','3','4','5','6','7','8','9','a','b','c','d','e','f'];
+		$leftguids = ['0', '1', '2', '3', '4', '5', '6', '7', '8', '9', 'a', 'b', 'c', 'd', 'e', 'f'];
 
 		// Prevent PreDB FT from always running
 		if ($this->workTypeOptions[0] === 'predbft') {

nntmux/Logger.php

Doc Comments +2 added lines, -2 removed lines

@@ -303,7 +303,7 @@ 
 	/**
 	 * Get resource usage string.
 	 *
-	 * @return bool|string
+	 * @return string|false
 	 *
 	 * @access public
 	 */
@@ -324,7 +324,7 @@ 
 	/**
 	 * Get system load.
 	 *
-	 * @return string|bool
+	 * @return string|false
 	 *
 	 * @access public
 	 */

Spacing +6 added lines, -6 removed lines

@@ -266,7 +266,7 @@ 
 	 */
 	public function showMemUsage($oldUsage = 0, $realUsage = false, $peak = false)
 	{
-		$currentUsage = ($peak ? memory_get_peak_usage($realUsage)  : memory_get_usage($realUsage));
+		$currentUsage = ($peak ? memory_get_peak_usage($realUsage) : memory_get_usage($realUsage));
 		$actualUsage = ($oldUsage > 0 ? $currentUsage - $oldUsage : $currentUsage);
 
 		$units = [
@@ -313,7 +313,7 @@ 
 			$usage = getrusage();
 
 			return
-				'USR: '  . $this->formatTimeString($usage['ru_utime.tv_sec']) .
+				'USR: ' . $this->formatTimeString($usage['ru_utime.tv_sec']) .
 				' SYS: ' . $this->formatTimeString($usage['ru_stime.tv_sec']) .
 				' FAULTS: ' . $usage['ru_majflt'] .
 				' SWAPS: ' . $usage['ru_nswap'];
@@ -333,7 +333,7 @@ 
 		if (!$this->isWindows) {
 			$string = '';
 			// Fix for single digits (2) or single float (2.1).
-			foreach(sys_getloadavg() as $load) {
+			foreach (sys_getloadavg() as $load) {
 				$strLen = strlen($load);
 				if ($strLen === 1) {
 					$string .= $load . '.00,';
@@ -604,7 +604,7 @@ 
 		asort($logs);
 
 		// Remove all new logs from array (all elements under the last 51 elements of the array).
-		array_splice($logs, -$this->maxLogs+1);
+		array_splice($logs, -$this->maxLogs + 1);
 
 		// Delete all the logs left in the array.
 		array_map('unlink', $logs);
@@ -697,11 +697,11 @@ 
 			$time .= '00H:';
 		}
 		if ($seconds > 60) {
-			$time .= str_pad(round((($seconds % 3600) / 60)), 2 , '0', STR_PAD_LEFT) . 'M:';
+			$time .= str_pad(round((($seconds % 3600) / 60)), 2, '0', STR_PAD_LEFT) . 'M:';
 		} else {
 			$time .= '00M:';
 		}
-		$time .= str_pad($seconds % 60, 2 , '0', STR_PAD_LEFT) . 'S';
+		$time .= str_pad($seconds % 60, 2, '0', STR_PAD_LEFT) . 'S';
 		return $time;
 	}
 

nntmux/Movie.php

Doc Comments +5 added lines, -5 removed lines

@@ -384,7 +384,7 @@ 
 	/**
 	 * Order types for movies page.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function getMovieOrdering()
 	{
@@ -556,7 +556,7 @@ 
 
 	/**
 	 * Get array of column keys, for inserting / updating.
-	 * @return array
+	 * @return string[]
 	 */
 	public function getColumnKeys()
 	{
@@ -612,7 +612,7 @@ 
 	 *
 	 * @param $variable
 	 *
-	 * @return string
+	 * @return boolean
 	 */
 	protected function checkVariable(&$variable)
 	{
@@ -646,7 +646,7 @@ 
 	/**
 	 * Fetch IMDB/TMDB/TRAKT info for the movie.
 	 *
-	 * @param $imdbId
+	 * @param string $imdbId
 	 *
 	 * @return bool
 	 */
@@ -1453,7 +1453,7 @@ 
 	/**
 	 * Get IMDB genres.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function getGenres()
 	{

Spacing +17 added lines, -17 removed lines

@@ -31,7 +31,7 @@ 
 	 * Current year of parsed search name.
 	 * @var string
 	 */
-	protected $currentYear  = '';
+	protected $currentYear = '';
 
 	/**
 	 * Current release id of parsed search name.
@@ -514,7 +514,7 @@ 
 	{
 		if (is_array($value) && !empty($value)) {
 			$temp = '';
-			foreach($value as $val) {
+			foreach ($value as $val) {
 				if (!is_array($val) && !is_object($val)) {
 					$temp .= (string)$val;
 				}
@@ -561,8 +561,8 @@ 
 	public function getColumnKeys()
 	{
 		return [
-			'actors','backdrop','cover','director','genre','imdbid','language',
-			'plot','rating','tagline','title','tmdbid', 'trailer','type','year'
+			'actors', 'backdrop', 'cover', 'director', 'genre', 'imdbid', 'language',
+			'plot', 'rating', 'tagline', 'title', 'tmdbid', 'trailer', 'type', 'year'
 		];
 	}
 
@@ -701,12 +701,12 @@ 
 		if ($this->checkVariable($fanart['banner'])) {
 			$mov['banner'] = $this->releaseImage->saveImage($imdbId . '-banner', $fanart['banner'], $this->imgSavePath);
 		}
-		$mov['title']   = $this->setTmdbImdbTraktVar($imdb['title']  , $tmdb['title'], $trakt['title']);
-		$mov['rating']  = $this->setTmdbImdbTraktVar($imdb['rating'] , $tmdb['rating'], $trakt['rating']);
-		$mov['plot']    = $this->setTmdbImdbTraktVar($imdb['plot']   , $tmdb['plot'], $trakt['overview']);
+		$mov['title']   = $this->setTmdbImdbTraktVar($imdb['title'], $tmdb['title'], $trakt['title']);
+		$mov['rating']  = $this->setTmdbImdbTraktVar($imdb['rating'], $tmdb['rating'], $trakt['rating']);
+		$mov['plot']    = $this->setTmdbImdbTraktVar($imdb['plot'], $tmdb['plot'], $trakt['overview']);
 		$mov['tagline'] = $this->setTmdbImdbTraktVar($imdb['tagline'], $tmdb['tagline'], $trakt['tagline']);
-		$mov['year']    = $this->setTmdbImdbTraktVar($imdb['year']   , $tmdb['year'], $trakt['year']);
-		$mov['genre']   = $this->setTmdbImdbTraktVar($imdb['genre']  , $tmdb['genre'], $trakt['genres']);
+		$mov['year']    = $this->setTmdbImdbTraktVar($imdb['year'], $tmdb['year'], $trakt['year']);
+		$mov['genre']   = $this->setTmdbImdbTraktVar($imdb['genre'], $tmdb['genre'], $trakt['genres']);
 
 		if ($this->checkVariable($imdb['type'])) {
 			$mov['type'] = $imdb['type'];
@@ -732,20 +732,20 @@ 
 			$mov['type'] = implode(', ', array_unique($mov['type']));
 		}
 
-		$mov['title']    = html_entity_decode($mov['title']   , ENT_QUOTES, 'UTF-8');
+		$mov['title'] = html_entity_decode($mov['title'], ENT_QUOTES, 'UTF-8');
 
 		$mov['title'] = str_replace(['/', '\\'], '', $mov['title']);
 		$movieID = $this->update([
-			'actors'    => html_entity_decode($mov['actors']  , ENT_QUOTES, 'UTF-8'),
+			'actors'    => html_entity_decode($mov['actors'], ENT_QUOTES, 'UTF-8'),
 			'backdrop'  => $mov['backdrop'],
 			'cover'     => $mov['cover'],
 			'director'  => html_entity_decode($mov['director'], ENT_QUOTES, 'UTF-8'),
-			'genre'     => html_entity_decode($mov['genre']   , ENT_QUOTES, 'UTF-8'),
+			'genre'     => html_entity_decode($mov['genre'], ENT_QUOTES, 'UTF-8'),
 			'imdbid'    => $mov['imdbid'],
 			'language'  => html_entity_decode($mov['language'], ENT_QUOTES, 'UTF-8'),
 			'plot'      => html_entity_decode(preg_replace('/\s+See full summary »/', ' ', $mov['plot']), ENT_QUOTES, 'UTF-8'),
 			'rating'    => round($mov['rating'], 1),
-			'tagline'   => html_entity_decode($mov['tagline'] , ENT_QUOTES, 'UTF-8'),
+			'tagline'   => html_entity_decode($mov['tagline'], ENT_QUOTES, 'UTF-8'),
 			'title'     => $mov['title'],
 			'tmdbid'    => $mov['tmdbid'],
 			'type'      => html_entity_decode(ucwords(preg_replace('/[\.\_]/', ' ', $mov['type'])), ENT_QUOTES, 'UTF-8'),
@@ -778,7 +778,7 @@ 
 	{
 		if ($this->fanartapikey != '')
 		{
-			$buffer = Utility::getUrl(['url' => 'https://webservice.fanart.tv/v3/movies/' . 'tt' . $imdbId . '?api_key=' . $this->fanartapikey , 'verifycert' => false]);
+			$buffer = Utility::getUrl(['url' => 'https://webservice.fanart.tv/v3/movies/' . 'tt' . $imdbId . '?api_key=' . $this->fanartapikey, 'verifycert' => false]);
 			if ($buffer !== false) {
 				$art = json_decode($buffer, true);
 				if (isset($art['status']) && $art['status'] === 'error') {
@@ -1196,8 +1196,8 @@ 
 
 		//If we found a year, try looking in a 4 year range.
 		if ($this->currentYear !== false) {
-			$start = (int) $this->currentYear - 2;
-			$end   = (int) $this->currentYear + 2;
+			$start = (int)$this->currentYear - 2;
+			$end   = (int)$this->currentYear + 2;
 			$andYearIn = 'AND year IN (';
 			while ($start < $end) {
 				$andYearIn .= $start . ',';
@@ -1295,7 +1295,7 @@ 
 	 */
 	protected function googleSearch()
 	{
-		$buffer =Utility::getUrl([
+		$buffer = Utility::getUrl([
 				'url' =>
 					'https://www.google.com/search?hl=en&as_q=&as_epq=' .
 					urlencode(

Braces +4 added lines, -4 removed lines

@@ -467,7 +467,7 @@ 
 	 * @return mixed|void
 	 */
 		public function parseTraktTv(&$data)
-	{
+		{
 		if (!isset($data['ids']['imdb']) || empty($data['ids']['imdb'])) {
 			return false;
 		}
@@ -572,7 +572,8 @@ 
 	 * @param array $values Array of keys/values to update. See $validKeys
 	 * @return int|bool
 	 */
-	public function update(array $values) {
+	public function update(array $values)
+	{
 		if (!count($values)) {
 			return false;
 		}
@@ -776,8 +777,7 @@ 
 	 */
 	protected function fetchFanartTVProperties($imdbId)
 	{
-		if ($this->fanartapikey != '')
-		{
+		if ($this->fanartapikey != '') {
 			$buffer = Utility::getUrl(['url' => 'https://webservice.fanart.tv/v3/movies/' . 'tt' . $imdbId . '?api_key=' . $this->fanartapikey , 'verifycert' => false]);
 			if ($buffer !== false) {
 				$art = json_decode($buffer, true);

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -302,7 +302,7 @@
 		$movieIDs = $releaseIDs = false;
 
 		if (is_array($movies['result'])) {
-			foreach ($movies['result'] AS $movie => $id) {
+			foreach ($movies['result'] as $movie => $id) {
 				$movieIDs[] = $id['imdbid'];
 				$releaseIDs[] = $id['grp_release_id'];
 			}

nntmux/Nfo.php

Doc Comments +2 added lines, -2 removed lines

@@ -192,7 +192,7 @@ 
 	 *
 	 * @param string $nfo     The nfo.
 	 * @param array  $release The SQL row for this release.
-	 * @param object $nntp    Instance of class NNTP.
+	 * @param NNTP $nntp    Instance of class NNTP.
 	 *
 	 * @return bool           True on success, False on failure.
 	 *
@@ -275,7 +275,7 @@ 
 	/**
 	 * Attempt to find NFO files inside the NZB's of releases.
 	 *
-	 * @param object $nntp           Instance of class NNTP.
+	 * @param NNTP $nntp           Instance of class NNTP.
 	 * @param string $groupID        (optional) Group ID.
 	 * @param string $guidChar       (optional) First character of the release GUID (used for multi-processing).
 	 * @param int    $processImdb    (optional) Attempt to find IMDB id's in the NZB?

Spacing +2 added lines, -2 removed lines

@@ -64,8 +64,8 @@
 
 	const NFO_FAILED = -9; // We failed to get a NFO after admin set max retries.
 	const NFO_UNPROC = -1; // Release has not been processed yet.
-	const NFO_NONFO  =  0; // Release has no NFO.
-	const NFO_FOUND  =  1; // Release has an NFO.
+	const NFO_NONFO  = 0; // Release has no NFO.
+	const NFO_FOUND  = 1; // Release has an NFO.
 
 	/**
 	 * Default constructor.

Braces +4 added lines, -5 removed lines

@@ -130,7 +130,8 @@ 
 	 *
 	 * @access public
 	 */
-	public function isNFO(&$possibleNFO, $guid) {
+	public function isNFO(&$possibleNFO, $guid)
+	{
 		if ($possibleNFO === false) {
 			return false;
 		}
@@ -141,8 +142,7 @@ 
 			$size > 11 &&
 			!preg_match(
 				'/\A(\s*<\?xml|=newz\[NZB\]=|RIFF|\s*[RP]AR|.{0,10}(JFIF|matroska|ftyp|ID3))|;\s*Generated\s*by.*SF\w/i'
-				, $possibleNFO))
-		{
+				, $possibleNFO)) {
 			// File/GetId3 work with files, so save to disk.
 			$tmpPath = $this->tmpPath . $guid . '.nfo';
 			file_put_contents($tmpPath, $possibleNFO);
@@ -158,8 +158,7 @@ 
 
 					// Or binary.
 				} else if (preg_match('/^(JPE?G|Parity|PNG|RAR|XML|(7-)?[Zz]ip)/', $result) ||
-					preg_match('/[\x00-\x08\x12-\x1F\x0B\x0E\x0F]/', $possibleNFO))
-				{
+					preg_match('/[\x00-\x08\x12-\x1F\x0B\x0E\x0F]/', $possibleNFO)) {
 					@unlink($tmpPath);
 					return false;
 				}