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

Форум

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

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

Эксперт

Сообщений: 684 Баллов: 92 Регистрация: 11.04.2013

#131

30.03.2020 00:44:19

Цитата
Евгений Жуков написал: Чтобы автоматически внутри обработчика получить контекст вызова

Если под контекстом подразумевается this, то на этот случай есть стрелочные функции.
С BX.delegate и BX.proxy дебажить (а я бы даже сказал реверс инжинерить - документации не хватает) битрикс то ещё удовольствие

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

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

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

#132

30.03.2020 10:16:26

Цитата
Ахат Баязи написал: В принципе несерьезный, но нервы может попортить пользователям.

про баг знаю, спасибо. Собственно, из-за него по-нормальному работать сложно.

Я бы предложил такой вариант: добавить к ссылке какой-то свой уникальный get-параметр, а в обработчике события его обрабатывать и вызывать метод

Код
$document->enableStamps(true);

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

Цитата
Ахат Баязи написал: По провайдеру смотреть, только не понял, как правильно в фильтре провайдер задать

Код
'PROVIDER'=> 'bitrix\\crm\\integration\\documentgenerator\\dataprovider\\deal'

вот кусок кода из компонента documentgenerator.documents

Код
$providerClassName = str_replace("\\", "\\\\", strtolower(get_class($this->provider))); $filter = [ 'PROVIDER' => $providerClassName, 'VALUE' => $this->provider->getSource(), ];

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

Эксперт

Сообщений: 455 Баллов: 93 Регистрация: 19.10.2012

#133

30.03.2020 15:01:54

Цитата
Александр Медведев написал: Ахат Баязи , всё хотел спросить - в чём практический смысл оборачивать обработчики в BX.delegate? В том же Firefox потом из-за этого нифига не понятно какой обработчик навешан на событие элемента

Вопрос, конечно, интересный... Сам не мог понять до недавнего времени ))) Делал так, только потому что в документации написано, и вся JS-логика у битрикса так написана.
В оф. инструкции о BX.proxy и BX.delegate сказано: ссылка

Цитата

Две родственных функции, генерирующих «функцию-делегата», которая будет вызывать требуемую функцию в нужном контексте. Это может использоваться, например, для назначения метода какого-либо объекта в качестве обработчика события с сохранением контекста объекта.Отличие между функциями в том, что BX.proxy при повторном вызове с теми же параметрами вернет не новую функцию-делегата, а ссылку на сгенерированную до этого функцию, что может быть полезно, например, если требуется отменить конкретный обработчик события.

До этого сам писал на чистом JS или использовал JQ. Но, со временем, понял, что проще для сложной кастомизации штатного функционала копипастнуть рабочий код ядра и работать с ним. А перекладывать штатную логику на JS или JQ оказалось себе дороже (проще написать заново, но тут приходится все равно вникать в штатную логику, чтоб не сломать штатный функционал). В общем, со временем, когда усложнилась логика реализуемого функционала, понял, что лучше писать в логике битрикса.
При создании объекта (как правило при загрузке страницы) в объекте инициализируются ОООГРОМНАЯ толпа переменных, методов и свойств объекта, которые могут динамически меняться в результате работы скриптов. Так вот самый простой способ получить к ним доступ в объекте через ключевое слово this.:

this.СВОЙСТВО, this.МЕТОД или this.ФУНКЦИЯ().

Все просто пока мы не начнем делать вложенность в объекте. Например, у нас есть объект, с набором методов и свойств, которые имеют свои свойства и методы.
Так вот this, будет ссылаться на объект, который стоит перед точкой, и таким образом получить данные которые имеет объект при инициализации недоступны.

А теперь живой кейс:
Объект: BX.CrmProductEditor (bitrix/components/bitrix/crm.product_row.list/templates/.default/script.map.js)

Код

