SASGIS - SAS.Планета
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0001497SAS.Планета[All Projects] Багpublic13-08-2012 10:1004-09-2012 13:10
Reportervasketsov 
Assigned Tovdemidov 
PriorityhighSeveritymajorReproducibilityalways
StatusresolvedResolutionfixed 
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 в форме настроек НЕ выполнять сохранение, если настройки не изменились. Кнопку Применить включать только при наличии изменений в настройках программы, при открытии формы с настройками её серить.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
13-08-2012 10:10vasketsovNew Issue
13-08-2012 10:22vdemidovNote Added: 0008415
13-08-2012 10:28vasketsovNote Added: 0008416
13-08-2012 10:30vdemidovNote Added: 0008417
13-08-2012 10:31vdemidovNote Added: 0008418
13-08-2012 10:48vasketsovNote Added: 0008422
13-08-2012 10:52vdemidovNote Added: 0008423
13-08-2012 11:55TolikNote Added: 0008424
14-08-2012 05:20ParasiteNote Added: 0008433
28-08-2012 08:01vdemidovStatusnew => confirmed
28-08-2012 13:17vdemidovProduct Version.Nightly => 110418
28-08-2012 13:17vdemidovTarget Version => 121010
04-09-2012 13:10vdemidovNote Added: 0008692
04-09-2012 13:10vdemidovStatusconfirmed => resolved
04-09-2012 13:10vdemidovFixed in Version => 121010
04-09-2012 13:10vdemidovResolutionopen => fixed
04-09-2012 13:10vdemidovAssigned To => vdemidov

Notes
(0008415)
vdemidov   
13-08-2012 10:22   
Мне кажется будет достаточно более интеллектуального сохранения меток. А еще нужно запихать чтение и сохранение в отдельный поток.
(0008416)
vasketsov   
13-08-2012 10:28   
Хм. А зачем вообще метки сохранять при сохранении настроек?
(0008417)
vdemidov   
13-08-2012 10:30   
Ну например поменялся путь хранения меток, то что такая настройка не вынесена в GUI это совсем другой вопрос.
(0008418)
vdemidov   
13-08-2012 10:31   
Хотя согласен можно и убрать. Но вот сложности при реализации второго пункта совсем не оправдывают полученный результат.
(0008422)
vasketsov   
13-08-2012 10:48   
>поменялся путь хранения меток
А по мне - так наоборот. Представь, что у нас уже есть разные папки с разными метками. И эта настройка вынесена в гуй. Если при смене пути сохранять метки - всё затрётся. Наоборот надо не сохранять (ну то есть если и сохранять - то только в старое место, и тут кстати да, оптимизация сохранения меток будет полезна), а заново перечитывать из нового места.

>сложности при реализации второго пункта
Он объективно менее приоритетен. Формально это вообще почти что "хотелка", если бы не проблема ожидания из-за пункта 1. И решения конечно хотелось бы универсального, типа ожидания события от "провайдеров" настроек при их изменении. Тогда можно было бы вешаться на это и включать кнопку.
(0008423)
vdemidov   
13-08-2012 10:52   
Ну события то есть, но просто провайдеров, которые могут поменяться, очень много. Задолбаешься подписываться на все изменения. Ну а сохранение при сохранении настроек можно уже сейчас выкидывать. Только нужно добавить признак, что менялась видимость у каких-то меток, а то она в отличие от всех остальных изменений не приводит к автоматическому сохранению.
(0008424)
Tolik   
13-08-2012 11:55   
Ну и надо не сохранять метки при закрытии, если они не изменились.
(0008433)
Parasite   
14-08-2012 05:20   
А если выдавать тупенький диалог "Метки изменились, сохранить? Д\Н" при выходе, не? Тогда и бэкап оных меток рядом не понадобится - пересохраняться будут уже не автоматом, а по подтверждению юзера, кой сам и дурак если что не так.
(0008692)
vdemidov   
04-09-2012 13:10   
Метки теперь сохраняются только если были не сохраненные изменения. Например, если вы меняли видимость отдельных меток, но не изменяли сами метки, то будет выполнено сохранение базы.