]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commitdiff
Removed the leading underscore for visitor_iterator property
authorAgam Rafaeli <agam.rafaeli@gmail.com>
Wed, 5 Dec 2018 21:12:59 +0000 (16:12 -0500)
committerMike Bayer <mike_mp@zzzcomputing.com>
Wed, 5 Dec 2018 21:12:59 +0000 (16:12 -0500)
A leading underscore usually denotes a private member. Since this
is a property and it is used in Query I removed the leading underscore

Change-Id: I8a35c09fd6d20ee0a13568ed7257a08b7bee2a08
Pull-request: https://github.com/sqlalchemy/sqlalchemy/pull/4398

lib/sqlalchemy/orm/query.py
lib/sqlalchemy/sql/visitors.py

index 93b9a85be8b799b473824bf437468fdfe5e04f81..df8add27d20fbd0c0eb0c66753cb40345e4e0d87 100644 (file)
@@ -311,7 +311,7 @@ class Query(object):
             orm_only = False
 
         if as_filter and self._filter_aliases:
-            for fa in self._filter_aliases._visitor_iterator:
+            for fa in self._filter_aliases.visitor_iterator:
                 adapters.append(
                     (
                         orm_only, fa.replace
index 22976921c0967cf7799096d747036994ca706b6d..b39ec8167aba9b051ed9d5f118ef19f41ac7b346 100644 (file)
@@ -115,7 +115,7 @@ class ClauseVisitor(object):
     __traverse_options__ = {}
 
     def traverse_single(self, obj, **kw):
-        for v in self._visitor_iterator:
+        for v in self.visitor_iterator:
             meth = getattr(v, "visit_%s" % obj.__visit_name__, None)
             if meth:
                 return meth(obj, **kw)
@@ -142,7 +142,7 @@ class ClauseVisitor(object):
         return visitors
 
     @property
-    def _visitor_iterator(self):
+    def visitor_iterator(self):
         """iterate through this visitor and each 'chained' visitor."""
 
         v = self
@@ -156,7 +156,7 @@ class ClauseVisitor(object):
         the chained visitor will receive all visit events after this one.
 
         """
-        tail = list(self._visitor_iterator)[-1]
+        tail = list(self.visitor_iterator)[-1]
         tail._next = visitor
         return self
 
@@ -200,7 +200,7 @@ class ReplacingCloningVisitor(CloningVisitor):
         """traverse and visit the given expression structure."""
 
         def replace(elem):
-            for v in self._visitor_iterator:
+            for v in self.visitor_iterator:
                 e = v.replace(elem)
                 if e is not None:
                     return e