instantsoft / icms2

Self-hosted Site Management System
https://instantcms.ru
GNU General Public License v2.0
295 stars 120 forks source link

Получение опций шаблона #1186

Closed asnenne closed 4 years ago

asnenne commented 4 years ago

Используемая версия InstantCMS: 2.13

https://github.com/instantsoft/icms2/blob/master/system/core/template.php

Предлагаю protected $options; заменить на public $options;

Я конечно не разбираюсь совсем, но не могу понять, что такого можно написать в опциях шаблона, что нужно прятать.

Зачем public? Ну вот я, например, пилю сейчас шаблон, в котором опции оформления. И к этому шаблону несколько виджетов. Элементарный вопрос: как получить в опциях виджета значения опций из шалона, например, цвет ссылок, чтобы сделать его значением по-умолчанию в опциях виджета? Ответ - никак, потому что всё защищено не понятно от какой угрозы.

Дальше. Все любят, чтобы стили были прописаны не внутри кода, а в отдельных css файлах. Хорошо, есть уже решение. Но блин эта защита не дает получить опции шаблона, чтобы сохранить всё в файл css.

Пожалуйста, уберите эту защиту!

Спасибо.

fuzegit commented 4 years ago

Я конечно не разбираюсь совсем

Но предлагаю)

Ответ - никак

Вообще-то никаких проблем нет https://github.com/instantsoft/icms2/blob/master/system/core/template.php#L2877

Пожалуйста, уберите эту защиту ... потому что всё защищено не понятно от какой угрозы.

Вы с чего решили, что это вообще какая-то защита?)

Я знаю откуда ветер дует и чьими мыслями вы пишите) Почему он сам это не пишет, неведомо. Наверное потому, что на все вопросы будут ответы.

Так же, если по существу, геттер и сеттер уже загруженных в объект опций, я добавлю. И это вопрос исключительно удобства. https://github.com/instantsoft/icms2/commit/108ac051217a2e9d29a2a66eeb95ec6e7ac39a87

asnenne commented 4 years ago

Я знаю откуда ветер дует и чьими мыслями вы пишите

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

Вы с чего решили, что это вообще какая-то защита?)

По Вашей рекомендации, воспользовался переводчиком 003