App\Http\Controllers\ClockworkController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	getPage()	0	4	1
A	sendTextMessage()	0	14	2

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Http\Requests;
use MJErwin\Clockwork\ClockworkClient;
use MJErwin\Clockwork\Message;

class ClockworkController extends Controller
{
    /**
     * @var mixed
     */
    protected $apiKey;

    /**
     * @var ClockworkClient
     */
    protected $client;

    const MSG_NUMBER = '07700900123';

    /**
     * Initialize ClockworkController
     */
    public function __construct()
    {
        $this->apiKey = env('CLOCKWORK_API_KEY');
        $this->client = new ClockworkClient($this->apiKey);
        $this->message = new Message();

The property message does not exist. Did you maybe forget to declare it?

        $this->message->setNumber(self::MSG_NUMBER);
        $this->message->setContent(trans('texts.message.sample_body'));

It seems like trans('texts.message.sample_body') targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, MJErwin\Clockwork\Message::setContent() does only seem to accept string, maybe add an additional type check?

    }

    /**
     * Return all data to the Clockwork API dashboard
     */
    public function getPage()
    {
        return view('api.clockwork');
    }

    /**
     * Send a Text Message
     *
     * @param  Request $request
     *
     * @return \Illuminate\Http\RedirectResponse
     */
    public function sendTextMessage(Request $request)
    {
        $this->validate($request, [
            'telephone'  => 'required'
        ]);

        $response = $this->client->sendMessage($this->message);

        if ($response->getMessageId()) {
            return redirect()->back()->with('info', trans('texts.message.sent_success'));
        }

        return redirect()->back()->with('errors', trans('texts.message.sent_failed'));
    }
}