BlogPostNotifications::updateNotificationRecipients() - Code Metrics - silverstripe/silverstripe-blog - Measure and Improve Code Quality continuously with Scrutinizer

updateNotificationRecipients() A
last analyzed 2021-05-14 09:33 UTC

↳ Parent: BlogPostNotifications

Complexity

Conditions	3
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	10
rs	9.9332
c	0
b	0
f	0
cc	3
nc	2
nop	2

<?php

namespace SilverStripe\Blog\Model;

use SilverStripe\Comments\Model\Comment;
use SilverStripe\Core\Config\Config;
use SilverStripe\ORM\DataExtension;
use SilverStripe\ORM\SS_List;
use SilverStripe\Security\Member;

/**
 * Customise blog post to support comment notifications.
 *
 * Extends {@see BlogPost} with extensions to {@see CommentNotifiable}.
 */
class BlogPostNotifications extends DataExtension
{
    /**
     * Configure whether to send notifications even for spam comments
     *
     * @config
     * @var boolean
     */
    private static $notification_on_spam = true;

    /**
     * Notify all authors of notifications.
     *
     * @param SS_List $list
     * @param mixed $comment
     */
    public function updateNotificationRecipients(&$list, &$comment)
    {
        //default is notification is on regardless of spam status
        $list = $this->owner->Authors();

        // If comment is spam and notification are set to not send on spam clear the recipient list
        if (Config::inst()->get(__CLASS__, 'notification_on_spam') == false && $comment->IsSpam) {
            $list = [];
        }
    }

    /**
     * Update comment to include the page title.
     *
     * @param string $subject
     * @param Comment $comment
     * @param Member|string $recipient
     */
    public function updateNotificationSubject(&$subject, &$comment, &$recipient)

    {
        $subject = sprintf('A new comment has been posted on %s', $this->owner->Title);
    }
}


1			<?php
2
3			namespace SilverStripe\Blog\Model;
4
5			use SilverStripe\Comments\Model\Comment;
6			use SilverStripe\Core\Config\Config;
7			use SilverStripe\ORM\DataExtension;
8			use SilverStripe\ORM\SS_List;
9			use SilverStripe\Security\Member;
10
11			/**
12			* Customise blog post to support comment notifications.
13			*
14			* Extends {@see BlogPost} with extensions to {@see CommentNotifiable}.
15			*/
16			class BlogPostNotifications extends DataExtension
17			{
18			/**
19			* Configure whether to send notifications even for spam comments
20			*
21			* @config
22			* @var boolean
23			*/
24			private static $notification_on_spam = true;
25
26			/**
27			* Notify all authors of notifications.
28			*
29			* @param SS_List $list
30			* @param mixed $comment
31			*/
32			public function updateNotificationRecipients(&$list, &$comment)
33			{
34			//default is notification is on regardless of spam status
35			$list = $this->owner->Authors();
36
37			// If comment is spam and notification are set to not send on spam clear the recipient list
38			if (Config::inst()->get(__CLASS__, 'notification_on_spam') == false && $comment->IsSpam) {
39			$list = [];
40			}
41			}
42
43			/**
44			* Update comment to include the page title.
45			*
46			* @param string $subject
47			* @param Comment $comment
48			* @param Member\|string $recipient
49			*/
50			public function updateNotificationSubject(&$subject, &$comment, &$recipient)
			0 ignored issues – show Unused Code introduced 2017-08-24 22:02 UTC by Report Bug Copy Issue Report The parameter `$comment` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2017-08-24 22:02 UTC by Report Bug Copy Issue Report The parameter `$recipient` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
51			{
52			$subject = sprintf('A new comment has been posted on %s', $this->owner->Title);
53			}
54			}
55

silverstripe / silverstripe-blog

updateNotificationRecipients() A last analyzed 2021-05-14 09:33 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

updateNotificationRecipients() A
last analyzed 2021-05-14 09:33 UTC