Fixed an errr when trying to display a variable description from an e…

…mpty or unselected row. Extracted a new private method ShowVariableDescription() in the FrmMainW class.
xvitaly · Jul 28, 2024 · 4facc77 · 4facc77
1 parent b4da816
commit 4facc77
Showing 1 changed file with 23 additions and 7 deletions.
diff --git a/src/srcrepair/FrmMainW.cs b/src/srcrepair/FrmMainW.cs
@@ -837,6 +837,23 @@ private bool GetRepairSelState()
             return PS_CleanPackages.Checked || PS_CleanRegistry.Checked || PS_ServiceRepair.Checked;
         }
 
+        /// <summary>
+        /// Shows the description of the specified variable or function.
+        /// </summary>
+        /// <param name="Variable">Variable name.</param>
+        private void ShowVariableDescription(string Variable)
+        {
+            string Description = CvarFetcher.GetString(Variable);
+            if (!string.IsNullOrEmpty(Description))
+            {
+                MessageBox.Show(Description, Properties.Resources.AppName, MessageBoxButtons.OK, MessageBoxIcon.Information);
+            }
+            else
+            {
+                MessageBox.Show(AppStrings.CE_ClNoDescr, Properties.Resources.AppName, MessageBoxButtons.OK, MessageBoxIcon.Warning);
+            }
+        }
+
         /// <summary>
         /// Handles with installed FPS-configs and shows special icon.
         /// </summary>
@@ -2649,22 +2666,21 @@ private void CE_ShowHint_Click(object sender, EventArgs e)
         {
             try
             {
-                string Buf = CE_Editor.Rows[CE_Editor.CurrentRow.Index].Cells[0].Value.ToString();
-                if (!string.IsNullOrEmpty(Buf))
+                if (CE_Editor.SelectedCells.Count > 0)
                 {
-                    Buf = CvarFetcher.GetString(Buf);
-                    if (!string.IsNullOrEmpty(Buf))
+                    DataGridViewCell Item = CE_Editor.Rows[CE_Editor.CurrentRow.Index].Cells[0];
+                    if (!Item.OwningRow.IsNewRow && (Item.Value != null))
                     {
-                        MessageBox.Show(Buf, Properties.Resources.AppName, MessageBoxButtons.OK, MessageBoxIcon.Information);
+                        ShowVariableDescription(Item.Value.ToString());
                     }
                     else
                     {
-                        MessageBox.Show(AppStrings.CE_ClNoDescr, Properties.Resources.AppName, MessageBoxButtons.OK, MessageBoxIcon.Warning);
+                        MessageBox.Show(AppStrings.CE_ClSelErr, Properties.Resources.AppName, MessageBoxButtons.OK, MessageBoxIcon.Warning);
                     }
                 }
                 else
                 {
-                    MessageBox.Show(AppStrings.CE_ClSelErr, Properties.Resources.AppName, MessageBoxButtons.OK, MessageBoxIcon.Warning);
+                    MessageBox.Show(AppStrings.CE_NoSelection, Properties.Resources.AppName, MessageBoxButtons.OK, MessageBoxIcon.Warning);
                 }
             }
             catch (Exception Ex)