From be8c1101f2df880471cd5fe3065e6ded5781cadc Mon Sep 17 00:00:00 2001
From: Thomas Gelf <thomas@gelf.net>
Date: Thu, 28 Nov 2019 06:52:09 +0100
Subject: [PATCH] travis: use phpcs 3.5.2

---
 .travis.yml                                            |  1 -
 application/controllers/ImportsourcesController.php    |  2 +-
 application/controllers/SelfServiceController.php      |  6 +++---
 application/forms/IcingaDependencyForm.php             |  2 +-
 application/forms/SyncPropertyForm.php                 |  4 ++--
 application/views/helpers/FormStoredPassword.php       |  2 +-
 library/Director/CheckPlugin/Check.php                 |  2 +-
 library/Director/Core/RestApiClient.php                |  2 +-
 library/Director/CustomVariable/CustomVariable.php     |  2 +-
 .../Director/CustomVariable/CustomVariableArray.php    |  2 +-
 .../CustomVariable/CustomVariableDictionary.php        |  2 +-
 .../Director/CustomVariable/CustomVariableString.php   |  2 +-
 library/Director/Daemon/ProcessList.php                |  2 +-
 .../DirectorObject/Automation/BasketSnapshot.php       |  4 ++--
 .../DirectorObject/Automation/ImportExport.php         |  4 ++--
 library/Director/Import/Import.php                     |  4 ++--
 library/Director/Objects/GroupMembershipResolver.php   |  4 ++--
 library/Director/Objects/IcingaTemplateResolver.php    |  2 +-
 library/Director/Objects/ImportSource.php              |  2 +-
 library/Director/Resolver/TemplateTree.php             | 10 +++++-----
 library/Director/Web/Form/DirectorObjectForm.php       | 10 +++++-----
 .../Web/Form/IplElement/ExtensibleSetElement.php       |  2 +-
 test/setup_vendor.sh                                   |  6 +++++-
 23 files changed, 41 insertions(+), 38 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index 604220fd..0f4aaa21 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -26,7 +26,6 @@ matrix:
       php: '5.6'
   allow_failures:
     - php: nightly
-    - php: '7.3'
 
 env:
   - CHECK=phpunit DB=mysql DIRECTOR_TESTDB_RES="Director MySQL TestDB" DIRECTOR_TESTDB="director_test"
diff --git a/application/controllers/ImportsourcesController.php b/application/controllers/ImportsourcesController.php
index ba6e3cc9..5d473778 100644
--- a/application/controllers/ImportsourcesController.php
+++ b/application/controllers/ImportsourcesController.php
@@ -57,7 +57,7 @@ class ImportsourcesController extends ActionController
      * @param $raw
      * @throws \Icinga\Exception\ConfigurationError
      */
-    protected function acceptImport(& $raw)
+    protected function acceptImport(&$raw)
     {
         (new ImportExport($this->db()))->unserializeImportSources(json_decode($raw));
     }
diff --git a/application/controllers/SelfServiceController.php b/application/controllers/SelfServiceController.php
index bce9785b..5d4393b5 100644
--- a/application/controllers/SelfServiceController.php
+++ b/application/controllers/SelfServiceController.php
@@ -336,7 +336,7 @@ class SelfServiceController extends ActionController
      * @throws \Zend_Controller_Request_Exception
      * @throws \Zend_Controller_Response_Exception
      */
