AdvertisementController::removealladvertisements() - Code Metrics - sunnysideup/silverstripe-advertisements - Measure and Improve Code Quality continuously with Scrutinizer

AdvertisementController::removealladvertisements() A
last analyzed 2018-08-23 23:08 UTC

↳ Parent: AdvertisementController

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	15
rs	9.7666
c	0
b	0
f	0
cc	2
nc	2
nop	1

<?php


class AdvertisementController extends Controller
namespace YourVendor;

class YourClass { }
{
    private static $allowed_actions = array(

        "removealladvertisements" => "ADMIN",
        "deletealladvertisements" => "ADMIN"
    );

    public function removealladvertisements($request)
    {
        $id = intval($request->param("ID"))-0;
        $page = SiteTree::get()->byID($id);
        if (!$page) {
            return "this page does not exist";
        }
        DB::query("DELETE FROM SiteTree_Advertisements WHERE SiteTreeID = ".$id);
        DB::query("UPDATE SiteTree SET AdvertisementsFolderID = 0 WHERE SiteTree.ID = ".$id);
        DB::query("UPDATE SiteTree_Live SET AdvertisementsFolderID = 0 WHERE SiteTree_Live.ID = ".$id);
        return sprintf(
            _t("AdvertisementController.REMOVEDALL", 'Removed all %1$s from this page, please reload page to see results.'),
            Config::inst()->get("Advertisement", "plural_name")
        );
    }

    public function deletealladvertisements($request)

    {
        DB::query("DELETE FROM \"Advertisement\"");
        return sprintf(
            _t("AdvertisementController.DELETEDALL", 'Deleted all %1$s from this website, please reload page to see results.'),
            Config::inst()->get("Advertisement", "plural_name")
        );
    }
}


1			<?php
2
3
4			class AdvertisementController extends Controller
			0 ignored issues – show Coding Style Compatibility introduced 2017-01-03 00:27 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
5			{
6			private static $allowed_actions = array(
			0 ignored issues – show Comprehensibility introduced 2017-01-03 00:27 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-01-03 00:27 UTC by Report Bug Copy Issue Report The property `$allowed_actions` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
7			"removealladvertisements" => "ADMIN",
8			"deletealladvertisements" => "ADMIN"
9			);
10
11			public function removealladvertisements($request)
12			{
13			$id = intval($request->param("ID"))-0;
14			$page = SiteTree::get()->byID($id);
15			if (!$page) {
16			return "this page does not exist";
17			}
18			DB::query("DELETE FROM SiteTree_Advertisements WHERE SiteTreeID = ".$id);
19			DB::query("UPDATE SiteTree SET AdvertisementsFolderID = 0 WHERE SiteTree.ID = ".$id);
20			DB::query("UPDATE SiteTree_Live SET AdvertisementsFolderID = 0 WHERE SiteTree_Live.ID = ".$id);
21			return sprintf(
22			_t("AdvertisementController.REMOVEDALL", 'Removed all %1$s from this page, please reload page to see results.'),
23			Config::inst()->get("Advertisement", "plural_name")
24			);
25			}
26
27			public function deletealladvertisements($request)
			0 ignored issues – show Unused Code introduced 2018-02-21 03:47 UTC by Report Bug Copy Issue Report The parameter `$request` 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...
28			{
29			DB::query("DELETE FROM \"Advertisement\"");
30			return sprintf(
31			_t("AdvertisementController.DELETEDALL", 'Deleted all %1$s from this website, please reload page to see results.'),
32			Config::inst()->get("Advertisement", "plural_name")
33			);
34			}
35			}
36

sunnysideup / silverstripe-advertisements

AdvertisementController::removealladvertisements() A last analyzed 2018-08-23 23:08 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

AdvertisementController::removealladvertisements() A
last analyzed 2018-08-23 23:08 UTC