Newsletter::createCampaign() - Code Metrics - Inspection of "Add default subscribe status config variable" - spatie/laravel-newsletter - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#30)

unknown

created 2016-05-04 16:35 UTC

Newsletter::createCampaign() B

↳ Parent: Newsletter

Complexity

Conditions	4
Paths	4

Size

Total Lines	34
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	34
rs	8.5806
cc	4
eloc	19
nc	4
nop	7

<?php

namespace Spatie\Newsletter;

use DrewM\MailChimp\MailChimp;
use Spatie\Newsletter\Exceptions\InvalidSubscribeStatus;

class Newsletter
{
    /** @var \DrewM\MailChimp\MailChimp */
    protected $mailChimp;

    /** * @var \Spatie\Newsletter\NewsletterListCollection */
    protected $lists;

    /**
     * @param \DrewM\MailChimp\MailChimp                  $mailChimp
     * @param \Spatie\Newsletter\NewsletterListCollection $lists
     */
    public function __construct(MailChimp $mailChimp, NewsletterListCollection $lists)
    {
        $this->mailChimp = $mailChimp;

        $this->lists = $lists;
    }

    /**
     * @param string $email
     * @param array  $mergeFields
     * @param string $listName
     * @param array  $options
     *
     * @return array|bool
     *
     * @throws \Spatie\Newsletter\Exceptions\InvalidNewsletterList
     */
    public function subscribe($email, $mergeFields = [], $listName = '', $options = [])
    {
        $list = $this->lists->findByName($listName);

        $defaultOptions = [
            'email_address' => $email,
            'status' => $this->getDefaultSubscribeStatus(),
            'email_type' => 'html',
        ];

        if (count($mergeFields)) {
            $defaultOptions['merge_fields'] = $mergeFields;
        }

        $options = array_merge($defaultOptions, $options);

        $response = $this->mailChimp->post("lists/{$list->getId()}/members", $options);

        if (!$this->lastActionSucceeded()) {
            return false;
        }

        return $response;
    }

    /**
     * @param string $email
     * @param string $listName
     *
     * @return array|bool
     *
     * @throws \Spatie\Newsletter\Exceptions\InvalidNewsletterList
     */
    public function getMember($email, $listName = '')
    {
        $list = $this->lists->findByName($listName);

        if (!$this->lastActionSucceeded()) {
            return false;
        }

        return $this->mailChimp->get("lists/{$list->getId()}/members/{$this->getSubscriberHash($email)}");
    }

    /**
     * @param string $email
     * @param string $listName
     *
     * @return bool
     */
    public function hasMember($email, $listName = '')
    {
        $response = $this->getMember($email, $listName);

        if (! isset($response['email_address'])) {
            return false;
        }

        if ($response['email_address'] != $email) {
            return false;
        }

        return true;
    }

    /**
     * @param $email
     * @param string $listName
     *
     * @return array|false
     *
     * @throws \Spatie\Newsletter\Exceptions\InvalidNewsletterList
     */
    public function unsubscribe($email, $listName = '')
    {
        $list = $this->lists->findByName($listName);

        $response = $this->mailChimp->delete("lists/{$list->getId()}/members/{$this->getSubscriberHash($email)}");

        return $response;
    }

    /**
     * @param string $fromName
     * @param string $replyTo
     * @param string $subject
     * @param string $html
     * @param string $listName
     * @param array  $options
     * @param array  $contentOptions
     *
     * @return array|bool
     *
     * @throws \Spatie\Newsletter\Exceptions\InvalidNewsletterList
     */
    public function createCampaign($fromName, $replyTo, $subject, $html = '', $listName = '', $options = [], $contentOptions = [])
    {
        $list = $this->lists->findByName($listName);

        $defaultOptions = [
            'type' => 'regular',
            'recipients' => [
                'list_id' => $list->getId(),
            ],
            'settings' => [
                'subject_line' => $subject,
                'from_name' => $fromName,
                'reply_to' => $replyTo,
            ],
        ];

        $options = array_merge($defaultOptions, $options);

        $response = $this->mailChimp->post('campaigns', $options);

        if (!$this->lastActionSucceeded()) {
            return false;
        }

        if ($html === '') {
            return $response;
        }

        if (!$this->updateContent($response['id'], $html, $contentOptions)) {
            return false;
        }

        return $response;
    }

