Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Fenom soft mode. #295

Closed
wants to merge 1 commit into from
Closed

Add Fenom soft mode. #295

wants to merge 1 commit into from

Conversation

touol
Copy link

@touol touol commented Nov 14, 2019

This option, when compiling a Fenom tag error, enables the output of this tag as is and output error in pdoTools for MODX log . To inverse JS / JSON code on MODX template.
Discussion in fenom-template/fenom#308 and https://modx.pro/help/19203
Изменения вносятся одновременно в 2 компонента. Сюда в pdoTools и в исходники Fenom. Чтоб опция работала, нужно, либо загрузить fenom с форка https://github.com/touol/fenom, либо дождаться примут PR и обновить исходники.
Опция pdotools_fenom_soft_mode включается системных настройках pdoTools. При ее включении, fenom феном, не распознанные теги, добавляет в текст шаблона(чанка) как есть и продолжает компилировать шаблон. При включении pdotools_fenom_save_on_errors, ошибки сохраняются в лог MODX.
Изменено поведение только самой критичной части компилятора fenom здесь. То есть, ошибки на валидном JS / JSON коде. В остальных случаях, феном так же выбрасывает исключения (На не закрытых {if ..},{block},{ignore} и т.д.).

…nables the output of this tag as is. To inverse JS / JSON.
@bezumkin
Copy link
Collaborator

bezumkin commented Feb 7, 2020

Ошибки нужно исправлять, а не прятать.

Если страница пустая, значит нужно посмотреть в лог системы, и всё поправить. А продолжать работу, если половина страницы не отработала - неправильно.

Поэтому извините, но я это принимать не буду.

@bezumkin bezumkin closed this Feb 7, 2020
@CrazyBoy49z
Copy link

@touol
Copy link
Author

touol commented Feb 8, 2020

Добрый день, Василий!
Извиняюсь, но это не ошибки :-). CSS и JS не должны ломать сайт. Для меня если компонент доставляет существенные неудобства юзерам значит компонент плохо сделан. Простите пожалуйста, но это мое мнение. pdoTools замечательный компонент, но на мой взгляд в нем есть недоработки которые желательно поправить. И еще, сейчас, я готовлю компонент в котором, грубо говоря, феном вызывается 2 раза. Хм.. и даже 10 раз бывает :-). На вашей версии pdoTools этот компонент скорее всего работать не будет. Скорее всего это в смысле я запустил свой компонент на вашей версии, что-то не заработало и я поставил доработанную, все заработало и дальше я не выяснял в чем конкретно проблема. Но вроде внутри инструкции феном есть еще инструкция феном которую не доработанный парсер фенома не пропускает. А в эту инструкцию компонент нужное добавляет и феном для нее запускает.
Извините что мутно объяснил. Альфа версию компонента, надеюсь, через 2-3 месяца выложу, а пока не хочу раскрывать подробности.
В общем, правка нужная. Подумайте пожалуйста еще раз.

@bezumkin
Copy link
Collaborator

bezumkin commented Feb 8, 2020

Мне кажется, я вполне ясно объяснил свою позицию: ошибки нужно исправлять, а не прятать.

Тем более, я сильно сомневаюсь, что автор Fenom примет изменения.

@touol
Copy link
Author

touol commented Feb 8, 2020

Ну ваше право :-). Придется свою версию продвигать. Отдельный форк вести.

ошибки нужно исправлять, а не прятать.

Надеюсь вы понимаете что тоже можете ошибаться? :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants