Бага в стандарте

[MironovI]

В SalesLine нашел дивный метод - используется после обработки документов по заказу - ставит в шапку минимальный статус из строк, только вот if (salesLine) никогда не отработает.. Как переписать думаю понятно кому захочется, просто времени нет..
static SalesStatus lowestSalesStatus(SalesId salesId)
{
SalesLine salesLine;
;

select minof(salesStatus) from salesLine
index hint SalesStatusIdx
where salesLine.salesId == salesId &&
salesLine.salesStatus > SalesStatus::None;

if (salesLine)
return salesLine.salesStatus;

return SalesStatus::Backorder;
}

[kashperuk]

Да нет. В этом коде все верно.

X++:

;
box::info(enum2str(SalesLine::lowestSalesStatus('00000010_061')));

По этому заказу мне выводит статус "Отгружено".

Вас смутило, видимо, то, что выбирается только maxOf() .

if (salesLine) будет проходить иногда.
(хотя в debugger показывает recId = 0)

Видимо, это еще одно доказательство того, что следует всегда писать именно
if (salesLine) а не if (salesLine.RecId != 0)

[MironovI]

Будет выбираться? Я уменя не выбирается.. правда на другом примере проверял.. хм..

[MironovI]

фигняс, у меня recid не выбирается.. и мне это кажется весьмалогичным..

[kashperuk]

Правильно. RecId и не выбирается.
А вот условие проходит.
Проверьте. Создайте заказ, и обработайте накладную.

На PurchLine то же самое, кстати.

[MironovI]

Цитата:

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

Видимо, это еще одно доказательство того, что следует всегда писать именно
if (salesLine) а не if (salesLine.RecId != 0)

Ну пару раз я нарывался на то, что при заполненном Recid условие if(table) проскакивало дальше. т.е. в упор не видело выбранного Recid, так что тож не факт..
Вообще это УЖОС какой-то, я перестал понимать как оно внутрях и что там себе думает..

[petr]

Да все же по-моему просто.
if (table) срабатывает в true если хоть одно поле таблицы имеет значение отличное от пустого. Т.е. србатывает и здесь если поле SalesLine.salesStatus выбрано.
Условие if (table.RecId) понятно дело срабатывает когда recId != 0, т.е. либо выбрана запись в таблице полностью, либо была выборка конкретно recId, например select RecId from table...

[petr]

А если ответить в тему, то, по-моему, баг в стандартном функционале присутствует в методе: PurchCalcTax_Invoice.initCursor()
Привожу этот метод с моими исправлениями.
select nofetch forupdate vendInvoiceTrans
index hint InvoiceIdx
where vendInvoiceTrans.purchID == vendInvoiceJour.purchId &&
vendInvoiceTrans.invoiceId == vendInvoiceJour.invoiceId &&
vendInvoiceTrans.invoiceDate == vendInvoiceJour.invoiceDate &&
// STM, BugFix, 25.11.2005, kpn, -->
vendInvoiceTrans.InternalInvoiceId == vendInvoiceJour.InternalInvoiceId &&
vendInvoiceTrans.numberSequenceGroup == vendInvoiceJour.numberSequenceGroup;
// STM, BugFix, 25.11.2005, kpn, <--
}

[MironovI]

Цитата:

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

Да все же по-моему просто.
if (table) срабатывает в true если хоть одно поле таблицы имеет значение отличное от пустого. Т.е. србатывает и здесь если поле SalesLine.salesStatus выбрано..

Э, вот тут самое интересное - SalesStatus::None тоже значение и тоже может быть в выбрано, но по вашей терминологии оно пустое..

тоже самое select minof(salesqty) from salesLine - тоже выдаст допустим 0(ноль) - но система знает что salesLine выбрана, хотя Recid будет пустой,

а вот select minof(salesqty) from salesLine where salesLine.SalesId = 'кря_кря_нет такого заказа' - система поймет что запись не выбрана

вопрос - как?

[petr]

На этот вопрос, наверное, смогут ответить только сотрудники MS. Согласн, что по виду самой записи (все поля пустые) нельзя определить выбрана она с пустыми значениями полей или она вообще не выбрана.

[AndyD]

Сама по себе запись - только видимая часть айсберга.

Не забывайте, табличная переменная может быть как связана с результатом выбоки, так и нет.

