From a4cfc2914b577c5857fea174f54db2e2018d37ab Mon Sep 17 00:00:00 2001
From: Martin Aspeli <optilude@gmail.com>
Date: Mon, 16 Mar 2009 16:57:33 +0000
Subject: [PATCH] Fix name errors. Thank you pyflakes

---
 CHANGES.txt                   | 5 +++++
 setup.py                      | 2 +-
 src/zc/relationship/index.py  | 9 +++------
 src/zc/relationship/shared.py | 2 +-
 4 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/CHANGES.txt b/CHANGES.txt
index 41be5da..20e9820 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -7,6 +7,11 @@ CHANGES
 
 (supports Zope 3.4/Zope 2.11/ZODB 3.8)
 
+1.1.1
+-----
+
+- Fix name errors caught by pyflakes.
+
 1.1.0
 -----
 
diff --git a/setup.py b/setup.py
index 1d15fc4..1af44e9 100644
--- a/setup.py
+++ b/setup.py
@@ -2,7 +2,7 @@
 
 setup(
     name="zc.relationship",
-    version="1.1",
+    version="1.1.1",
     packages=find_packages('src'),
     include_package_data=True,
     package_dir= {'':'src'},
diff --git a/src/zc/relationship/index.py b/src/zc/relationship/index.py
index b781673..49014ac 100644
--- a/src/zc/relationship/index.py
+++ b/src/zc/relationship/index.py
@@ -1,6 +1,3 @@
-import re
-import types
-
 import persistent
 import persistent.interfaces
 from BTrees import OOBTree, IFBTree, IOBTree, Length
@@ -119,7 +116,7 @@ def __init__(self, attrs, defaultTransitiveQueriesFactory=None,
             res['attrname'] = val.__name__
             res['name'] = data.get('name', res['attrname'])
             if res['name'] in _attrs or val in seen:
-                raise ValueError('Duplicate in attrs', name, val)
+                raise ValueError('Duplicate in attrs', res['name'], val)
             seen.add(val)
             _attrs[res['name']] = res
             res['dump'] = data.get('dump', generateToken)
@@ -356,7 +353,7 @@ def tokenizeValues(self, values, name):
     def resolveValueTokens(self, tokens, name):
         load = self._attrs[name]['load']
         if load is None:
-            return values
+            return tokens
         cache = {}
         return (load(t, self, cache) for t in tokens)
 
@@ -483,7 +480,7 @@ def findValueTokens(self, resultName, query, maxDepth=None, filter=None,
                         targetQuery=None, targetFilter=None,
                         transitiveQueriesFactory=None):
         if resultName not in self._attrs:
-            raise ValueError('name not indexed', nm)
+            raise ValueError('name not indexed', resultName)
         return self._yieldValueTokens(
             resultName, *self._parse(
                 query, maxDepth, filter, targetQuery, targetFilter,
diff --git a/src/zc/relationship/shared.py b/src/zc/relationship/shared.py
index 74952bb..b0b1570 100644
--- a/src/zc/relationship/shared.py
+++ b/src/zc/relationship/shared.py
@@ -190,7 +190,7 @@ def minDepthFilter(depth):
         return None
     if not isinstance(depth, (int, long)) or depth < 1:
         raise ValueError('invalid minDepth', depth)
-    return lambda relchain, query, index, cache: len(relchain) >= minDepth
+    return lambda relchain, query, index, cache: len(relchain) >= depth
 
 class AbstractContainer(persistent.Persistent):
     def __init__(self,