fix-#842: Fix ignored chained options

Author: DavidBord

docs/changelog.rst

@@ -5,6 +5,7 @@ Changelog
 Changes in 0.10.1 - DEV
 =======================
 - Fix infinite recursion with CASCADE delete rules under specific conditions. #1046
+- Fix ignored chained options #842
 
 Changes in 0.10.0
 =================

mongoengine/queryset/base.py

@@ -684,11 +684,7 @@ def limit(self, n):
         :param n: the maximum number of objects to return
         """
         queryset = self.clone()
-        if n == 0:
-            queryset._cursor.limit(1)
-        else:
-            queryset._cursor.limit(n)
-        queryset._limit = n
+        queryset._limit = n if n != 0 else 1
         # Return self to allow chaining
         return queryset
 
@@ -699,7 +695,6 @@ def skip(self, n):
         :param n: the number of objects to skip before returning results
         """
         queryset = self.clone()
-        queryset._cursor.skip(n)
         queryset._skip = n
         return queryset
 
@@ -717,7 +712,6 @@ def hint(self, index=None):
         .. versionadded:: 0.5
         """
         queryset = self.clone()
-        queryset._cursor.hint(index)
         queryset._hint = index
         return queryset
 

tests/document/indexes.py

@@ -581,7 +581,7 @@ def invalid_index():
             self.assertRaises(TypeError, invalid_index)
 
         def invalid_index_2():
-            return BlogPost.objects.hint(('tags', 1))
+            return BlogPost.objects.hint(('tags', 1)).next()
         self.assertRaises(Exception, invalid_index_2)
 
     def test_unique(self):

tests/queryset/queryset.py

@@ -188,6 +188,8 @@ def test_find(self):
             "[<Person: Person object>, <Person: Person object>]",  "%s" % self.Person.objects[1:3])
         self.assertEqual(
             "[<Person: Person object>, <Person: Person object>]",  "%s" % self.Person.objects[51:53])
+        # Test only after limit
+        self.assertIsNone(self.Person.objects().limit(2).only('name')[0].age)
 
     def test_find_one(self):
         """Ensure that a query using find_one returns a valid result.