BX.CrmProductEditor.prototype =
{
...
инициализация, и объявление переменных
...
w4aSaveProductRows: function(){
    // alert('tutut-' + this._serviceUrl)
    let isDirector = false;
    let w4aDirOptions = [];
    if(this.isDirector())
    {
        isDirector = true;

        w4aDirOptions = [
            {
                'id': 'W4A_DIR_TOTAL_DDP_KZT_BOTTOM',
                'value': BX('W4A_DIR_ADD_PAY_KZT_BOTTOM').dataset.value
            },
            {
                'id': 'W4A_DIR_TOTAL_SUM_KZT_BOTTOM',
                'value': BX('W4A_DIR_TOTAL_SUM_KZT_BOTTOM').dataset.value
            },
            {
                'id': 'W4A_DIR_ADD_PAY_KZT_BOTTOM',
                'value': BX('W4A_DIR_ADD_PAY_KZT_BOTTOM').dataset.value
            },
            {
                'id': 'W4A_DIR_MARGE_KZT_BOTTOM',
                'value': BX('W4A_DIR_MARGE_KZT_BOTTOM').dataset.value
            },
            {
                'id': 'W4A_DIR_COMMENT_BOTTOM',
                'value': BX('W4A_DIR_COMMENT_BOTTOM').value
            },
        ];
    }


    BX.ajax({
        'url': this._serviceUrl,
        'method': 'POST',
        'dataType': 'json',
        'data':
            {
                'MODE': 'W4A_SAVE_PRODUCTS',
                'OWNER_TYPE': this.getSetting('ownerType', ''),
                'OWNER_ID': this.getSetting('ownerID', 0),
                'PERMISSION_ENTITY_TYPE': this.getSetting('permissionEntityType', ''),
                'PRODUCT_ROW_DATA': this.productsToJson(),
                'PRODUCT_ROW_SETTINGS': {
                    'ENABLE_DISCOUNT': this.isDiscountEnabled() ? 'Y' : 'N',
                    'ENABLE_TAX': this.isTaxEnabled() ? 'Y' : 'N'
                },
                'SITE_ID': this.getSetting('siteId', ''),
                'W4A_IS_DIRECTOR': isDirector,
                'DIR_OPTIONS': JSON.stringify(w4aDirOptions)
            },
        // onsuccess: BX.delegate(this._onSaveProductsRequestSuccess, this), // original
        onsuccess: BX.delegate(this._w4aOnSaveProductsRequestSuccess, this),
        onfailure: BX.delegate(this._onSaveProductsRequestFailure, this)
    });
},
...
})

В вышеприведенном примере, данные из AJAX прилетают в onsuccess: & onfailure:, и если там вызвать к this, то мы не получим доступ к свойствам объекта,
а очень хочется.
Вот тогда помогает делегирование ( onsuccess: BX.delegate(this._w4aOnSaveProductsRequestSuccess, this))
И мы делегируем методу _w4aOnSaveProductsRequestSuccess обработку результата, с передачей ему полученных результатов.
=============================
ЗЫ Не знаю правильно ли я объяснил, тут наверное про this лучше написано тут

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

Эксперт

Сообщений: 455 Баллов: 93 Регистрация: 19.10.2012

#134

30.03.2020 15:11:31

Цитата
Евгений Жуков написал: Но лучше все же использовать BX.proxy

Я так и не понял в чем отличия между BX.proxy и BX.delegate, сколько не тестил, разницы нее увидел.
Можете привести конкретный кейс, где это может быть лучше?

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

Эксперт

Сообщений: 684 Баллов: 92 Регистрация: 11.04.2013

#135

30.03.2020 18:03:55

