MAbadir\ElasticLaravel\Console\IndexInitializationTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	handle()	0	7	2
A	confirmPassword()	0	11	2
A	buildIndex()	0	13	3

<?php
/**
 * Created by PhpStorm.
 * User: mina
 * Date: 3/26/17
 * Time: 3:38 PM
 */

namespace MAbadir\ElasticLaravel\Console;

use MAbadir\ElasticLaravel\ElasticClient;

trait IndexInitializationTrait
{
    /**
     * Basic standard configuration
     * @var array
     */
    protected $params = [
        "body" => [
            "settings" => [
                "number_of_shards" => 1,
                "number_of_replicas" => 0,
            ],
        ],
    ];

    /**
     * Holds the ElasticClient
     * @var \MAbadir\ElasticLaravel\ElasticClient
     */
    protected $client;

    /**
     * IndexInitializationTrait constructor.
     *
     * @param \MAbadir\ElasticLaravel\ElasticClient $client
     */
    public function __construct(ElasticClient $client)
    {
        parent::__construct();
        $this->client = $client;
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        if ($this->confirm('Are you sure you want to proceed, this will destroy the Search Index? [y|N]'))

The method confirm() does not exist on MAbadir\ElasticLaravel\Console\IndexInitializationTrait. Did you maybe mean confirmPassword()?

        {
            return $this->confirmPassword();
        }
    }

    /**
     * Confirms user awareness before proceeding
     *
     * @return    mixed
     * @author    Mina Abadir <mina@abadir.email>
     * @copyright Copyright (c) 2016, Mina Abadir
     */
    protected function confirmPassword()
    {
        $password = str_random(7);
        $input = $this->ask('For safety please enter this text "'.$password.'"');

It seems like ask() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if($input == $password){
            return $this->buildIndex();
        }
        else {
            return $this->error('Password is incorrect!');

It seems like error() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }

    /**
     * Deletes, Creates the Index then puts the settings
     *
     * @return    mixed
     * @author    Mina Abadir <mina@abadir.email>
     * @copyright Copyright (c) 2016, Mina Abadir
     */
    protected function buildIndex()
    {
        $this->client->dropIndex();
        $response = $this->client->createIndex($this->params);

        if(is_array($response) && $response['acknowledged'] == true)
        {
            $this->info('Index Initialization Successful');

It seems like info() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
        else{
            $this->error('Index Initialization Failed');

It seems like error() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }
}