Дмитрий

"Правильность" зависит от точки зрения, требований, ограничений и прочих особенностей решения. Ваш код решает вашу проблему? Удовлетворяет вашим требованиям, предъявляемым к нему? Значит всё отлично. Если нет - то тут уже надо думать и искать более правильное решение по конкретным критериям конкретного проекта.

Blade в этом плане сложен - он "компилируется" в настоящий php и кэшируется, поэтому реальный стек сложно сопоставить с исходником.

Поэтому надо писать в нём минимум логики, а всю работу производить в контроллере. В шаблоне у вас не должно быть никаких $_GET и уж конечно никаких

$cats=$site->qa("select * from categories where catid=0  order by name asc");

Тогда они будут простыми и ошибку будет сложнее совершить и гораздо легче найти. А если она случится в контроллере, то там уже трейс будет нормальный.
Использовать нормальный фреймворк, но при этом писать в нём код так, как будто на дворе до сих пор PHP 5.2 - это странно.

Вопрос: повлияет ли удаление из кода (нашел в корне сайта в html.php) на работу сайта или нет?

Конечно повлияет: ведь это иконочный шрифт - при его удалении у вас на сайте перестанут работать все иконки. Если вас это устраивает - удаляйте, конечно. Ну или замените другим иконочным шрифтом. Можно убрать из шрифта не используемые иконки. Или использовать векторные иконки на сайте.

я в кодинге и в программировании вообще ничего не понимаю.

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

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

100500 раз отвечено - git

Да. Минимальная задача верстальщика сверстать этот блок. Более опытный (и если есть такая необходимость) может еще сделать js-слайдер.

Теоретически функционал Недавно просмотренные можно реализовать полностью на фронтенде (список просмотренных: названия, цена, картинка хранит в локалстордже), но тогда этого разработчика я не стал бы называться простым верстальщиком. Это уже фронтенд-разработчик/программист/инженер. Да и есть у этого ограничения. Например если цена на товар поменялась, то в браузере клиента будет старая. Или если товар снят с продажи, то он все равно его увидит. Ну и надо понимать, что это совершенно разный объем работ.

В классическом случае этот функционал + интеграцию верстки в шаблон делает бекенд-разработчик.

Самое время познакомиться с темной стороной программирования.
Начинающие вайтишники искренне думают, что программист - это типа такой художник. Берет мольберт, поллитру, кисти и начинает ВАЯТЬ. Потом отходит на шаг, любуется делом рук своих, и снова. Ваять. А потом сразу заказчику, за большие деньги.

Так вот, в реальности это всё не так.
Большую часть времени программист не пишет код.
А пытается разобраться, почему он не работает.

Так что мы будем сейчас учиться это делать.
Тем более, что это в принципе несложно.
Главное не думать, что чем-то поможет сидеть и тупить в свой кодик. И приглашать других людей потупить в него тоже бессмысленно. Потому что причина может быть совсем не в нем. но даже если проблема и в коде, то искать её всё равно надо по-другому.
В код не надо тупить. Его надо ЗАПУСКАТЬ.
И выводить промежуточные результаты. Проверять его работу.
Заранее выяснить, какие должны быть значения у переменных, и проверять их на каждом этапе.
Где не совпадут - там и проблема.
В идеале IDE сама покажет содержание всех переменных при трассировке, но если пишешь код в блокнотике, то даже тупо писать var_dump($bar1,$var2,$var3...); и смотреть что там лежит.
Условия проверять еще проще, тупо echo 'зашли в условие if (!empty($user))';
И если лежит не то, или эхо не выводится - вот тогда уже смотреть в код и думать, почему так получилось.

В частности, при авторизации надо проверить две вещи:
1. Находится ли юзер по логину
2. если находится, то проверить корректность хэша. Для этого при регистрации надо вывести полученный через password_hash пароль и записать на бумажке
потом запросить сохраненный из БД и сравнить

Кроме того
Разумеется, отладка невозможна без сообщений об ошибках.
В половине случаев РНР человеческим голосом сообщает в чем проблема.
Поэтому всегда, в любом окружении должно стоять error_reporting(E_ALL);
плюс на домашнем компике полезно прописать ini_set('display_errors', 1); чтобы сразу видеть ошибки на экране.
На боевом сервере разумеется поставить 0 вместо 1, и добавить ini_set('log_errors', 1);

