silverstripe / comment-notifications

tests/Model/CommentNotifiableTestDataObject.php

Indentation +32 added lines, -32 removed lines

@@ -11,36 +11,36 @@
 
 class CommentNotifiableTestDataObject extends DataObject implements TestOnly
 {
-    private static $db = [
-        "Title" => "Varchar(255)",
-        "URLSegment" => "Varchar(255)",
-    ];
-
-    private static $has_one = [
-        'Author' => 'SilverStripe\Security\Member'
-    ];
-
-    private static $extensions = [
-        CommentNotifiable::class,
-        CommentsExtension::class
-    ];
-
-    private static $table_name = 'CommentNotifiableTestDataObject';
-
-    public function updateNotificationRecipients(&$list, $comment)
-    {
-        $author = $this->Author();
-
-        if ($author && $author->exists()) {
-            $list[] = $author->Email;
-        }
-    }
-
-    public function Link($action = false)
-    {
-        return Controller::join_links(
-            Director::baseURL(),
-            $this->URLSegment
-        );
-    }
+	private static $db = [
+		"Title" => "Varchar(255)",
+		"URLSegment" => "Varchar(255)",
+	];
+
+	private static $has_one = [
+		'Author' => 'SilverStripe\Security\Member'
+	];
+
+	private static $extensions = [
+		CommentNotifiable::class,
+		CommentsExtension::class
+	];
+
+	private static $table_name = 'CommentNotifiableTestDataObject';
+
+	public function updateNotificationRecipients(&$list, $comment)
+	{
+		$author = $this->Author();
+
+		if ($author && $author->exists()) {
+			$list[] = $author->Email;
+		}
+	}
+
+	public function Link($action = false)
+	{
+		return Controller::join_links(
+			Director::baseURL(),
+			$this->URLSegment
+		);
+	}
 }

tests/CommentNotifierTest.php

Indentation +71 added lines, -71 removed lines

