App\Controller\ContactsController

Complexity

Total Complexity

15

Size/Duplication

Total Lines	115
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	13

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	beforeFilter()	0	6	1
A	owner()	0	14	2
B	user()	0	27	6
B	_contact()	0	37	6

<?php

namespace App\Controller;

use App\Form\ContactForm;
use App\Form\ContactFormOwner;
use Cake\Datasource\Exception\RecordNotFoundException;
use Cake\Event\Event;
use Cake\Form\Form;
use Cake\Http\Exception\BadRequestException;
use Cake\ORM\TableRegistry;
use Saito\Exception\Logger\ExceptionLogger;

class ContactsController extends AppController
{

    /**
     * {@inheritDoc}
     */
    public function beforeFilter(Event $event)
    {
        parent::beforeFilter($event);
        $this->set('showDisclaimer', true);
        $this->Auth->allow('owner');
    }

    /**
     * Contacts forum's owner via contact address
     *
     * @return void
     */
    public function owner()
    {
        $recipient = 'contact';
        if ($this->CurrentUser->isLoggedIn()) {
            $user = $this->CurrentUser;
            $sender = $user->getId();
            $this->request = $this->request->withData('sender_contact', $user->get('user_email'));
        } else {
            $senderContact = $this->request->getData('sender_contact');
            $sender = [$senderContact => $senderContact];
        }

        $this->_contact(new ContactFormOwner(), $recipient, $sender);
    }

    /**
     * Contacts individual user
     *
     * @param string $id user-ID
     * @return void
     * @throws \InvalidArgumentException
     * @throws BadRequestException
     */
    public function user($id = null)
    {
        if (empty($id) || !$this->CurrentUser->isLoggedIn()) {
            throw new BadRequestException();
        }

        $Users = TableRegistry::get('Users');

The method Cake\ORM\TableRegistry::get() has been deprecated with message: 3.6.0 Use \Cake\ORM\Locator\TableLocator::get() instead.

        try {
            $recipient = $Users->get($id);
        } catch (RecordNotFoundException $e) {
            throw new BadRequestException();
        }
        $this->set('user', $recipient);

        if (!$recipient->get('personal_messages')
            && !$this->CurrentUser->permission('saito.core.user.contact')) {
            throw new BadRequestException(null, 1562415010);
        }

        $this->set(
            'titleForPage',
            __('user_contact_title', $recipient->get('username'))
        );

        $sender = $this->CurrentUser->getId();
        $this->_contact(new ContactForm(), $recipient, $sender);
    }

    /**
     *  contact form validating and email sending
     *
     * @param Form $contact contact-form
     * @param mixed $recipient recipient
     * @param mixed $sender sender
     * @return \Cake\Network\Response|void
     */
    protected function _contact(Form $contact, $recipient, $sender)
    {
        if ($this->request->is('get')) {
            if ($this->request->getData('cc') === null) {
                $this->request = $this->request->withData('cc', true);
            }
        }

        if ($this->request->is('post')) {
            $isValid = $contact->validate($this->request->getData());

It seems like $this->request->getData() targeting Cake\Http\ServerRequest::getData() can also be of type null or string; however, Cake\Form\Form::validate() does only seem to accept array, maybe add an additional type check?

            if ($isValid) {
                try {
                    $email = [
                        'recipient' => $recipient,
                        'sender' => $sender,
                        'subject' => $this->request->getData('subject'),
                        'message' => $this->request->getData('text'),
                        'template' => 'user_contact',
                        'ccsender' => (bool)$this->request->getData('cc'),
                    ];
                    $this->SaitoEmail->email($email);
                    $message = __('Message was send.');
                    $this->Flash->set($message, ['element' => 'success']);

                    return $this->redirect('/');
                } catch (\Exception $e) {
                    $Logger = new ExceptionLogger();
                    $Logger->write('Contact email failed', ['e' => $e]);
                    $message = $e->getMessage();
                    $message = __('Message couldn\'t be send: {0}', $message);
                    $this->Flash->set($message, ['element' => 'error']);
                }
            }
        }

        $this->set(compact('contact'));
    }
}