From ec6930da8226b9879eb2d7289af739aa96de43ef Mon Sep 17 00:00:00 2001
From: Sean Molenaar <sean@seanmolenaar.eu>
Date: Wed, 28 Oct 2020 21:33:17 +0100
Subject: DB: stop overloading getID

Signed-off-by: Sean Molenaar <sean@seanmolenaar.eu>
---
 lib/Db/Feed.php                 | 21 ---------------------
 lib/Db/Folder.php               | 15 ---------------
 lib/Db/Item.php                 | 15 ---------------
 lib/Migration/MigrateConfig.php | 18 +++++++++++++-----
 4 files changed, 13 insertions(+), 56 deletions(-)

(limited to 'lib')

diff --git a/lib/Db/Feed.php b/lib/Db/Feed.php
index 80d425117..852de4c78 100644
--- a/lib/Db/Feed.php
+++ b/lib/Db/Feed.php
@@ -183,14 +183,6 @@ class Feed extends Entity implements IAPI, \JsonSerializable
         return $this->httpLastModified;
     }
 
-    /**
-     * @return int
-     */
-    public function getId(): int
-    {
-        return $this->id;
-    }
-
     /**
      * @return string|null
      */
@@ -475,19 +467,6 @@ class Feed extends Entity implements IAPI, \JsonSerializable
         return $this;
     }
 
-    /**
-     * @param int $id
-     */
-    public function setId(int $id): Feed
-    {
-        if ($this->id !== $id) {
-            $this->id = $id;
-            $this->markFieldUpdated('id');
-        }
-
-        return $this;
-    }
-
     /**
      * @param string|null $lastModified
      */
diff --git a/lib/Db/Folder.php b/lib/Db/Folder.php
index e4dbf0fe7..2efd65e96 100644
--- a/lib/Db/Folder.php
+++ b/lib/Db/Folder.php
@@ -52,11 +52,6 @@ class Folder extends Entity implements IAPI, \JsonSerializable
         return $this->deletedAt;
     }
 
-    public function getId(): int
-    {
-        return $this->id;
-    }
-
     /**
      * @return string|null
      */
@@ -115,16 +110,6 @@ class Folder extends Entity implements IAPI, \JsonSerializable
         return $this;
     }
 
-    public function setId(int $id): self
-    {
-        if ($this->id !== $id) {
-            $this->id = $id;
-            $this->markFieldUpdated('id');
-        }
-
-        return $this;
-    }
-
     public function setLastModified(?string $lastModified = null): self
     {
 
diff --git a/lib/Db/Item.php b/lib/Db/Item.php
index 555bcbb61..1dab1b96c 100644
--- a/lib/Db/Item.php
+++ b/lib/Db/Item.php
@@ -222,11 +222,6 @@ class Item extends Entity implements IAPI, \JsonSerializable
         return $this->guidHash;
     }
 
-    public function getId(): string
-    {
-        return $this->id;
-    }
-
     public function getIntro(): string
     {
         return strip_tags($this->getBody());
@@ -440,16 +435,6 @@ class Item extends Entity implements IAPI, \JsonSerializable
         return $this;
     }
 
-    public function setId(int $id): self
-    {
-        if ($this->id !== $id) {
-            $this->id = $id;
-            $this->markFieldUpdated('id');
-        }
-
-        return $this;
-    }
-
     public function setLastModified(string $lastModified = null): self
     {
         if ($this->lastModified !== $lastModified) {
diff --git a/lib/Migration/MigrateConfig.php b/lib/Migration/MigrateConfig.php
index 9f3b32291..4ba34fb5e 100644
--- a/lib/Migration/MigrateConfig.php
+++ b/lib/Migration/MigrateConfig.php
@@ -11,7 +11,6 @@
 
 namespace OCA\News\Migration;
 
-use OCA\News\AppInfo\Application;
 use OCA\News\Config\LegacyConfig;
 use OCP\IConfig;
 use OCP\Migration\IRepairStep;
@@ -40,13 +39,22 @@ class MigrateConfig implements IRepairStep
     /**
      * @param LegacyConfig $config
      * @param IConfig      $iConfig
-     * @param Application  $application To make sure the class is found below
      */
-    public function __construct(LegacyConfig $config, IConfig $iConfig, Application $application)
+    public function __construct(LegacyConfig $config, IConfig $iConfig)
     {
         $this->config = $config;
         $this->iConfig = $iConfig;
-        $this->defaults = $application::DEFAULT_SETTINGS;
+
+        // copied from Application::default_settings
+        $this->defaults = [
+            'autoPurgeMinimumInterval' => 60,
+            'autoPurgeCount'           => 200,
+            'maxRedirects'             => 10,
+            'feedFetcherTimeout'       => 60,
+            'useCronUpdates'           => true,
+            'exploreUrl'               => '',
+            'updateInterval'           => 3600,
+        ];
     }
 
     public function getName()
@@ -57,7 +65,7 @@ class MigrateConfig implements IRepairStep
     public function run(IOutput $output)
     {
         $version = $this->iConfig->getAppValue('news', 'installed_version', '0.0.0');
-        if (version_compare($version, '15.0.0', '>')) {
+        if (version_compare($version, '15.0.6', '>')) {
             return;
         }
 
-- 
cgit v1.2.3