Сторно по Подотчетнику

[BOAL]

На АХ4Сп2ХФ (не помню какой, но от лета 2008 + лос слой от зимы 2009)

В АО есть кнопка Отклонить, если делать через нее, то проводки ГК идут коррекцией
Если делать руками с минусом в строке будет реверс.

С Отменой через кнопку есть два дефекта (доделать).
1. Проводка по сотруднику делается без галки коррекция, тогда как внутри ГК все со сторно
Можно дописать
\Classes\EmplAdvReject_RU\runFreeLines
emplTransStorno.Correct = NoYes::Yes; // fix

2. Имеет смысл на форму АО добавить в Запросы менюИтем EmplTrans_RU, тогда поэтому запросу будет видно все проводки по сотруднику по всем строкам этого АО (просто удобно) и (ВАЖНО) проводки по отмене АО. Иначе их не видно из АО, что есть дефект.

Если очень нужно, то можно и сторно делать при минусе в АО, но зачем, если есть кнопка Отменить?

[vanokh]

Цитата:

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

На АХ4Сп2ХФ (не помню какой, но от лета 2008 + лос слой от зимы 2009)

В АО есть кнопка Отклонить, если делать через нее, то проводки ГК идут коррекцией
Если делать руками с минусом в строке будет реверс.

С Отменой через кнопку есть два дефекта (доделать).
1. Проводка по сотруднику делается без галки коррекция, тогда как внутри ГК все со сторно
Можно дописать
\Classes\EmplAdvReject_RU\runFreeLines
emplTransStorno.Correct = NoYes::Yes; // fix

2. Имеет смысл на форму АО добавить в Запросы менюИтем EmplTrans_RU, тогда поэтому запросу будет видно все проводки по сотруднику по всем строкам этого АО (просто удобно) и (ВАЖНО) проводки по отмене АО. Иначе их не видно из АО, что есть дефект.

Если очень нужно, то можно и сторно делать при минусе в АО, но зачем, если есть кнопка Отменить?

Да, именно эти хотфиксы и стоят. По ГК понятно что уже есть сторно, нужно было именно сторно по подотчетникам - галочка коррекция в проводках EmplTrans. Для АО так и сделали, только добавив галочку в настройки:

        if (EmplParameters_RU::find().TransactionReversalCorrection)
        {
            emplTransStorno.Correct = NoYes::Yes;
        }

Про запросы из АО - спасибо, недоглядели, теперь добавим

А теперь про вторую и основную часть АО это конечно хорошо, но еще есть возможность делать проводки EmplTrans из журнала ГК. Это и была основная проблема. Завязавшись на ту же добавленную настройку, сделали следующее:

EmplVoucher_RU.post()
...
    if (EmplParameters_RU::find().TransactionReversalCorrection)
    {
        emplTrans.Correct = ledgerVoucher.parmCorrection(emplTrans.Voucher);
    }

Скопировано из CustVendVoucher Также по образцу CustTrans в EmplTrans добавлен метод postLoad.

[BOAL]

Галочки такой в стандарте нет (смутило отсутствие суфикса).
И использование ее в обоих местах сомнительно, тк в EmplVoucher_RU.post() там уже сидит корректное заполнение Да-нет
А в отмене нужно тогда тянуть ее в ГК проводку.
То есть, сторно должно быть в паре - ГК +сущность

ЖГК не есть проблема, там, если делать руками, знак в дебете-кредите определяет сторно или нет.
Это в АО знак минус дает реверс, а сторно только кнопкой.

[vanokh]

Цитата:

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

Галочки такой в стандарте нет (смутило отсутствие суфикса).

Я ж сказал, галочку добавили.

Цитата:

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

И использование ее в обоих местах сомнительно, тк в EmplVoucher_RU.post() там уже сидит корректное заполнение Да-нет
А в отмене нужно тогда тянуть ее в ГК проводку.
То есть, сторно должно быть в паре - ГК +сущность

Нет, в post() до этого EmplTrans.Correct никак не заполнялось. Только в LedgerTrans - заполняется стандартным механизмом по знаку в дебете-кредите. Поэтому мы и добавили галочку в настройку и заполнение Correct.

Цитата:

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

ЖГК не есть проблема, там, если делать руками, знак в дебете-кредите определяет сторно или нет.

Да, определяет, но только для проводок ГК, в EmplTrans всегда шел реверс.

Цитата:

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

Это в АО знак минус дает реверс, а сторно только кнопкой.

Реверс да, уже была функциональность, а сторно не было реализовано, для этого в EmplAdvReject_RU\runFreeLines добавили явное заполнение Correct, если в настройках стоит галочка Сторно вместо Реверса (см. выше)

[BOAL]

Мысль не постиг

Штатно (без всяких правок) в
EmplVoucher_RU.post()
была строка, которую вы зачем-то обернули в Если с галкой.

При этом вы нарушили (осознано) целостность системы
Эта галка врубает режим, когда в ГК красное сторно, а в Сотрудники реверс
Что есть баг

Был в стандарте баг в отмене АО, когда в ГК было красное сторно, а в сотрудники реверс - это лечится приведенной мной строкой, без всяких Если с галками.

[vanokh]

Цитата:

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

Мысль не постиг
Штатно (без всяких правок) в
EmplVoucher_RU.post()
была строка, которую вы зачем-то обернули в Если с галкой.

В том то и дело, что этой строки не было - Ax 4.0 sp2 fp1 + зимние хотфиксы по курсовым разницам (37376, 39274), версия приложения 4.0.2501.347.
То есть было всегда по ГК - сторно, по сотрудникам - реверс. В постановке задачи сказано (см. первый пост), что иногда (!) надо делать сторно, а в остальных случаях реверс. Поэтому и обернул в Если с галкой. Аналогично в отмене АО.

Цитата:

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

При этом вы нарушили (осознано) целостность системы
Эта галка врубает режим, когда в ГК красное сторно, а в Сотрудники реверс
Что есть баг

А в настройках ГК аналогичная галочка Реверсирование операции-Коррекция не нарушает целостность? ;-)
Ага, нашел. Не нарушает - просто не дает сторнировать операции по подотчетникам
Подумаем еще насчет целостности...