0025009: Draw Harness, ViewerTest - Incorrect handling of comma and period keys pressed in a 3D view

ID	Project	Category	View Status	Date Submitted	Last Update

0025009	Open CASCADE	OCCT:DRAW	public	2014-06-15 19:18	2015-05-14 15:33

Reporter	~~san~~	Assigned To	bugmaster
Priority	normal	Severity	minor
Status	closed	Resolution	fixed
Platform	Windows	OS	VC++ 2010
Product Version	6.7.1
Target Version	6.9.0	Fixed in Version	6.9.0

Summary	0025009: Draw Harness, ViewerTest - Incorrect handling of comma and period keys pressed in a 3D view
Description	Under Windows, ViewerWindowProc() function in ViewerTest_ViewerCommands.cxx tries to process almost all keyboard shortcuts passed through wParam argument of WM_KEYDOWN message as if they are true human-readable string constants. However, this is not the case for comma (,) and period (.) keys: wParam contains VK_OEM_COMMA and VK_OEM_PERIOD, respectively. Thus pressing comma and period keys in a test 3D view has unexpected effect, and this should be corrected (virtual key codes should be translated to ASCII characters before further processing, or virtual key codes should be used for all keyboard shortcuts).
Tags	No tags attached.

Test case number

Date Modified	Username	Field	Change
2014-06-15 19:18	~~san~~	New Issue
2014-06-15 19:18	~~san~~	Assigned To	=> mkv
2014-06-20 10:53	~~mkv~~	Assigned To	mkv => kgv
2014-06-20 10:53	~~mkv~~	Status	new => assigned
2014-06-20 11:02	kgv	Note Added: 0029840
2014-06-20 11:03	kgv	Target Version	=> 6.8.0
2014-09-26 13:21	~~abv~~	Target Version	6.8.0 => 7.1.0
2014-12-01 10:07	kgv	Note Added: 0034859
2014-12-01 10:07	kgv	Assigned To	kgv => bugmaster
2014-12-01 10:07	kgv	Status	assigned => feedback
2014-12-01 10:07	kgv	Resolution	open => fixed
2014-12-02 10:22	bugmaster	Status	feedback => tested
2014-12-02 10:23	bugmaster	Status	tested => verified
2015-01-19 16:23	bugmaster	Target Version	7.1.0 => 6.9.0
2015-05-14 15:29	~~aiv~~	Status	verified => closed
2015-05-14 15:33	~~aiv~~	Fixed in Version	=> 6.9.0

kgv 2014-06-20 11:02 developer ~0029840	The work-in-progress patch is currently located in branch CR24394_1.

kgv 2014-12-01 10:07 developer ~0034859	Dear bugmaster, please switch issue to verified state - the problem has been solved within patch for #0025436.