Цитата
Ахат Баязи написал: BX.CrmProductEditor.prototype = { ... инициализация, и объявление переменных ... w4aSaveProductRows: function(){

А для чего перезаписывать/расширять родной объект BX.CrmProductEditor?
Или я просто не понял что делаешь)

BX.CrmProductEditor я получаю так

Код

/**
 * @type {BX.Crm.EntityEditor}
 */
const entityEditor = BX.Crm.EntityEditor.get('{editor_guid}');

/**
 * @type {BX.CrmProductEditor}
 */
const productEditor = entityEditor
  ._controllers
  .find(controller => controller._id === 'PRODUCT_ROW_PROXY')
  ._externalEditor;

и использую его методы.

Цитата
Ахат Баязи написал: В вышеприведенном примере, данные из AJAX прилетают в onsuccess: & onfailure:, и если там вызвать к this, то мы не получим доступ к свойствам объекта,

Так сделай проще

Код
onsuccess: () => this._w4aOnSaveProductsRequestSuccess(),

или если метод короткий то прямо так

Код
onsuccess(){ },

Я понимаю какие проблемы решают BX.delegate и BX.proxy, но эти проблемы уже не актуальны несколько лет.
Загляни сам в "новое" ядро
bitrix/js/main/core/src/lib
никаких делегирований и прокси и правильно

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

Эксперт

Сообщений: 455 Баллов: 93 Регистрация: 19.10.2012

#136

31.03.2020 09:24:44

Цитата
Александр Медведев написал: А для чего перезаписывать/расширять родной объект BX.CrmProductEditor?Или я просто не понял что делаешь)

Ну у меня, откровенно говоря, стоит по ТЗ более широкая задача, чем была описана выше.
вот 1-на из них:
расширить таблицу товаров:
было добавлено 11 доп. полей, такие как:

артикул,
срок поставки,
Цена закупа в локальной валюте (KZT),
Цена закупа в валюте (с возможностью изменять валюту: RUR|EUR|USD),
наценка,
Цена для клиента в локальной валюте (KZT)
Цена для клиента в валюте
Маржа
Срок поставки
Гарантия
Комментарий (Примечание)

А расчет ширины полей находится в BX.CrmProductEditor, причем жестко зашит в массиве, в относительных величинах
В итоге из такого массива (штатный):

Код

/** Original case, when added 0 cols*/ 
 var setWidthTax =  [ 0, 0, 0, 0, 0, 0, 0, 0, 0],
        setWidthSale = [29, 5, 5,10,10,10,10,18, 3],
        setWidthAll =  [ 0, 0, 0, 0, 0, 0, 0, 0, 0],
        resetWidth =   [29, 8, 8,16,16, 0, 0,19, 3];
}

получился такой массив:

Код

var setWidthTax =  [ 0, 0, 0, 0, 0, 0,   0, 0, 0, 0, 0, 0, 0,   0, 0, 0, 0,  0, 0,   0, 0, 0, 0],
    setWidthSale = [ 5,24, 5, 0, 0, 0,   0, 0, 0, 0, 0, 0, 0,   5, 3,10,10,  5,18,   5, 8, 0, 3],
    setWidthAll =  [ 0, 0, 0, 0, 0, 0,   0, 0, 0, 0, 0, 0, 0,   0, 0, 0, 0,  0, 0,   0, 0, 0, 0],
    resetWidth =   [ 8,30, 5, 0, 0, 0,   0, 0, 0, 0, 0, 0, 0,   5, 3, 0, 0,  5,19,   8,13, 0, 3],
    setCalcMode =  [10,14, 6, 0, 0, 0,   7, 7, 7, 7, 7, 7, 8,   0, 0, 0, 0,  6,11,   0, 0, 0, 3]; // shown with calc mode;

Код

/** In SPECIAL CASE for XXXX, when added 11 cols (positions: 1st, 4-10 & 3 last cols)
 *  @description
 * 0-2: P/N, Name, Quantity
 * 3-5: w4a-crm-item-spacer ACHTUNG!! Spacer don't remove!!!
 * 6-7: price_ddp_usd, total_ddp_usd
 * 8-9: price_ddp_kzt, total_ddp_kzt
 * 10: marge
 * 11-12: price_usd, total_usd
 *
 * 13: Unit (ед. измерениия)
 * 14-16: Discount (without Taxes Fields)
 *
 * 17-18: Price & Total for Client
 *
 * 19-20: delivery_time, warranty, comment
 * 21: remove button
 * */

