ProductExtension::getRemoveLink() - Code Metrics - Inspection of "SS4 upgrade" - dynamic/silverstripe-product-wishlist - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#2)

by Matthew

created 2018-12-21 19:03 UTC

ProductExtension::getRemoveLink() A

↳ Parent: ProductExtension

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
eloc	10
dl	0
loc	18
ccs	0
cts	11
cp	0
rs	9.9332
c	0
b	0
f	0
cc	3
nc	3
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();
        if ($request->param('Action') != 'view') {
            return false;
        }

        $list = ProductWishList::get()->filter([
            'URLSegment' => $request->param('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			if ($request->param('Action') != 'view') {
50			return false;
51			}
52
53			$list = ProductWishList::get()->filter([
54			'URLSegment' => $request->param('ID'),
55			])->first();
56
57			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...
58			return false;
59			}
60
61			return $list->canEdit(Security::getCurrentUser());
62			}
63			}
64

dynamic / silverstripe-product-wishlist

Pull Request — master (#2)

ProductExtension::getRemoveLink() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like