ProductExtension::canRemoveFromWishList() - Code Metrics - dynamic/silverstripe-product-wishlist - Measure and Improve Code Quality continuously with Scrutinizer

ProductExtension::canRemoveFromWishList() A
last analyzed 2019-09-16 16:03 UTC

↳ Parent: ProductExtension

Complexity

Conditions	3
Paths	4

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
eloc	11
c	2
b	0
f	0
dl	0
loc	18
ccs	0
cts	12
cp	0
rs	9.9
cc	3
nc	4
nop	0
crap	12

<?php

namespace Dynamic\Wishlist\Extensions;


use Dynamic\Wishlist\Model\ProductWishList;
use SilverStripe\Control\Controller;
use SilverStripe\Core\Config\Config;
use SilverStripe\Core\Extension;
use SilverStripe\Security\Security;

/**
 * Class ProductExtension
 * @package Dynamic\Wishlist\Extensions
 */
class ProductExtension extends Extension
{

    /**
     * @return bool|String
     */
    public function getRemoveLink()
    {
        $wishListPage = Config::inst()->get(ProductWishList::class, 'listing_page_class');
        /** @var \SilverStripe\CMS\Model\SiteTree $page */
        $page = $page = $wishListPage::get()->first();


        if (!$page) {

            return false;
        }

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

        return Controller::join_links(
            $page->Link('remove'),
            Controller::curr()->getRequest()->param('ID'),
            $this->owner->ID
        );
    }

    /**
     * @return bool
     */
    protected function canRemoveFromWishList()
    {
        $request = Controller::curr()->getRequest();
        $list = ProductWishList::get()->filter([
            'URLSegment' => $request->param('ID'),
        ])->first();

        if (!$list) {

            $list = ProductWishList::get()->filter([
                'MemberID' => Security::getCurrentUser()->ID,
            ])->first();
        }

        if (!$list) {

            return false;
        }

        return $list->canEdit(Security::getCurrentUser());
    }
}


1			<?php
2
3			namespace Dynamic\Wishlist\Extensions;
4
5
6			use Dynamic\Wishlist\Model\ProductWishList;
7			use SilverStripe\Control\Controller;
8			use SilverStripe\Core\Config\Config;
9			use SilverStripe\Core\Extension;
10			use SilverStripe\Security\Security;
11
12			/**
13			* Class ProductExtension
14			* @package Dynamic\Wishlist\Extensions
15			*/
16			class ProductExtension extends Extension
17			{
18
19			/**
20			* @return bool\|String
21			*/
22			public function getRemoveLink()
23			{
24			$wishListPage = Config::inst()->get(ProductWishList::class, 'listing_page_class');
25			/** @var \SilverStripe\CMS\Model\SiteTree $page */
26			$page = $page = $wishListPage::get()->first();
			0 ignored issues – show Unused Code introduced 2018-12-21 19:02 UTC by Report Bug Copy Issue Report The assignment to `$page` is dead and can be removed. Loading history...
27
28			if (!$page) {
			0 ignored issues – show introduced 2018-12-21 19:02 UTC by Report Bug Copy Issue Report `$page` is of type `SilverStripe\CMS\Model\SiteTree`, thus it always evaluated to `true`. Loading history...
29			return false;
30			}
31
32			if (!$this->canRemoveFromWishList()) {
33			return false;
34			}
35
36			return Controller::join_links(
37			$page->Link('remove'),
38			Controller::curr()->getRequest()->param('ID'),
39			$this->owner->ID
40			);
41			}
42
43			/**
44			* @return bool
45			*/
46			protected function canRemoveFromWishList()
47			{
48			$request = Controller::curr()->getRequest();
49			$list = ProductWishList::get()->filter([
50			'URLSegment' => $request->param('ID'),
51			])->first();
52
53			if (!$list) {
			0 ignored issues – show introduced 2019-09-16 16:00 UTC by Report Bug Copy Issue Report `$list` is of type `SilverStripe\ORM\DataObject`, thus it always evaluated to `true`. Loading history...
54			$list = ProductWishList::get()->filter([
55			'MemberID' => Security::getCurrentUser()->ID,
56			])->first();
57			}
58
59			if (!$list) {
			0 ignored issues – show introduced 2018-12-21 17:22 UTC by Report Bug Copy Issue Report `$list` is of type `SilverStripe\ORM\DataObject`, thus it always evaluated to `true`. Loading history...
60			return false;
61			}
62
63			return $list->canEdit(Security::getCurrentUser());
64			}
65			}
66

dynamic / silverstripe-product-wishlist

ProductExtension::canRemoveFromWishList() A last analyzed 2019-09-16 16:03 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

ProductExtension::canRemoveFromWishList() A
last analyzed 2019-09-16 16:03 UTC