diff -r 8312e0e76795 -r 0f70a4ef4592 src/eric7/Plugins/CheckerPlugins/CodeStyleChecker/Security/SecurityNodeVisitor.py --- 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): """