sashanazarov: Manipulating the Marking form from code

[Blog bot]

Источник: http://sashanazarov.blogspot.com/201...from-code.html
==============

Businesses buy AX, because they would like to automate their processes. Sometimes they would like to automate more, than standard AX allows. And if business logic is built into forms, it is sort of hard to automate.

One of our clients had to migrate open sales and purchase orders to AX, and mark their lines against each other afterwards. Apparently, calling InventTransOrigin::updateMarking method was not enough, as sales and purchase lines were still not pointing to each other in the reference fields. So, I tried to use the standard Marking form for that ad-hoc task.

Below is the code sample that takes a purchase line lot ID, opens the Marking form, selects the target sales order line and marks it (or unmarks, if it was marked before). Just like you do it by hand.

static void main(Args _args)
{
#define.TmpInventTransMarkDsNo(2)
#define.InventTransOriginDsNo(4)
#define.InventTransOriginMarkDsNo(5)
#define.MarkNowControlName('markNow')

#define.PurchLineLotId("GSC-000983")
#define.SalesLineLotId("GSC-000982")

FormRun formRun;
Args args;
TmpInventTransMark tmpInventTransMark;
SalesLine salesLine;
InventTransOrigin salesLineOrigin;
PurchLine purchLine;
InventTransOrigin inventTransOriginMark;
FormCheckBoxControl markNowCheckBox;

purchLine = PurchLine::findInventTransId(#PurchLineLotId);

args = new Args();
args.name(formStr(InventMarking));
args.record(purchLine);

// Work-around: first if-statement in
// Classes\ReqCalc\argsItemId method expects a caller
args.caller(new InventMarkingEmulator());

formRun = classfactory.formRunClass(args);
formRun.init();
formRun.run();

// The temp table is filled in when this active() is called
formRun.dataSource(#InventTransOriginDsNo).active();

tmpInventTransMark =
formRun.dataSource(#TmpInventTransMarkDsNo).cursor();
inventTransOriginMark =
formRun.dataSource(#InventTransOriginMarkDsNo).cursor();

ttsbegin;

// Find the required reference and mark it
formRun.dataSource(#TmpInventTransMarkDsNo).first();
Debug::assert(tmpInventTransMark.RecId != 0);
do
{
if (inventTransOriginMark.InventTransId == #SalesLineLotId)
{
markNowCheckBox =
formRun.design().controlName(#MarkNowControlName);
markNowCheckBox.value(1);
markNowCheckBox.clicked();

formRun.closeOk();
break;
}
} while (formRun.dataSource(#TmpInventTransMarkDsNo).next());

ttscommit;
}


Источник: http://sashanazarov.blogspot.com/201...from-code.html

[Maxim Gorbunov]

Совет из разряда тех, которыми никогда не стоит пользоваться. Проблема в том, что он основан на изначально неверном утверждении, что вся логика маркинга определена в форме InventMarking. Это, мягко говоря, не совсем так.

Правильный способ решения поставленной задачи - использовать метод TmpInventTransMark::updateTmpMark(). Он, кстати, не только поля InventRef* в SalesLine и PurchLine заполнит, но и резервирование обновит.

[Stitch_MS]

За отзыв спасибо, но хотелось бы пояснить

• Это был не совет, а описание решения некоторой задачи.
• Утверждения, что вся логика маркинга определена в форме - не было. Было "Apparently, calling InventTransOrigin::updateMarking method was not enough... So, I tried to use the standard Marking form for that ad-hoc task. "
• Использование метода updateTmpMark напрямую из кода подразумевает подготовку параметров, один из которых - контейнер c записями. Я такие вещи, мягко говоря, недолюбливаю. Судя по перекрестным ссылкам, единственное место, где этот метод используется - это та самая форма. Форма же и занимается подготовкой записей для метода. Я не хотел писать всё это руками для своей задачи, если это уже сделано на форме.

[Raven Melancholic]

Я думаю, что маркировка из кода все таки задача для выполнения на сервере. Использование формы, её датасорсов для заполнения данных переводит задачу на клиента, что не очень хорошо.

[Maxim Gorbunov]

Цитата:

Сообщение от Stitch_MS

Это был не совет, а описание решения некоторой задачи.

Мы с вами это понимаем. А хотелось бы, чтобы понимали и начинающие программисты. Из оригинального сообщения не следует, что его не стоит использовать в качестве руководства к действию.

Цитата:

Сообщение от Stitch_MS

Утверждения, что вся логика маркинга определена в форме - не было. Было "Apparently, calling InventTransOrigin::updateMarking method was not enough... So, I tried to use the standard Marking form for that ad-hoc task. "

Напрямую - не было. Но вот после этого - "And if business logic is built into forms, it is sort of hard to automate" - и последующего описания процесса маркировки, можно запутаться и сделать неправильные выводы.

Цитата:

Сообщение от Stitch_MS

Использование метода updateTmpMark напрямую из кода подразумевает подготовку параметров, один из которых - контейнер c записями. Я такие вещи, мягко говоря, недолюбливаю. Судя по перекрестным ссылкам, единственное место, где этот метод используется - это та самая форма. Форма же и занимается подготовкой записей для метода. Я не хотел писать всё это руками для своей задачи, если это уже сделано на форме.

Так вы посмотрите, как этот контейнер с записями готовится. Попробуйте через него. Код получится проще и понятнее, чем у вас. И, кстати, чтобы вызвать форму, тоже много параметров приходится подготовить, не так ли?

[Blog bot]

Источник: http://sashanazarov.blogspot.com/201...continued.html
==============

After publishing a post about manipulating the Marking form from code, I got some feedback (special thanks to Maxim Gorbunov).

I would like my readers to remember, that forms should never be used like that in business logic executed on a daily basis. If this is something that should be run more often than once, this logic should preferably be run on the server, and in the case with the Marking form, it should rather be TmpInventTransMark::updateTmpMark() method called directly. You will need to prepare parameters for it, though, so you will first need to figure out how the form does that.

In other words, please use out-of-the-box solutions you find on the Internet with care.


Источник: http://sashanazarov.blogspot.com/201...continued.html

[Stitch_MS]

Спасибо. Я написал еще один пост, специально для начинающих программистов.

А насчет "посмотреть, как этот контейнер с записями готовится" - я ж и не спорю, что после изучения, как это делает форма, я написал бы более понятный и эффективный код. Очень возможно.

Там же как получилось: посмотрел я на форму, увидел там N-ное количество методов и переменных на уровне формы и сразу вспомнил картинку:



А поскольку я уже знал, как работать с формой из кода, я решил съэкономить время. Вот и всё .