PersonalizeHtmlAction::execute() - Code Metrics - Inspection of "add failing tests" - spatie/laravel-email-campaigns - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 153a0b...cff926 )

by Freek

created 2019-10-10 07:46 UTC

PersonalizeHtmlAction::execute() A

↳ Parent: PersonalizeHtmlAction

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.8333
c	0
b	0
f	0
cc	1
nc	1
nop	2

<?php

namespace Spatie\EmailCampaigns\Actions;

use Spatie\EmailCampaigns\Models\Campaign;
use Spatie\EmailCampaigns\Models\CampaignSend;
use Spatie\EmailCampaigns\Http\Controllers\UnsubscribeController;
use Spatie\EmailCampaigns\Models\Subscriber;

class PersonalizeHtmlAction
{
    public function execute($html, CampaignSend $pendingSend)
    {
        $subscription = $pendingSend->subscription;

        $html = str_replace('::campaignSendUuid::', $pendingSend->uuid, $html);
        $html = str_replace('::subscriptionUuid::', $subscription->uuid, $html);
        $html = str_replace('::subscriber.uuid::', $subscription->subscriber->uuid, $html);
        $html = str_replace('::unsubscribeUrl::', action(UnsubscribeController::class, $subscription->uuid), $html);

        $html = $this->replaceSubscriberAttributes($html, $subscription->subscriber);

        return $html;
    }

    protected function replaceSubscriberAttributes(string $html, Subscriber $subscriber): string

    {
        /*
        $html = str_replace('::subscriber.uuid::', $subscriber->uuid, $html);
        $html = str_replace('::subscriber.extra_attributes.first_name::', 'John', $html);
        */

        return $html;
    }
}


1			<?php
2
3			namespace Spatie\EmailCampaigns\Actions;
4
5			use Spatie\EmailCampaigns\Models\Campaign;
6			use Spatie\EmailCampaigns\Models\CampaignSend;
7			use Spatie\EmailCampaigns\Http\Controllers\UnsubscribeController;
8			use Spatie\EmailCampaigns\Models\Subscriber;
9
10			class PersonalizeHtmlAction
11			{
12			public function execute($html, CampaignSend $pendingSend)
13			{
14			$subscription = $pendingSend->subscription;
15
16			$html = str_replace('::campaignSendUuid::', $pendingSend->uuid, $html);
17			$html = str_replace('::subscriptionUuid::', $subscription->uuid, $html);
18			$html = str_replace('::subscriber.uuid::', $subscription->subscriber->uuid, $html);
19			$html = str_replace('::unsubscribeUrl::', action(UnsubscribeController::class, $subscription->uuid), $html);
20
21			$html = $this->replaceSubscriberAttributes($html, $subscription->subscriber);
22
23			return $html;
24			}
25
26			protected function replaceSubscriberAttributes(string $html, Subscriber $subscriber): string
			0 ignored issues – show Unused Code introduced 2019-10-10 07:47 UTC by Report Bug Copy Issue Report The parameter `$subscriber` 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...
27			{
28			/*
29			$html = str_replace('::subscriber.uuid::', $subscriber->uuid, $html);
30			$html = str_replace('::subscriber.extra_attributes.first_name::', 'John', $html);
31			*/
32
33			return $html;
34			}
35			}
36

spatie / laravel-email-campaigns

Push — master ( 153a0b...cff926 )

PersonalizeHtmlAction::execute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like