ActionTimeoutChanges

Ginger-Automation · Jun 12, 2024 · 20f1a95 · 20f1a95
1 parent 4aa24b7
commit 20f1a95
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 3 deletions.
diff --git a/Ginger/Ginger/Actions/ActionEditPage.xaml b/Ginger/Ginger/Actions/ActionEditPage.xaml
@@ -151,7 +151,7 @@
                                     <Label Content="Wait:" Style="{StaticResource $LabelStyle}" />
                                     <Actions:UCValueExpression x:Name="xWaitVeUC" ToolTip="Action will wait this seconds long before start to run" HorizontalAlignment="Stretch" Width="80" />
                                     <Label Content="Timeout:" Style="{StaticResource $LabelStyle}" Margin="10,0,0,0"/>
-                                    <TextBox x:Name="xTimeoutTextBox" TextChanged="txtTimeout_TextChanged" ToolTip="Timeout field is in seconds. Default is 30 minutes (1800 seconds). Use 0 for no timeout, allowing actions to run indefinitely. If an action exceeds the specified timeout duration, it will be stopped and marked as failed." Style="{StaticResource @TextBoxStyle}" Width="50" TextAlignment="Center"  HorizontalAlignment="Left" />
+                                    <TextBox x:Name="xTimeoutTextBox" TextChanged="txtTimeout_TextChanged" PreviewTextInput="xTimeoutTextBox_PreviewTextInput" ToolTip="Timeout field is in seconds. Default is 30 minutes (1800 seconds). Use 0 for no timeout, allowing actions to run indefinitely. If an action exceeds the specified timeout duration, it will be stopped and marked as failed." Style="{StaticResource @TextBoxStyle}" Width="50" TextAlignment="Center"  HorizontalAlignment="Left" />
                                 </StackPanel>
 
                                 <Expander x:Name="xRetryExpander" DockPanel.Dock="Top" Margin="0,10,0,0" ExpandDirection="Down" IsExpanded="False" Style="{StaticResource $ExpanderStyle}">

diff --git a/Ginger/Ginger/Actions/ActionEditPage.xaml.cs b/Ginger/Ginger/Actions/ActionEditPage.xaml.cs
@@ -1668,7 +1668,7 @@ private void txtTimeout_TextChanged(object sender, TextChangedEventArgs e)
         {
             if (xTimeoutTextBox.Text == String.Empty || xTimeoutTextBox.Text == null)
             {
-                xTimeoutTextBox.Text = "0";
+                xTimeoutTextBox.Text = null;
                 xTimeoutTextBox.CaretIndex = 1;
             }
         }
@@ -2259,5 +2259,34 @@ protected override void IsVisibleChangedHandler(object sender, DependencyPropert
                 base.IsVisibleChangedHandler(sender, e);
             }
         }
+
+        private void xTimeoutTextBox_PreviewTextInput(object sender, TextCompositionEventArgs e)
+        {
+
+            bool isEmptyString = string.IsNullOrEmpty(e.Text);
+            if (isEmptyString)
+            {
+                //allow
+                e.Handled = false;
+                return;
+            }
+
+            string currentText = xTimeoutTextBox.Text != null ? xTimeoutTextBox.Text : string.Empty;
+            bool isValidInteger = int.TryParse(currentText + e.Text, out _);
+            if (isValidInteger)
+            {
+                //allow
+                e.Handled = false;
+                return;
+            }
+            else
+            {
+                //don't allow
+                e.Handled = true;
+            }
+
+        }
+
+
     }
 }
diff --git a/Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs b/Ginger/GingerCoreNET/Drivers/CoreDrivers/Web/Selenium/SeleniumDriver.cs
@@ -2359,7 +2359,7 @@ private int NewSmartSyncGetMaxTimeout(ActNewSmartSync act)
             }
             else
             {
-                return 300;
+                return (int)ImplicitWait;
             }
 
         }