Crate\PDO\PDOStatementImplementationPhp8

Complexity

Total Complexity

2

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setFetchMode()	0	3	1
A	fetchAll()	0	3	1

<?php
/**
 * Licensed to CRATE Technology GmbH("Crate") under one or more contributor
 * license agreements.  See the NOTICE file distributed with this work for
 * additional information regarding copyright ownership.  Crate licenses
 * this file to you under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.  You may
 * obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
 * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.  See the
 * License for the specific language governing permissions and limitations
 * under the License.
 *
 * However, if you have executed another commercial license agreement
 * with Crate these terms will supersede the license and you may use the
 * software solely pursuant to the terms of the relevant commercial agreement.
 */

declare(strict_types=1);

namespace Crate\PDO;

/**
 * @internal
 */
trait PDOStatementImplementationPhp8
{
    /**
     * @deprecated Use one of the fetch- or iterate-related methods.
     *
     * @param int   $mode
     * @param mixed ...$args
     *
     * @return bool
     */
    public function setFetchMode($mode, ...$args): bool
    {
        return $this->doSetFetchMode($mode, ...$args);

The method doSetFetchMode() does not exist on Crate\PDO\PDOStatementImplementationPhp8. Did you maybe mean setFetchMode()?

    }

    /**
     * @deprecated Use fetchAllNumeric(), fetchAllAssociative() or fetchFirstColumn() instead.
     *
     * @param int|null $mode
     * @param mixed    ...$args
     *
     * @return mixed[]
     */
    public function fetchAll($mode = null, ...$args): mixed
    {
        return $this->doFetchAll($mode, ...$args);

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

    }
}