jeancsil / flight-spy

src/Service/ElasticSearch/Client.php

<?php
/**
 * @author Jean Silva <me@jeancsil.com>
 * @license MIT
 */
namespace Jeancsil\FlightSpy\Service\ElasticSearch;

use Elasticsearch\ClientBuilder;

final class Client

Client does not seem to conform to the naming convention (Utils?$).

The property $ELASTICSEARCH_DEFAULT_HOST is not named in camelCase.

{
    private static $ELASTICSEARCH_DEFAULT_HOST = 'elasticsearch:9200';

$ELASTICSEARCH_DEFAULT_HOST does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    /**
     * @var \Elasticsearch\Client
     */
    private static $instance;

    /**
     * To avoid the direct construction
     */
    private function __construct()
    {
    }

    /**
     * Returns ElasticSearch client object
     * @return \Elasticsearch\Client
     */
    public static function getInstance()
    {
        if (is_null(static::$instance)) {

Since Jeancsil\FlightSpy\Service\ElasticSearch\Client is declared final, using late-static binding will have no effect. You might want to replace static with self instead.

            $logger = ClientBuilder::defaultLogger('flightspy_elasticsearch.log');

            static::$instance = ClientBuilder::create()

Since Jeancsil\FlightSpy\Service\ElasticSearch\Client is declared final, using late-static binding will have no effect. You might want to replace static with self instead.

                ->setHosts([static::$ELASTICSEARCH_DEFAULT_HOST])

Since Jeancsil\FlightSpy\Service\ElasticSearch\Client is declared final, using late-static binding will have no effect. You might want to replace static with self instead.

                ->setRetries(2)
                ->setLogger($logger)
                ->build();
        }

        return static::$instance;

Since Jeancsil\FlightSpy\Service\ElasticSearch\Client is declared final, using late-static binding will have no effect. You might want to replace static with self instead.

    }
}