У меня только один вопрос.
Какой смысл вообще делать парольную защиту, если любой придурок сможет спокойно авторизоваться через SQL инъекцию?

Валидация это обычно что? Простыня) Поэтому свой Form Request. Написать php artisan make:request и потом импортировать в ide это буквально секунды времени. Зато в этом классе ты можешь кучу всего делать. Например prepareForValidation, сообщения, логику со своими правилами и все это будет в одном месте. Запихивать валидацию в middleware или в свои классы в Ларавел - это извращение.

В таких случаях всегда такие вычисления перекладывают на БД, за 1 запрос ты получаешь ответ и возвращаешь его в качестве результата.
В случае, если ты попытаешься получить записи для подсчета в PHP, то на больших выборках велика вероятность получить слишком большой массив данных, который не поместится в память, отведенную под PHP, не говоря уже о времени перебора данных.

phpstorm для основной разработки, любой редактор под руками (vsc, atom, sublime, nano, vim...) для посмотреть/быстрых маленьких правок.

сервер: все (за редким исключением) ставлю из пакетов, затем допиливаю ручками.

phpmyadmin/аналоги–для–других–баз последний раз видел году в 2010. с тех пор использую либо консоль, либо гуевые тулзы (для удаленного доступа через ssh/http–тунель).

докеры, вагранты, "идентичное проду окружение", "сборки – гавно" – это все типичный булшит и базвордс.
иногда перечисленное реально нужно и полезно (сам пользую где надо), но далеко не всегда/везде и не всем.

разрабатываете в одиночку – вообще пофигу на чем, лишь бы вас устраивало и было вам удобно.
разрабатываете в команде – пока ваш код запускается у других участников (что не проблема, если понимать что делаешь/особенности языка под разные платформы/следование принятым соглашениям), всем плевать на чем вы его у себя крутите. хоть стоя в лыжах в гамаке и на утюге.

4 пробела – везде и всегда равны 4 пробелам, а размер таб'а плавает.
при неумелом обращении, особенно при командной разработке, это может привести к неожиданному виду кода.

дело личных привычек/предпочтений и/или договоренности.
в умелых руках проблемы "space vs tab" вообще не существует.

const array = [
  [
    { name: 'A', age: 10 },
    { name: 'B', age: 10 },
    { name: 'A', age: 10 },
    { name: 'B', age: 10 },
  ],
  [
    { name: 'C', age: 10 },
    { name: 'D', age: 10 },
    { name: 'C', age: 10 },
    { name: 'D', age: 10 },
  ],
];

const groupBy = (collection, extractKey) => {
  const cache = new Map();
  
  return collection.reduce((accumulator, entry) => {
    const key = extractKey(entry);
    if (!cache.has(key)) {
      const group = [];

      cache.set(key, group);
      accumulator.push(group);
    }
    cache.get(key).push(entry);

    return accumulator;
  }, []);
};

const newArray = array.map(entry => groupBy(entry, item => item.name));

console.log(newArray);
/*
[
  [
    [ { name: 'A', age: 10 }, { name: 'A', age: 10 } ],
    [ { name: 'B', age: 10 }, { name: 'B', age: 10 } ]
  ],
  [
    [ { name: 'C', age: 10 }, { name: 'C', age: 10 } ],
    [ { name: 'D', age: 10 }, { name: 'D', age: 10 } ]
  ]
]
*/

Также провел тесты, нашел самый шустрый вариант.

const groupBy = (collection, extractKey) => {
  const cache = new Map();
  
  for (const entry of collection) {
    const key = extractKey(entry);
    if (!cache.has(key)) {
      cache.set(key, []);
    }
    cache.get(key).push(entry);
  }
  
  return [...cache.values()];
};

код завязан на браузерные события: переписывать его на пхп нет ни смысла, ни необходимости.

get("/categories/category/destroy/{category}"
href="/categories/category/delete/{{$category->id}}"

Да ну ёпрст же… Попробуйте себя в авиамоделировании.