Дыра в запрете редактирования шапок заказов на продажу/покупку

[gl00mie]

Стандартный подход к тому, как регулировать доступ к полям записи, состоит в том, чтобы на обработчик formDS.active() повесить некую логику, которая, скажем, будет управлять свойством formDS.allowEdit(). Примерно так же сделано в заказах на продажу и покупку, скажем, в SalesTable_DS.active() вызывается

element.editSales(salesTableForm.editHeaderAllowed());

и почти наверняка на каждом проекте в этой логике есть свои модификации. Так вот, в формах SalesTable/PurchTable создание новых заказов реализовано таким образом, что в этом стандартном подходе появляется зияющая дыра. Взглянем на код SalesTable_DS.create():

void create(boolean  append = true)
{
    SalesTable   newSalesTable;
    ;
    salesTableForm.interCompanyAutoCreateOrders();
    element.editSales(true);
    if (salesTableForm.create())
    {
        newSalesTable = SalesTable::find(salesTableForm.newSalesId());
        if (newSalesTable)
        {
            super(append);
            salesTable.data(newSalesTable);
            salesTable_ds.reread();
            tabLineOverview.setFocus();
        }
    }
    else
    {
        element.editSales(salesTableForm.editHeaderAllowed(), salesTableForm.deleteHeaderAllowed());
    }
}

Занятно здесь то, что возврат из salesTableForm.create() после безусловного разрешения редактирования текущего заказа происходит отнюдь не мгновенно, а лишь после того, как пользователь закроет форму создания заказа. В это время можно переключиться обратно на форму заказов и - о, чудо! - увидеть, что благодаря вызову element.editSales(true) шапка заказа независимо от его статуса стала доступна для редактирования. Конечно, на validateWrite() и write() остались проверки, которые могут воспрепятствовать изменению, к примеру, заказа в статусе "Отгружено", но сам факт того, что есть штатная возможность обойти логику, управляющую доступом к шапке заказа, как-то удручает. Эти рассуждения в равной степени касаются PurchTable, причем что в 3.0, что в AX 2009.

Решение у проблемы весьма простое: нужно в create() перенести вызов element.editSales(true) внутрь if непосредственно перед super() и аналогично поступить с вызовом element.editPurch(true) в форме PurchTable.

а что, service request в Microsoft автоматом с форума создается для исправления багов?

[Maximin]

Мы эту проблему решили просто - сделав окно создания заказа SalesCreateOrder немодальной с параметром TopMost. В результате, переключиться никуда не выйдет. Кстати, описанная проблема - не единичная, в кач-ве направления раскопок могу привести форму сопоставления платежей - там тоже есть дырка наподобие. Да и вообще, разработчики часто забывают о том, что текущее окно - м.б. не единственным в системе.

[gl00mie]

Цитата:

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

а что, service request в Microsoft автоматом с форума создается для исправления багов?

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

Цитата:

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

Мы эту проблему решили просто - сделав окно создания заказа SalesCreateOrder немодальной с параметром TopMost. В результате, переключиться никуда не выйдет.

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

[Maximin]

Цитата:

Т.е. вы решили закрыть дыру за счет того, что форма создания заказа закрывает собой другие формы? А если у меня монитор с очень большим разрешением, и я могу просто сдвинуть форму создания заказа вбок, чтоб ничего не заслоняла?

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

[AlexSD]

Всю жизнь считал, что с модального окна нельзя увести фокус... А тут вот как оказывается...

[Maximin]

Уел , сейчас исправлю, писал одно - думал другое.

Цитата:

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

Всю жизнь считал, что с модального окна нельзя увести фокус... А тут вот как оказывается...