При чём здесь условие на поле name в общем случае?
Это конечно такой хитрый лайфхак, использование поля not null для имитации джойна при использовании лефт джойна, но в общем случае это не работает.

А я предложил его убрать?

Сомневаюсь, что у деды все аттачи в посте имеют одинаковые наименования.

Я уже работать уйду.

Понятия не имею, я такие запросы не пишу.

Он прав, можно, но не только лефтами и было это давно.
Впрочем сейчас тоже есть возможность включить сию заебатую возможность используя хинт.
ЗЫ сцуко, как же меня заебал этот хинт во всех подряд запросах, даже если там всего 2 таблицы соединяются.

В моём случае это наследие от которого я избавляюсь, где встречаю.
Пока ни один ревьювер не завернул конкретно это исправление.

Ещё не очень понятно, зачем значение для user_id передавать в апострофах? Если это число, то и передавать его числом.

Может и так.
Но мне кажется это базя просто перестраховался или пример неправильно понял.
Скорее всего для id дед использовал автоинкрементное поле, но оно вряд ли строковое.