Steema Issues Database

Note: This database is for bugs and wishes only. For technical support help, if you are a customer please visit our online forums;
otherwise you can use StackOverflow.
Before using this bug-tracker we recommend a look at this document, Steema Bug Fixing Policy.



Bug 692 - SurfaceNearestTool method GetRowCol, never return the first column of Surface
Summary: SurfaceNearestTool method GetRowCol, never return the first column of Surface
Status: RESOLVED FIXED
Alias: None
Product: VCL TeeChart
Classification: Unclassified
Component: Tools (show other bugs)
Version: 140220
Hardware: PC Windows
: Normal normal
Target Milestone: ---
Assignee: Steema Issue Manager
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-01 07:09 EDT by sandra pazos
Modified: 2014-04-09 10:19 EDT (History)
1 user (show)

See Also:
Chart Series: ---
Delphi / C++ Builder RAD IDE Version:

Attachments
Nearest Surface Tool Problem (57.03 KB, application/x-zip-compressed)
2014-04-01 07:09 EDT, sandra pazos 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description sandra pazos 2014-04-01 07:09:37 EDT 
Created attachment 145 [details]
Nearest Surface Tool Problem

The SurfaceNearestTool method GetRowCol returns all Surface rows and columns selected, except the first column. Attached project show the problem. This behaviour isn't produced in TeeChart.Net.
Comment 1 david berneda 2014-04-09 10:16:25 EDT 
The first column you mean it returns columns always from 2 to 30 in this example?

I guess the problem is its returning rows from 1 to 29, but columns from 2 to 30.

There is an internal code that controls if the surface is "back faced", ie: if you rotate the surface 180 degrees, the tool should work and return the same values as if not rotated.

The solution might be always returning the Col number -1 in all cases.
Comment 2 david berneda 2014-04-09 10:19:32 EDT 
Fixed. Now the column returned in GetRowCol function is always decremented by 1.

Warning: This change can break existing code assuming the column is always +1.