Показать сообщение отдельно
Старый 30.08.2019, 11:06   #392  
dech is offline
dech
Участник
Аватар для dech
Самостоятельные клиенты AX
 
548 / 247 (10) ++++++
Регистрация: 25.06.2009
Адрес: Омск
Записей в блоге: 3
Цитата:
Сообщение от George Nordic Посмотреть сообщение
Что должен вернуть void???
Я не про тип, а про название. В названии должен быть хотя бы префикс set, иначе метод выглядит как возвращающий нечто (в данном случае похожее на запись InventDim).
Второе, зачем в private-метод пихать единственную операцию присваивания? В контексте вызов подобного метода только дизориентирует. Другое дело, если бы метод вызывался наследниками, хотя все равно над названием надо было лучше думать.
Еще нарыл в VendInvoiceDocument.main() 2 условия по Voucher
X++:
            select firstOnly TableId, RecId from taxEngineLedgerJournalTransHeader
                join RecId from ledgerJournalTransLoc
                    where ledgerJournalTransLoc.AccountType    == LedgerJournalACType::Vend
                        && ledgerJournalTransLoc.JournalNum    == taxEngineLedgerJournalTransHeader.JournalNum
                        && ledgerJournalTransLoc.Voucher       == taxEngineLedgerJournalTransHeader.Voucher
                        && ledgerJournalTransLoc.Invoice       == vendInvoiceJournal.InvoiceId
                        && ledgerJournalTransLoc.Voucher       == vendInvoiceJournal.LedgerVoucher;
Кроме того, TableId не нужно указывать в списке полей селекта.
__________________
Незаменимых людей нет. Но и в отпуск мы тебя не отпустим. © Начальник

Последний раз редактировалось dech; 30.08.2019 в 11:10.