    public function updateContent($campaignId, $html, $options = [])
    {
        $defaultOptions = compact('html');

        $options = array_merge($defaultOptions, $options);

        $response = $this->mailChimp->put("campaigns/{$campaignId}/content", $options);

        if (!$this->lastActionSucceeded()) {
            return false;
        }

        return $response;
    }

    /**
     * @return \DrewM\MailChimp\MailChimp
     */
    public function getApi()
    {
        return $this->mailChimp;
    }

    /**
     * @return array|false
     */
    public function getLastError()
    {
        return $this->mailChimp->getLastError();
    }

    /**
     * @return bool
     */
    public function lastActionSucceeded()
    {
        return !$this->mailChimp->getLastError();
    }

    /**
     * @param string $email
     *
     * @return string
     */
    protected function getSubscriberHash($email)
    {
        return $this->mailChimp->subscriberHash($email);
    }

    protected function getDefaultSubscribeStatus()
    {
        $defaultSubscribeStatus = $this->app['config']['laravel-newsletter.defaultSubscribeStatus'];
class MyClass { }

$x = new MyClass();
$x->foo = true;

        if (! in_array($defaultSubscribeStatus, ['subscribed', 'pending'])) {
            throw InvalidSubscribeStatus::invalidStatus($defaultSubscribeStatus);
        }

        return $defaultSubscribeStatus;
    }
}


