MarkupTable.checkCustAccount(), баг?

[S.Kuskov]

Не вдаваясь в смысл проверок, могу предложить моё видение этого кода

if (A && B) // (1) сюда попадём если условие A = true и B = true
else if (B) // (2) сюда попадём если условие A = false, a B = true

получается для возникновения ошибки условие B должно быть всегда истино. Т.е. логично было бы написать так:

if (B)
{
    if (A) {/* (1) сюда попадём если условие A = true и B = true */}
    else   {/* (2) сюда попадём если условие A = false, a B = true */}
}

В итоге получаем, что-то вроде

    if (this.CustType != MarkupType::Item || !PurchParameters::find().PostChargeAccount)
    {
        if ((this.CustType != MarkupType::LedgerAccount) && this.CustAccount)
            ok = checkFailed("@SYS25165");
        if ((this.CustType == MarkupType::LedgerAccount) && !this.CustAccount)
            ok = checkFailed("@SYS25989");
    }