Разработчикам

Форум

Генератор документов - феррари на которой можно ездить только вокруг дома, Новый модуль получился хорошим но не предназначен для использования разработчиками.

Антон Горбылев

Администратор

Сообщений: 365 Баллов: 55 Регистрация: 22.01.2014

#11

30.01.2019 09:37:45

final у ряда основных классов проставлен специально, как раз чтобы не было наследников (хотя до выпуска в коробку его там не было). Версия ещё достаточно сырая, так у меня нет необходимости строго следовать текущей сигнатуре. Вы можете его убрать, но с выходом каждого обновления придётся руками проверять, что ничего не сломается.

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

Пользователь 17644

Эксперт

Сообщений: 762 Баллов: 70 Регистрация: 14.11.2007

#12

30.01.2019 10:50:12

Цитата
Антон Горбылев написал: Простейшие условия ... - есть в планах.

можете привести пример?

Пользователь 136059

Гуру

Сообщений: 5443 Баллов: 649 Регистрация: 16.07.2012

#13

30.01.2019 12:09:03

Цитата
Антон Горбылев написал: final у ряда основных классов проставлен специально, как раз чтобы не было наследников (хотя до выпуска в коробку его там не было).

Эм... а с какой целью? Вот мне например final в классах битрикса доставляют очень большие неудобства и поэтому приходится извращаться.

Голосуй за идеи по развитию API Bitrix:
https://idea.1c-bitrix.ru/26707/
https://idea.1c-bitrix.ru/26709/
https://idea.1c-bitrix.ru/the-local-extension-folder-js/

Антон Горбылев

Администратор

Сообщений: 365 Баллов: 55 Регистрация: 22.01.2014

#14

30.01.2019 12:09:12

Цитата

Alexey Andreev написал:

Цитата
Антон Горбылев написал: Простейшие условия ... - есть в планах.

можете привести пример?

Детали на стадии обсуждения, в ближайшую итерацию добавления каких-либо операций в шаблоны не входит. Пока что анонсировать что-то в этом направлении рано, но оно обязательно будет.

Антон Горбылев

Администратор

Сообщений: 365 Баллов: 55 Регистрация: 22.01.2014

#15

30.01.2019 17:03:26

Цитата

Андрей Николаев написал:

Цитата
Антон Горбылев написал: final у ряда основных классов проставлен специально, как раз чтобы не было наследников (хотя до выпуска в коробку его там не было).

Потому что класс и сигнатура методов ещё не приняли своего окончательного вида.
Представьте, что у класса нет final. Вы спокойно сделали наследника, переопределили часть методов, добавили свои.
Потом вышло обновление, где я какой-нибудь из методов сделал private, или добавил обязательных параметров.

И всё, у вас fatal каждый раз при использовании вашего класса.
Если это ваш портал с какими-то специфическими доработками - можно оперативно исправить и ущерба будет немного.
А вдруг кто-нибудь сделает приложение со своим наследником?

Одна из проблем битрикса, как продукта - это большое legacy. А большое legacy обусловлено политикой максимальной обратной совместимости. Поддержка обратной совместимости замедляет скорость обновления кода.

Проставив final, я, в первую очередь, оставляю себе возможность вносить изменения в классы без оглядки на пользовательский код.
Говорю только за себя, но коллеги часто руководствуются этим же принципом.

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

Пользователь 272757

Заглянувший

Сообщений: 32 Баллов: 1 Регистрация: 27.07.2014

#16

02.04.2019 17:40:33

Цитата

Антон Горбылев написал:

Цитата

Андрей Николаев написал:

Цитата
Антон Горбылев написал: final у ряда основных классов проставлен специально, как раз чтобы не было наследников (хотя до выпуска в коробку его там не было).

Антон, здравствуйте. На самом деле, есть потребность в генерации нормальных вложенных списков. Если docx предполагает какой-то такой формат:

Код

<w:p w:rsidR="002A0243" w:rsidRDefault="00AB6E60" w:rsidP="00AB6E60">
    <w:pPr>
        <w:pStyle w:val="a3"/>
        <w:numPr>
            <w:ilvl w:val="0"/>
            <w:numId w:val="1"/>
        </w:numPr>
    </w:pPr>
    <w:r>
        <w:rPr>
            <w:lang w:val="en-US"/>
        </w:rPr>
        <w:t>Текст</w:t>
    </w:r>
