Skip to content
This repository has been archived by the owner on Mar 8, 2021. It is now read-only.

error with plugin ajaxSubmit #1107

Closed
Dmi3yy opened this issue Dec 14, 2016 · 0 comments
Closed

error with plugin ajaxSubmit #1107

Dmi3yy opened this issue Dec 14, 2016 · 0 comments
Assignees
Labels

Comments

@Dmi3yy
Copy link
Collaborator

Dmi3yy commented Dec 14, 2016

Во всех релизах после 1.1-d8.0 перестал работать плагин ajaxSubmit.
Проблема в том, что при попытке получить содержимое страницы (45 строка плагина)
$html = $modx->documentContent;

возвращается NULL, а это приводит к фатальной ошибке при получении нужного DOM-узла
(строка 63 плагина)
$results = $dom->query($xpath);


Провел небольшое исследование файла document.parser.class.inc.php
Выяснилось, что на момент возникновения события OnLoadWebPageCache
свойство $this->documentContent имеет значение NULL.
(т.е. по сути плагин работал, но только для еще некэшированных документов, по событию OnLoadWebDocument, там все ок)
В общем добавил в методе getDocumentObjectFromCache, перед публикацией события OnLoadWebPageCache в 636 строке это:
$this->documentContent = $result;

Плагин заработал.

Правда по хорошему нужно бы и в методе prepareResponse публикацию события OnLoadWebDocument перенести после 2186 строки, когда в свойстве documentContent лежит уже распарсеный шаблон (как и в переменной $result метода getDocumentObjectFromCache)

@Dmi3yy Dmi3yy added the 1.2.1 label Dec 14, 2016
@Dmi3yy Dmi3yy self-assigned this Dec 14, 2016
@Dmi3yy Dmi3yy closed this as completed in 7dbcc9c Dec 15, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant