View Issue Details

IDProjectCategoryView StatusLast Update
0001497SAS.ПланетаБаг / Bugpublic04-09-2012 13:10
Reportervasketsov Assigned Tovdemidov  
PriorityhighSeveritymajorReproducibilityalways
Status resolvedResolutionfixed 
PlatformWindowsOSVistaOS VersionUltimate
Product Version110418 
Target Version121010Fixed in Version121010 
Summary0001497: Сохранение настроек программы неоптимально
DescriptionИтак по часам (GPS отключен, никаких приколов типа сложной текущей области выделения):
1. Запускаем сас. Запустился. Закрываем по крестику. Время закрытия - 13-15 секунд.
2. Запускаем сас. Запустился. Открываем настройки программы. Жмём OK. Настройки закрываются те же 13-15 секунд. Плюс-минус секунда-две.

Теперь закрываем сас и удаляем файлы с метками (меток под 100 МБ). Повторяем:
1. Пара секунд.
2. Мгновенно.

Итого необходимо:
1. Удалить сохранение меток при сохранении (Применить или OK) настроек программы.
2. При нажатии на OK в форме настроек НЕ выполнять сохранение, если настройки не изменились. Кнопку Применить включать только при наличии изменений в настройках программы, при открытии формы с настройками её серить.
TagsNo tags attached.

Activities

vdemidov

13-08-2012 10:22

manager   ~0008415

Мне кажется будет достаточно более интеллектуального сохранения меток. А еще нужно запихать чтение и сохранение в отдельный поток.

vasketsov

13-08-2012 10:28

manager   ~0008416

Хм. А зачем вообще метки сохранять при сохранении настроек?

vdemidov

13-08-2012 10:30

manager   ~0008417

Ну например поменялся путь хранения меток, то что такая настройка не вынесена в GUI это совсем другой вопрос.

vdemidov

13-08-2012 10:31

manager   ~0008418

Хотя согласен можно и убрать. Но вот сложности при реализации второго пункта совсем не оправдывают полученный результат.

vasketsov

13-08-2012 10:48

manager   ~0008422

>поменялся путь хранения меток
А по мне - так наоборот. Представь, что у нас уже есть разные папки с разными метками. И эта настройка вынесена в гуй. Если при смене пути сохранять метки - всё затрётся. Наоборот надо не сохранять (ну то есть если и сохранять - то только в старое место, и тут кстати да, оптимизация сохранения меток будет полезна), а заново перечитывать из нового места.

>сложности при реализации второго пункта
Он объективно менее приоритетен. Формально это вообще почти что "хотелка", если бы не проблема ожидания из-за пункта 1. И решения конечно хотелось бы универсального, типа ожидания события от "провайдеров" настроек при их изменении. Тогда можно было бы вешаться на это и включать кнопку.

vdemidov

13-08-2012 10:52

manager   ~0008423

Ну события то есть, но просто провайдеров, которые могут поменяться, очень много. Задолбаешься подписываться на все изменения. Ну а сохранение при сохранении настроек можно уже сейчас выкидывать. Только нужно добавить признак, что менялась видимость у каких-то меток, а то она в отличие от всех остальных изменений не приводит к автоматическому сохранению.

Tolik

13-08-2012 11:55

manager   ~0008424

Ну и надо не сохранять метки при закрытии, если они не изменились.

Parasite

14-08-2012 05:20

administrator   ~0008433

А если выдавать тупенький диалог "Метки изменились, сохранить? Д\Н" при выходе, не? Тогда и бэкап оных меток рядом не понадобится - пересохраняться будут уже не автоматом, а по подтверждению юзера, кой сам и дурак если что не так.

vdemidov

04-09-2012 13:10

manager   ~0008692

Метки теперь сохраняются только если были не сохраненные изменения. Например, если вы меняли видимость отдельных меток, но не изменяли сами метки, то будет выполнено сохранение базы.

Issue History

Date Modified Username Field Change
13-08-2012 10:10 vasketsov New Issue
13-08-2012 10:22 vdemidov Note Added: 0008415
13-08-2012 10:28 vasketsov Note Added: 0008416
13-08-2012 10:30 vdemidov Note Added: 0008417
13-08-2012 10:31 vdemidov Note Added: 0008418
13-08-2012 10:48 vasketsov Note Added: 0008422
13-08-2012 10:52 vdemidov Note Added: 0008423
13-08-2012 11:55 Tolik Note Added: 0008424
14-08-2012 05:20 Parasite Note Added: 0008433
28-08-2012 08:01 vdemidov Status new => confirmed
28-08-2012 13:17 vdemidov Product Version .Nightly => 110418
28-08-2012 13:17 vdemidov Target Version => 121010
04-09-2012 13:10 vdemidov Note Added: 0008692
04-09-2012 13:10 vdemidov Status confirmed => resolved
04-09-2012 13:10 vdemidov Fixed in Version => 121010
04-09-2012 13:10 vdemidov Resolution open => fixed
04-09-2012 13:10 vdemidov Assigned To => vdemidov
08-08-2025 13:22 zed Category Баг => Баг / Bug