From 90d0e4e006e78b14366ff28a757c6ddae425c3a1 Mon Sep 17 00:00:00 2001
From: megachriz <megachriz@654114.no-reply.drupal.org>
Date: Thu, 8 Mar 2018 12:19:53 +0100
Subject: [PATCH] Issue #2943953 by MegaChriz: Fixed fatal error "No more
 accounts to revert to." when trying to unlock a feed.

---
 feeds.info                                 |  1 +
 includes/FeedsAccountSwitcher.inc          |  2 +-
 includes/FeedsAccountSwitcherException.inc | 12 +++++
 includes/FeedsAccountSwitcherInterface.inc |  2 +-
 includes/FeedsSource.inc                   |  7 ++-
 tests/feeds_processor_node.test            | 51 ++++++++++++++++++++++
 6 files changed, 72 insertions(+), 3 deletions(-)
 create mode 100644 includes/FeedsAccountSwitcherException.inc

diff --git a/feeds.info b/feeds.info
index bf3784b5..14a04b0c 100644
--- a/feeds.info
+++ b/feeds.info
@@ -15,6 +15,7 @@ test_dependencies[] = rules:rules
 test_dependencies[] = variable:variable
 
 files[] = includes/FeedsAccountSwitcher.inc
+files[] = includes/FeedsAccountSwitcherException.inc
 files[] = includes/FeedsAccountSwitcherInterface.inc
 files[] = includes/FeedsConfigurable.inc
 files[] = includes/FeedsHTTPCache.inc
diff --git a/includes/FeedsAccountSwitcher.inc b/includes/FeedsAccountSwitcher.inc
index 5bc77394..476e2eb8 100644
--- a/includes/FeedsAccountSwitcher.inc
+++ b/includes/FeedsAccountSwitcher.inc
@@ -78,7 +78,7 @@ class FeedsAccountSwitcher implements FeedsAccountSwitcherInterface {
       $this->activateCurrentUser();
     }
     else {
-      throw new RuntimeException('No more accounts to revert to.');
+      throw new FeedsAccountSwitcherException('No more accounts to revert to.');
     }
     // Restore original session saving status if all account switches are
     // reverted.
diff --git a/includes/FeedsAccountSwitcherException.inc b/includes/FeedsAccountSwitcherException.inc
new file mode 100644
index 00000000..fc806df2
--- /dev/null
+++ b/includes/FeedsAccountSwitcherException.inc
@@ -0,0 +1,12 @@
+<?php
+
+/**
+ * @file
+ * Contains FeedsAccountSwitcherException.
+ */
+
+/**
+ * Thrown when there is no account to switch back to.
+ */
+class FeedsAccountSwitcherException extends RuntimeException {
+}
diff --git a/includes/FeedsAccountSwitcherInterface.inc b/includes/FeedsAccountSwitcherInterface.inc
index dd7fab35..517ef220 100644
--- a/includes/FeedsAccountSwitcherInterface.inc
+++ b/includes/FeedsAccountSwitcherInterface.inc
@@ -31,7 +31,7 @@ interface FeedsAccountSwitcherInterface {
    * @return FeedsAccountSwitcherInterface
    *   $this.
    *
-   * @throws RuntimeException
+   * @throws FeedsAccountSwitcherException
    *   When there are no more account switches to revert.
    */
   public function switchBack();
diff --git a/includes/FeedsSource.inc b/includes/FeedsSource.inc
index 307c83d8..ad2d46f3 100644
--- a/includes/FeedsSource.inc
+++ b/includes/FeedsSource.inc
@@ -933,7 +933,12 @@ class FeedsSource extends FeedsConfigurable {
   public function unlock() {
     $this->clearStates();
     $this->save();
-    $this->releaseLock();
+    try {
+      $this->releaseLock();
+    }
+    catch (FeedsAccountSwitcherException $exception) {
+      // Ignore switch back exceptions.
+    }
   }
 
   /**
diff --git a/tests/feeds_processor_node.test b/tests/feeds_processor_node.test
index 73e76aa7..94c0a637 100644
--- a/tests/feeds_processor_node.test
+++ b/tests/feeds_processor_node.test
@@ -650,6 +650,57 @@ class FeedsRSStoNodesTest extends FeedsWebTestCase {
     $this->assertEqual(0, db_query("SELECT COUNT(*) FROM {queue} WHERE name = 'feeds_source_clear'")->fetchField());
   }
 
+  /**
+   * Tests unlocking a feed.
+   */
+  public function testUnlock() {
+    // Just remove the mappings rather than creating a new importer.
+    $this->removeMappings('syndication', $this->getCurrentMappings('syndication'));
+
+    $this->setPlugin('syndication', 'FeedsFileFetcher');
+    $this->setPlugin('syndication', 'FeedsCSVParser');
+
+    $this->setSettings('syndication', NULL, array(
+      'content_type' => '',
+      'process_in_background' => TRUE,
+      'import_period' => FEEDS_SCHEDULE_NEVER,
+    ));
+
+    $this->addMappings('syndication', array(
+      0 => array(
+        'source' => 'title',
+        'target' => 'title',
+      ),
+      1 => array(
+        'source' => 'GUID',
+        'target' => 'guid',
+        'unique' => TRUE,
+      ),
+    ));
+
+    $this->importFile('syndication', $this->absolutePath() . '/tests/feeds/many_nodes_ordered.csv', 'Schedule import');
+    $this->assertEqual(0, db_query("SELECT COUNT(*) FROM {node}")->fetchField());
+
+    // Assert that the import button is disabled.
+    $this->assertFieldDisabled('op');
+
+    // Assert that there is one import task in the queue.
+    $this->assertEqual(1, db_query("SELECT COUNT(*) FROM {queue} WHERE name = 'feeds_source_import'")->fetchField());
+
+    // Now unlock the feed.
+    $this->drupalPost('import/syndication/unlock', array(), 'Unlock');
+    $this->assertText('Importer unlocked.');
+
+    // Assert that the unlock button is now disabled.
+    $this->drupalGet('import/syndication/unlock');
+    $this->assertFieldDisabled('op');
+    $this->assertText('This importer is not locked, therefore it cannot be unlocked.');
+
+    // And assert that the import button is no longer disabled.
+    $this->drupalGet('import/syndication');
+    $this->assertFieldEnabled('op');
+  }
+
   /**
    * Tests skip new items.
    */
-- 
GitLab