Mercurial Repositories > eric / file diff

--- a/src/eric7/Plugins/CheckerPlugins/CodeStyleChecker/Security/SecurityNodeVisitor.py	Tue Aug 29 16:55:01 2023 +0200
+++ b/src/eric7/Plugins/CheckerPlugins/CodeStyleChecker/Security/SecurityNodeVisitor.py	Tue Aug 29 16:55:18 2023 +0200
@@ -183,43 +183,19 @@
         @type ast.Constant
         """
         if isinstance(node.value, str):
-            self.visit_Str(node)
+            self.__context["str"] = node.value
+            if not isinstance(node._securityParent, ast.Expr):  # docstring
+                self.__context["linerange"] = SecurityUtils.linerange_fix(
+                    node._securityParent
+                )
+                self.__runChecks("Str")
         elif isinstance(node.value, bytes):
-            self.visit_Bytes(node)
-
-    def visit_Str(self, node):
-        """
-        Public method defining a visitor for String nodes.
-
-        This adds relevant information about node to
-        the context for use in tests which inspect strings.
-
-        @param node reference to the node being inspected
-        @type ast.Str
-        """
-        self.__context["str"] = node.s
-        if not isinstance(node._securityParent, ast.Expr):  # docstring
-            self.__context["linerange"] = SecurityUtils.linerange_fix(
-                node._securityParent
-            )
-            self.__runChecks("Str")
-
-    def visit_Bytes(self, node):
-        """
-        Public method defining a visitor for Bytes nodes.
-
-        This adds relevant information about node to
-        the context for use in tests which inspect strings.
-
-        @param node reference to the node being inspected
-        @type ast.Bytes
-        """
-        self.__context["bytes"] = node.s
-        if not isinstance(node._securityParent, ast.Expr):  # docstring
-            self.__context["linerange"] = SecurityUtils.linerange_fix(
-                node._securityParent
-            )
-            self.__runChecks("Bytes")
+            self.__context["bytes"] = node.value
+            if not isinstance(node._securityParent, ast.Expr):  # docstring
+                self.__context["linerange"] = SecurityUtils.linerange_fix(
+                    node._securityParent
+                )
+                self.__runChecks("Bytes")
 
     def __preVisit(self, node):
         """