Changeset - 43649002cb03
[Not reviewed]
0 2 0
Christopher Neugebauer - 8 years ago 2016-09-03 06:18:27
chrisjrn@gmail.com
Makes ProductCondition work if you have both valid and cancelled instances of a product. Fixes #68
2 files changed with 16 insertions and 2 deletions:
0 comments (0 inline, 0 general)
registrasion/controllers/conditions.py
Show inline comments
...
 
@@ -174,2 +174,4 @@ class ProductConditionController(IsMetByFilter, ConditionController):
 
        released = commerce.Cart.STATUS_RELEASED
 
        paid = commerce.Cart.STATUS_PAID
 
        active = commerce.Cart.STATUS_ACTIVE
 
        in_released_carts = Q(
...
 
@@ -177,4 +179,11 @@ class ProductConditionController(IsMetByFilter, ConditionController):
 
        )
 
        not_in_paid_or_active_carts = ~(
 
            Q(enabling_products__productitem__cart__status=paid) |
 
            Q(enabling_products__productitem__cart__status=active)
 
        )
 

	
 
        queryset = queryset.filter(in_user_carts)
 
        queryset = queryset.exclude(in_released_carts)
 
        queryset = queryset.exclude(
 
            in_released_carts & not_in_paid_or_active_carts
 
        )
 

	
registrasion/tests/test_flag.py
Show inline comments
...
 
@@ -354,3 +354,3 @@ class FlagTestCases(RegistrationCartTestCase):
 

	
 
    def test_oops(self):
 
    def test_product_stays_enabled_even_if_some_are_cancelled(self):
 
        ''' Flags should be enabled, even if *some* enabling products are cnx.
...
 
@@ -379 +379,6 @@ class FlagTestCases(RegistrationCartTestCase):
 
        cart2.add_to_cart(self.PROD_1, 1)
 

	
 
        cart2.set_quantity(self.PROD_2, 0)
 

	
 
        with self.assertRaises(ValidationError):
 
            cart2.add_to_cart(self.PROD_1, 1)
0 comments (0 inline, 0 general)