digitalstars / database Goto Github PK

Оригинальные заполнители PDO (и mysqli) отправляют в БД null, если передано значение null.
И это очень логичное поведение.
Было бы неплохо видеть его и в этой библиотеке.
То, есть, скажем, функцию getValueIntType я бы предложил переписать так

private function getValueIntType($value) {
    if (is_integer($value))
        return $value;
    if (is_numeric($value) || is_bool($value)) {
        return (int)$value;
    }
    if (is_null($value)) {
        return 'null';
    }
    throw new Exception($this->createErrorMessage('integer', $value));
}

Прекрасная библиотека и прекрасное развитие идеи именованных плейсхолдеров.
И я хотел бы рекомендовать её к использованию, но предвижу критику из-за нарушения публичного контракта PDO, а конкретнее - LSP: "Функции, которые используют базовый тип, должны иметь возможность использовать подтипы базового типа, не зная об этом".
То есть взять и подставить DataBase вместо PDO не получится.
С одной стороны это не то чтобы проблема, но с другой все-таки ограничивает обрасть применения.

Нет ли в планах разделить PDO и DataBase, сделав PDO зависимостью для DataBase, применив старое доброе #prefer composition over inheritance#?