</w:p>

То есть мы видим, что это параграфы с Paragraph Properties, уровни нумерации, то в Вашей реализации мы видим просто строки с текстом и пробелами как отступ:

Код
'<w:r><w:t xml:space="preserve">' . ' '.$context['currentNumber'] . '</w:t></w:r>'.....

Скажите, есть ли в планах приведение списков в более удобный и правильный вид?
Как раз столкнулись с генерацией приказов, где регламентирован чуть ли не каждый отступ в документе. Пришлось пока что отложить, решаем что делать, писать некую свою реализацию или Битрикс будет семимильными шагами развивать генератор и стоит подождать. Опять же final запретит отнаследоваться и реализовать свой метод htmlNodeToXml (оно в принципе и к лучшему)

Антон Горбылев

Администратор

Сообщений: 365 Баллов: 55 Регистрация: 22.01.2014

#17

03.04.2019 14:34:53

Здравствуйте.

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

Пока что можно попробовать подвеситься на событие onCreateDocument, в обработчике доставать Body и менять контент файла с помощью регулярных выражений (для поиска нужного места можно расставить метки внутри xml). Это не очень удобно, но должно работать.

Пользователь 3092471

Заглянувший

Сообщений: 2 Регистрация: 09.04.2019

#18

09.04.2019 15:14:55

Цитата

Антон Горбылев написал:
Здравствуйте.

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

Пока что можно попробовать подвеситься на событие onCreateDocument, в обработчике доставать Body и менять контент файла с помощью регулярных выражений (для поиска нужного места можно расставить метки внутри xml). Это не очень удобно, но должно работать.

Здравствуйте, Антон.
Вы не могли бы привести пример, как можно в обработчике события onCreateDocument поменять контент файла?

Антон Горбылев

Администратор

Сообщений: 365 Баллов: 55 Регистрация: 22.01.2014

#19

09.04.2019 15:57:49

Артем Пищулин,

Точно надо напрямую изменить контент файла?
Если надо изменить значения полей, то есть событие onBeforeProcessDocument. В обработчике можно вызвать \Bitrix\DocumentGenerator\Document::setValues(). Пример есть в документации https://dev.1c-bitrix.ru/api_d7/bitrix/documentgenerator/customization.php

Если всё-таки нужен именно контент, то алгоритм будет такой:
В обработчик события приходит объект \Bitrix\DocumentGenerator\Document.
Из него можно достать FILE_ID
По этому ид дергаем \Bitrix\DocumentGenerator\Model\FileTable::getContent()
Меняем контент, а потом обновляем его методом \Bitrix\DocumentGenerator\Model\FileTable::updateContent()

Код

\Bitrix\Main\EventManager::getInstance()->addEventHandler('documentgenerator', 'onCreateDocument', function($event)
{
   $document = $event->getParameter('document');
   $fileId = $document->FILE_ID;
   $content = \Bitrix\DocumentGenerator\Model\FileTable::getContent();
   $content = $content; // some modification
   \Bitrix\DocumentGenerator\Model\FileTable::updateContent($fileId, $content);
});

Пользователь 3092471 Заглянувший Сообщений: 2 Регистрация: 09.04.2019	#20 0 10.04.2019 10:31:22 Антон Горбылев, спасибо. На самом деле, мне нужно вставить на место плейсхолдера таблицу (или любой код, как есть, без его дополнительной обработки). В первом случае, фильтруются значения полей перед вставкой в документ и от таблицы остается только текст. Во втором случае, в переменную $content попадает не содержимое файла (word/document.xml), а его структура ([Content_Types].xml). Возможно, я что-то не так понял. Подскажите, пожалуйста, возможно ли плейсхолдер заменять на произвольный код?

Форум

Центр поддержки

Продукты

Управление сайтом

Битрикс24

Интернет-магазин + CRM

Решения

Для интернет-магазинов

Каталог готовых решений

Внедрение

Выбрать партнера

Проверить партнера

Стать партнером