PDFUploadField::__construct() - Code Metrics - sunnysideup/silverstripe-pdf_upload_field - Measure and Improve Code Quality continuously with Scrutinizer

PDFUploadField::__construct() B
last analyzed 2018-03-20 01:26 UTC

↳ Parent: PDFUploadField

Complexity

Conditions	2
Paths	2

Size

Total Lines	24
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	24
rs	8.9713
c	0
b	0
f	0
cc	2
eloc	17
nc	2
nop	4

<?php

/**
 * allow a pdf to be uploaded...
 *
 * Usage:
 *     $field = PDFUploadField::create(
 *         "ReportField",
 *         "Title",
 *         null,
 *         "folder-name"
 *     );
 */
class PDFUploadField extends UploadField
namespace YourVendor;

class YourClass { }
{
    private static $pdf_folder_name = "pdfs";

    /**
     * @param string  $name
     * @param string  $title
     * @param SS_List $items If no items are defined, the field will try to auto-detect an existing relation on

     *
     *                       @link $record}, with the same name as the field name.
     *
     * @param string $folderName
     *
     * @return ReportUploadField

     */
    public function __construct(
        $name,
        $title,
        SS_List $items = null,
        $folderName = ""
    ) {
        parent::__construct(
            $name,
            $title,
            $items
        );
        $this->setRightTitle('Only PDF files are accepted.');
        //create folder
        if (!$folderName) {
            $folderName = Config::inst()->get('PDFUploadField', 'pdf_folder_name');
        }
        Folder::find_or_make(Director::baseFolder()."/assets/".$folderName);
        //set folder
        $this->setFolderName($folderName);
        $this->setAllowedExtensions(array('pdf'));
        $this->setAllowedMaxFileNumber(1);

        return $this;

    }
}


1			<?php
2
3			/**
4			* allow a pdf to be uploaded...
5			*
6			* Usage:
7			* $field = PDFUploadField::create(
8			* "ReportField",
9			* "Title",
10			* null,
11			* "folder-name"
12			* );
13			*/
14			class PDFUploadField extends UploadField
			0 ignored issues – show Coding Style Compatibility introduced 2017-01-03 01:28 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...
15			{
16			private static $pdf_folder_name = "pdfs";
			0 ignored issues – show Unused Code introduced 2017-01-03 01:28 UTC by Report Bug Copy Issue Report The property `$pdf_folder_name` 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...
17			/**
18			* @param string $name
19			* @param string $title
20			* @param SS_List $items If no items are defined, the field will try to auto-detect an existing relation on
			0 ignored issues – show Documentation introduced 2017-01-03 01:28 UTC by Report Bug Copy Issue Report Should the type for parameter `$items` not be `null\|SS_List`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
21			*
22			* @link $record}, with the same name as the field name.
23			*
24			* @param string $folderName
25			*
26			* @return ReportUploadField
			0 ignored issues – show Comprehensibility Best Practice introduced 2017-01-03 01:28 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
27			*/
28			public function __construct(
29			$name,
30			$title,
31			SS_List $items = null,
32			$folderName = ""
33			) {
34			parent::__construct(
35			$name,
36			$title,
37			$items
38			);
39			$this->setRightTitle('Only PDF files are accepted.');
40			//create folder
41			if (!$folderName) {
42			$folderName = Config::inst()->get('PDFUploadField', 'pdf_folder_name');
43			}
44			Folder::find_or_make(Director::baseFolder()."/assets/".$folderName);
45			//set folder
46			$this->setFolderName($folderName);
47			$this->setAllowedExtensions(array('pdf'));
48			$this->setAllowedMaxFileNumber(1);
49
50			return $this;
			0 ignored issues – show Bug introduced 2017-01-03 01:28 UTC by Report Bug Copy Issue Report Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct? Loading history...
51			}
52			}
53

sunnysideup / silverstripe-pdf_upload_field

PDFUploadField::__construct() B last analyzed 2018-03-20 01:26 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

PDFUploadField::__construct() B
last analyzed 2018-03-20 01:26 UTC