- Code cleaning

- Coding standards fixes

.scrutinizer.yml

@@ -5,6 +5,8 @@ build:
     environment:
         php:
             version: 7.0.7
+        variables:
+            NETTE: default
 
     dependencies:
         before:

.travis.yml

@@ -7,9 +7,11 @@ env:
 php:
   - 7.0
 
+before_install:
+  - composer self-update
+
 before_script:
   - php .travis.composer.php
-  - composer self-update
   - composer install --no-interaction --prefer-source --dev
 
 script:

composer.json

@@ -22,15 +22,17 @@
 	"require": {
 		"php"	: ">=7.0.0",
 
-		"nette/application"	: "~2.2"
+		"nette/application"	: "~2.3",
+
+		"pds/skeleton"	: "@dev"
 	},
 
 	"require-dev": {
-		"nette/bootstrap"		: "~2.3@dev",
-		"nette/di"				: "~2.3@dev",
-		"nette/mail"			: "~2.3@dev",
-		"nette/safe-stream"		: "~2.3@dev",
-		"nette/robot-loader"	: "~2.3@dev",
+		"nette/bootstrap"		: "~2.3",
+		"nette/di"				: "~2.3",
+		"nette/mail"			: "~2.3",
+		"nette/safe-stream"		: "~2.3",
+		"nette/robot-loader"	: "~2.3",
 		"nette/tester"			: "@dev",
 
 		"tracy/tracy"	: "@dev"

src/IPub/Application/UI/TRedirect.php

@@ -52,12 +52,11 @@ final public function go($destination, $args = [], $snippets = [])
 		$presenter = ($this instanceof Application\UI\Presenter) ? $this : $this->getPresenter();
 
 		if ($presenter->isAjax()) {
-			if ($destination === 'this') {
-				foreach ($snippets as $snippet) {
-					$this->redrawControl($snippet);
-				}
+			foreach ($snippets as $snippet) {
+				$this->redrawControl($snippet);
+			}
 
-			} else {
+			if ($destination !== 'this') {
 				$this->forward($destination, $args);
 			}
 

tests/IPubTests/Application/RedirectTest.phpt

@@ -64,7 +64,7 @@ class RedirectTest extends Tester\TestCase
 	}
 
 	/**
-	 * Set up
+	 * {@inheritdoc}
 	 */
 	public function setUp()
 	{
@@ -184,7 +184,7 @@ class TestPresenter extends UI\Presenter
 	/**
 	 * {@inheritdoc}
 	 */
-	public function isAjax()
+	public function isAjax() : bool
 	{
 		if (in_array($this->action, ['ajaxRedirect', 'forward'], TRUE)) {
 			return TRUE;

tests/IPubTests/Application/TranslatorTest.phpt

@@ -65,7 +65,7 @@ class TranslatorTest extends Tester\TestCase
 	}
 
 	/**
-	 * Set up
+	 * {@inheritdoc}
 	 */
 	public function setUp()
 	{

tests/IPubTests/bootstrap.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types = 1);
+
 define('DS', DIRECTORY_SEPARATOR);
 
 require __DIR__ . DS . '..' . DS . '..' . DS . 'vendor' . DS . 'autoload.php';
@@ -13,7 +15,7 @@
 date_default_timezone_set('Europe/Prague');
 
 // Create temporary directory
-define('TEMP_DIR', __DIR__ . DS . '..' . DS . 'tmp' . DS . (isset($_SERVER['argv']) ? md5(serialize($_SERVER['argv'])) : getmypid()));
+define('TEMP_DIR', __DIR__ . '/../tmp/' . (isset($_SERVER['argv']) ? md5(serialize($_SERVER['argv'])) : getmypid()));
 Tester\Helpers::purge(TEMP_DIR);
 \Tracy\Debugger::$logDirectory = TEMP_DIR;