byjg / anydataset

src/Database/DBMysqlFunctions.php

<?php

namespace ByJG\AnyDataset\Database;

use ByJG\AnyDataset\Repository\DBDataset;

class DBMySQLFunctions extends DBBaseFunctions
{

    function concat($s1, $s2 = null)

It is generally recommended to explicitly declare the visibility for methods.

    {
        return "concat(" . implode(func_get_args(), ', ') . ")";
    }

    /**
     * Given a SQL returns it with the proper LIMIT or equivalent method included
     * @param string $sql
     * @param int $start
     * @param int $qty
     * @return string
     */
    function limit($sql, $start, $qty)

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        if (strpos($sql, ' LIMIT ') === false) {
            return $sql . " LIMIT $start, $qty ";
        }

        return $sql;
    }

    /**
     * Given a SQL returns it with the proper TOP or equivalent method included
     * @param string $sql
     * @param int $qty
     * @return string
     */
    function top($sql, $qty)

It is generally recommended to explicitly declare the visibility for methods.

    {
        return $this->limit($sql, 0, $qty);
    }

    /**
     * Return if the database provider have a top or similar function
     * @return bool
     */
    function hasTop()

It is generally recommended to explicitly declare the visibility for methods.

    {
        return true;
    }

    /**
     * Return if the database provider have a limit function
     * @return bool
     */
    function hasLimit()

It is generally recommended to explicitly declare the visibility for methods.

    {
        return true;
    }

    /**
     * Format date column in sql string given an input format that understands Y M D
     * @param string $fmt
     * @param string|bool $col
     * @return string
     * @example $db->getDbFunctions()->SQLDate("d/m/Y H:i", "dtcriacao")
     */
    function sqlDate($fmt, $col = false)

It is generally recommended to explicitly declare the visibility for methods.

    {
        if (!$col) $col = 'now()';
        $s = 'DATE_FORMAT(' . $col . ",'";
        $concat = false;
        $len = strlen($fmt);
        for ($i = 0; $i < $len; $i++) {
            $ch = $fmt[$i];
            switch ($ch) {
                case 'Y':
                case 'y':
                    $s .= '%Y';
                    break;
                case 'Q':
                case 'q':
                    $s .= "'),Quarter($col)";

                    if ($len > $i + 1) $s .= ",DATE_FORMAT($col,'";
                    else $s .= ",('";
                    $concat = true;
                    break;
                case 'M':
                    $s .= '%b';
                    break;

                case 'm':
                    $s .= '%m';
                    break;
                case 'D':
                case 'd':
                    $s .= '%d';
                    break;

                case 'H':
                    $s .= '%H';
                    break;

                case 'h':
                    $s .= '%I';
                    break;

                case 'i':
                    $s .= '%i';
                    break;

                case 's':
                    $s .= '%s';
                    break;

                case 'a':
                case 'A':
                    $s .= '%p';
                    break;

                default:

This code seems to be duplicated across your project.

The default body in a switch statement must start on the line following the statement.

                    if ($ch == '\\') {
                        $i++;
                        $ch = substr($fmt, $i, 1);
                    }
                    $s .= $ch;
                    break;
            }
        }
        $s.="')";
        if ($concat) $s = "CONCAT($s)";
        return $s;
    }

    /**
     * Format a string date to a string database readable format.
     *
     * @param string $date
     * @param string $dateFormat
     * @return string
     */
    function toDate($date, $dateFormat)

It is generally recommended to explicitly declare the visibility for methods.

    {
        return parent::toDate($date, $dateFormat);
    }

    /**
     * Format a string database readable format to a string date in a free format.
     *
     * @param string $date
     * @param string $dateFormat
     * @return string
     */
    function fromDate($date, $dateFormat)

It is generally recommended to explicitly declare the visibility for methods.

    {
        return parent::fromDate($date, $dateFormat);
    }

    /**
     *
     * @param DBDataset $dbdataset
     * @param string $sql
     * @param array $param
     * @return int
     */
    function executeAndGetInsertedId($dbdataset, $sql, $param)

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        $id = parent::executeAndGetInsertedId($dbdataset, $sql, $param);
        $it = $dbdataset->getIterator("select LAST_INSERT_ID() id");
        if ($it->hasNext()) {
            $sr = $it->moveNext();
            $id = $sr->getField("id");
        }

        return $id;
    }
}