FanatPHP

Перед соединением с БД надо написать строчку

mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);

после этого пхп будет писать реальную причину ошибки, а не эту ерунду.

Колупаться, как тут предлагают, с mysqli_error() - это треш и прошлый век.

Подробный пример того как правильно соединяться с БД, с выставлением всех необходимых опций

Не нужно осваивать всё сразу. У тебя не будет работать тоже все сразу. И ты не будешь знать, что именно.

Всегда надо решать только одну задачу за раз

А у тебя тут и ООП, и ПДО, а неймспейсы ,и автолоад, и ни одну из этих вещей ты не понимаешь.

Собрался писать класс для работы с БД? Отлично, пиши класс для работы с БД. БЕЗ неймспейсов и автолоадов. Не переломишься, добавишь один инклюд. Но зато хотя бы не будешь бегать по коду и искать, в каком из 10 мест у тебя ошибка.

Только после того, как класс у тебя за заработает, добавляешь неймспейсы. И мучаешься уже с ними.
После того как заработает с неймспейсами, осваиваешь автолоад.

Свой велосипед, чтобы он был хоть на что-то годен и добавлял хоть что-то к оригинальному ПДО, переписываешь так

class DB
{
    public $pdo;

    public function __construct() {
        $host = "localhost";
        $user = "root";
        $password = "";
        $dbname = "dbtest";
        $charset = "utf8";
        
        $dsn = "mysql:host=$host;dbname=$dbname;charset=$charset";
        $this->opt = [
            PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
            PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
            PDO::ATTR_EMULATE_PREPARES => FALSE
        ];
        $this->pdo = new PDO($dsn, $user, $password, $opt);
    }
    
    public function query($sql, $args = NULL)
    {
        if (!$args)
        {
             return $this->pdo->query($sql);
        }
        $stmt = $this->pdo->prepare($sql);
        $stmt->execute($args);
        return $stmt;
    }
    public function __destruct() {
        $pdo = NULL;
    }
}

Объяснения по коду (на английском) - Детские болезни моего первого класса для работы с БД

Дальше.
Класс Settings не должен наследовать классу БД. Это совершенно разные сущности. Класс человек не должен наследовать классу Карман только потому что у всех людей есть карманы обычно.

Если человеку нужен карман, то карман передается в конструктор добавляется в свойства класса. Поэтому

сlass Settings {
    public function __construct($db) {
        this->db = $db;
    }
    public function getSiteName() {
        $stmt = $this->db->query("SELECT Value FROM configs WHERE Name = 'SiteName'");
        return $stmt->fetchColumn();
    }
}

Данные надо передавать отдельно от запроса

$sql = $conn->prepare("INSERT INTO MyGuests2 (text2) VALUES (?)");
$sql->bind_param("s", $_POST["name"]);
$sql->exeсute();
echo "Успешно созданы новые записи";

И вдогонку, как правильно соединяться

То что написано в ответе выше, плюс

Выяснить, есть ли в сервисе возможность запрашивать сразу много записей.
По возможности кэшировать информацию.

В пресловутом BOM, конечно.

При любых проблемах с "невидимыми" символами надо использовать urlencode()

после того как на месте невидимых символов выведет что-то вроде %EF%BB%BF, тупо заменить проценты на слеши, и вырезать из начала строки

if(substr($str, 0, 3) == "\xEF\xBB\xBF") {
    $str = substr($str, 3);
}

Для начала query() меняем на это

public function query($sql, $params = []) {
    $stmt = $this->db->prepare($sql);
    $stmt->execute($params);
    return $stmt;
}

это не принципиально для ответа на вопрос, но просто чтобы убрать бессмысленный код, который к тому же ухудшает функциональность.

А для получения ответа на вопрос надо научиться правильно соединяться с PDO.
Для этого там, где создается инстанс класса $this->db, пишем вот такой код, в котором самой главной является строчка

PDO::ATTR_ERRMODE            => PDO::ERRMODE_EXCEPTION,

После этого запускаем свой код, и PHP сам, человеческим голосом, сообщает нам, в чем проблема этого кода.
Таким образом мы сразу убиваем двух зайцев:

- во-первых, мы сразу перестаем нуждаться в услугах гадалок с тостера, которые начинают натужно выдумывать, по какой причине этот код может не работать
- во-вторых, получаем точное описание проблемы, а при наличии минимального умения пользоваться гуглом - еще и множество рецептов её решения.

На будущее. Сидеть тупить в код совершенно бессмысленно. проблема может быть где угодно - в другом коде, в настройках БД, в свойствах таблицы, в опечатках, которые в реальном коде есть, а в подредактированном варианте для тостера нету. И еще примерно в миллиарде причин. Все равно все перебрать нереально.

Поэтому никогда не надо уподобляться бабкам-гадалкам, которые пишут "а вы уверены" и "спробуй заячий помет, может поможет", а просто всегда настраивать пхп так, чтобы он сам сообщал, в чем конкретно заключается ошибка.

Это бессмысленный вопрос, и ответа на него не существует.

Чисто процент строк с комментариями можно посчитать примитивным грепом, но следйет отдавать себе отчет, что эта метрика абсолютно бессмысленная, и ничего, кроме вреда, не принесет.

Я правильно понимаю, что из всего этого адского говнокода тебе непонятен только оператор добавления в массив []?

Учебник разорвать, сжечь и выкинуть на помойку.
На будущее, этот код пишется вот так

$data = mysqli_query($link, "SELECT * FROM workers")->fetch_all(MYSQLI_ASSOC);
var_dump($data);

Чтобы было понятно.

Проблема в том, что админ наследует модели, а не использует её как сервис.
И в итоге обращение идет не к Model:query(), а к PDO::query().

Плюс в этом "ООП и MVC" еще примерно 100500 косяков.
Например, этот сайт ляжет, как только выйдет в продакшен, и ты перестанешь быть его единственным посетителем.

Класс Model надо переименовать в DB и сделать нормальный код соединения. Наследников у него быть не должно.
после этого создать единственный экземпляр класса, и передавать его в конструктор всем классам, которым нужна работа с БД.

Для общего развития: Детские болезни моего первого класса для работы с БД (англ.)