# -*- coding: utf-8 -*-
##############################################################################
#
-# Remembership module for OpenERP, Overload membership module
-# Copyright (C) 2012 L'Heureux Cyclage (<http://www.heureux-cyclage.org>) Ludovic CHEVALIER
+# Remembership module for OpenERP, Membership module enhancement and bug fixes
+# Copyright (C) 2012-2018 L'Heureux Cyclage (<http://www.heureux-cyclage.org>)
#
# This file is a part of Remembership
#
#
##############################################################################
-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'
- _columns = {
- 'member_ident': fields.char('Member identifier', size=64),
- 'associate_members': fields.one2many('res.partner', 'associate_member', 'Associate members', help='Members who are associated to this partner.'),
- }
-
- _sql_constraints = [
- ('member_ident_uniq', 'unique(member_ident, company_id)', 'The member identifier must be unique !'),
- ]
-
+ # XXX: copied from openerp/addons/membership/membership.py
def _membership_date(self, cr, uid, ids, name, args, context=None):
"""Return date of membership"""
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:
# partner_id = partner.associate_member.id
#else:
'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']
+ 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']
+ 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':
- # NOTE: hmm, partner_id et pas partner.id ?
- line_id2 = member_line_obj.search(cr, uid, [('partner', '=', partner_id)], limit=1, order='date_cancel', context=context)
+ line_id2 = member_line_obj.search(cr, uid, [('partner', '=', partner.id)], limit=1, order='date_cancel', context=context)
if line_id2:
res[partner.id]['membership_cancel'] = member_line_obj.read(cr, uid, line_id2[0], ['date_cancel'], context=context)['date_cancel']
return res
+ # XXX: copied from openerp/addons/membership/membership.py
+ def _get_partner_id(self, cr, uid, ids, context=None):
+ member_line_obj = self.pool.get('membership.membership_line')
+ res_obj = self.pool.get('res.partner')
+ data_inv = member_line_obj.browse(cr, uid, ids, context=context)
+ list_partner = []
+ for data in data_inv:
+ list_partner.append(data.partner.id)
+ ids2 = list_partner
+ while ids2:
+ ids2 = res_obj.search(cr, uid, [('associate_member', 'in', ids2)], context=context)
+ list_partner += ids2
+ return list_partner
+ # XXX: copied from openerp/addons/membership/membership.py
+ def _get_invoice_partner(self, cr, uid, ids, context=None):
+ inv_obj = self.pool.get('account.invoice')
+ res_obj = self.pool.get('res.partner')
+ data_inv = inv_obj.browse(cr, uid, ids, context=context)
+ list_partner = []
+ for data in data_inv:
+ list_partner.append(data.partner_id.id)
+ ids2 = list_partner
+ while ids2:
+ ids2 = res_obj.search(cr, uid, [('associate_member', 'in', ids2)], context=context)
+ list_partner += ids2
+ return list_partner
+
+ _columns = {
+ 'member_ident': fields.char('Member identifier', size=64),
+ 'associate_members': fields.one2many('res.partner', 'associate_member', 'Associate members', help='Members who are associated to this partner.'),
+ # XXX: copied from openerp/addons/membership/membership.py
+ 'membership_start': fields.function(
+ _membership_date, multi = 'membership_start',
+ string = 'Membership Start Date', type = 'date',
+ store = {
+ 'account.invoice': (_get_invoice_partner, ['state'], 10),
+ 'membership.membership_line': (_get_partner_id, ['state'], 10, ),
+ 'res.partner': (lambda self, cr, uid, ids, c={}: ids, ['free_member'], 10)
+ }, help="Date from which membership becomes active."),
+ # XXX: copied from openerp/addons/membership/membership.py
+ 'membership_stop': fields.function(
+ _membership_date,
+ string = 'Membership End Date', type='date', multi='membership_stop',
+ store = {
+ 'account.invoice': (_get_invoice_partner, ['state'], 10),
+ 'membership.membership_line': (_get_partner_id, ['state'], 10),
+ 'res.partner': (lambda self, cr, uid, ids, c={}: ids, ['free_member'], 10)
+ }, help="Date until which membership remains active."),
+ # XXX: copied from openerp/addons/membership/membership.py
+ 'membership_cancel': fields.function(
+ _membership_date,
+ string = 'Cancel Membership Date', type='date', multi='membership_cancel',
+ store = {
+ 'account.invoice': (_get_invoice_partner, ['state'], 11),
+ 'membership.membership_line': (_get_partner_id, ['state'], 10),
+ 'res.partner': (lambda self, cr, uid, ids, c={}: ids, ['free_member'], 10)
+ }, help="Date on which membership has been cancelled"),
+ }
+
+ _sql_constraints = [
+ ('member_ident_uniq', 'unique(member_ident, company_id)', 'The member identifier must be unique !'),
+ ]
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()
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: [write] date_from: %s" % str(date_from))
- print ("DEV: [write] date_to : %s" % str(date_to))
+ 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
, context=context)
if line.product_id.membership_grouped:
if line.invoice_id.partner_id.associate_members:
- for associate_member in line.invoice_id.partner_id.associate_members:
- print ("DEV: [write] [associate] date_from: %s" % str(date_from))
- print ("DEV: [write] [associate] date_to : %s" % str(date_to))
- member_line_obj.create(cr, uid, {
- 'partner': associate_member.id,
- 'membership_id': line.product_id.id,
- 'member_price': line.price_unit,
- 'date': time.strftime('%Y-%m-%d'),
- 'date_from': date_from,
- 'date_to': date_to,
- 'account_invoice_line': line.id,
- }, context=context)
+ associate_member_line_ids = member_line_obj.search(cr, uid
+ , [ ('account_invoice_line', '=', line.id)
+ , ('partner', '!=', line.invoice_id.partner_id.id)
+ ]
+ , 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
+ }
+ , context=context)
else:
print("DEV: mettre une contrainte pour l'objet\
membership.membership_line interdisant les\
adhésions groupées reliées à des partenaires\
sans membres associés")
else:
- associate_ml_ids = member_line_obj.search(cr, uid, [('account_invoice_line', '=', line.id), ('partner', '!=', line.invoice_id.partner_id.id)], context=context)
- member_line_obj.unlink(cr, uid, associate_ml_ids, context=context)
-
+ associate_member_line_ids = member_line_obj.search(cr, uid
+ , [ ('account_invoice_line', '=', line.id)
+ , ('partner', '!=', line.invoice_id.partner_id.id)
+ ]
+ , 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:
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:
date_from = member_line.date_from
date_to = member_line.date_to
if line.product_id.membership_date2date:
- print "DEV: 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: [create] date_from: %s" % str(date_from))
- print ("DEV: [create] date_to : %s" % str(date_to))
+ 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
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: [create] [associate] date_from: %s" % str(date_from))
- print ("DEV: [create] [associate] date_to : %s" % str(date_to))
+ 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,
'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: