HasSubscriptions::subscribeTo() - Code Metrics - Inspection of "Merge pull request #1 from sagitarius29/develop" - sagitarius29/laravel-subscriptions - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e68c06...8ce3f9 )

by Adolfo

created 2019-10-29 02:57 UTC

HasSubscriptions::subscribeTo() A

↳ Parent: HasSubscriptions

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	19
rs	9.6333
c	0
b	0
f	0
cc	3
nc	4
nop	1

<?php

namespace Sagitarius29\LaravelSubscriptions\Traits;

use Sagitarius29\LaravelSubscriptions\Entities\Subscription;
use Sagitarius29\LaravelSubscriptions\Contracts\PlanContract;
use Sagitarius29\LaravelSubscriptions\Contracts\SubscriptionContact;

trait HasSubscriptions
{
    public function subscriptions()
    {
        return $this->morphMany(config('subscriptions.entities.plan_subscription'), 'subscriber');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    public function subscribeTo(PlanContract $plan)
    {
        $currentSubscription = $this->getCurrentSubscription();
        $start_at = null;
        $end_at = null;

        if ($currentSubscription == null) {
            $start_at = now();
        }

        if ($plan->isFree()) {
            $end_at = null;
        }

        $subscription = Subscription::make($plan, $start_at, $end_at);
        $subscription = $this->subscriptions()->save($subscription);

        return $subscription;
    }

    public function changePlanTo(PlanContract $plan)

    {
    }

    public function renewSubscription(): bool
    {
    }

    public function cancelSubscription(): bool
    {
    }

    public function getCurrentSubscription(): ?SubscriptionContact
    {
        return $this->subscriptions()
            ->current(now())
            ->first();
    }

    public function getConsumables()
    {
    }
}


1			<?php
2
3			namespace Sagitarius29\LaravelSubscriptions\Traits;
4
5			use Sagitarius29\LaravelSubscriptions\Entities\Subscription;
6			use Sagitarius29\LaravelSubscriptions\Contracts\PlanContract;
7			use Sagitarius29\LaravelSubscriptions\Contracts\SubscriptionContact;
8
9			trait HasSubscriptions
10			{
11			public function subscriptions()
12			{
13			return $this->morphMany(config('subscriptions.entities.plan_subscription'), 'subscriber');
			0 ignored issues – show Bug introduced 2019-10-29 02:10 UTC by Report Bug Copy Issue Report It seems like `morphMany()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
14			}
15
16			public function subscribeTo(PlanContract $plan)
17			{
18			$currentSubscription = $this->getCurrentSubscription();
19			$start_at = null;
20			$end_at = null;
21
22			if ($currentSubscription == null) {
23			$start_at = now();
24			}
25
26			if ($plan->isFree()) {
27			$end_at = null;
28			}
29
30			$subscription = Subscription::make($plan, $start_at, $end_at);
31			$subscription = $this->subscriptions()->save($subscription);
32
33			return $subscription;
34			}
35
36			public function changePlanTo(PlanContract $plan)
			0 ignored issues – show Unused Code introduced 2019-10-29 02:10 UTC by Report Bug Copy Issue Report The parameter `$plan` 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...
37			{
38			}
39
40			public function renewSubscription(): bool
41			{
42			}
43
44			public function cancelSubscription(): bool
45			{
46			}
47
48			public function getCurrentSubscription(): ?SubscriptionContact
49			{
50			return $this->subscriptions()
51			->current(now())
52			->first();
53			}
54
55			public function getConsumables()
56			{
57			}
58			}
59

sagitarius29 / laravel-subscriptions

Push — master ( e68c06...8ce3f9 )

HasSubscriptions::subscribeTo() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like