т.к. колонок много, то пришлось еще вводить функционал отображения режима калькулятора.
Так вот все эти доп. поля участвуют в пересчете значений других полей (в том числе и штатных) в процесе ввода значений поля.
Опять же этот функционал сидит в BX.CrmProductEditor
Задача усложнилась тем, что в других сущностях: Лид, Предложение должен был остаться штатный функционал
Поэтому, ни чего другого, как перенести штатный компонент: bitrix:crm.product_row.list
в свое пространство имен:w4a.bitrix:crm.product_row.list.calc и там его кастомизировать, я не придумал.
Если есть менее затратные способы кастомизации, прошу поделиться.

PSСорри за офтоп, но эти доп. поля как раз и выносятся в последствии в генератор документов.

Прикрепленные файлы

2020-03-31_121018.png (134.22 КБ)

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

Эксперт

Сообщений: 455 Баллов: 93 Регистрация: 19.10.2012

#137

31.03.2020 10:41:02

Цитата
Александр Медведев написал: Так сделай проще onsuccess: () => this._w4aOnSaveProductsRequestSuccess(),

Имхо: Не вижу упрощения. В чем разница, от приведенной ниже конструкции:

Код
onsuccess: BX.delegate(this._w4aOnSaveProductsRequestSuccess, this),

ИМХО2: Да, использование стрелочной функции возможно решит данную задачу, но усложнит чтение кода(читать как сопровождение проекта) другим программистом. Т.к. в Битрикс- ядре я не встречал использование стрелочных функций. А тут: BX.delegete(), по названию. понятно, что мы кому-то что-то делегируем.
пишу ВОЗМОЖНО, потому что, У стрелочных функций нет «this», и как следствие полученные доп. данные из AJAX она не передаст их в вызываемую функцию. Попробую в др. проекте, отпишусь.

Цитата
Стрелочные функции особенные: у них нет своего «собственного» this. Если мы используем this внутри стрелочной функции, то его значение берётся из внешней «нормальной» функции.

Это искл. мое мнение.

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

Эксперт

Сообщений: 684 Баллов: 92 Регистрация: 11.04.2013

#138

01.04.2020 10:52:50

Цитата

Ахат Баязи написал:
Поэтому, ни чего другого, как перенести штатный компонент: bitrix:crm.product_row.list в свое пространство имен:w4a.bitrix:crm.product_row.list.calc и там его кастомизировать, я не придумал.
Если есть менее затратные способы кастомизации, прошу поделиться.

Была чуть похожая задача, тоже вмешаться в работу продуктов сделки - применять персональные скидки для контактов и компаний + депозитный счёт.
Не придумал ничего лучше, чем на vue написать собственный "компонент" работы с продуктами сделки.
Мне показалось проще, чем переделывать нативный

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

Эксперт

Сообщений: 455 Баллов: 93 Регистрация: 19.10.2012

#139

01.04.2020 12:29:15

Цитата
Александр Медведев написал: Не придумал ничего лучше, чем на vue написать собственный "компонент" работы с продуктами сделки.Мне показалось проще, чем переделывать нативный

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

Цитата
Александр Медведев написал: Мне показалось проще, чем переделывать нативный

Да, пришлось потанцевать с бубном!!! Хвала шамана за этот инструмент

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

Эксперт

Сообщений: 684 Баллов: 92 Регистрация: 11.04.2013

#140

01.04.2020 12:36:12

Цитата
Ахат Баязи написал: По идее, да можно было бы свой компонент написать, но пришлось бы в него переносить весь (штатный функционал).

Не обязательно. Я штатный оставил, просто скрыл.
И по факту свой компонент управлял штатным.
К примеру, добавил в свой компонент товар - он добавился и в штатном, поменял количество - в штатном всё пересчиталось.
Опять же так показалось проще в плане сохранения товаров.
Не пришлось его писать - штатный это итак умеет))

Форум

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

Продукты

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

Битрикс24

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

Решения

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

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

Внедрение

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

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

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