more style fixes

articleenhancer/xpatharticleenhancer.php

@@ -25,6 +25,7 @@ class XPathArticleEnhancer implements ArticleEnhancer {
 	private $fileFactory;
 	private $maximumTimeout;
 	private $config;
+	private $regexXPathPair;
 
 
 	/**

db/mapperfactory.php

@@ -21,6 +21,7 @@ use \OCA\News\Core\Db;
 class MapperFactory {
 
 	private $dbType;
+	private $db;
 
 	public function __construct($dbType, Db $db) {
 		$this->dbType = $dbType;

tests/unit/businesslayer/BusinessLayerTest.php

@@ -54,7 +54,7 @@ class BusinessLayerTest extends \PHPUnit_Framework_TestCase {
 			->with($this->equalTo($id), $this->equalTo($user))
 			->will($this->returnValue($folder));
 
-		$result = $this->newsBusinessLayer->delete($id, $user);
+		$this->newsBusinessLayer->delete($id, $user);
 	}
 
 

tests/unit/businesslayer/ItemBusinessLayerTest.php

@@ -296,8 +296,7 @@ class ItemBusinessLayerTest extends \PHPUnit_Framework_TestCase {
 			->method('deleteReadOlderThanThreshold')
 			->with($this->equalTo($this->threshold));
 
-		$result = $this->itemBusinessLayer->autoPurgeOld();
-
+		$this->itemBusinessLayer->autoPurgeOld();
 	}
 
 

tests/unit/db/ItemMapperTest.php

@@ -359,7 +359,7 @@ class ItemMapperTest extends \OCA\News\Utility\MapperTestUtility {
 		$this->setMapperResult($sql1, $params1, array($row));
 		$this->setMapperResult($sql2, $params2);
 
-		$result = $this->mapper->deleteReadOlderThanThreshold($threshold);
+		$this->mapper->deleteReadOlderThanThreshold($threshold);
 	}
 
 

tests/unit/db/postgres/ItemMapperTest.php

@@ -148,7 +148,7 @@ class ItemMapperTest extends \OCA\News\Utility\MapperTestUtility {
 		$this->setMapperResult($sql1, $params1, array($row));
 		$this->setMapperResult($sql2, $params2);
 
-		$result = $this->mapper->deleteReadOlderThanThreshold($threshold);
+		$this->mapper->deleteReadOlderThanThreshold($threshold);
 	}
 
 

utility/controllertestutility.php

@@ -58,7 +58,7 @@ abstract class ControllerTestUtility extends \PHPUnit_Framework_TestCase {
 	 * @param Response $response the response which we want to test for headers
 	 */
 	protected function assertHeaders(array $expected=array(), Response $response){
-		$headers = $reponse->getHeaders();
+		$headers = $response->getHeaders();
 		foreach($expected as $header){
 			$this->assertTrue(in_array($header, $headers));
 		}