@@ -12,102 +12,102 @@
 
 class CommentNotifierTest extends SapphireTest
 {
-    protected static $fixture_file = 'CommentNotifications.yml';
+	protected static $fixture_file = 'CommentNotifications.yml';
 
-    protected $oldhost = null;
+	protected $oldhost = null;
 
-    protected static $extra_dataobjects = [
-        CommentNotifiableTestDataObject::class
-    ];
+	protected static $extra_dataobjects = [
+		CommentNotifiableTestDataObject::class
+	];
 
-    protected static $extra_controllers = [
-        CommentNotifierTestController::class
-    ];
+	protected static $extra_controllers = [
+		CommentNotifierTestController::class
+	];
 
-    protected function setUp()
-    {
-        parent::setUp();
+	protected function setUp()
+	{
+		parent::setUp();
 
-        Config::modify()->set('Email', 'admin_email', 'myadmin@mysite.com');
-        $this->oldhost = isset($_SERVER['HTTP_HOST']) ? $_SERVER['HTTP_HOST'] : null;
-        $_SERVER['HTTP_HOST'] = 'www.mysite.com';
-    }
+		Config::modify()->set('Email', 'admin_email', 'myadmin@mysite.com');
+		$this->oldhost = isset($_SERVER['HTTP_HOST']) ? $_SERVER['HTTP_HOST'] : null;
+		$_SERVER['HTTP_HOST'] = 'www.mysite.com';
+	}
 
-    protected function tearDown()
-    {
-        $_SERVER['HTTP_HOST'] = $this->oldhost;
+	protected function tearDown()
+	{
+		$_SERVER['HTTP_HOST'] = $this->oldhost;
 
-        parent::tearDown();
-    }
+		parent::tearDown();
+	}
 
-    public function testSendEmail()
-    {
-        $author = $this->objFromFixture(Member::class, 'author');
-        $item1 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item1');
-        $item2 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item2');
-        $comment1 = $this->objFromFixture(Comment::class, 'comment1');
-        $comment2 = $this->objFromFixture(Comment::class, 'comment2');
-        $comment3 = $this->objFromFixture(Comment::class, 'comment3');
-        $controller = new CommentNotifierTestController();
+	public function testSendEmail()
+	{
+		$author = $this->objFromFixture(Member::class, 'author');
+		$item1 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item1');
+		$item2 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item2');
+		$comment1 = $this->objFromFixture(Comment::class, 'comment1');
+		$comment2 = $this->objFromFixture(Comment::class, 'comment2');
+		$comment3 = $this->objFromFixture(Comment::class, 'comment3');
+		$controller = new CommentNotifierTestController();
 
-        // Comment 1
-        $result = $controller->notifyCommentRecipient($comment1, $item1, $author);
-        $this->assertEmailSent('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
+		// Comment 1
+		$result = $controller->notifyCommentRecipient($comment1, $item1, $author);
+		$this->assertEmailSent('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
 
-        $email = $this->findEmail('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
+		$email = $this->findEmail('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
 
-        $this->assertContains('<li>Bob Bobberson</li>', $email['Content']);
-        $this->assertContains('<li>bob@address.com</li>', $email['Content']);
-        $this->assertContains('<blockquote>Hey what a lovely comment</blockquote>', $email['Content']);
+		$this->assertContains('<li>Bob Bobberson</li>', $email['Content']);
+		$this->assertContains('<li>bob@address.com</li>', $email['Content']);
+		$this->assertContains('<blockquote>Hey what a lovely comment</blockquote>', $email['Content']);
 
-        $this->clearEmails();
+		$this->clearEmails();
 
-        // Comment 2
-        $result = $controller->notifyCommentRecipient($comment2, $item2, $author);
-        $this->assertEmailSent('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
+		// Comment 2
+		$result = $controller->notifyCommentRecipient($comment2, $item2, $author);
+		$this->assertEmailSent('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
 
-        $email = $this->findEmail('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
-        $this->assertContains('<li>Secret</li>', $email['Content']);
-        $this->assertContains('<li>secret@notallowed.com</li>', $email['Content']);
-        $this->assertContains('<blockquote>I don&#039;t want to disclose my details</blockquote>', $email['Content']);
+		$email = $this->findEmail('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
+		$this->assertContains('<li>Secret</li>', $email['Content']);
+		$this->assertContains('<li>secret@notallowed.com</li>', $email['Content']);
+		$this->assertContains('<blockquote>I don&#039;t want to disclose my details</blockquote>', $email['Content']);
 
-        $this->clearEmails();
+		$this->clearEmails();
 
-        // Comment 3
-        $result = $controller->notifyCommentRecipient($comment3, $item1, $author);
-        $this->assertEmailSent('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
+		// Comment 3
+		$result = $controller->notifyCommentRecipient($comment3, $item1, $author);
+		$this->assertEmailSent('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
 
-        $email = $this->findEmail('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
+		$email = $this->findEmail('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
 
-        $this->assertContains('<li>Anonymous</li>', $email['Content']);
-        $this->assertContains('<li>notlogged@in.com</li>', $email['Content']);
-        $this->assertContains('<blockquote>I didn&#039;t log in</blockquote>', $email['Content']);
+		$this->assertContains('<li>Anonymous</li>', $email['Content']);
+		$this->assertContains('<li>notlogged@in.com</li>', $email['Content']);
+		$this->assertContains('<blockquote>I didn&#039;t log in</blockquote>', $email['Content']);
 
-        $this->clearEmails();
+		$this->clearEmails();
 
-        // Comment 3 without an author
-        $result = $controller->notifyCommentRecipient($comment3, $item1, 'foobar@silverstripe.org');
-        $this->assertEmailSent('foobar@silverstripe.org', 'noreply@mysite.com', 'A new comment has been posted');
+		// Comment 3 without an author
+		$result = $controller->notifyCommentRecipient($comment3, $item1, 'foobar@silverstripe.org');
+		$this->assertEmailSent('foobar@silverstripe.org', 'noreply@mysite.com', 'A new comment has been posted');
 
-        $this->clearEmails();
+		$this->clearEmails();
 
-        // Comment 3 without a valid email
-        $result = $controller->notifyCommentRecipient($comment3, $item1, '<foobar1>');
-        $noEmail = (bool) $this->findEmail('<foobar1>', 'noreply@mysite.com', 'A new comment has been posted');
+		// Comment 3 without a valid email
+		$result = $controller->notifyCommentRecipient($comment3, $item1, '<foobar1>');
+		$noEmail = (bool) $this->findEmail('<foobar1>', 'noreply@mysite.com', 'A new comment has been posted');
 
-        $this->assertFalse($noEmail);
-    }
+		$this->assertFalse($noEmail);
+	}
 
-    public function testOnAfterPostComment()
-    {
-        $this->clearEmails();
+	public function testOnAfterPostComment()
+	{
+		$this->clearEmails();
 
-        $comment1 = $this->objFromFixture(Comment::class, 'comment1');
+		$comment1 = $this->objFromFixture(Comment::class, 'comment1');
 
-        $controller = new CommentNotifierTestController();
-        $controller->invokeWithExtensions('onAfterPostComment', $comment1);
+		$controller = new CommentNotifierTestController();
+		$controller->invokeWithExtensions('onAfterPostComment', $comment1);
 
-        // test that after posting a comment the notifications are sent.
-        $this->assertEmailSent('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
-    }
+		// test that after posting a comment the notifications are sent.
+		$this->assertEmailSent('andrew@address.com', 'noreply@mysite.com', 'A new comment has been posted');
+	}
 }

tests/Control/CommentNotifierTestController.php

Indentation +4 added lines, -4 removed lines

@@ -8,9 +8,9 @@
 
 class CommentNotifierTestController extends Controller implements TestOnly
 {
-    private static $extensions = array(
-        CommentNotifier::class
-    );
+	private static $extensions = array(
+		CommentNotifier::class
+	);
 
-    private static $url_segment = 'commentnotifiertest';
+	private static $url_segment = 'commentnotifiertest';
 }

tests/CommentNotifiableTest.php

Indentation +56 added lines, -56 removed lines

@@ -11,60 +11,60 @@
 
 class CommentNotifiableTest extends SapphireTest
 {
-    protected static $fixture_file = 'CommentNotifications.yml';
-
-    protected $oldhost = null;
-
-    protected static $extra_dataobjects = [
-        CommentNotifiableTestDataObject::class
-    ];
-
-    protected function setUp()
-    {
-        parent::setUp();
-
-        Config::modify()->set(Email::class, 'admin_email', 'myadmin@mysite.com');
-
-        $this->oldhost = isset($_SERVER['HTTP_HOST']) ? $_SERVER['HTTP_HOST'] : null;
-        $_SERVER['HTTP_HOST'] = 'www.mysite.com';
-    }
-
-    protected function tearDown()
-    {
-        $_SERVER['HTTP_HOST'] = $this->oldhost;
-
-        parent::tearDown();
-    }
-
-    public function testGetRecipients()
-    {
-        $comment1 = $this->objFromFixture(Comment::class, 'comment1');
-        $comment2 = $this->objFromFixture(Comment::class, 'comment2');
-        $item1 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item1');
-        $item2 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item2');
-
-        $this->assertEquals(['myadmin@mysite.com', 'andrew@address.com'], $item1->notificationRecipients($comment1));
-        $this->assertEquals(['myadmin@mysite.com'], $item2->notificationRecipients($comment2));
-    }
-
-    public function testNotificationSubject()
-    {
-        $recipient = $this->objFromFixture(Member::class, 'author');
-        $comment1 = $this->objFromFixture(Comment::class, 'comment1');
-        $comment2 = $this->objFromFixture(Comment::class, 'comment2');
-        $item1 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item1');
-        $item2 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item2');
-
-        $this->assertEquals('A new comment has been posted', $item1->notificationSubject($comment1, $recipient));
-        $this->assertEquals('A new comment has been posted', $item2->notificationSubject($comment2, $recipient));
-    }
-
-    public function testNotificationSender()
-    {
-        $comment1 = $this->objFromFixture(Comment::class, 'comment1');
-        $author = $this->objFromFixture(Member::class, 'author');
-        $item1 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item1');
-
-        $this->assertEquals('noreply@mysite.com', $item1->notificationSender($comment1, $author));
-    }
+	protected static $fixture_file = 'CommentNotifications.yml';
+
+	protected $oldhost = null;
+
+	protected static $extra_dataobjects = [
+		CommentNotifiableTestDataObject::class
+	];
+
+	protected function setUp()
+	{
+		parent::setUp();
+
+		Config::modify()->set(Email::class, 'admin_email', 'myadmin@mysite.com');
+
+		$this->oldhost = isset($_SERVER['HTTP_HOST']) ? $_SERVER['HTTP_HOST'] : null;
+		$_SERVER['HTTP_HOST'] = 'www.mysite.com';
+	}
+
+	protected function tearDown()
+	{
+		$_SERVER['HTTP_HOST'] = $this->oldhost;
+
+		parent::tearDown();
+	}
+
+	public function testGetRecipients()
+	{
+		$comment1 = $this->objFromFixture(Comment::class, 'comment1');
+		$comment2 = $this->objFromFixture(Comment::class, 'comment2');
+		$item1 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item1');
+		$item2 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item2');
+
+		$this->assertEquals(['myadmin@mysite.com', 'andrew@address.com'], $item1->notificationRecipients($comment1));
+		$this->assertEquals(['myadmin@mysite.com'], $item2->notificationRecipients($comment2));
+	}
+
+	public function testNotificationSubject()
+	{
+		$recipient = $this->objFromFixture(Member::class, 'author');
+		$comment1 = $this->objFromFixture(Comment::class, 'comment1');
+		$comment2 = $this->objFromFixture(Comment::class, 'comment2');
+		$item1 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item1');
+		$item2 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item2');
+
+		$this->assertEquals('A new comment has been posted', $item1->notificationSubject($comment1, $recipient));
+		$this->assertEquals('A new comment has been posted', $item2->notificationSubject($comment2, $recipient));
+	}
+
+	public function testNotificationSender()
+	{
+		$comment1 = $this->objFromFixture(Comment::class, 'comment1');
+		$author = $this->objFromFixture(Member::class, 'author');
+		$item1 = $this->objFromFixture(CommentNotifiableTestDataObject::class, 'item1');
+
+		$this->assertEquals('noreply@mysite.com', $item1->notificationSender($comment1, $author));
+	}
 }

src/Extensions/CommentNotifiable.php

Indentation +108 added lines, -108 removed lines

@@ -11,112 +11,112 @@
 class CommentNotifiable extends DataExtension
 {
 
-    /**
-     * Default subject line if the owner doesn't override it
-     *
-     * @config
-     * @var string
-     */
-    private static $default_notification_subject = 'A new comment has been posted';
-
-    /**
-     * Default sender
-     *
-     * @config
-     * @var string
-     */
-    private static $default_notification_sender = 'noreply@{host}';
-
-    /**
-     * Default template to use for comment notifications
-     *
-     * @config
-     * @var string
-     */
-    private static $default_notification_template = 'SilverStripe\\CommentNotifications\\CommentEmail';
-
-    /**
-     * Return the list of members or emails to send comment notifications to
-     *
-     * @param Comment $comment
-     * @return array|Traversable
-     */
-    public function notificationRecipients($comment)
-    {
-        $list = [];
-
-        if ($adminEmail = Email::config()->admin_email) {
-            $list[] = $adminEmail;
-        }
-
-        $this->owner->invokeWithExtensions('updateNotificationRecipients', $list, $comment);
-
-        return $list;
-    }
-
-    /**
-     * Gets the email subject line for comment notifications
-     *
-     * @param Comment $comment Comment
-     * @param Member|string $recipient
-     * @return string
-     */
-    public function notificationSubject($comment, $recipient)
-    {
-        $subject = $this->owner->config()->default_notification_subject;
-
-        $this->owner->invokeWithExtensions('updateNotificationSubject', $subject, $comment, $recipient);
-
-        return $subject;
-    }
-
-    /**
-     * Get the sender email address to use for email notifications
-     *
-     * @param Comment $comment
-     * @param Member|string $recipient
-     * @return string
-     */
-    public function notificationSender($comment, $recipient)
-    {
-        $sender = $this->owner->config()->default_notification_sender;
-
-        // Do hostname substitution
-        $host = isset($_SERVER['HTTP_HOST'])
-            ? preg_replace('/^www\./i', '', $_SERVER['HTTP_HOST'])
-            : 'localhost';
-        $sender = preg_replace('/{host}/', $host, $sender);
-
-        $this->owner->invokeWithExtensions('updateNotificationSender', $sender, $comment, $recipient);
-
-        return $sender;
-    }
-
-    /**
-     * Determine the template to use for this email
-     *
-     * @param Comment $comment
-     * @param Member|string $recipient
-     * @return string Template name (excluding .ss extension)
-     */
-    public function notificationTemplate($comment, $recipient)
-    {
-        $template = $this->owner->config()->default_notification_template;
-
-        $this->owner->invokeWithExtensions('updateNotificationTemplate', $template, $comment, $recipient);
-
-        return $template;
-    }
-
-    /**
-     * Update the notification email
-     *
-     * @param Email $email
-     * @param Comment $comment
-     * @param Member|string $recipient
-     */
-    public function updateCommentNotification($email, $comment, $recipient)
-    {
-        //
-    }
+	/**
+	 * Default subject line if the owner doesn't override it
+	 *
+	 * @config
+	 * @var string
+	 */
+	private static $default_notification_subject = 'A new comment has been posted';
+
+	/**
+	 * Default sender
+	 *
+	 * @config
+	 * @var string
+	 */
+	private static $default_notification_sender = 'noreply@{host}';
+
+	/**
+	 * Default template to use for comment notifications
+	 *
+	 * @config
+	 * @var string
+	 */
+	private static $default_notification_template = 'SilverStripe\\CommentNotifications\\CommentEmail';
+
+	/**
+	 * Return the list of members or emails to send comment notifications to
+	 *
+	 * @param Comment $comment
+	 * @return array|Traversable
+	 */
+	public function notificationRecipients($comment)
+	{
+		$list = [];
+
+		if ($adminEmail = Email::config()->admin_email) {
+			$list[] = $adminEmail;
+		}
+
+		$this->owner->invokeWithExtensions('updateNotificationRecipients', $list, $comment);
+
+		return $list;
+	}
+
+	/**
+	 * Gets the email subject line for comment notifications
+	 *
+	 * @param Comment $comment Comment
+	 * @param Member|string $recipient
+	 * @return string
+	 */
+	public function notificationSubject($comment, $recipient)
+	{
+		$subject = $this->owner->config()->default_notification_subject;
+
+		$this->owner->invokeWithExtensions('updateNotificationSubject', $subject, $comment, $recipient);
+
+		return $subject;
+	}
+
+	/**
+	 * Get the sender email address to use for email notifications
+	 *
+	 * @param Comment $comment
+	 * @param Member|string $recipient
+	 * @return string
+	 */
+	public function notificationSender($comment, $recipient)
+	{
+		$sender = $this->owner->config()->default_notification_sender;
+
+		// Do hostname substitution
+		$host = isset($_SERVER['HTTP_HOST'])
+			? preg_replace('/^www\./i', '', $_SERVER['HTTP_HOST'])
+			: 'localhost';
+		$sender = preg_replace('/{host}/', $host, $sender);
+
+		$this->owner->invokeWithExtensions('updateNotificationSender', $sender, $comment, $recipient);
+
+		return $sender;
+	}
+
+	/**
+	 * Determine the template to use for this email
+	 *
+	 * @param Comment $comment
+	 * @param Member|string $recipient
+	 * @return string Template name (excluding .ss extension)
+	 */
+	public function notificationTemplate($comment, $recipient)
+	{
+		$template = $this->owner->config()->default_notification_template;
+
+		$this->owner->invokeWithExtensions('updateNotificationTemplate', $template, $comment, $recipient);
+
+		return $template;
+	}
+
+	/**
+	 * Update the notification email
+	 *
+	 * @param Email $email
+	 * @param Comment $comment
+	 * @param Member|string $recipient
+	 */
+	public function updateCommentNotification($email, $comment, $recipient)
+	{
+		//
+	}
 }

src/Extensions/CommentNotifier.php

Indentation +65 added lines, -65 removed lines

@@ -15,79 +15,79 @@
 class CommentNotifier extends Extension
 {
 
-    /**
-     * Notify Members of the post there is a new comment.
-     *
-     * @param Comment $comment
-     */
-    public function onAfterPostComment(Comment $comment)
-    {
-        $parent = $comment->Parent();
+	/**
+	 * Notify Members of the post there is a new comment.
+	 *
+	 * @param Comment $comment
+	 */
+	public function onAfterPostComment(Comment $comment)
+	{
+		$parent = $comment->Parent();
 
-        if (!$parent || !$parent->hasMethod('notificationRecipients')) {
-            return;
-        }
+		if (!$parent || !$parent->hasMethod('notificationRecipients')) {
+			return;
+		}
 
-        // Ask parent to submit all recipients
-        $recipients = $parent->notificationRecipients($comment);
+		// Ask parent to submit all recipients
+		$recipients = $parent->notificationRecipients($comment);
 
-        foreach ($recipients as $recipient) {
-            $this->notifyCommentRecipient($comment, $parent, $recipient);
-        }
-    }
+		foreach ($recipients as $recipient) {
+			$this->notifyCommentRecipient($comment, $parent, $recipient);
+		}
+	}
 
-    /**
-     * Send comment notification to a given recipient
-     *
-     * @param Comment $comment
-     * @param DataObject $parent Object with the {@see CommentNotifiable} extension applied
-     * @param Member|string $recipient Either a member object or an email address to which notifications should be sent
-     */
-    public function notifyCommentRecipient($comment, $parent, $recipient)
-    {
-        $subject = $parent->notificationSubject($comment, $recipient);
-        $sender = $parent->notificationSender($comment, $recipient);
-        $template = $parent->notificationTemplate($comment, $recipient);
+	/**
+	 * Send comment notification to a given recipient
+	 *
+	 * @param Comment $comment
+	 * @param DataObject $parent Object with the {@see CommentNotifiable} extension applied
+	 * @param Member|string $recipient Either a member object or an email address to which notifications should be sent
+	 */
+	public function notifyCommentRecipient($comment, $parent, $recipient)
+	{
+		$subject = $parent->notificationSubject($comment, $recipient);
+		$sender = $parent->notificationSender($comment, $recipient);
+		$template = $parent->notificationTemplate($comment, $recipient);
 
-        // Validate email
-        // Important in case of the owner being a default-admin or a username with no contact email
-        $to = ($recipient instanceof Member) ? $recipient->Email : $recipient;
+		// Validate email
+		// Important in case of the owner being a default-admin or a username with no contact email
+		$to = ($recipient instanceof Member) ? $recipient->Email : $recipient;
 
-        if (!Email::is_valid_address($to)) {
-            return;
-        }
+		if (!Email::is_valid_address($to)) {
+			return;
+		}
 
-        // Prepare the email
-        $email = Email::create();
-        $email->setSubject($subject);
-        $email->setFrom($sender);
-        $email->setTo($to);
-        $email->setHTMLTemplate($template);
+		// Prepare the email
+		$email = Email::create();
+		$email->setSubject($subject);
+		$email->setFrom($sender);
+		$email->setTo($to);
+		$email->setHTMLTemplate($template);
 
-        if ($recipient instanceof Member) {
-            $email->setData([
-                'Parent' => $parent,
-                'Comment' => $comment,
-                'Recipient' => $recipient,
-                'ApproveLink' => $comment->ApproveLink($recipient),
-                'HamLink' => $comment->HamLink($recipient),
-                'SpamLink' => $comment->SpamLink($recipient),
-                'DeleteLink' => $comment->DeleteLink($recipient),
-            ]);
-        } else {
-            $email->setData([
-                'Parent' => $parent,
-                'Comment' => $comment,
-                'ApproveLink' => false,
-                'SpamLink' => false,
-                'DeleteLink' => false,
-                'HamLink' => false,
-                'Recipient' => $recipient
-            ]);
-        }
+		if ($recipient instanceof Member) {
+			$email->setData([
+				'Parent' => $parent,
+				'Comment' => $comment,
+				'Recipient' => $recipient,
+				'ApproveLink' => $comment->ApproveLink($recipient),
+				'HamLink' => $comment->HamLink($recipient),
+				'SpamLink' => $comment->SpamLink($recipient),
+				'DeleteLink' => $comment->DeleteLink($recipient),
+			]);
+		} else {
+			$email->setData([
+				'Parent' => $parent,
+				'Comment' => $comment,
+				'ApproveLink' => false,
+				'SpamLink' => false,
+				'DeleteLink' => false,
+				'HamLink' => false,
+				'Recipient' => $recipient
+			]);
+		}
 
-        $this->owner->invokeWithExtensions('updateCommentNotification', $email, $comment, $recipient);
+		$this->owner->invokeWithExtensions('updateCommentNotification', $email, $comment, $recipient);
 
-        return $email->send();
-    }
+		return $email->send();
+	}
 }