UserDefinedForm_EmailRecipient::getCMSFields() - Code Metrics - Inspection of "BUG Fix can* permissions on non-top level objects." - silverstripe/silverstripe-userforms - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Pull Request — master (#441)

by Damian
created 2016-03-14 01:02 UTC
UserDefinedForm_EmailRecipient::getCMSFields() B

↳ Parent: UserDefinedForm_EmailRecipient
Complexity

Conditions	2
Paths	2
Size

Total Lines	152
Code Lines	101
Duplication

Lines	0
Ratio	0 %
Metric	Value
dl	0
loc	152
rs	8.2857
cc	2
eloc	101
nc	2
nop	0
How to fix Long Method

<?php


/**
 * A Form can have multiply members / emails to email the submission
 * to and custom subjects
 *
 * @package userforms
 */
class UserDefinedForm_EmailRecipient extends DataObject {
namespace YourVendor;

class YourClass { }

	private static $db = array(

		'EmailAddress' => 'Varchar(200)',
		'EmailSubject' => 'Varchar(200)',
		'EmailFrom' => 'Varchar(200)',
		'EmailReplyTo' => 'Varchar(200)',
		'EmailBody' => 'Text',
		'EmailBodyHtml' => 'HTMLText',
		'EmailTemplate' => 'Varchar',
		'SendPlain' => 'Boolean',
		'HideFormData' => 'Boolean',
		'CustomRulesCondition' => 'Enum("And,Or")'
	);

	private static $has_one = array(

		'Form' => 'UserDefinedForm',
		'SendEmailFromField' => 'EditableFormField',
		'SendEmailToField' => 'EditableFormField',
		'SendEmailSubjectField' => 'EditableFormField'
	);

	private static $has_many = array(

		'CustomRules' => 'UserDefinedForm_EmailRecipientCondition'
	);

	private static $summary_fields = array(

		'EmailAddress',
		'EmailSubject',
		'EmailFrom'
	);

	public function summaryFields() {
		$fields = parent::summaryFields();
		if(isset($fields['EmailAddress'])) {
			$fields['EmailAddress'] = _t('UserDefinedForm.EMAILADDRESS', 'Email');
		}
		if(isset($fields['EmailSubject'])) {
			$fields['EmailSubject'] = _t('UserDefinedForm.EMAILSUBJECT', 'Subject');
		}
		if(isset($fields['EmailFrom'])) {
			$fields['EmailFrom'] = _t('UserDefinedForm.EMAILFROM', 'From');
		}
		return $fields;
	}

	/**
	 * Get instance of UserDefinedForm when editing in getCMSFields
	 *
	 * @return UserDefinedFrom
	 */
	protected function getFormParent() {
		$formID = $this->FormID
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			? $this->FormID
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			: Session::get('CMSMain.currentPage');
		return UserDefinedForm::get()->byID($formID);
	}

	public function getTitle() {
		if($this->EmailAddress) {
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			return $this->EmailAddress;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
		}
		if($this->EmailSubject) {
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			return $this->EmailSubject;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
		}
		return parent::getTitle();
	}

	/**
	 * Generate a gridfield config for editing filter rules
	 *
	 * @return GridFieldConfig
	 */
	protected function getRulesConfig() {
		$formFields = $this->getFormParent()->Fields();

		$config = GridFieldConfig::create()
			->addComponents(
				new GridFieldButtonRow('before'),
				new GridFieldToolbarHeader(),
				new GridFieldAddNewInlineButton(),
				new GridFieldDeleteAction(),
				$columns = new GridFieldEditableColumns()
			);

		$columns->setDisplayFields(array(
			'ConditionFieldID' => function($record, $column, $grid) use ($formFields) {

				return DropdownField::create($column, false, $formFields->map('ID', 'Title'));
			},
			'ConditionOption' => function($record, $column, $grid) {

				$options = UserDefinedForm_EmailRecipientCondition::config()->condition_options;
				return DropdownField::create($column, false, $options);
			},
			'ConditionValue' => function($record, $column, $grid) {

				return TextField::create($column);
			}
		));

		return $config;
	}

	/**
	 * @return FieldList
	 */
	public function getCMSFields() {
		Requirements::javascript(USERFORMS_DIR . '/javascript/Recipient.js');

		// Determine optional field values
		$form = $this->getFormParent();

		// predefined choices are also candidates
		$multiOptionFields = EditableMultipleOptionField::get()->filter('ParentID', $form->ID);

		// if they have email fields then we could send from it
		$validEmailFromFields = EditableEmailField::get()->filter('ParentID', $form->ID);

		// For the subject, only one-line entry boxes make sense
		$validSubjectFields = ArrayList::create(
			EditableTextField::get()
				->filter('ParentID', $form->ID)
				->exclude('Rows:GreaterThan', 1)
				->toArray()
		);
		$validSubjectFields->merge($multiOptionFields);

		// To address cannot be unbound, so restrict to pre-defined lists
		$validEmailToFields = $multiOptionFields;

		// Build fieldlist
		$fields = FieldList::create(Tabset::create('Root')->addExtraClass('EmailRecipientForm'));

		// Configuration fields
		$fields->addFieldsToTab('Root.EmailDetails', array(
			// Subject
			FieldGroup::create(
				TextField::create('EmailSubject', _t('UserDefinedForm.TYPESUBJECT', 'Type subject'))
					->setAttribute('style', 'min-width: 400px;'),
				DropdownField::create(
					'SendEmailSubjectFieldID',
					_t('UserDefinedForm.SELECTAFIELDTOSETSUBJECT', '.. or select a field to use as the subject'),
					$validSubjectFields->map('ID', 'Title')
				)->setEmptyString('')
			)
				->setTitle(_t('UserDefinedForm.EMAILSUBJECT', 'Email subject')),

			// To
			FieldGroup::create(
				TextField::create('EmailAddress', _t('UserDefinedForm.TYPETO', 'Type to address'))
					->setAttribute('style', 'min-width: 400px;'),
				DropdownField::create(
					'SendEmailToFieldID',
					_t('UserDefinedForm.ORSELECTAFIELDTOUSEASTO', '.. or select a field to use as the to address'),
					$validEmailToFields->map('ID', 'Title')
				)->setEmptyString(' ')
			)
				->setTitle(_t('UserDefinedForm.SENDEMAILTO','Send email to'))
				->setDescription(_t(
					'UserDefinedForm.SENDEMAILTO_DESCRIPTION',
					'You may enter multiple email addresses as a comma separated list.'
				)),


			// From
			TextField::create('EmailFrom', _t('UserDefinedForm.FROMADDRESS','Send email from'))
				->setDescription(_t(
					'UserDefinedForm.EmailFromContent',
					"The from address allows you to set who the email comes from. On most servers this ".
					"will need to be set to an email address on the same domain name as your site. ".
					"For example on yoursite.com the from address may need to be [email protected]. ".
					"You can however, set any email address you wish as the reply to address."
				)),


			// Reply-To
			FieldGroup::create(
				TextField::create('EmailReplyTo', _t('UserDefinedForm.TYPEREPLY', 'Type reply address'))
					->setAttribute('style', 'min-width: 400px;'),
				DropdownField::create(
					'SendEmailFromFieldID',
					_t('UserDefinedForm.ORSELECTAFIELDTOUSEASFROM', '.. or select a field to use as reply to address'),
					$validEmailFromFields->map('ID', 'Title')
				)->setEmptyString(' ')
			)
				->setTitle(_t('UserDefinedForm.REPLYADDRESS', 'Email for reply to'))
				->setDescription(_t(
					'UserDefinedForm.REPLYADDRESS_DESCRIPTION',
					'The email address which the recipient is able to \'reply\' to.'
				))
		));

		// Only show the preview link if the recipient has been saved.
		if (!empty($this->EmailTemplate)) {
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			$preview = sprintf(
				'<p><a href="%s" target="_blank" class="ss-ui-button">%s</a></p><em>%s</em>',
				"admin/pages/edit/EditForm/field/EmailRecipients/item/{$this->ID}/preview",
				_t('UserDefinedForm.PREVIEW_EMAIL', 'Preview email'),
				_t('UserDefinedForm.PREVIEW_EMAIL_DESCRIPTION', 'Note: Unsaved changes will not appear in the preview.')
			);
		} else {
			$preview = sprintf(
				'<em>%s</em>',
				_t(
					'UserDefinedForm.PREVIEW_EMAIL_UNAVAILABLE',
					'You can preview this email once you have saved the Recipient.'
				)
			);
		}

		// Email templates
		$fields->addFieldsToTab('Root.EmailContent', array(
			CheckboxField::create('HideFormData', _t('UserDefinedForm.HIDEFORMDATA', 'Hide form data from email?')),
			CheckboxField::create(
				'SendPlain',
				_t('UserDefinedForm.SENDPLAIN', 'Send email as plain text? (HTML will be stripped)')
			),
			DropdownField::create(
				'EmailTemplate',
				_t('UserDefinedForm.EMAILTEMPLATE', 'Email template'),
				$this->getEmailTemplateDropdownValues()
			)->addExtraClass('toggle-html-only'),
			HTMLEditorField::create('EmailBodyHtml', _t('UserDefinedForm.EMAILBODYHTML','Body'))
				->addExtraClass('toggle-html-only'),
			TextareaField::create('EmailBody', _t('UserDefinedForm.EMAILBODY','Body'))
				->addExtraClass('toggle-plain-only'),
			LiteralField::create(
				'EmailPreview',
				'<div id="EmailPreview" class="field toggle-html-only">' . $preview . '</div>'
			)
		));

		// Custom rules for sending this field
		$grid = new GridField(
			"CustomRules",
			_t('EditableFormField.CUSTOMRULES', 'Custom Rules'),
			$this->CustomRules(),
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
			$this->getRulesConfig()
		);
		$grid->setDescription(_t(
			'UserDefinedForm.RulesDescription',
			'Emails will only be sent to the recipient if the custom rules are met. If no rules are defined, this receipient will receive notifications for every submission.'
		));
		$fields->addFieldsToTab('Root.CustomRules', array(
			new DropdownField(
				'CustomRulesCondition',
				_t('UserDefinedForm.SENDIF', 'Send condition'),
				array(
					'Or' => 'Any conditions are true',
					'And' => 'All conditions are true'
				)
			),
			$grid
		));

		$this->extend('updateCMSFields', $fields);
		return $fields;
	}

	/**
	 * Return whether a user can create an object of this type
	 *
     * @param Member $member
     * @param array $context Virtual parameter to allow context to be passed in to check
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
	 * @return bool
	 */
	public function canCreate($member = null) {

		// Check parent page
        $parent = $this->getCanCreateContext(func_get_args());
        if($parent) {
            return $parent->canEdit($member);
        }

        // Fall back to secure admin permissions
        return parent::canCreate($member);
	}

    /**
     * Helper method to check the parent for this object
     *
     * @param array $args List of arguments passed to canCreate
     * @return SiteTree Parent page instance
     */
    protected function getCanCreateContext($args) {

        // Inspect second parameter to canCreate for a 'Parent' context
        if(isset($args[1]['Form'])) {
            return $args[1]['Form'];
        }
        // Hack in currently edited page if context is missing
        if(Controller::has_curr() && Controller::curr() instanceof CMSMain) {
            return Controller::curr()->currentPage();
        }

        // No page being edited
        return null;
    }

	/**
	 * @param Member
	 *
	 * @return boolean
	 */
	public function canView($member = null) {
		return $this->Form()->canView($member);
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
	}

	/**
	 * @param Member
	 *
	 * @return boolean
	 */
	public function canEdit($member = null) {
		return $this->Form()->canEdit($member);
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
	}

	/**
	 * @param Member
	 *
	 * @return boolean
	 */
	public function canDelete($member = null) {
		return $this->canEdit($member);
	}

	/*
	 * Determine if this recipient may receive notifications for this submission
	 *
	 * @param array $data
	 * @param Form $form
	 * @return bool
	 */
	public function canSend($data, $form) {
		// Skip if no rules configured
		$customRules = $this->CustomRules();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
		if(!$customRules->count()) {
			return true;
		}

		// Check all rules
		$isAnd = $this->CustomRulesCondition === 'And';
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
		foreach($customRules as $customRule) {
			$matches = $customRule->matches($data, $form);
			if($isAnd && !$matches) {
				return false;
			}
			if(!$isAnd && $matches) {
				return true;
			}
		}

		// Once all rules are checked
		return $isAnd;
	}

	/**
	 * Make sure the email template saved against the recipient exists on the file system.
	 *
	 * @param string
	 *
	 * @return boolean
	 */
	public function emailTemplateExists($template = '') {
		$t = ($template ? $template : $this->EmailTemplate);
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

		return in_array($t, $this->getEmailTemplateDropdownValues());
	}

	/**
	 * Get the email body for the current email format
	 *
	 * @return string
	 */
	public function getEmailBodyContent() {
		return $this->SendPlain ? $this->EmailBody : $this->EmailBodyHtml;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
	}

	/**
	 * Gets a list of email templates suitable for populating the email template dropdown.
	 *
	 * @return array
	 */
	public function getEmailTemplateDropdownValues() {
		$templates = array();

		$finder = new SS_FileFinder();
		$finder->setOption('name_regex', '/^.*\.ss$/');

		$found = $finder->find(BASE_PATH . '/' . UserDefinedForm::config()->email_template_directory);

		foreach ($found as $key => $value) {
			$template = pathinfo($value);

			$templates[$template['filename']] = $template['filename'];
		}

		return $templates;
	}
}

silverstripe / silverstripe-userforms

Pull Request — master (#441)

UserDefinedForm_EmailRecipient::getCMSFields() B

Complexity

Size

Duplication

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like