-    protected function addHostToParams(IcingaHost $host, array & $params)
+    protected function addHostToParams(IcingaHost $host, array &$params)
     {
         if (! $host->isObject()) {
             return;
@@ -391,7 +391,7 @@ class SelfServiceController extends ActionController
         $params['accept_config']    = $host->getSingleResolvedProperty('accept_config')=== 'y';
     }
 
-    protected function addStringSettingsToParams(Settings $settings, array $keys, array & $params)
+    protected function addStringSettingsToParams(Settings $settings, array $keys, array &$params)
     {
         foreach ($keys as $key) {
             $value = $settings->get("self-service/$key");
@@ -401,7 +401,7 @@ class SelfServiceController extends ActionController
         }
     }
 
-    protected function addBooleanSettingsToParams(Settings $settings, array $keys, array & $params)
+    protected function addBooleanSettingsToParams(Settings $settings, array $keys, array &$params)
     {
         foreach ($keys as $key) {
             $value = $settings->get("self-service/$key");
diff --git a/application/forms/IcingaDependencyForm.php b/application/forms/IcingaDependencyForm.php
index 3a48eecf..14913865 100644
--- a/application/forms/IcingaDependencyForm.php
+++ b/application/forms/IcingaDependencyForm.php
@@ -287,7 +287,7 @@ class IcingaDependencyForm extends DirectorObjectForm
         return $this;
     }
 
-    protected function handleProperties(DbObject $object, & $values)
+    protected function handleProperties(DbObject $object, &$values)
     {
         if ($this->hasBeenSent()) {
             if (isset($values['parent_host'])
diff --git a/application/forms/SyncPropertyForm.php b/application/forms/SyncPropertyForm.php
index 735cebfc..ba2404a4 100644
--- a/application/forms/SyncPropertyForm.php
+++ b/application/forms/SyncPropertyForm.php
@@ -223,7 +223,7 @@ class SyncPropertyForm extends DirectorObjectForm
         return $this;
     }
 
-    protected function addIcingaTempateColumns(& $columns)
+    protected function addIcingaTempateColumns(&$columns)
     {
         $funcTemplates = 'enum' . ucfirst($this->rule->get('object_type')) . 'Templates';
         if (method_exists($this->db, $funcTemplates)) {
@@ -238,7 +238,7 @@ class SyncPropertyForm extends DirectorObjectForm
         }
     }
 
-    protected function addDatalistsColumns(& $columns)
+    protected function addDatalistsColumns(&$columns)
     {
         // Clear other columns, we don't allow them right now
         $columns = [];
diff --git a/application/views/helpers/FormStoredPassword.php b/application/views/helpers/FormStoredPassword.php
index be063896..bc0e9aef 100644
--- a/application/views/helpers/FormStoredPassword.php
+++ b/application/views/helpers/FormStoredPassword.php
@@ -46,7 +46,7 @@ class Zend_View_Helper_FormStoredPassword extends Zend_View_Helper_FormElement
         return $res;
     }
 
-    protected function stripAttribute(& $attribs, $name, $default = null)
+    protected function stripAttribute(&$attribs, $name, $default = null)
     {
         if (\array_key_exists($name, $attribs)) {
             if (\strlen($attribs[$name])) {
diff --git a/library/Director/CheckPlugin/Check.php b/library/Director/CheckPlugin/Check.php
index bba518b4..d05f5a7f 100644
--- a/library/Director/CheckPlugin/Check.php
+++ b/library/Director/CheckPlugin/Check.php
@@ -39,7 +39,7 @@ class Check extends CheckResults
         return $this;
     }
 
-    protected function makeBool($check, & $message)
+    protected function makeBool($check, &$message)
     {
         if (is_callable($check)) {
             try {
diff --git a/library/Director/Core/RestApiClient.php b/library/Director/Core/RestApiClient.php
index d8649b7c..b0854ff4 100644
--- a/library/Director/Core/RestApiClient.php
+++ b/library/Director/Core/RestApiClient.php
@@ -172,7 +172,7 @@ class RestApiClient
      * @param  $data
      * @return int
      */
-    protected function readPart($curl, & $data)
+    protected function readPart($curl, &$data)
     {
         $length = strlen($data);
         $this->readBuffer .= $data;
diff --git a/library/Director/CustomVariable/CustomVariable.php b/library/Director/CustomVariable/CustomVariable.php
index e440afd4..98eda84d 100644
--- a/library/Director/CustomVariable/CustomVariable.php
+++ b/library/Director/CustomVariable/CustomVariable.php
@@ -109,7 +109,7 @@ abstract class CustomVariable implements IcingaConfigRenderer
         ));
     }
 
-    public function flatten(array & $flat, $prefix)
+    public function flatten(array &$flat, $prefix)
     {
         $flat[$prefix] = $this->getDbValue();
     }
diff --git a/library/Director/CustomVariable/CustomVariableArray.php b/library/Director/CustomVariable/CustomVariableArray.php
index c5428ba1..7e430a4e 100644
--- a/library/Director/CustomVariable/CustomVariableArray.php
+++ b/library/Director/CustomVariable/CustomVariableArray.php
@@ -69,7 +69,7 @@ class CustomVariableArray extends CustomVariable
         return $this;
     }
 
-    public function flatten(array & $flat, $prefix)
+    public function flatten(array &$flat, $prefix)
     {
         foreach ($this->value as $k => $v) {
             $v->flatten($flat, sprintf('%s[%d]', $prefix, $k));
diff --git a/library/Director/CustomVariable/CustomVariableDictionary.php b/library/Director/CustomVariable/CustomVariableDictionary.php
index c5ce7bda..45a29864 100644
--- a/library/Director/CustomVariable/CustomVariableDictionary.php
+++ b/library/Director/CustomVariable/CustomVariableDictionary.php
@@ -75,7 +75,7 @@ class CustomVariableDictionary extends CustomVariable implements Countable
         return $ret;
     }
 
-    public function flatten(array & $flat, $prefix)
+    public function flatten(array &$flat, $prefix)
     {
         foreach ($this->value as $k => $v) {
             $v->flatten($flat, sprintf('%s["%s"]', $prefix, $k));
diff --git a/library/Director/CustomVariable/CustomVariableString.php b/library/Director/CustomVariable/CustomVariableString.php
index 16b2f210..2d509681 100644
--- a/library/Director/CustomVariable/CustomVariableString.php
+++ b/library/Director/CustomVariable/CustomVariableString.php
@@ -37,7 +37,7 @@ class CustomVariableString extends CustomVariable
         return $this;
     }
 
-    public function flatten(array & $flat, $prefix)
+    public function flatten(array &$flat, $prefix)
     {
         // TODO: we should get rid of type=string and always use JSON
         $flat[$prefix] = json_encode($this->getValue());
diff --git a/library/Director/Daemon/ProcessList.php b/library/Director/Daemon/ProcessList.php
index f90ae182..70e2a083 100644
--- a/library/Director/Daemon/ProcessList.php
+++ b/library/Director/Daemon/ProcessList.php
@@ -79,7 +79,7 @@ class ProcessList
 
         $timer = $this->loop->addPeriodicTimer($timeout / 20, function () use (
             $deferred,
-            & $timer,
+            &$timer,
             $killTimer
         ) {
             $stopped = [];
diff --git a/library/Director/DirectorObject/Automation/BasketSnapshot.php b/library/Director/DirectorObject/Automation/BasketSnapshot.php
index 8c8b3425..77e8b24b 100644
--- a/library/Director/DirectorObject/Automation/BasketSnapshot.php
+++ b/library/Director/DirectorObject/Automation/BasketSnapshot.php
@@ -258,7 +258,7 @@ class BasketSnapshot extends DbObject
      * @throws \Zend_Db_Adapter_Exception
      */
     protected function restoreType(
-        & $all,
+        &$all,
         $typeName,
         BasketSnapshotFieldResolver $fieldResolver,
         Db $connection,
@@ -314,7 +314,7 @@ class BasketSnapshot extends DbObject
      * @param $list
      * @throws \Icinga\Module\Director\Exception\DuplicateKeyException
      */
-    protected function recursivelyStore(IcingaObject $object, & $list)
+    protected function recursivelyStore(IcingaObject $object, &$list)
     {
         foreach ($object->listImportNames() as $parent) {
             if (array_key_exists($parent, $list)) {
diff --git a/library/Director/DirectorObject/Automation/ImportExport.php b/library/Director/DirectorObject/Automation/ImportExport.php
index 540de5f5..422eaead 100644
--- a/library/Director/DirectorObject/Automation/ImportExport.php
+++ b/library/Director/DirectorObject/Automation/ImportExport.php
@@ -120,7 +120,7 @@ class ImportExport
     public function unserializeImportSources($objects)
     {
         $count = 0;
-        $this->connection->runFailSafeTransaction(function () use ($objects, & $count) {
+        $this->connection->runFailSafeTransaction(function () use ($objects, &$count) {
             foreach ($objects as $object) {
                 ImportSource::import($object, $this->connection)->store();
                 $count++;
@@ -133,7 +133,7 @@ class ImportExport
     public function unserializeSyncRules($objects)
     {
         $count = 0;
-        $this->connection->runFailSafeTransaction(function () use ($objects, & $count) {
+        $this->connection->runFailSafeTransaction(function () use ($objects, &$count) {
             foreach ($objects as $object) {
                 SyncRule::import($object, $this->connection)->store();
             }
diff --git a/library/Director/Import/Import.php b/library/Director/Import/Import.php
index 3b5f879b..46f9da4b 100644
--- a/library/Director/Import/Import.php
+++ b/library/Director/Import/Import.php
@@ -443,7 +443,7 @@ class Import
      *
      * @param array $array
      */
-    protected function sortArrayObject(& $array)
+    protected function sortArrayObject(&$array)
     {
         foreach ($array as $key => $val) {
             $this->sortElement($val);
@@ -455,7 +455,7 @@ class Import
      *
      * @param mixed $el
      */
-    protected function sortElement(& $el)
+    protected function sortElement(&$el)
     {
         if (is_array($el)) {
             $this->sortArrayObject($el);
diff --git a/library/Director/Objects/GroupMembershipResolver.php b/library/Director/Objects/GroupMembershipResolver.php
index 34921c6c..c7639728 100644
--- a/library/Director/Objects/GroupMembershipResolver.php
+++ b/library/Director/Objects/GroupMembershipResolver.php
@@ -406,7 +406,7 @@ abstract class GroupMembershipResolver
         );
     }
 
-    protected function getDifference(& $left, & $right)
+    protected function getDifference(&$left, &$right)
     {
         $diff = array();
 
@@ -476,7 +476,7 @@ abstract class GroupMembershipResolver
      * @param IcingaObject[]|int[] $objects
      * @return ZfSelect
      */
-    protected function addMembershipWhere(ZfSelect $query, $column, & $objects)
+    protected function addMembershipWhere(ZfSelect $query, $column, &$objects)
     {
         if (empty($objects)) {
             return $query;
diff --git a/library/Director/Objects/IcingaTemplateResolver.php b/library/Director/Objects/IcingaTemplateResolver.php
index e210f6dc..61122a00 100644
--- a/library/Director/Objects/IcingaTemplateResolver.php
+++ b/library/Director/Objects/IcingaTemplateResolver.php
@@ -300,7 +300,7 @@ class IcingaTemplateResolver
         return array_keys($list);
     }
 
-    protected function assertNotInList($id, & $list)
+    protected function assertNotInList($id, &$list)
     {
         if (array_key_exists($id, $list)) {
             $list = array_keys($list);
diff --git a/library/Director/Objects/ImportSource.php b/library/Director/Objects/ImportSource.php
index a58aa195..c2c6a028 100644
--- a/library/Director/Objects/ImportSource.php
+++ b/library/Director/Objects/ImportSource.php
@@ -299,7 +299,7 @@ class ImportSource extends DbObjectWithSettings implements ExportInterface
         return null;
     }
 
-    public function applyModifiers(& $data)
+    public function applyModifiers(&$data)
     {
         $modifiers = $this->fetchFlatRowModifiers();
 
diff --git a/library/Director/Resolver/TemplateTree.php b/library/Director/Resolver/TemplateTree.php
index af4411dc..f8d8fedd 100644
--- a/library/Director/Resolver/TemplateTree.php
+++ b/library/Director/Resolver/TemplateTree.php
@@ -215,7 +215,7 @@ class TemplateTree
      * @param $list
      * @throws NestingError
      */
-    protected function assertNotInList($id, & $list)
+    protected function assertNotInList($id, &$list)
     {
         if (array_key_exists($id, $list)) {
             $list = array_keys($list);
@@ -265,7 +265,7 @@ class TemplateTree
      * @return array
      * @throws NestingError
      */
-    public function getAncestorsById($id, & $ancestors = [], $path = [])
+    public function getAncestorsById($id, &$ancestors = [], $path = [])
     {
         $path[$id] = true;
         foreach ($this->getParentsById($id) as $pid => $name) {
@@ -325,7 +325,7 @@ class TemplateTree
         }
     }
 
-    public function getDescendantsById($id, & $children = [], & $path = [])
+    public function getDescendantsById($id, &$children = [], &$path = [])
     {
         $path[$id] = true;
         foreach ($this->getChildrenById($id) as $pid => $name) {
@@ -359,14 +359,14 @@ class TemplateTree
     protected function returnFullTree()
     {
         $result = $this->rootNodes;
-        foreach ($result as $id => & $node) {
+        foreach ($result as $id => &$node) {
             $this->addChildrenById($id, $node);
         }
 
         return $result;
     }
 
-    protected function addChildrenById($pid, array & $base)
+    protected function addChildrenById($pid, array &$base)
     {
         foreach ($this->getChildrenById($pid) as $id => $name) {
             $base['children'][$id] = [
diff --git a/library/Director/Web/Form/DirectorObjectForm.php b/library/Director/Web/Form/DirectorObjectForm.php
index b0eaa301..3e645ac4 100644
--- a/library/Director/Web/Form/DirectorObjectForm.php
+++ b/library/Director/Web/Form/DirectorObjectForm.php
@@ -286,7 +286,7 @@ abstract class DirectorObjectForm extends DirectorForm
     }
 
     // TODO: move to a subform
-    protected function handleRanges(IcingaObject $object, & $values)
+    protected function handleRanges(IcingaObject $object, &$values)
     {
         if (! $object->supportsRanges()) {
             return;
@@ -369,7 +369,7 @@ abstract class DirectorObjectForm extends DirectorForm
         return $this->displayGroups[$group];
     }
 
-    protected function handleProperties(DbObject $object, & $values)
+    protected function handleProperties(DbObject $object, &$values)
     {
         if ($this->hasBeenSent()) {
             foreach ($values as $key => $value) {
@@ -752,17 +752,17 @@ abstract class DirectorObjectForm extends DirectorForm
         return $this->objectName;
     }
 
-    protected function removeFromSet(& $set, $key)
+    protected function removeFromSet(&$set, $key)
     {
         unset($set[$key]);
     }
 
-    protected function moveUpInSet(& $set, $key)
+    protected function moveUpInSet(&$set, $key)
     {
         list($set[$key - 1], $set[$key]) = array($set[$key], $set[$key - 1]);
     }
 
-    protected function moveDownInSet(& $set, $key)
+    protected function moveDownInSet(&$set, $key)
     {
         list($set[$key + 1], $set[$key]) = array($set[$key], $set[$key + 1]);
     }
diff --git a/library/Director/Web/Form/IplElement/ExtensibleSetElement.php b/library/Director/Web/Form/IplElement/ExtensibleSetElement.php
index 8c440cda..76b2c939 100644
--- a/library/Director/Web/Form/IplElement/ExtensibleSetElement.php
+++ b/library/Director/Web/Form/IplElement/ExtensibleSetElement.php
@@ -113,7 +113,7 @@ class ExtensibleSetElement extends BaseHtmlElement
         return $this;
     }
 
-    protected function extractZfInfo(& $attribs = null)
+    protected function extractZfInfo(&$attribs = null)
     {
         if ($attribs === null) {
             return;
diff --git a/test/setup_vendor.sh b/test/setup_vendor.sh
index 62d7bd5a..c53982f8 100755
--- a/test/setup_vendor.sh
+++ b/test/setup_vendor.sh
@@ -8,7 +8,11 @@ PHP_VERSION="$(php -r 'echo phpversion();')"
 ICINGAWEB_VERSION=${ICINGAWEB_VERSION:=2.7.1}
 ICINGAWEB_GITREF=${ICINGAWEB_GITREF:=}
 
-PHPCS_VERSION=${PHPCS_VERSION:=3.3.2}
+if [ "$PHP_VERSION" '<' 7.1.0 ]; then
+  PHPCS_VERSION=${PHPCS_VERSION:=3.3.2}
+else
+  PHPCS_VERSION=${PHPCS_VERSION:=3.5.2}
+fi
 
 if [ "$PHP_VERSION" '<' 5.6.0 ]; then
   PHPUNIT_VERSION=${PHPUNIT_VERSION:=4.8}