X-Git-Url: http://git.cyclocoop.org/?p=burette%2Fremembership.git;a=blobdiff_plain;f=remembership.py;h=5560fbf84f4ad5b57555b52fa28f1bbd45de05d7;hp=733f00d8088259664b4e3188869e897609496bf6;hb=HEAD;hpb=435ec7d1f0a1d180e9f76512b64c43f61ff5e295 diff --git a/remembership.py b/remembership.py index 733f00d..c8d8b46 100644 --- a/remembership.py +++ b/remembership.py @@ -1,8 +1,8 @@ # -*- coding: utf-8 -*- ############################################################################## # -# Remembership module for OpenERP, Overload membership module -# Copyright (C) 2012 L'Heureux Cyclage () Ludovic CHEVALIER +# Remembership module for OpenERP, Membership module enhancement and bug fixes +# Copyright (C) 2012-2018 L'Heureux Cyclage () # # This file is a part of Remembership # @@ -21,12 +21,12 @@ # ############################################################################## -from osv import osv -from osv import fields +from openerp.osv import osv +from openerp.osv import fields import time from datetime import datetime, date from dateutil.relativedelta import relativedelta - +from openerp.tools.translate import _ class Partner(osv.osv): _inherit = 'res.partner' @@ -37,6 +37,7 @@ class Partner(osv.osv): name = name[0] res = {} member_line_obj = self.pool.get('membership.membership_line') + print ("DEV: [remembership] [_membership_date]") for partner in self.browse(cr, uid, ids, context=context): # XXX: commented part from openerp/addons/membership/membership.py #if partner.associate_member: @@ -50,18 +51,26 @@ class Partner(osv.osv): 'membership_cancel': False } if name == 'membership_start': - line_id = member_line_obj.search(cr, uid, [('partner', '=', partner_id),('date_cancel','=',False)], + line_id = member_line_obj.search(cr, uid, [('partner', '=', partner_id),('state','!=','canceled')], limit=1, order='date_from', context=context) if line_id: res[partner.id]['membership_start'] = member_line_obj.read(cr, uid, line_id[0], ['date_from'], context=context)['date_from'] + if partner.associate_member: + print ("DEV: [_membership_date] [associate] date_from: %s" % str(res[partner.id]['membership_start'])) + else: + print ("DEV: [_membership_date] date_from: %s" % str(res[partner.id]['membership_start'])) if name == 'membership_stop': - line_id1 = member_line_obj.search(cr, uid, [('partner', '=', partner_id),('date_cancel','=',False)], + line_id1 = member_line_obj.search(cr, uid, [('partner', '=', partner_id),('state','!=','canceled')], limit=1, order='date_to desc', context=context) if line_id1: res[partner.id]['membership_stop'] = member_line_obj.read(cr, uid, line_id1[0], ['date_to'], context=context)['date_to'] + if partner.associate_member: + print ("DEV: [_membership_date] [associate] date_to: %s" % str(res[partner.id]['membership_stop'])) + else: + print ("DEV: [_membership_date] date_to: %s" % str(res[partner.id]['membership_stop'])) if name == 'membership_cancel': if partner.membership_state == 'canceled': @@ -134,9 +143,85 @@ class Partner(osv.osv): def create_membership_invoice(self, cr, uid, ids, product_id=None, datas=None, context=None): # NOTE: use context to supply date_from to account_invoice_line.create - ctx = context and context.copy() or {} - ctx['date_from'] = datas.get('date_from', None) - return super(Partner, self).create_membership_invoice(cr, uid, ids, product_id=product_id, datas=datas, context=ctx) + context = context and context.copy() or {} + context.update({'date_from': datas.get('date_from', None)}); + # XXX: copied from openerp/addons/membership/membership.py to fix account_invoice_line creation + print ("DEV: [remembership] [partner] [create_membership_invoice] ids=%s" % str(ids)) + invoice_obj = self.pool.get('account.invoice') + invoice_line_obj = self.pool.get('account.invoice.line') + invoice_tax_obj = self.pool.get('account.invoice.tax') + product_id = product_id or datas.get('membership_product_id', False) + amount = datas.get('amount', 0.0) + invoice_list = [] + if type(ids) in (int, long,): + ids = [ids] + for partner in self.browse(cr, uid, ids, context=context): + print ("DEV: [remembership] [partner] [create_membership_invoice] ids=%s partner=%s" % (str(ids),str(partner.id))) + account_id = partner.property_account_receivable and partner.property_account_receivable.id or False + fpos_id = partner.property_account_position and partner.property_account_position.id or False + addr = self.address_get(cr, uid, [partner.id], ['invoice']) + if partner.free_member: + raise osv.except_osv(_('Error!'), + _("Partner is a free Member.")) + if not addr.get('invoice', False): + raise osv.except_osv(_('Error!'), + _("Partner doesn't have an address to make the invoice.")) + quantity = 1 + line_value = { + 'product_id': product_id, + } + + print ("DEV: [remembership] [partner] [create_membership_invoice] ids=%s partner=%s [invoice_line_obj] [product_id_change]" % (str(ids),str(partner.id))) + line_dict = invoice_line_obj.product_id_change(cr, uid, {}, + product_id, False, quantity, '', 'out_invoice', partner.id, fpos_id, price_unit=amount, context=context) + line_value.update(line_dict['value']) + line_value['price_unit'] = amount + if line_value.get('invoice_line_tax_id', False): + tax_tab = [(6, 0, line_value['invoice_line_tax_id'])] + line_value['invoice_line_tax_id'] = tax_tab + + print ("DEV: [remembership] [partner] [create_membership_invoice] ids=%s partner=%s [invoice_obj] [create] account=%s" % (str(ids),str(partner.id),str(account_id))) + # NOTE: let account_invoice_obj.create() call account_invoice_line.create() + # to have a correct membership_state: becauce now fields.function 'store' watchdogs + # are triggered after account_invoice_line.create(). + invoice_id = invoice_obj.create(cr, uid, { + 'partner_id': partner.id, + 'account_id': account_id, + 'fiscal_position': fpos_id or False, + 'invoice_line':[(0, 0, line_value)] + }, context=context) + #line_value['invoice_id'] = invoice_id + #print ("DEV: [remembership] [partner] [create_membership_invoice] ids=%s partner=%s [invoice_line_obj] [create]" % (str(ids),str(partner.id))) + #invoice_line_id = invoice_line_obj.create(cr, uid, line_value, context=context) + #invoice_obj.write(cr, uid, invoice_id, {'invoice_line': line_value}, context=context) + print ("DEV: [remembership] [partner] [create_membership_invoice] : invoice_list.append(invoice_id) : pre") + invoice_list.append(invoice_id) + print ("DEV: [remembership] [partner] [create_membership_invoice] : invoice_list.append(invoice_id) : post") + if line_value['invoice_line_tax_id']: + tax_value = invoice_tax_obj.compute(cr, uid, invoice_id).values() + for tax in tax_value: + invoice_tax_obj.create(cr, uid, tax, context=context) + # NOTE: this wasn't working, and is not needed now. + #recompute the membership_state of those partners + #print ("DEV: [remembership] [partner] [create_membership_invoice] ids=%s [recompute states]" % str(ids)) + #self.pool.get('res.partner').write(cr, uid, ids, {}, context=context) + print ("DEV: [remembership] [partner] [create_membership_invoice] return=%s" % str(invoice_list)) + return invoice_list + + def update_membership_status(self, cr, uid): + """Action for updating old members status with scheduled cron job""" + partner_obj = self.pool.get('res.partner') + today = date.today() + ids = partner_obj.search(cr, uid, [('membership_state', '=', 'paid'), ('membership_stop', '<', today)], context=None) + for id in ids: + self.write(cr, uid, id, {'membership_state': 'old'}, context=None) + + def copy(self, cr, uid, id, default=None, context=None): + if default is None: + default = {} + default['member_ident'] = None + return super(Partner, self).copy(cr, uid, id, default, context) + Partner() @@ -172,10 +257,15 @@ class account_invoice_line(osv.osv): date_from = member_line.date_from date_to = member_line.date_to if line.product_id.membership_date2date: - date_from = context['date_from'] and datetime.strptime(context['date_from'], "%Y-%m-%d") or date.today() - date_to = date_from + relativedelta(months = +12) # TODO: parameterize this delta? + date_from = ('date_from' in context + and context.get('date_from') + and datetime.strptime(context('date_from'), "%Y-%m-%d") + or date.today()) + date_to = date_from + relativedelta(months=+12, days=-1) # TODO: parameterize this delta? date_from = date_from.strftime("%Y-%m-%d") date_to = date_to .strftime("%Y-%m-%d") + print ("DEV: [remembership] [account_invoice_line] [write] date_from: %s" % str(date_from)) + print ("DEV: [remembership] [account_invoice_line] [write] date_to : %s" % str(date_to)) member_line_obj.write(cr, uid, member_line.id , {'date_from': date_from ,'date_to' : date_to @@ -189,6 +279,8 @@ class account_invoice_line(osv.osv): ] , context=context) for associate_member_line in member_line_obj.browse(cr, uid, associate_member_line_ids, context=context): + print ("DEV: [remembership] [account_invoice_line] [write] [associate] date_from: %s" % str(date_from)) + print ("DEV: [remembership] [account_invoice_line] [write] [associate] date_to : %s" % str(date_to)) member_line_obj.write(cr, uid, associate_member_line.id , {'date_from': date_from ,'date_to' : date_to @@ -206,7 +298,6 @@ class account_invoice_line(osv.osv): ] , context=context) member_line_obj.unlink(cr, uid, associate_member_line_ids, context=context) - #Define member ident if it's necessary partners = [line.invoice_id.partner_id] if line.invoice_id.partner_id.associate_members: @@ -218,11 +309,13 @@ class account_invoice_line(osv.osv): if line.product_id and not line.product_id.membership and ml_ids: # Product line has changed to a non membership product member_line_obj.unlink(cr, uid, ml_ids, context=context) + print ("DEV: [remembership] [account_invoice_line] [write] : return=%s" % str(res)) return res def create(self, cr, uid, vals, context=None): member_line_obj = self.pool.get('membership.membership_line') res = super(account_invoice_line, self).create(cr, uid, vals, context=context) + print ("DEV: [remembership] [account_invoice_line] [create] : begin res=%s" % str(res)) line = self.browse(cr, uid, res, context=context) if line.invoice_id.type == 'out_invoice' and line.product_id and line.product_id.membership: @@ -236,10 +329,16 @@ class account_invoice_line(osv.osv): date_from = member_line.date_from date_to = member_line.date_to if line.product_id.membership_date2date: - date_from = context['date_from'] and datetime.strptime(context['date_from'], "%Y-%m-%d") or date.today() - date_to = date_from + relativedelta(months = +12) # TODO: parameterize this delta? + date_from = ('date_from' in context + and context.get('date_from') + and datetime.strptime(context.get('date_from'), "%Y-%m-%d") + or date.today()) + print("#DEBUG: Date de début - %s" % str(date_from)) + date_to = date_from + relativedelta(months=+12, days=-1) # TODO: parameterize this delta? date_from = date_from.strftime("%Y-%m-%d") date_to = date_to .strftime("%Y-%m-%d") + print ("DEV: [remembership] [account_invoice_line] [create] date_from: %s" % str(date_from)) + print ("DEV: [remembership] [account_invoice_line] [create] date_to : %s" % str(date_to)) member_line_obj.write(cr, uid, member_line.id , {'date_from': date_from ,'date_to' : date_to @@ -250,6 +349,8 @@ class account_invoice_line(osv.osv): partners.extend(line.invoice_id.partner_id.associate_members) #Adding membership lines just for associate partners for associate_member in line.invoice_id.partner_id.associate_members: + print ("DEV: [remembership] [account_invoice_line] [create] [associate] date_from: %s" % str(date_from)) + print ("DEV: [remembership] [account_invoice_line] [create] [associate] date_to : %s" % str(date_to)) member_line_obj.create(cr, uid, { 'partner': associate_member.id, 'membership_id': line.product_id.id, @@ -259,30 +360,15 @@ class account_invoice_line(osv.osv): 'date_to': date_to, 'account_invoice_line': line.id, }, context=context) + elif line.product_id.membership_grouped and not line.invoice_id.partner_id.associate_members: + raise osv.except_osv(_('Error!!!'), _('You try to invoice grouped membership product to a partner who hasn\'t associated partners.')) #Define member ident if it's necessary for i in partners: if not i.member_ident: mbr_id = self.pool.get('ir.sequence').get(cr, uid, 'member_ident') self.pool.get('res.partner').write(cr, uid, i.id, {'member_ident': mbr_id}) + print ("DEV: [remembership] [account_invoice_line] [create] : return=%s" % str(res)) return res account_invoice_line() - - -class pos_make_payment(osv.osv_memory): - _inherit = 'pos.make.payment' - - def check(self, cr, uid, ids, context=None): - """ - Auto invoice orders. Temporarly solution before make partner members just with orders, without invoice. - """ - res = super(pos_make_payment, self).check(cr, uid, ids, context=context) - - order_obj = self.pool.get('pos.order') - order_obj.action_invoice(cr, uid, ids, context=context) - - return res - -pos_make_payment() - # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4: