Ошибка в классе SysMailerAddressField в Dynamics Ax 4 SP1. Метод appendAddress.

[Demetrio]

Обнаружил, что при добавлении нескольких адресов для отправки письма - отправляется последний. Заглянул в код метода appendAddress:

void appendAddress(str addr, str name='')
{
    str address;
    str addresses;

    if (prmisdefault(name))
    {
        address = addr;
    }
    else
    {
        address = name + '[' + addr + ']';
    }

    addresses = this.get();

    if (strlen(addresses) > 0)
    {
        address = ',' + address;
    }

    this.set(address);

    addressCount++;
}

Насколько я понимаю, здесь закралась ошибка. Наверное, нужно примерно так:

void appendAddress(str addr, str name='')
{
    ...
    if (strlen(addresses) > 0)
    {
        address =  addresses + ',' + address;
    }
   ...
}

И если это ошибка - не исправлена ли она в SP 2?

[kashperuk]

В 5ке исправлено - правда не так, как Вы предложили:

    if (strlen(addresses) > 0)
    {
        address = ',' + address;
    }

    this.set(addresses + address);

Но суть, в принципе, одна и та же.

[Demetrio]

В любом случае удивляешься порой такому коду.
В таких ситуациях прозрачнее и проще, на мой взгляд, использовать контейнер.

[vladz]

обычно "лучшие практики" советуют для слияния строк исрользовать оператор s += s1 или , например, метод strFmt("%1%2", s, s1). Хотя, наверное, для данного кода разработчики посчитали скорость некритичным параметром.