Sheer, С точки зрения сферического вакуума, Вы абсолютно правы, как и любая паническая программа валидации (работающая в том же вакууме и созданная людьми неглупыми, но людьми не могущими всё предусмотреть) показывающая обсолютно правильные в большинстве случаев вещи, но есть и исключения. Ибо слава богу мы в вакууме этом не живём, иначе бы задохнулись.
И так аргументирую мои правки, сразу скажу на истину последней инстанции я не претендовал и не претендую. Лишь показал как сделал у себя, может кто то с таким же технологическим складом ума примет те же решения, и с экономит время, любой другой может скачать оригинал, я его не замещаю.
Дано, расширение используемое на ВСЕХ страницах форума кроме админки, и ВСЕМИ пользователями, ботами и гостями. Посему его минимальная ресурсоемкость крайне важна.
Начнём с простого.
Sheer писал(а): Это очень нехорошо, следует все же использовать языковые переменные.
Отсутствие титле, или вообще их присутствие не каждый пользователь у стрелок и заметит, мобильные устройства сразу идут стороной ибо не кажут их, там подвёл палец и перешёл, пользователи стационарных компов тоже не будут долго висеть мышкой читая что там, поэтому эти переменные в данном расширении в подавляющем большинстве случаев (сноска на статистическую погрешность запрещающию говорить 0) не затребованы, и лишь попусту тратят ресурсы серверной части. По идеи эти титле вообще можно стереть, и лишь параноидальная программа валидации заметит их отсутствие (и то заметит ли?). Я писал их для русского форума. поэтому у меня на русском. Но их вообще отсутствие для меня не проблема, или английский язык в них.
Посложней.
Sheer писал(а): Совсем плохо. У автора сделано правильно. А если уж применять подобный метод, то вместо {BOARD_URL} нужно использовать {ROOT_PATH} и добавлять assets_version={T_ASSETS_VERSION}
{BOARD_URL} Стандартная переменная форума и наверняка где то по логике должна кешироваться не затрагивая расширение (не проверял) и из за этого ест меньше ресурсов, Ваша же решение с 2 мя переменными есть в 2 раза больше ресурсов с тем же результатом, что имхо не кошерно.
Так что в целом мои правки больше чем в 2 раза (с учётом языковых переменных во много раз) экономят ресурсы сервера с тем же визуальным и работоспособным результатом.
Perfecthus писал(а): О каких запросах речь? Вроде итак ничего лишнего не было.
Собственно на большинство ответил выше, кроме переноса скрипта из отдельного файла в само тело
overall_footer_after.html, чтобы из за незначительного текста лишний раз не мучить сервер поиском и подгрузкой дополнительного файла.
Ещё раз повторю это сугубо моё имхо не претендующие на конечную истину, и для меня важна минимальная нагрузка на сервер, при максимальном достигнутом результате.
).
