Smile-SA/elasticsuite

Method __construct has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

        DataCheckResultFactory $checkResultFactory,
        Builder $searchRequestBuilder,
        QueryFactory $queryFactory,
        SearchEngineInterface $searchEngine,
        ?DataFixerInterface $dataFixer = null

Argument 5 (query) is null but \Smile\ElasticsuiteCore\Search\Request\Builder::create() takes \Smile\ElasticsuiteCore\Search\Request\QueryInterface|string defined at /code/src/module-elasticsuite-core/Search/Request/Builder.php:136

Open

        return $this->searchRequestBuilder->create($storeId, EventIndexInterface::INDEX_IDENTIFIER, 0, 0, null, [], [], $queryFilters);

• Exclude checks
  • Disable engine
  • Disable check

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public function check($storeId): DataCheckResult
    {
        /** @var DataCheckResult $checkResult */
        $checkResult = $this->checkResultFactory->create([]);
 
 

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

            $this->queryFactory->create(
                QueryInterface::TYPE_BOOL,
                [
                    'should' => [
                        $this->queryFactory->create(QueryInterface::TYPE_MISSING, ['field' => 'session.uid']),

• Read up
  Read up