diff --git a/odex25_sales/dev_membership/models/dev_membership.py b/odex25_sales/dev_membership/models/dev_membership.py index 5f405c6de..00471da28 100644 --- a/odex25_sales/dev_membership/models/dev_membership.py +++ b/odex25_sales/dev_membership/models/dev_membership.py @@ -143,15 +143,18 @@ class DevMembership(models.Model): self.to_date = self.from_date + relativedelta(months=+self.duration) - relativedelta(days=1) elif self.interval == 'days': self.to_date = self.from_date + relativedelta(days=+self.duration) - + def action_confirm_membership(self): - if self.partner_id.nationality_id not in self.product_id.nationality_ids and self.prodcut_id.nationality_ids!=False: + if self.partner_id.nationality_id not in self.product_id.nationality_ids and self.product_id.nationality_ids != False: raise ValidationError(_("The member's nationality does not meet the membership requirements")) - if self.partner_id.age <18: + if self.partner_id.age < 18: raise ValidationError(_('The member must be at least 18 years old.')) - if self.partner_id.gender != self.product_id.gender and self.product_id.gender!=False: + if self.partner_id.gender != self.product_id.gender and self.product_id.gender != False: raise ValidationError(_('Membership is only allowed for %s') % self.product_id.gender) + + + self.state = 'confirm' def action_active_membership(self): if not self.invoice_id and not self.is_free: