Сторнирование заказа на продажу

[Cardagant]

При сторнировании Заказа на продажу, по которому цена продажи была равна 0 грн, складкие проводки не маркируются как Сторнированные и проводка по Главной книге проходит "реверсом", а не сторнированием.
Подскажите пожалуйста в каком направлении искать. Спасибо! DAX 2009

[d&m]

1. сторно по Заказу на продажу - это не маркировка, а возвращенный лот. Возвращенные лоты автоматически проставляются по кнопке Функции / Создать кредит-ноту (вы так сторнируете?)
2. реверс/сторно регулируется флажком "Корректировка по кредиту" (передаётся из параметров модуля "Клиент" на форму разноски накладной, где может редактироваться ручками). Корректировка по кредиту = да - будет сторно.

[Cardagant]

Цитата:

Сообщение от d&m

1. сторно по Заказу на продажу - это не маркировка, а возвращенный лот. Возвращенные лоты автоматически проставляются по кнопке Функции / Создать кредит-ноту (вы так сторнируете?)
.

Да, сторнирую так.

Цитата:

Сообщение от d&m

2. реверс/сторно регулируется флажком "Корректировка по кредиту" (передаётся из параметров модуля "Клиент" на форму разноски накладной, где может редактироваться ручками). Корректировка по кредиту = да - будет сторно.

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

UPD: Кажется, будто какая-то проверка на наличие то ли цены, то ли суммы стоит, но могу ошибаться.

[mnt_dx]

А что у вас пошло в проводку ГК и как вы её вообще получили с нулем?

[Pustik]

Цитата:

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

А что у вас пошло в проводку ГК и как вы её вообще получили с нулем?

Наверное, речь идет о проводках по себестоимости.

[Pustik]

Да, действительно. Вспомнил про эту багу.Столкнулся при переходе.Особенно неприятно, когда она всплывает после пересчетов: Счета переворачиваются.
Вся беда в методе класса SalesFormLetter :

protected void initFromSalesTotals(SalesTotals _salesTotals)
{
    ........................................................
    if (this.canBeCreditnote())
    {
        if (TaxParameters::find().CashDiscOnInvoice)
        {
            cashDiscAmount = salesTotals.cashDiscAmount();

            if (salesTotals.totalAmountCashDiscOnInvoice(true) < 0)
                creditNote = NoYes::Yes;
            else
                creditNote = NoYes::No;
        }
        else
        {
            if (salesTotals.totalAmount() < 0)
                creditNote = NoYes::Yes;
            else
                creditNote = NoYes::No;
        }
    }
    else
    {
        creditNote = NoYes::No;
    }
}

Здесь видно, что сторно это или нет, определяется исключительно только по сумме.

Можно обойтись небольшой кровью:
В классе SalesFormLetter_Invoice в методе UpdateNow() вместо

inventMovement.parmStorno_RU(creditNote && salesParmUpdate.Storno);

можно сделать так

inventMovement.parmStorno_RU((creditNote || salesParmLine.DeliverNow < 0) && salesParmUpdate.Storno);
ledgerVoucher.findLedgerVoucherObject().parmCorrection((creditNote || salesParmLine.DeliverNow < 0) && salesParmUpdate.Storno);

кстати и сторнирование через немедленное получение тоже не помечает проводки как сторно, в случае, когда сумма нулевая

[Pustik]

Сейчас потестировал, все-таки можно и глобально подойти к этому вопросу:
в методе initFromSalesTotals класса SalesFormLetter
вместо

if (salesTotals.totalAmount() < 0)
                creditNote = NoYes::Yes;
            else
                creditNote = NoYes::No;

можно поставить

if (salesTotals.totalAmount() < 0 || salesTotals.totalQty() < 0)
                creditNote = NoYes::Yes;
            else
                creditNote = NoYes::No;

[Cardagant]

2Pustik : Благодарю Вас за помощь! Уже нашёл это место, Вы подтвердили мои мысли по этому поводу!