ContactController

Complexity

Total Complexity

9

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

3 Methods

Rating	Name	Duplication	Size	Complexity
A	set_head_data()	0	8	1
A	set_body_data()	0	7	1
C	process_form()	0	40	7

<?

It is not recommend to use PHP's short opening tag <?, better use <?php, or <?= in case of outputting.

Loader::load('controller', 'home/DefaultPageController');

final class ContactController extends DefaultPageController
{

	private static $TITLE = 'Contact | Jacob Emerick';
	private static $DESCRIPTION = 'Contact page to reach Jacob Emerick by email or social networks';

	private static $KEYWORD_ARRAY = array(
		'Jacob Emerick',
		'jacobemerick',
		'jpemeric',
		'contact',
		'email',
		'connect');

	protected function set_head_data()
	{
		$this->set_title(self::$TITLE);
		$this->set_description(self::$DESCRIPTION);
		$this->set_keywords(self::$KEYWORD_ARRAY);
		
		parent::set_head_data();
	}

	protected function set_body_data()
	{
		$this->set_body('form_container', $this->process_form());
		$this->set_body_view('Contact');
		
		parent::set_body_data();
	}

	private function process_form()
	{
		if(!Request::hasPost() || Request::getPost('submit') != 'Send Message!')
			return (object) array('display' => 'normal');
		
		Loader::load('utility', 'Validate');
		$error_result = array();
		if(!Validate::checkRequest('post', 'name', 'string'))
			$error_result['name'] = 'please enter your name';
		if(!Validate::checkRequest('post', 'email', 'string'))
			$error_result['email'] = 'please enter a valid email';
		if(!Validate::checkRequest('post', 'message', 'string'))
			$error_result['message'] = 'please write a message';
		
		$values = (object) array(
			'name' => Request::getPost('name'),
			'email' => Request::getPost('email'),
			'message' => Request::getPost('message'));
		
		if(count($error_result) > 0)
		{
			return (object) array(
				'display' => 'error',
				'messages' => $error_result,
				'values' => $values);
		}

    global $container;
    $sent = $container['mail']

$sent is not used, you could remove the assignment.

      ->addTo($container['config']->admin_email)
      ->setSubject('Home Page Contact')
      ->setPlainMessage(
        "Name: {$values->name}\n" .
        "Email: {$values->email}\n" .
        "Message: {$values->message}"
      )
      ->send();		

		return (object) array('display' => 'success');
	}

}