silverstripe / comment-notifications

code/CommentNotifier.php

Braces +9 added lines, -3 removed lines

@@ -14,7 +14,9 @@ 
 	 */
 	public function onAfterPostComment(Comment $comment) {
 		$parent = $comment->getParent();
-		if(!$parent) return;
+		if(!$parent) {
+			return;
+		}
 		
 		// Ask parent to submit all recipients
 		$recipients = $parent->notificationRecipients($comment);
@@ -33,7 +35,9 @@ 
 	 * @return boolean
 	 */
 	public function isValidEmail($email) {
-		if(!$email) return false;
+		if(!$email) {
+			return false;
+		}
 
 		$pcrePattern = '^[a-z0-9!#$%&\'*+/=?^_`{|}~-]+(?:\\.[a-z0-9!#$%&\'*+/=?^_`{|}~-]+)*'
 			. '@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?$';
@@ -61,7 +65,9 @@ 
 		$to = $recipient instanceof Member
 			? $recipient->Email
 			: $recipient;
-		if(!$this->isValidEmail($to)) return;
+		if(!$this->isValidEmail($to)) {
+			return;
+		}
 
 		// Prepare the email
 		$email = new Email();

tests/CommentNotifiableTest.php

Braces +3 added lines, -1 removed lines

@@ -75,7 +75,9 @@
 
 	public function notificationRecipients($comment) {
 		$author = $this->Author();
-		if($author && $author->exists()) return new ArrayList(array($author));
+		if($author && $author->exists()) {
+			return new ArrayList(array($author));
+		}
 		return parent::notificationRecipients($comment);
 	}