1			<?php
2
3			namespace Spatie\Newsletter;
4
5			use DrewM\MailChimp\MailChimp;
6			use Spatie\Newsletter\Exceptions\InvalidSubscribeStatus;
7
8			class Newsletter
9			{
10			/** @var \DrewM\MailChimp\MailChimp */
11			protected $mailChimp;
12
13			/** * @var \Spatie\Newsletter\NewsletterListCollection */
14			protected $lists;
15
16			/**
17			* @param \DrewM\MailChimp\MailChimp $mailChimp
18			* @param \Spatie\Newsletter\NewsletterListCollection $lists
19			*/
20			public function __construct(MailChimp $mailChimp, NewsletterListCollection $lists)
21			{
22			$this->mailChimp = $mailChimp;
23
24			$this->lists = $lists;
25			}
26
27			/**
28			* @param string $email
29			* @param array $mergeFields
30			* @param string $listName
31			* @param array $options
32			*
33			* @return array\|bool
34			*
35			* @throws \Spatie\Newsletter\Exceptions\InvalidNewsletterList
36			*/
37			public function subscribe($email, $mergeFields = [], $listName = '', $options = [])
38			{
39			$list = $this->lists->findByName($listName);
40
41			$defaultOptions = [
42			'email_address' => $email,
43			'status' => $this->getDefaultSubscribeStatus(),
44			'email_type' => 'html',
45			];
46
47			if (count($mergeFields)) {
48			$defaultOptions['merge_fields'] = $mergeFields;
49			}
50
51			$options = array_merge($defaultOptions, $options);
52
53			$response = $this->mailChimp->post("lists/{$list->getId()}/members", $options);
54
55			if (!$this->lastActionSucceeded()) {
56			return false;
57			}
58
59			return $response;
60			}
61
62			/**
63			* @param string $email
64			* @param string $listName
65			*
66			* @return array\|bool
67			*
68			* @throws \Spatie\Newsletter\Exceptions\InvalidNewsletterList
69			*/
70			public function getMember($email, $listName = '')
71			{
72			$list = $this->lists->findByName($listName);
73
74			if (!$this->lastActionSucceeded()) {
75			return false;
76			}
77
78			return $this->mailChimp->get("lists/{$list->getId()}/members/{$this->getSubscriberHash($email)}");
79			}
80
81			/**
82			* @param string $email
83			* @param string $listName
84			*
85			* @return bool
86			*/
87			public function hasMember($email, $listName = '')
88			{
89			$response = $this->getMember($email, $listName);
90
91			if (! isset($response['email_address'])) {
92			return false;
93			}
94
95			if ($response['email_address'] != $email) {
96			return false;
97			}
98
99			return true;
100			}
101
102			/**
103			* @param $email
104			* @param string $listName
105			*
106			* @return array\|false
107			*
108			* @throws \Spatie\Newsletter\Exceptions\InvalidNewsletterList
109			*/
110			public function unsubscribe($email, $listName = '')
111			{
112			$list = $this->lists->findByName($listName);
113
114			$response = $this->mailChimp->delete("lists/{$list->getId()}/members/{$this->getSubscriberHash($email)}");
115
116			return $response;
117			}
118
119			/**
120			* @param string $fromName
121			* @param string $replyTo
122			* @param string $subject
123			* @param string $html
124			* @param string $listName
125			* @param array $options
126			* @param array $contentOptions
127			*
128			* @return array\|bool
129			*
130			* @throws \Spatie\Newsletter\Exceptions\InvalidNewsletterList
131			*/
132			public function createCampaign($fromName, $replyTo, $subject, $html = '', $listName = '', $options = [], $contentOptions = [])
133			{
134			$list = $this->lists->findByName($listName);
135
136			$defaultOptions = [
137			'type' => 'regular',
138			'recipients' => [
139			'list_id' => $list->getId(),
140			],
141			'settings' => [
142			'subject_line' => $subject,
143			'from_name' => $fromName,
144			'reply_to' => $replyTo,
145			],
146			];
147
148			$options = array_merge($defaultOptions, $options);
149
150			$response = $this->mailChimp->post('campaigns', $options);
151
152			if (!$this->lastActionSucceeded()) {
153			return false;
154			}
155
156			if ($html === '') {
157			return $response;
158			}
159
160			if (!$this->updateContent($response['id'], $html, $contentOptions)) {
161			return false;
162			}
163
164			return $response;
165			}
166
167			public function updateContent($campaignId, $html, $options = [])
168			{
169			$defaultOptions = compact('html');
170
171			$options = array_merge($defaultOptions, $options);
172
173			$response = $this->mailChimp->put("campaigns/{$campaignId}/content", $options);
174
175			if (!$this->lastActionSucceeded()) {
176			return false;
177			}
178
179			return $response;
180			}
181
182			/**
183			* @return \DrewM\MailChimp\MailChimp
184			*/
185			public function getApi()
186			{
187			return $this->mailChimp;
188			}
189
190			/**
191			* @return array\|false
192			*/
193			public function getLastError()
194			{
195			return $this->mailChimp->getLastError();
196			}
197
198			/**
199			* @return bool
200			*/
201			public function lastActionSucceeded()
202			{
203			return !$this->mailChimp->getLastError();
204			}
205
206			/**
207			* @param string $email
208			*
209			* @return string
210			*/
211			protected function getSubscriberHash($email)
212			{
213			return $this->mailChimp->subscriberHash($email);
214			}
215
216			protected function getDefaultSubscribeStatus()
217			{
218			$defaultSubscribeStatus = $this->app['config']['laravel-newsletter.defaultSubscribeStatus'];
			0 ignored issues – show Bug introduced 2016-05-04 16:37 UTC by Report Bug Copy Issue Report The property `app` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
219
220			if (! in_array($defaultSubscribeStatus, ['subscribed', 'pending'])) {
221			throw InvalidSubscribeStatus::invalidStatus($defaultSubscribeStatus);
222			}
223
224			return $defaultSubscribeStatus;
225			}
226			}
227

spatie / laravel-newsletter

Pull Request — master (#30)

Newsletter::createCampaign() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like