Если не связана, то, на сколько я понимаю, проверка курсора эквивилентна проверке на RecId (если даже заполнить другие поля, короме RecId, то проверка не пройдет).

Если связана (результат select или QueryRun), то проверяется результат выбоки, хранящейся в памяти и не доступный из кода. Табличная переменная показывает лишь полученные данные.
Как пример - можно сделать выборку из любой таблицы, что бы в курсор вернулась хоть одна запись (с заполненным recId) и обнулить recId - проверка пройдет успешно.

Есть еще один момент - если в результате выбоки не была получена ни одна запись, то табличная переменная получается не связанная (если изменить recId, то проверка пройдет успешно).

Вообще, вывод из этого - проверяйте табличную переменную,а не RecId

[MironovI]

Цитата:

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

А если ответить в тему, то, по-моему, баг в стандартном функционале присутствует в методе: PurchCalcTax_Invoice.initCursor()
Привожу этот метод с моими исправлениями.
select nofetch forupdate vendInvoiceTrans
index hint InvoiceIdx
where vendInvoiceTrans.purchID == vendInvoiceJour.purchId &&
vendInvoiceTrans.invoiceId == vendInvoiceJour.invoiceId &&
vendInvoiceTrans.invoiceDate == vendInvoiceJour.invoiceDate &&
// STM, BugFix, 25.11.2005, kpn, -->
vendInvoiceTrans.InternalInvoiceId == vendInvoiceJour.InternalInvoiceId &&
vendInvoiceTrans.numberSequenceGroup == vendInvoiceJour.numberSequenceGroup;
// STM, BugFix, 25.11.2005, kpn, <--
}

Вот какие бывают совпадения, сегодня поймал аналогичную багу в SalesCalcTax_Invoice\initCursor и впомнил про этот пост Правда поймать ее имеют шанс далеко не все - те кто обрабатывают накладные с одинаковыми номерами но с разными группами номерных серий

void initCursor()
{;
select nofetch forupdate custInvoiceTrans
index hint InvoiceIdx
where custInvoiceTrans.salesId == custInvoiceJour.salesId &&
custInvoiceTrans.invoiceId == custInvoiceJour.invoiceId &&
custInvoiceTrans.invoiceDate == custInvoiceJour.invoiceDate &&
// фикса, imir, 03.10.06 -->
custInvoiceTrans.numberSequenceGroup == custInvoiceJour.numberSequenceGroup;
// фикса, imir, 03.10.06 <--
}

[petr]

А интересно зафиксирован ли этот баг в MS. Ведь исправить - плевое дело. Может правда он уже исправлен в свежих SP?
В принципе этот баг мало на что влияет. Я наткнулся на него, когда стал делать отчет по строкам накладных поставщиков и увидел неправильные сумма НДС.

[NetBus]

Цитата:

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

Может правда он уже исправлен в свежих SP?

По крайне мере в Axapta 3.0 sp4 исправлено

PHP код:

void  initCursor()
{;
    select nofetch forupdate vendInvoiceTrans
        index hint InvoiceIdx
        where vendInvoiceTrans.purchID             == vendInvoiceJour.purchId             &&
              vendInvoiceTrans.invoiceId           == vendInvoiceJour.invoiceId           &&
              vendInvoiceTrans.InternalInvoiceId   == vendInvoiceJour.InternalInvoiceId   &&
              vendInvoiceTrans.numberSequenceGroup == vendInvoiceJour.numberSequenceGroup &&
              vendInvoiceTrans.invoiceDate         == vendInvoiceJour.invoiceDate;
} 


[MironovI]

На SP3 точно нет

[36AC]

у нас ax 3.0 sp3 cu1

PHP код:

void  initCursor()
{;
    select nofetch forupdate vendInvoiceTrans
        index hint InvoiceIdx
        where vendInvoiceTrans.purchID             == vendInvoiceJour.purchId             &&
              vendInvoiceTrans.invoiceId           == vendInvoiceJour.invoiceId           &&
              vendInvoiceTrans.InternalInvoiceId   == vendInvoiceJour.InternalInvoiceId   &&
              vendInvoiceTrans.numberSequenceGroup == vendInvoiceJour.numberSequenceGroup &&
              vendInvoiceTrans.invoiceDate         == vendInvoiceJour.invoiceDate;
}