App\Http\Controllers\ClockworkController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	1
Bugs	0	Features	1

3 Methods

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

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;

use App\Http\Requests;
use App\Http\Controllers\Controller;
use MJErwin\Clockwork\ClockworkClient;
use MJErwin\Clockwork\Message;

class ClockworkController extends Controller
{

    protected $apiKey;
    protected $client;

    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('07700900123');
        $this->message->setContent('Check out this message!');

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

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

        $number = $request->input('number');

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

        $message = 'Testing Clockwork SMS #LaravelHackathonStarter';

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

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

        //return redirect()->back()->with('info','Your Message has been sent successfully');

77% of this comment could be valid code. Did you maybe forget this after debugging?

    }
}