From 53aa7a4d0dad11551947523aaf503174e7431917 Mon Sep 17 00:00:00 2001
From: "J. King" <jking@jkingweb.ca>
Date: Thu, 5 Sep 2019 10:21:36 -0400
Subject: [PATCH] Style fixes

---
 RoboFile.php                       | 11 +++++------
 lib/Db/Driver.php                  |  2 +-
 lib/REST/Fever/API.php             |  5 ++---
 tests/cases/REST/Fever/TestAPI.php |  4 ++--
 4 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/RoboFile.php b/RoboFile.php
index 8350730d..93c94443 100644
--- a/RoboFile.php
+++ b/RoboFile.php
@@ -2,7 +2,6 @@
 
 use Robo\Result;
 
-
 const BASE = __DIR__.\DIRECTORY_SEPARATOR;
 const BASE_TEST = BASE."tests".\DIRECTORY_SEPARATOR;
 define("IS_WIN", defined("PHP_WINDOWS_VERSION_MAJOR"));
@@ -190,7 +189,7 @@ class RoboFile extends \Robo\Tasks {
     }
 
     /** Generates static manual pages in the "manual" directory
-     * 
+     *
      * The resultant files are suitable for offline viewing and inclusion into release builds
      */
     public function manual(array $args): Result {
@@ -209,7 +208,7 @@ class RoboFile extends \Robo\Tasks {
     }
 
     /** Rebuilds the entire manual theme
-     * 
+     *
      * This requires Node and Yarn to be installed, and only needs to be done when
      * Daux's theme changes
      */
@@ -223,7 +222,7 @@ class RoboFile extends \Robo\Tasks {
         // rebuild the stylesheet
         $t->addCode([$this, "manualCss"]);
         // copy JavaScript files from the Daux theme
-        foreach(glob($dauxjs."daux*") as $file) {
+        foreach (glob($dauxjs."daux*") as $file) {
             $t->taskFilesystemStack()->copy($file, $themeout.basename($file), true);
         }
         // download highlight.js
@@ -231,7 +230,7 @@ class RoboFile extends \Robo\Tasks {
             // compile the list of desired language (enumerated above) into an application/x-www-form-urlencoded body
             $post = http_build_query((function($langs) {
                 $out = [];
-                foreach($langs as $l) {
+                foreach ($langs as $l) {
                     $out[$l.".js"] = "on";
                 }
                 return $out;
@@ -274,7 +273,7 @@ class RoboFile extends \Robo\Tasks {
     }
 
     /** Rebuilds the manual theme's stylesheet only
-     * 
+     *
      * This requires Node and Yarn to be installed.
      */
     public function manualCss(): Result {
diff --git a/lib/Db/Driver.php b/lib/Db/Driver.php
index a456fba3..44a1e989 100644
--- a/lib/Db/Driver.php
+++ b/lib/Db/Driver.php
@@ -84,7 +84,7 @@ interface Driver {
     public function literalString(string $str): string;
 
     /** Performs implementation-specific database maintenance to ensure good performance
-     * 
+     *
      * This should be restricted to quick maintenance; in SQLite terms it might include ANALYZE, but not VACUUM
      */
     public function maintenance(): bool;
diff --git a/lib/REST/Fever/API.php b/lib/REST/Fever/API.php
index 4bb3c233..1401d63a 100644
--- a/lib/REST/Fever/API.php
+++ b/lib/REST/Fever/API.php
@@ -203,7 +203,7 @@ class API extends \JKingWeb\Arsse\REST\AbstractHandler {
             } elseif (isset($v[0])) {
                 // this is a very simplistic check for an indexed array
                 // it would not pass muster in the face of generic data,
-                // but we'll assume our code produces only well-ordered 
+                // but we'll assume our code produces only well-ordered
                 // indexed arrays
                 $p->appendChild($this->makeXMLIndexed($v, $d->createElement($k), substr($k, 0, strlen($k) - 1)));
             } else {
@@ -227,7 +227,6 @@ class API extends \JKingWeb\Arsse\REST\AbstractHandler {
             }
         }
         return $p;
-
     }
 
     protected function logIn(string $hash): bool {
@@ -317,7 +316,7 @@ class API extends \JKingWeb\Arsse\REST\AbstractHandler {
         $lastUnread = Date::normalize($lastUnread, "sql");
         $since = Date::sub("PT15S", $lastUnread);
         $c->unread(false)->markedSince($since);
-        Arsse::$db->articleMark(Arsse::$user->id, ['read' => false], $c); 
+        Arsse::$db->articleMark(Arsse::$user->id, ['read' => false], $c);
     }
 
     protected function getRefreshTime() {
diff --git a/tests/cases/REST/Fever/TestAPI.php b/tests/cases/REST/Fever/TestAPI.php
index 83116cc9..e453151e 100644
--- a/tests/cases/REST/Fever/TestAPI.php
+++ b/tests/cases/REST/Fever/TestAPI.php
@@ -318,7 +318,7 @@ class TestAPI extends \JKingWeb\Arsse\Test\AbstractTest {
         ]);
         $act = $this->h->dispatch($this->req("api&feeds"));
         $this->assertMessage($exp, $act);
-    }       
+    }
 
     /** @dataProvider provideItemListContexts */
     public function testListItems(string $url, Context $c, bool $desc) {
@@ -518,7 +518,7 @@ class TestAPI extends \JKingWeb\Arsse\Test\AbstractTest {
     public function testListFeedIcons() {
         $act = $this->h->dispatch($this->req("api&favicons"));
         $exp = new JsonResponse(['favicons' => [['id' => 0, 'data' => API::GENERIC_ICON_TYPE.",".API::GENERIC_ICON_DATA]]]);
-        $this->assertMessage($exp, $act);        
+        $this->assertMessage($exp, $act);
     }
 
     public function testAnswerOptionsRequest() {