Еще один импорт курсов валют

[oip]

Еще совсем мелкие замечание. У формы неплохо бы сделать column width и column height, у таблицы GM_AxCurCode2CBRCurCode поля CurrencyCode и CBRCurrencyCode по логике должны быть мандатори. В проекте в трех местах используются метки @GMC*, однако меточного файла не приложено, да и в других местах метки не используются. Надо бы однообразие. Или метки и меточный файл или везде текст.

Форма сейчас работает по совсем неочевидной для пользователя логике. Создаем записи, задав только CurremcyCode, но при следующем открытии формы она откроется пустой из-за ренджа в ините, но создавать новые записи уже не получится, так как в таблице они есть. Надо бы как-то иначе.

Upd. Ой, не так с мандатори . CBRCurrencyCode == "" - это RUR? Тоже не очень прозрачно и хорошо.

[Zabr]

Цитата:

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

В проекте в трех местах используются метки @GMC*

Не нашел. Утром проект скачивал. Может поменяли уже ?

[Zabr]

Цитата:

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

Надо бы как-то иначе.

Вряд ли у ЦБ поменяются коды валют, так что можно было бы облегчить жизнь юзерам: при первом запуске формы сразу заполнять справочник (зашить список в код). Либо к проекту приложить файл для импорта, скачанный с сайта ЦБ.

[Zabr]

Цитата:

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

у таблицы GM_AxCurCode2CBRCurCode поля CurrencyCode и CBRCurrencyCode по логике должны быть мандатори

.. а еще и уникальными. сейчас можно для нескольких кодов валют ЦБ установить одинаковый код валюты Аксапты.

А еще: и при импорте списка валют, и при импорте курсов - можно указать неправильное (несуществующее) имя файла, нажать ОК и получить вместо сообщения об ошибке сообщение "импорт завершен". Как будто все в порядке. Какая тут есть опасность: например, кто-то регулярно выгружает файл с курсами, а задание на загрузку в Аксапте стоит в пакете. Если изменится местоположение каталога для выгрузки или файл назовут неправильно, а пакет продолжает работать, то никто не узнает, что файл перестал загружаться.