Refactored the code

javiereguiluz · web-flow · commit cba04784e7ca · 2018-01-04T11:46:46.000+01:00
diff --git a/service_container.rst b/service_container.rst
@@ -365,7 +365,8 @@ made. To do that, you create a new class::
                 ->addPart(
                     'Someone just updated the site. We told them: '.$happyMessage
                 );
-            $this->mailer->send($message);
+            
+            return $this->mailer->send($message) > 0;
         }
     }
 
@@ -379,8 +380,10 @@ you can use the service immediately::
     {
         // ...
 
-        $siteUpdateManager->notifyOfSiteUpdate();
-        $this->addFlash('success', 'Notification mail was sent successfully.');
+        if ($siteUpdateManager->notifyOfSiteUpdate()) {
+            $this->addFlash('success', 'Notification mail was sent successfully.');
+        }
+        
         // ...
     }
 

Original file line number	Diff line number	Diff line change
`@@ -365,7 +365,8 @@ made. To do that, you create a new class::`
`365`	`365`	`->addPart(`
`366`	`366`	`'Someone just updated the site. We told them: '.$happyMessage`
`367`	`367`	`);`
`368`		`- $this->mailer->send($message);`
	`368`	`+`
	`369`	`+ return $this->mailer->send($message) > 0;`
`369`	`370`	`}`
`370`	`371`	`}`
`371`	`372`
`@@ -379,8 +380,10 @@ you can use the service immediately::`
`379`	`380`	`{`
`380`	`381`	`// ...`
`381`	`382`
`382`		`- $siteUpdateManager->notifyOfSiteUpdate();`
`383`		`- $this->addFlash('success', 'Notification mail was sent successfully.');`
	`383`	`+ if ($siteUpdateManager->notifyOfSiteUpdate()) {`
	`384`	`+ $this->addFlash('success', 'Notification mail was sent successfully.');`
	`385`	`+ }`
	`386`	`+`
`384`	`387`	`// ...`
`385`	`388`	`}`
`386`	`389`