Dynamic\Jobs\Model\JobSubmission

Complexity

Total Complexity

12

Size/Duplication

Total Lines	229
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	fieldLabels()	0	18	1
A	getEditLink()	0	10	1
A	canCreate()	0	3	1
A	getFrontEndFields()	0	30	1
A	getCMSFields()	0	27	1
A	getTitle()	0	3	1
A	getRequiredFields()	0	8	1
A	canEdit()	0	3	1
A	canDelete()	0	3	1
A	getName()	0	6	2
A	canView()	0	3	1

<?php

namespace Dynamic\Jobs\Model;

use Dynamic\Jobs\Admin\JobAdmin;
use Dynamic\Jobs\Forms\SimpleHtmlEditorField;
use Dynamic\Jobs\Page\Job;
use SilverStripe\Assets\File;
use SilverStripe\Control\Controller;
use SilverStripe\Control\Director;
use SilverStripe\Forms\DateField;
use SilverStripe\Forms\EmailField;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\FileField;
use SilverStripe\Forms\ReadonlyField;
use SilverStripe\Forms\RequiredFields;
use SilverStripe\Forms\TextField;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\Permission;

/**
 * Class JobSubmission
 * @package Dynamic\Jobs\Model
 */
class JobSubmission extends DataObject
{
    /**
     * @var string
     */
    private static $table_name = 'Dynamic_JobSubmission';

The private property $table_name is not used, and could be removed.

    /**
     * @var array
     */
    private static $db = [

The private property $db is not used, and could be removed.

        'FirstName' => 'Varchar(255)',
        'LastName' => 'Varchar(255)',
        'Email' => 'Varchar(255)',
        'Phone' => 'Varchar(255)',
        'Available' => 'Date',
        'Content' => 'HTMLText',
    ];

    /**
     * @var array
     */
    private static $has_one = [

The private property $has_one is not used, and could be removed.

        'Job' => Job::class,
        'Resume' => File::class,
    ];

    /**
     * @var string
     */
    private static $default_sort = 'Created DESC';

The private property $default_sort is not used, and could be removed.

    /**
     * @var array
     */
    private static $summary_fields = [

The private property $summary_fields is not used, and could be removed.

        'Name',
        'Job.Title',
        'Created.Nice',
    ];

    /**
     * @var array
     */
    private static $searchable_fields = [

The private property $searchable_fields is not used, and could be removed.

        'FirstName',
        'LastName',
        'Job.ID',
        'Email',
        'Phone',
        'Content',
    ];

    /**
     * @param bool $includerelations
     * @return array
     */
    public function fieldLabels($includerelations = true)
    {
        $labels = parent::fieldLabels($includerelations);

        $labels['Name'] = _t(__CLASS__ . '.NameLabel', 'Applicant');
        $labels['Job.Title'] = _t(__CLASS__ . '.JobLabel', 'Job');
        $labels['Job.ID'] = _t(__CLASS__ . '.JobLabel', 'Job');
        $labels['Created'] = _t(__CLASS__ . '.CreatedLabel', 'Application Date');
        $labels['Created.Nice'] = _t(__CLASS__ . '.CreatedLabel', 'Application Date');
        $labels['FirstName'] = _t(__CLASS__ . '.FirstNameLabel', 'First');
        $labels['LastName'] = _t(__CLASS__ . '.LastNameLabel', 'Last');
        $labels['Email'] = _t(__CLASS__ . '.EmailLabel', 'Email');
        $labels['Phone'] = _t(__CLASS__ . '.PhoneLabel', 'Phone');
        $labels['Available'] = _t(__CLASS__ . '.AvailableLabel', 'Date Available');
        $labels['Resume'] = _t(__CLASS__ . '.ResumeLabel', 'Resume');
        $labels['Content'] = _t(__CLASS__ . '.ContentLabel', 'Cover Letter');

        return $labels;
    }

    /**
     * @return string
     */
    public function getName()
    {
        if ($this->FirstName) {

The property FirstName does not exist on Dynamic\Jobs\Model\JobSubmission. Since you implemented __get, consider adding a @property annotation.

            return $this->FirstName . ' ' . $this->LastName;

The property LastName does not exist on Dynamic\Jobs\Model\JobSubmission. Since you implemented __get, consider adding a @property annotation.

        } else {
            return 'No Name';
        }
    }

