unreal4u\MQTT\Internals\DisconnectCleanup

Complexity

Total Complexity

1

Size/Duplication

Total Lines	14
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	performSpecialActions()	0	7	1

<?php

declare(strict_types=1);

namespace unreal4u\MQTT\Internals;

use unreal4u\MQTT\Client;

/**
 * Performs some cleanup on the socket after disconnecting, this class is NOT a part of the MQTT protocol
 */
final class DisconnectCleanup extends ProtocolBase implements ReadableContentInterface
{
    use ReadableContent;

    /**
     * @inheritdoc
     */
    public function performSpecialActions(Client $client, WritableContentInterface $originalRequest): bool
    {
        $successFullyClosed = stream_socket_shutdown($client->socket, STREAM_SHUT_RDWR);
        $information['successfullyClosed'] = $successFullyClosed;

$information was never initialized. Although not strictly required by PHP, it is generally a good practice to add $information = array(); before regardless.

        $this->logger->info('Sent shutdown signal to socket', $information);
        $client->setConnected(false);
        return true;
    }
}