Fix static tests and remove return before redirect

nuzil · ihor-sviziev · commit ac4a2fa7216b · 2018-07-22T04:56:15.000Z
diff --git a/app/code/Magento/Newsletter/Controller/Manage/Save.php b/app/code/Magento/Newsletter/Controller/Manage/Save.php
@@ -96,9 +96,8 @@ public function execute()
                             ->unsubscribeCustomerById($customerId);
                         $this->messageManager->addSuccess(__('We removed the subscription.'));
                     }
-                }else{
+                } else {
                     $this->messageManager->addSuccess(__('We updated the subscription.'));
-                    return;
                 }
             } catch (\Exception $e) {
                 $this->messageManager->addError(__('Something went wrong while saving your subscription.'));

Original file line number	Diff line number	Diff line change
`@@ -96,9 +96,8 @@ public function execute()`
`96`	`96`	`->unsubscribeCustomerById($customerId);`
`97`	`97`	`$this->messageManager->addSuccess(__('We removed the subscription.'));`
`98`	`98`	`}`
`99`		`- }else{`
	`99`	`+ } else {`
`100`	`100`	`$this->messageManager->addSuccess(__('We updated the subscription.'));`
`101`		`- return;`
`102`	`101`	`}`
`103`	`102`	`} catch (\Exception $e) {`
`104`	`103`	`$this->messageManager->addError(__('Something went wrong while saving your subscription.'));`