VerstkaShopcms

Мной была замечена какая-то околесица условий для запроса выборки товаров для категории, а именно в случае когда есть товары отображаемые в "не родной" категории(в том смысле что в товаре добавлена дополнительная категория) и плюс стоит галочка выводить товары из подкатегорий. Кажется так, надеюсь не перепутал. У самого руки не дошли распутать данный клубок. Знаю что данная фишка не сильно используется народом в шопцмс. Но косяк есть косяк. Извиняюсь если об этом уже писали где-то на форуме.

Но если так уж хочется, то, скорее всего, обрамление <!--{php}.....{/php}--> уберет лишний код при просмотре результата в редакторе и никак не повлияет на smarty-обработку.

Я немного другое имел ввиду - а именно написать функцию которая бы при сохранении вырезала теги {php}{/php} и всё что прописано внутри. чтобы сделать более безопаснее данную доработку. Хотя в общем никто не запрещал использовать пхп функции в смарти через @. То есть можно просто прописать в смарти {массив|@count} и получить количество элементов массива. Думаю с другими функциями пхп можно делать тоже самое. Так что хоть так хоть этак - дать возможность прописывать код в шаблонах - дать возможность злым рукам сделать какую-то бяку.
В общем можно долго дискутировать на эту тему. И сколько людей - столько и мнений. Каждый выбирает для себя сам путь решения.

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

Безусловно можно разобраться как записывать смарти-теги в обычные блоки. Также можно скорее всего написать функцию фильтрации опасных функций и выражений в тексте дабы усилить безопасность и никто не смог бы записать зловредный код в коде(тексте) блока. Кстати тоже идея. Нужно только определится с набором для фильтрования.

Что мешает сделать то же самое, но не на уровне вставки блока, добавляя (как я понял) редактор блокового файла в админку блоков, а на уровне вставки статической страницы из менеджера страниц, где этот редактор уже есть? Код для вставки на ту же строчку-две, только ничего не надо менять в админке блоков.

Не совсем понял как можно вставить через статическую страницу информацию для страницы Обратная связь(feedback.html) к примеру? Второй момент использование блока-файла вставленного в любой шаблон даёт возможность использовать возможности смарти(прописать условия вывода информации как пример). Чего не сделаешь ни через доп страницу ни через обычный блок.

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

Допустим на странице контакты должен выводится блок с информацией которая:
1. Должно быть 2-3 варианта в зависимости от города определённого геотаргетингом.
2. Эту информацию нужно редактировать из админки удобным способом. Как пример телефон и адрес для 1-го города, 2-го города, и для всех остальных.

Возможно как-то можно сделать фильтрацию {php}{/php} при редактировании файла-блока?

PS.: Имея админский доступ в магазин можно много чего плохого сделать и так. Согласен что чем меньше дыр тем лучше.

Спасибо за критику. Данное дополнение было как раз сделано для того чтобы редактировать блок который является файлом и лежит в папке шаблон/blocks.
Порой приходится делать часть вёрстки шаблона блоком, к примеру шапку в которой заказчик хочет менять картинку логотипа не утруждая себя заходить через фтп и подливать картинку, или футер. Или что прийдется. К примеру в шаблоне feedback.tpl.html нужно вставить какие-то данные и по каким-то причинах стандартным решением через блоки сделать это не просто. В этом случае в сам шаблон(feedback.tpl.html) вставляем {include file="block/test.tpl.html"}, где test.tpl.html нужный нам блок с к примеру адресом и телефонами. И добавив его в управлении блоками как блок из файла - получаем доступ к быстрой и лёгкой правки информации содержащейся в блоке через вивисигн редактор.

По поводу бить по голове за изменение прав к файлам - отвечу. Согласен что с этим нужно разбираться. Но на некоторых хостингах почему-то скрипт выполняется не от рута(админа не знаю как точно это назвать) и соответственно прав 644 не достаточно для перезаписи файла. И иногда хостинг никак не хочет давать такие возможности. Безусловно можно найти другого хостера и еще много чего применить к данной ситуации. Я по поводу прав написал потому что сталкивался с такой проблемой и если кто-то с форума захочет попробовать это дополнение и столкнётся с этой ситуацией по поводу прав - то я уже описал эту проблему. Возможно моих знаний не достаточно но я не вижу ничего криминального в том что у файлов в папке {шаблон}/blocks будут права хоть 777. Я не призываю всем файлам скрипта выдать права 777

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