Энхансмент vs ключ на объект

Автор ysichov, Липень 31, 2017, 05:38:59 ПП

Попередня тема - Наступна тема

0 Користувачі і 1 Гість дивляться цю тему.

ysichov

Доброго времени суток!

Была задача добавить немного кода где-то внутри стандартного функционального модуля.

Как обычно все срочно :). Сделал энхансмент в начале ФМ, скопировал весь ФМ, в конце добавил EXIT, переименовал все объявления переменных, ну и в самом коде в полтыщи строк, добавил несколько строк своего кода.

Хочу спросить у экспертов есть ли смысл такое делать? С точки зрения будущих апгрейдов и поддержки, лучше взять ключ разработчика и вставить несколько строк своего кода или без привлечения базисников делать такие не очень красивые энхансменты?

Спасибо!

Uukrul

Цитата: ysichov від Липень 31, 2017, 05:38:59 ПП
Как обычно все срочно :). Сделал энхансмент в начале ФМ, скопировал весь ФМ, в конце добавил EXIT, переименовал все объявления переменных, ну и в самом коде в полтыщи строк, добавил несколько строк своего кода.

Хочу спросить у экспертов есть ли смысл такое делать? С точки зрения будущих апгрейдов и поддержки, лучше взять ключ разработчика и вставить несколько строк своего кода или без привлечения базисников делать такие не очень красивые энхансменты?
Ни первое, ни второе делать не рекомендуется. Я бы это... при всем уважении, наверное за такое пристрелил бы... хотя может №1 подойдет и выскажет мнение более объективно  8)

ysichov

#2
Всех не перестреляешь :)

Ну не придумал какой-то неведомый архитектор возможности изменения. Ни настроечной таблички не сделал, ни бадишки, ничего. Тупой линейный код, который не устраивает бизнес-заказчика.
Я, конечно в жирные годы, копировал все в Z или писал Z с нуля. Но это и мне надоело и клиенты внезапно научились деньги считать :)

В общем альтернатив нет? :)

ФМ HRHAP_C_DOCUMENT_UPD_GOAL. Надо иногда не выполнять какую-то часть логики, а возвращать сообщение с типом E.

Вставил несколько строчек в строке 350.

Uukrul

Цитата: ysichov від Серпень 07, 2017, 03:48:08 ПП
Я, конечно в жирные годы, копировал все в Z или писал Z с нуля. Но это и мне надоело и клиенты внезапно научились деньги считать :)
Да тут как раз они их считать не научились, вот сейчас один такой клиент, ищет почему у него данные в определенных случаях не сохраняются.  А потому что до этого одни умные консультанты модулек подшаманили твоим методом. Так что сначала они попали когда платили первому консалту, а сейчас попадут за раскопки этой хрени другому. Так что как раз нихрена они считать не умеют.

Цитата: ysichov від Серпень 07, 2017, 03:48:08 ПП
В общем альтернатив нет? :)
Есть, послать на OSS пусть с SAP-ом договариваются, кто из них умный, а кто красивый.

ysichov

Цитата: Uukrul від Серпень 07, 2017, 04:23:44 ПП
Есть, послать на OSS пусть с SAP-ом договариваются, кто из них умный, а кто красивый.

Спасибо! будем посылать на OSS.

Uukrul

Цитата: ysichov від Серпень 08, 2017, 08:31:15 ДП
Спасибо! будем посылать на OSS.
Посылать то можно, но вот пойдут ли они туда :-)

SMF spam blocked by CleanTalk