From bf082d1aeda02ac2d2e77c5c142dde5a0a6b9605 Mon Sep 17 00:00:00 2001
From: James Maslek <jmaslek11@gmail.com>
Date: Wed, 20 Sep 2023 17:09:35 -0400
Subject: [PATCH] Hotfix stock screener (#5440)

* correct screener

* fix another .strip
---
 .../stocks/screener/screener_controller.py    | 28 +++++++++----------
 .../stocks/screener/screener_view.py          |  4 +--
 2 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/openbb_terminal/stocks/screener/screener_controller.py b/openbb_terminal/stocks/screener/screener_controller.py
index 36e32e42e02..ec40c1d2c07 100644
--- a/openbb_terminal/stocks/screener/screener_controller.py
+++ b/openbb_terminal/stocks/screener/screener_controller.py
@@ -57,7 +57,7 @@ class ScreenerController(BaseController):
     if PRESETS_PATH.exists():
         preset_choices.update(
             {
-                filepath.name.strip(".ini"): filepath
+                filepath.name.replace(".ini", ""): filepath
                 for filepath in PRESETS_PATH.iterdir()
                 if filepath.suffix == ".ini"
             }
@@ -66,7 +66,7 @@ class ScreenerController(BaseController):
     if PRESETS_PATH_DEFAULT.exists():
         preset_choices.update(
             {
-                filepath.name.strip(".ini"): filepath
+                filepath.name.replace(".ini", ""): filepath
                 for filepath in PRESETS_PATH_DEFAULT.iterdir()
                 if filepath.suffix == ".ini"
             }
@@ -243,8 +243,8 @@ class ScreenerController(BaseController):
 
         if ns_parser:
             preset = (
-                self.preset.strip(".ini")
-                if self.preset.strip(".ini") in finviz_model.d_signals
+                self.preset.replace(".ini", "")
+                if self.preset.replace(".ini", "") in finviz_model.d_signals
                 else self.preset
             )
 
@@ -321,8 +321,8 @@ class ScreenerController(BaseController):
 
         if ns_parser:
             preset = (
-                self.preset.strip(".ini")
-                if self.preset.strip(".ini") in finviz_model.d_signals
+                self.preset.replace(".ini", "")
+                if self.preset.replace(".ini", "") in finviz_model.d_signals
                 else self.preset
             )
             sort_map = screener_helper.finviz_map("valuation")
@@ -398,8 +398,8 @@ class ScreenerController(BaseController):
 
         if ns_parser:
             preset = (
-                self.preset.strip(".ini")
-                if self.preset.strip(".ini") in finviz_model.d_signals
+                self.preset.replace(".ini", "")
+                if self.preset.replace(".ini", "") in finviz_model.d_signals
                 else self.preset
             )
             sort_map = screener_helper.finviz_map("financial")
@@ -475,8 +475,8 @@ class ScreenerController(BaseController):
 
         if ns_parser:
             preset = (
-                self.preset.strip(".ini")
-                if self.preset.strip(".ini") in finviz_model.d_signals
+                self.preset.replace(".ini", "")
+                if self.preset.replace(".ini", "") in finviz_model.d_signals
                 else self.preset
             )
             sort_map = screener_helper.finviz_map("ownership")
@@ -553,8 +553,8 @@ class ScreenerController(BaseController):
         sort_map = screener_helper.finviz_map("performance")
         if ns_parser:
             preset = (
-                self.preset.strip(".ini")
-                if self.preset.strip(".ini") in finviz_model.d_signals
+                self.preset.replace(".ini", "")
+                if self.preset.replace(".ini", "") in finviz_model.d_signals
                 else self.preset
             )
             self.screen_tickers = finviz_view.screener(
@@ -628,8 +628,8 @@ class ScreenerController(BaseController):
 
         if ns_parser:
             preset = (
-                self.preset.strip(".ini")
-                if self.preset.strip(".ini") in finviz_model.d_signals
+                self.preset.replace(".ini", "")
+                if self.preset.replace(".ini", "") in finviz_model.d_signals
                 else self.preset
             )
             sort_map = screener_helper.finviz_map("technical")
diff --git a/openbb_terminal/stocks/screener/screener_view.py b/openbb_terminal/stocks/screener/screener_view.py
index 08287fe5b78..1ae79ef0a39 100644
--- a/openbb_terminal/stocks/screener/screener_view.py
+++ b/openbb_terminal/stocks/screener/screener_view.py
@@ -19,7 +19,7 @@ preset_choices = {}
 if PRESETS_PATH.exists():
     preset_choices.update(
         {
-            filepath.name.strip(".ini"): filepath
+            filepath.name.replace(".ini", ""): filepath
             for filepath in PRESETS_PATH.iterdir()
             if filepath.suffix == ".ini"
         }
@@ -28,7 +28,7 @@ if PRESETS_PATH.exists():
 if PRESETS_PATH_DEFAULT.exists():
     preset_choices.update(
         {
-            filepath.name.strip(".ini"): filepath
+            filepath.name.replace(".ini", ""): filepath
             for filepath in PRESETS_PATH_DEFAULT.iterdir()
             if filepath.suffix == ".ini"
         }