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

Как целиком выпилить использование eval() из Fenom? #326

Open
eugene20237 opened this issue Jul 25, 2021 · 9 comments

Comments

@eugene20237
Copy link

Возможно ли убрать использование функции eval() из шаблонизатора? Что при этом потеряется и как примерно это сделать? Хочу сделать форк без этой функции.

@WinterSilence
Copy link
Contributor

@eugene20237 ты конечно можешь заменить eval на include временных php файлов, но что это тебе даст я хз..

@whatafunc
Copy link

@WinterSilence привет, а что посоветуете с интеграцией в modx делать?

в modx 3 и php8 вот это ловится:
Fatal error: Uncaught TypeError: Fenom\Render::__construct(): Argument #1 ($fenom) must be of type Fenom, null given, called in /var/www/html/core/components/pdotools/src/Parsing/Fenom/Fenom.php(116) : eval()'d code on line 52 and defined in /var/www/html/core/components/pdotools/vendor/fenom/fenom/src/Fenom/Render.php:83 Stack trace:
#0 /var/www/html/core/components/pdotools/src/Parsing/Fenom/Fenom.php(116) : eval()'d code(52): Fenom\Render->__construct(NULL, Object(Closure), Array)
#1 /var/www/html/core/components/pdotools/src/Parsing/Fenom/Fenom.php(116): eval()
#2 /var/www/html/core/components/pdotools/src/Parsing/Parser.php(69): ModxPro\PdoTools\Parsing\Fenom\Fenom->process('{var $thisId = ...', Array)
#3 /var/www/html/core/src/Revolution/modResource.php(521): ModxPro\PdoTools\Parsing\Parser->processElementTags('', '{var $thisId = ...', true, false, '[[', ']]', Array, 10)
#4 /var/www/html/core/src/Revolution/modResource.php(469): MODX\Revolution\modResource->parseContent()
#5 /var/www/html/core/src/Revolution/modResponse.php(72): MODX\Revolution\modResource->prepare()
#6 /var/www/html/core/src/Revolution/modRequest.php(154): MODX\Revolution\modResponse->outputContent(Array)
#7 /var/www/html/core/src/Revolution/modX.php(1242): MODX\Revolution\modRequest->prepareResponse()
#8 /var/www/html/core/src/Revolution/modX.php(1281): MODX\Revolution\modX->sendForward('2', Array, false)
#9 /var/www/html/core/src/Revolution/modRequest.php(134): MODX\Revolution\modX->sendErrorPage()
#10 /var/www/html/core/src/Revolution/modX.php(1499): MODX\Revolution\modRequest->handleRequest()
#11 /var/www/html/index.php(63):
MODX\Revolution\modX->handleRequest()
#12 {main} thrown in /var/www/html/core/components/pdotools/vendor/fenom/fenom/src/Fenom/Render.php on line 83

@WinterSilence
Copy link
Contributor

WinterSilence commented Jan 12, 2023

@modxcustomize я посоветую создавать новые issue для вопросов не по теме.

Uncaught TypeError: Fenom\Render::__construct(): Argument #1 ($fenom) must be of type Fenom, null given

текст ошибки по моему достаточно точно описывает суть проблемы. если не сможешь сам справится, то пиши на мне мыло - попробую помочь

@whatafunc
Copy link

: Fenom\Render::__construct(): Argument #1 ($fenom) must be of type Fenom, null given

спасибо, сделал

@bzick
Copy link
Member

bzick commented Jan 17, 2023

На неделе выкачу фикс без eval

@whatafunc
Copy link

На неделе выкачу фикс без eval
а этот баг не пофиксится?

@bzick
Copy link
Member

bzick commented Jan 17, 2023

@modxcustomize думаю починю по пути

@Cheizerman
Copy link

Cheizerman commented Feb 7, 2023

: Fenom\Render::__construct(): Argument #1 ($fenom) must be of type Fenom, null given

спасибо, сделал

Hello, please tell me how you solved this problem? I have the same error

@bzick
Copy link
Member

bzick commented Feb 8, 2023

in progress

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

No branches or pull requests

5 participants