    /**
     * @return string
     */
    public function getTitle()
    {
        return $this->getName();
    }

    /**
     * @param null $params

Are you sure the doc-type for parameter $params is correct as it would always require null to be passed?

     * @return FieldList
     */
    public function getFrontEndFields($params = null)
    {
        // Resume Upload
        $ResumeField = FileField::create('Resume')->setTitle('Resume');
        $ResumeField->getValidator()->setAllowedExtensions([
            'pdf',
            'doc',
            'docx',
        ]);
        $ResumeField->setFolderName('Uploads/Resumes');
        $ResumeField->setRelationAutoSetting(false);
        $ResumeField->setAttribute('required', true);

        $fields = FieldList::create(
            TextField::create('FirstName', 'First Name')
                ->setAttribute('required', true),
            TextField::create('LastName', 'Last Name')
                ->setAttribute('required', true),
            EmailField::create('Email')
                ->setAttribute('required', true),
            TextField::create('Phone')
                ->setAttribute('required', true),
            DateField::create('Available', 'Date Available'),
            $ResumeField,
            SimpleHtmlEditorField::create('Content', 'Cover Letter')
        );

        $this->extend('updateFrontEndFields', $fields);

        return $fields;
    }

    /**
     * @return RequiredFields
     */
    public function getRequiredFields()
    {
        return new RequiredFields([
            'FirstName',
            'LastName',
            'Email',
            'Phone',
            'Resume',
        ]);
    }

    /**
     * @return FieldList
     */
    public function getCMSFields()
    {
        $fields = parent::getCMSFields();

        $fields->removeByName([
            'JobID',
        ]);

        $fields->insertBefore(
            ReadonlyField::create('JobTitle', $this->fieldLabel('Job.Title'), $this->Job()->getTitle()),

The method Job() does not exist on Dynamic\Jobs\Model\JobSubmission. Since you implemented __call, consider adding a @method annotation.

            'Content'

'Content' of type string is incompatible with the type SilverStripe\Forms\FormField expected by parameter $item of SilverStripe\Forms\FieldList::insertBefore().

        );

        $fields->insertBefore(
            ReadonlyField::create(
                'Created',
                $this->fieldLabel('Created'),
                $this->dbObject('Created')->FormatFromSettings()
            ),
            'Content'
        );

        $resume = $fields->dataFieldByName('Resume')
            ->setFolderName('Uploads/Resumes');
        $fields->insertBefore($resume, 'Content');

        return $fields;
    }

    public function getEditLink()
    {
        $link = Controller::join_links(
            Director::absoluteBaseURL(),
            singleton(JobAdmin::class)->Link(),
            'Dynamic-Jobs-Model-JobSubmission/EditForm/field/Dynamic-Jobs-Model-JobSubmission/item/' .
                $this->ID . '/edit'
        );

        return $link;
    }

    /**
     * @param null $member

Are you sure the doc-type for parameter $member is correct as it would always require null to be passed?

     *
     * @return bool|int
     */
    public function canEdit($member = null)
    {
        return Permission::check('JOB_MANAGE', 'any', $member);
    }

    /**
     * @param null $member

Are you sure the doc-type for parameter $member is correct as it would always require null to be passed?

     *
     * @return bool|int
     */
    public function canDelete($member = null)
    {
        return Permission::check('JOB_MANAGE', 'any', $member);
    }

    /**
     * @param null $member

Are you sure the doc-type for parameter $member is correct as it would always require null to be passed?

     *
     * @return bool|int
     */
    public function canCreate($member = null, $contect = [])
    {
        return Permission::check('JOB_MANAGE', 'any', $member);
    }

    /**
     * @param null $member

Are you sure the doc-type for parameter $member is correct as it would always require null to be passed?

     *
     * @return bool
     */
    public function canView($member = null)
    {
        return Permission::check('JOB_MANAGE', 'any', $member);
    }
}