SASGIS - SAS.Планета
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0002985SAS.Планета[All Projects] Багpublic13-03-2016 06:5216-03-2016 10:50
ReporterGarl 
Assigned Tozed 
PrioritynormalSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformWindowsOS7OS VersionProfessional
Product Version.Nightly 
Target Version160606Fixed in Version160606 
Summary0002985: Access violation при изменении метки
DescriptionБерём любую метку, жмём пкм - изменить кликаем в 20 метрах рядом и получаем вот это.
метки в Marks.db3

Steps To Reproduce
Additional Informationночнушка последняя доступная для скачивания.
TagsNo tags attached.
Relationships
has duplicate 0002989closed zed Ошибка при изменении метки 
Attached Files? 13-03-2016SASPlanet.Debug.elf (188,650) 13-03-2016 06:52
http://www.sasgis.org/mantis/file_download.php?file_id=2034&type=bug
Issue History
Date ModifiedUsernameFieldChange
13-03-2016 06:52GarlNew Issue
13-03-2016 06:52GarlFile Added: 13-03-2016SASPlanet.Debug.elf
16-03-2016 10:14zedRelationship addedhas duplicate 0002989
16-03-2016 10:22zedNote Added: 0017082
16-03-2016 10:22zedStatusnew => confirmed
16-03-2016 10:45zedNote Added: 0017084
16-03-2016 10:45zedStatusconfirmed => resolved
16-03-2016 10:45zedFixed in Version => 160606
16-03-2016 10:45zedResolutionopen => fixed
16-03-2016 10:45zedAssigned To => zed
16-03-2016 10:45zedTarget Version => 160606
16-03-2016 10:48vdemidovNote Added: 0017085
16-03-2016 10:50vdemidovNote Edited: 0017085bug_revision_view_page.php?bugnote_id=17085#r6881
16-03-2016 10:50vdemidovNote Edited: 0017085bug_revision_view_page.php?bugnote_id=17085#r6882

Notes
(0017082)
zed   
16-03-2016 10:22   
Там в вызывающем коде то ли опечатка, то ли с логикой какая проблема.



    VVisible := FMarkSystem.MarkDb.GetMarkVisible(AMark);

    if AAsNewMark then begin

      VSourceMark := nil;

    end else begin

      VSourceMark := AMark;

    end;

    if ADescription <> '' then begin

      VMark := FMarkSystem.MarkDb.Factory.ModifyName(AMark, '', ADescription);

    end;

    VMark := FMarkSystem.MarkDb.Factory.ModifyGeometry(VMark, AGeometry);

Скорее всего, в последней строчке в ModifyGeometry должно передаваться AMark. Но тогда не понятно, зачем там мелькает по коду выше VSourceMark?
(0017084)
zed   
16-03-2016 10:45   
Кажется исправил: https://bitbucket.org/sas_team/sas.planet.src/commits/684854b79c4d365193ae81662a002875e3073fa8
(0017085)
vdemidov   
16-03-2016 10:48   
(edited on: 16-03-2016 10:50)
Мда, что-то я тут наглючил.
PS: Да именно так и должно было быть.

Спасибо.