Spinen\Discourse\Listeners\LogoutDiscourseUser

Complexity

Total Complexity

4

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Test Coverage

Coverage

11.76%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	handle()	0	26	3

<?php

namespace Spinen\Discourse\Listeners;

use GuzzleHttp\Client;
use Illuminate\Auth\Events\Logout;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Support\Facades\Log;

/**
 * Class LogoutDiscourseUser
 *
 * When logging out a Laravel user, send a Logout request to Discourse for that user also.
 *
 * @package Spinen\Discourse\Listeners
 */
class LogoutDiscourseUser implements ShouldQueue
{
    /**
     * @var Client
     */
    public $client;

    /**
     * Create the event listener.
     *
     * @param Client $client
     *
     * @return void
     */
    public function __construct(Client $client)
    {
        $this->client = $client;
    }

    /**
     * Handle the event.
     *
     * @param Logout $event
     *
     * @return void
     */
    public function handle(Logout $event)
    {
        if (!$event->user) {
            return;
        }

        $configs = [
            'base_uri' => config('services.discourse.url'),

The function config was not found. Maybe you did not declare it correctly or list all dependencies?

            'headers'  => [
                'Api-Key'      => config('services.discourse.api.key'),
                'Api-Username' => config('services.discourse.api.user'),
            ],
        ];

        // Get Discourse user to match this one, and send a Logout request to Discourse and get the response
        $user = json_decode(
            $this->client->get("users/by-external/{$event->user->id}.json", $configs)

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

                         ->getBody()
        )->user;

        $response = $this->client->post("admin/users/{$user->id}/log_out");

        if ($response->getStatusCode() !== 200) {
            Log::notice(
                "When logging out user {$event->user->id} Discourse returned status code {$response->getStatusCode()}:",
                ['reason' => $response->getReasonPhrase()]
            );
        }
    }
}