MantisBT - Community
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0027893Community[OCCT] OCCT:Visualizationpublic2016-09-22 13:432016-12-09 16:38
Reportersolomin_s 
Assigned Tokgv 
PrioritynormalSeveritytrivial 
StatusclosedResolutionfixed 
PlatformAOSLOS VersionL
Product Version[OCCT] 7.0.0 
Target Version[OCCT] 7.1.0Fixed in Version[OCCT] 7.1.0 
Test case numberbugs vis bug27893
Summary0027893: Visualization - AIS_InteractiveContext::SetSelected does not work
Descriptionvoid AIS_InteractiveContext::SetSelected (const Handle(AIS_InteractiveObject)& theObject, const Standard_Boolean theToUpdateViewer) does not work.
Seems code contains typo.
...
  if (theObject->HasSelection (0))
    return;
...
  // added to avoid untimely viewer update...
  const Handle(SelectMgr_Selection)& aSel = theObject->Selection (0);
...

Seems, the first condition should be negate.

p.s. according to the #0026272
- now each selectable object can define own selection mode for "global" selection of the whole object;
- whole object selection mode is 0 by default for all standard interactive objects;
Why is mode 0 used by default instead of the own selection mode?
Steps To Reproducebugs vis bug27893
Additional information
and documentation updates
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-09-22 13:43solomin_sNew Issue
2016-09-22 13:43solomin_sAssigned To => kgv
2016-09-22 13:49solomin_sDescription Updatedbug_revision_view_page.php?rev_id=14762#r14762
2016-09-22 13:49solomin_sSummaryAIS_InteractiveContext::SetSelected does not work => Visualization - AIS_InteractiveContext::SetSelected does not work
2016-09-23 14:32kgvAssigned Tokgv => vpa
2016-09-23 14:32kgvSeverityminor => trivial
2016-09-23 14:32kgvStatusnew => assigned
2016-09-23 15:40gitNote Added: 0058080
2016-09-23 15:41vpaNote Added: 0058081
2016-09-23 15:41vpaAssigned Tovpa => kgv
2016-09-23 15:41vpaStatusassigned => resolved
2016-09-23 15:41vpaSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=14770#r14770
2016-09-23 15:43kgvNote Added: 0058082
2016-09-23 15:43kgvAssigned Tokgv => bugmaster
2016-09-23 15:43kgvStatusresolved => reviewed
2016-09-23 16:19mkvAssigned Tobugmaster => mkv
2016-09-26 12:12mkvNote Added: 0058150
2016-09-26 12:13mkvNote Added: 0058151
2016-09-26 12:13mkvAssigned Tomkv => vpa
2016-09-26 12:13mkvStatusreviewed => assigned
2016-09-26 12:13mkvTest case number => bugs vis bug27893
2016-09-26 20:34gitNote Added: 0058187
2016-09-26 20:35vpaStatusassigned => resolved
2016-09-26 20:35vpaNote Added: 0058188
2016-09-26 20:35vpaAssigned Tovpa => bugmaster
2016-09-26 20:35vpaStatusresolved => reviewed
2016-09-27 12:02mkvAssigned Tobugmaster => mkv
2016-09-28 15:31mkvNote Added: 0058267
2016-09-28 15:33mkvNote Added: 0058269
2016-09-28 15:33mkvAssigned Tomkv => bugmaster
2016-09-28 15:33mkvStatusreviewed => tested
2016-10-03 13:47kgvChangeset attached => occt master 0d5880e1
2016-10-03 13:47kgvAssigned Tobugmaster => kgv
2016-10-03 13:47kgvStatustested => verified
2016-10-03 13:47kgvResolutionopen => fixed
2016-10-28 21:42gitNote Added: 0059456
2016-12-09 16:30aivStatusverified => closed
2016-12-09 16:38aivFixed in Version => 7.1.0

Notes
(0058080)
git   
2016-09-23 15:40   
Branch CR27893 has been created by vpa.

SHA-1: 133d7b582c10cbd0857574881021a0652f8ded43


Detailed log of new commits:

Author: vpa
Date: Fri Sep 23 15:39:56 2016 +0300

    0027893: Visualization - AIS_InteractiveContext::SetSelected does not work
    
    - misprint in check of global selection existence in AIS_InteractiveContext::SetSelected was corrected;
    - remaining occurrencies of hard-coded 0 selection mode were corrected to use global selection mode;
    - test case for issue #27983
(0058081)
vpa   
2016-09-23 15:41   
Dear Kirill,

please review patch in branch CR27893.
(0058082)
kgv   
2016-09-23 15:43   
Please test the patch.
(0058150)
mkv   
2016-09-26 12:12   
Dear BugMaster,
Branch CR27893 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms on Release mode.
SHA-1: 133d7b582c10cbd0857574881021a0652f8ded43

There are following compilation errors:

Windows:
http://jenkins-test-07.nnov.opencascade.com:8080/view/CR27893-master/job/CR27893-master-OCCT-Windows-64-VC10-opt-compile/1/parsed_console/ [^]

59>..\..\..\src\QABugs\QABugs_19.cxx(5290): error C2668: 'AIS_InteractiveContext::SetSelected' : ambiguous call to overloaded function
...

1>..\..\..\src\QABugs\QABugs_19.cxx(5290): error C2668: 'AIS_InteractiveContext::SetSelected' : ambiguous call to overloaded function
....
(0058151)
mkv   
2016-09-26 12:13   
Dear vpa,
Branch CR27893 has been rejected due to:
- compilation errors
(0058187)
git   
2016-09-26 20:34   
Branch CR27893 has been updated forcibly by vpa.

SHA-1: 2bfeb5ab802c6804151a786fefb84668d7053661
(0058188)
vpa   
2016-09-26 20:35   
Dear Bugmaster,

please test updated patch in branch CR27893.
(0058267)
mkv   
2016-09-28 15:31   
Dear BugMaster,
Branch CR27893 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested on Release mode.
SHA-1: 2bfeb5ab802c6804151a786fefb84668d7053661

Number of compiler warnings:

occt component :
Linux: 0 (0 on master)
Windows: 0 (0 on master)
MacOS : 0 (0 on master)

products component :
Linux: 64 (64 on master)
Windows: 0 (0 on master)
MacOS : 1144

Regressions/Differences/Improvements:
No regressions/differences

Testing cases:
http://occt-tests/CR27893-master-OCCT/Debian70-64/bugs/vis/bug27893.html [^]
http://occt-tests/CR27893-master-OCCT/Windows-64-VC10/bugs/vis/bug27893.html [^]
bugs vis bug27893: OK

Testing on Linux:
occt component :
Total MEMORY difference: 91002176 / 90234529 [+0.85%]
Total CPU difference: 19421.73999999987 / 19251.709999999875 [+0.88%]
products component :
Total MEMORY difference: 30039679 / 30046985 [-0.02%]
Total CPU difference: 5131.029999999967 / 5151.099999999975 [-0.39%]

Testing on Windows:
occt component :
Total MEMORY difference: 57214174 / 57221541 [-0.01%]
Total CPU difference: 18318.46422529868 / 18310.78897609872 [+0.04%]
products component :
Total MEMORY difference: 21271108 / 21235672 [+0.17%]
Total CPU difference: 4900.381412499953 / 4988.631178199945 [-1.77%]

There are no differences in images found by testdiff.
(0058269)
mkv   
2016-09-28 15:33   
Dear BugMaster,
Branch CR27893 is TESTED.
(0059456)
git   
2016-10-28 21:42   
Branch CR27893 has been deleted by kgv.

SHA-1: 2bfeb5ab802c6804151a786fefb84668d7053661