Опасный метод PurchLine в VendInvoiceTrans

[Brave]

Здравствуйте! Axapta 3.0
Недавно накололся на методе в таблице VendInvoiceTrans.PurchLine()

Ожидалось, что данный метод вернёт мне строку закупки по накладной, но в некоторых случаях этого не происходило.

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

PurchLine purchLine()
{
  PurchLine purchLine;
  ;
  select purchLine
    where purchLine.purchId == this.purchID &&
               purchLine.inventTransId == this.inventTransId;
  return purchLine;
}

Собственно в методе VendInvoiceTrans.PurchLine() осуществляется выборка по PurchId и InventTransId. Зачем?! Достаточно одного InventTransId, так как он является первичным ключиком в строках закупки.
Глянул этот метод в Ax2009 - аналогичный код.
В 2012 глянуть возможности нет.

Как вы считаете? Старый баг? Или специальный метод?
Как выглядит метод в Ax2012?

[oip]

В AX2012 только InventTransId.

public PurchLine purchLine()
{
    return PurchLine::findInventTransId(this.InventTransId);
}

static PurchLine findInventTransId(TradeInventTransId   _inventTransId,
                                   boolean              _update = false)
{
    PurchLine purchLine;

    if (_inventTransId)
    {
        purchLine.selectForUpdate(_update);

        select firstonly purchLine
            where purchLine.InventTransId == _inventTransId
               && !purchLine.IsDeleted;
    }

    return purchLine;
}

[Brave]

Спасибо! Значит это была ошибка. Исправлю у себя.

 select firstonly purchLine
            where purchLine.InventTransId == _inventTransId
               && !purchLine.IsDeleted;

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

[dech]

В АХ 4.0 тот же код, что и в 3.0. Не думаю, что это баг. Тем более, что и в 5 версии то же самое, и никто ничего не исправил. 2012 очень сильно нормализована, поэтому там достаточно ссылки лишь на InventTransId.
Мне кажется, у вас не заполнен VendInvoiceTrans.PurchId, либо заполнен чем-то другим, раз уж метод у вас возвращает что-то не то.

[Brave]

VendInvoiceTrans.PurchID заполнен кодом закупки, из которой формируется накладная (приходный ордер).
VendInvoiceTrans.OrigPurchId как раз указывает на нужную закупку, откуда выбрали строку.
И более того - зачем искать с кодом закупки и лотом, когда лот первичный ключ в строках. Не оптимальный запрос получается в итоге.

[Ivanhoe]

Думаю, ошибка в том, что не Orig используется - действительно, в просто PurchID писалась "первая" закупка при обработке накладной по нескольким закупкам.
А вот зачем в принципе связи по коду закупки помимо лота - перестраховка, наверное. Можно и убрать.

[dech]

Цитата:

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

А вот зачем в принципе связи по коду закупки помимо лота - перестраховка, наверное. Можно и убрать.

Согласен, InventTransId - уникальный ключ, можно искать строку закупки только по одному этому полю.

Цитата:

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

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

Смотрите, получается, если ничего не менять в коде метода purchLine(), то он вам возвращает не ту запись, которую вы ожидали. Как это так получилось? Логично было, если бы метод вообще ничего не вернул. На этом основании я полагаю, что у вас где-то нарушена бизнес-логика.

[Brave]

Вы немного не так интерпретировали

Цитата:

Ожидалось, что данный метод вернёт мне строку закупки по накладной, но в некоторых случаях этого не происходило.

Я получал пустой курсор.

[IvanS]

В 2009 тоже недавно с этим столкнулся. Либо нужно ставить OrigPurchId или просто оставлять InventTransId.