Carno\Database\Chips\ParamsBind::binding()

Complexity

Conditions	5
Paths	9

Size

Total Lines	18
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function binding(string $sql, array $params) : string
    {
        // sort by key length (make correct of ":test,:test2")
        uksort($params, static function ($a, $b) {
            return strlen($b) - strlen($a);
        });

        // prepare slots
        foreach ($params as $name => $data) {
            is_scalar($data) && $sql = str_ireplace([":{$name}", "?{$name}"], sprintf($this->pbks, $name), $sql);
        }

        // data replaced
        foreach ($params as $name => $data) {
            is_scalar($data) && $sql = str_ireplace(sprintf($this->pbks, $name), '"'.$this->escape($data).'"', $sql);

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

        }

        return $sql;
    }