ImportExamples::download() - Code Metrics - sunnysideup/silverstripe-cms_tricks_for_apps - Measure and Improve Code Quality continuously with Scrutinizer

ImportExamples::download() A
last analyzed 2018-03-20 00:57 UTC

↳ Parent: ImportExamples

Complexity

Conditions	4
Paths	4

Size

Total Lines	18
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	18
rs	9.2
c	0
b	0
f	0
cc	4
eloc	13
nc	4
nop	1

<?php


class ImportExamples extends ContentController
namespace YourVendor;

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

        "download" => "Admin"
    );

    public function download($request)
    {
        $importerClassName = $request->param("ID");
        $objectClassName = str_replace("Import", "", $importerClassName);
        if (class_exists($objectClassName)) {
            $obj = new $importerClassName($objectClassName);
            if ($obj instanceof CsvBulkLoader) {
                if ($fileData = $obj->getExportExampleData()) {
                    $fileName = $objectClassName.".csv";
                    return SS_HTTPRequest::send_file($fileData, $fileName, 'text/csv');
                } else {
                    user_error("Could not export import example", E_USER_ERROR);
                }
            } else {
                user_error("$obj->class is not an instance of CsvBulkLoader");
            }
        }
    }

    public static function get_link($className)

    {
        return "ImportExamples/download/{$className}Import/";
    }
}


1			<?php
2
3
4			class ImportExamples extends ContentController
			0 ignored issues – show Coding Style Compatibility introduced 2017-01-03 00:26 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:26 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:26 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			"download" => "Admin"
8			);
9
10			public function download($request)
11			{
12			$importerClassName = $request->param("ID");
13			$objectClassName = str_replace("Import", "", $importerClassName);
14			if (class_exists($objectClassName)) {
15			$obj = new $importerClassName($objectClassName);
16			if ($obj instanceof CsvBulkLoader) {
17			if ($fileData = $obj->getExportExampleData()) {
18			$fileName = $objectClassName.".csv";
19			return SS_HTTPRequest::send_file($fileData, $fileName, 'text/csv');
20			} else {
21			user_error("Could not export import example", E_USER_ERROR);
22			}
23			} else {
24			user_error("$obj->class is not an instance of CsvBulkLoader");
25			}
26			}
27			}
28
29			public static function get_link($className)
			0 ignored issues – show Documentation introduced 2017-01-03 00:26 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
30			{
31			return "ImportExamples/download/{$className}Import/";
32			}
33			}
34

sunnysideup / silverstripe-cms_tricks_for_apps

ImportExamples::download() A last analyzed 2018-03-20 00:57 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ImportExamples::download() A
last analyzed 2018-03-20 00:57 UTC