Merge pull request #1 from xobb1t/develop

Fix cart creation for anonymous user
trilan · Sep 16, 2011 · dab686e · dab686e
2 parents ce5dd37 + 7a976e4
commit dab686e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/tinycart/models.py b/tinycart/models.py
@@ -19,7 +19,7 @@ def get_for_request(self, request):
                 return self.get(pk=request.session['cart'])
             except self.model.DoesNotExist:
                 pass
-        cart = self.get_or_create(user=None)[0]
+        cart = self.create(user=None)
         request.session['cart'] = cart.pk
         return cart
 

diff --git a/tinycart/tests/test_models.py b/tinycart/tests/test_models.py
@@ -26,6 +26,10 @@ def test_cart_for_anonymous_user(self):
         self.assertEqual(self.request.session['cart'], cart.pk)
         self.assertEqual(Cart.objects.get_for_request(self.request), cart)
 
+        self.request.session = {}
+        Cart.objects.get_for_request(self.request)
+        self.assertFalse(cart.pk==self.request.session['cart'])
+
     def test_cart_for_authenticated_user(self):
         self.request.user = User.objects.create_user('john', 'john@example.com')
         cart = Cart.objects.get_for_request(self.request)