MAGETWO-53307: Checkout is broken when purchasing products on persisted session

Stanislav Idolov · Stanislav Idolov · commit 4740771eb844 · 2016-05-24T12:09:27.000+03:00
diff --git a/app/code/Magento/Persistent/Observer/SetQuotePersistentDataObserver.php b/app/code/Magento/Persistent/Observer/SetQuotePersistentDataObserver.php
@@ -79,7 +79,7 @@ public function execute(\Magento\Framework\Event\Observer $observer)
             && $this->quoteManager->isPersistent()
         ) {
             //Quote is not actual customer's quote, just persistent
-            $quote->setIsActive(false)->setIsPersistent(true);
+            $quote->setIsPersistent(true);
         }
     }
 }
diff --git a/app/code/Magento/Persistent/Test/Unit/Observer/SetQuotePersistentDataObserverTest.php b/app/code/Magento/Persistent/Test/Unit/Observer/SetQuotePersistentDataObserverTest.php
@@ -101,7 +101,6 @@ public function testExecuteWhenSessionIsPersistent()
         $this->customerSessionMock->expects($this->once())->method('isLoggedIn')->will($this->returnValue(false));
         $this->helperMock->expects($this->once())->method('isShoppingCartPersist')->will($this->returnValue(false));
         $this->quoteManagerMock->expects($this->once())->method('isPersistent')->will($this->returnValue(true));
-        $this->quoteMock->expects($this->once())->method('setIsActive')->with(false)->will($this->returnSelf());
         $this->quoteMock->expects($this->once())->method('setIsPersistent')->with(true);
         $this->model->execute($this->observerMock);
     }

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ public function execute(\Magento\Framework\Event\Observer $observer)`
`79`	`79`	`&& $this->quoteManager->isPersistent()`
`80`	`80`	`) {`
`81`	`81`	`//Quote is not actual customer's quote, just persistent`
`82`		`- $quote->setIsActive(false)->setIsPersistent(true);`
	`82`	`+ $quote->setIsPersistent(true);`
`83`	`83`	`}`
`84`	`84`	`}`
`85`	`85`	`}`
Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,6 @@ public function testExecuteWhenSessionIsPersistent()`
`101`	`101`	`$this->customerSessionMock->expects($this->once())->method('isLoggedIn')->will($this->returnValue(false));`
`102`	`102`	`$this->helperMock->expects($this->once())->method('isShoppingCartPersist')->will($this->returnValue(false));`
`103`	`103`	`$this->quoteManagerMock->expects($this->once())->method('isPersistent')->will($this->returnValue(true));`
`104`		`- $this->quoteMock->expects($this->once())->method('setIsActive')->with(false)->will($this->returnSelf());`
`105`	`104`	`$this->quoteMock->expects($this->once())->method('setIsPersistent')->with(true);`
`106`	`105`	`$this->model->execute($this->observerMock);`
`107`	`106`	`}`