ProductExtension::canRemoveFromWishList() - 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 17:08 UTC

ProductExtension::canRemoveFromWishList() A

↳ Parent: ProductExtension

Complexity

Conditions	3
Paths	3

Size

Total Lines	16
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	9
dl	0
loc	16
rs	9.9666
c	0
b	0
f	0
cc	3
nc	3
nop	0

<?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');
        if ($this->canRemoveFromWishList()) {
            /** @var \SilverStripe\CMS\Model\SiteTree $page */
            if ($page = $wishListPage::get()->first()) {
                return Controller::join_links(
                    $page->Link('remove'),
                    Controller::curr()->getRequest()->param('ID'),
                    $this->owner->ID
                );
            }
        }

        return false;
    }

    /**
     * @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			if ($this->canRemoveFromWishList()) {
26			/** @var \SilverStripe\CMS\Model\SiteTree $page */
27			if ($page = $wishListPage::get()->first()) {
28			return Controller::join_links(
29			$page->Link('remove'),
30			Controller::curr()->getRequest()->param('ID'),
31			$this->owner->ID
32			);
33			}
34			}
35
36			return false;
37			}
38
39			/**
40			* @return bool
41			*/
42			protected function canRemoveFromWishList()
43			{
44			$request = Controller::curr()->getRequest();
45			if ($request->param('Action') != 'view') {
46			return false;
47			}
48
49			$list = ProductWishList::get()->filter([
50			'URLSegment' => $request->param('ID'),
51			])->first();
52
53			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...
54			return false;
55			}
56
57			return $list->canEdit(Security::getCurrentUser());
58			}
59			}
60

dynamic / silverstripe-product-wishlist

Pull Request — master (#2)

ProductExtension::canRemoveFromWishList() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like