Grids: Push some repeatedly used Methods into the Base Class

wolfganguhr

Hello

Acutally the routine SelectByIndex which is assigned to the datagrid in den ManualCode-Segement shows the following ...

        private void SelectByIndex(int rowIndex)
        {
            // ... Some Code (allways indetical)
            if ((rowIndex >= 1) && (grdDetails.Rows.Count > 1))
            {
                // Some Code in case of a valid selection ...
                // Partly different ..
            }
            else
            {
                // Code in Case of an invalid selection ...
                // Partly different ..

            // added!
        	grdDetails.Selection.ResetSelection(false);
        }
    }

In this invalid selection seqment grdDetails.Selection.ResetSelection(true); shall be integrated in all this routines. Otherwise it looks like a selection has been done but it isn't.

Actual this problem occurs in the following classes ...

[]UC_GLBatches.ManualCode.cs (my problem)[/]
[]SetupAnalysisTypes.ManualCode.cs[/]
[]UC_AccountAnalysisAttributes.ManualCode.cs[/]
[]UC_SetupAnalysisValues.ManualCode.cs/][/*:m][/list:u]

Following options:
- []move the routine inside the TSgrdDataGrid-Code[/]
  []change all functions[/]
  []change only my function[/]
  [/list:u]
  
  I prefer the first version where SelectByIndex is moved into TSgrdDataGrid code and the possibility to added by the possibility to add a SelectByIndexManual-Routine
  
  Best regards
  Wolfgang

pokorra

Would it be an option to add the SelectByIndex method to Ict.Common.Controls.TSgrdDataGridPaged.
We don't want to change the SourceGrid directly.

wolfganguhr

pokorra wrote
Would it be an option to add the SelectByIndex method to Ict.Common.Controls.TSgrdDataGridPaged.

I think that this may be the best way. And CurrentRowIndex is another candidate for this operation too.

But I request a "common go" for to change this files. No one shall work with them actually. And we shall have a look at grdDetails.Rows.Count for a moment:

grdDetails.Rows.Count is the number of rows including the headlines of a table. Actually the following numbers are relevant ...

[]-1 = Select no Row (unselect)[/]
[]0 = Select the header row[/]
[]n = Select the (n-1)th data row[/]
[/list:u]

Actually I'll assume that all the tables will have exact one header row. No table with zero rows or 2 or more shall exist in the future?