what is the use of _line_column and _tax_column? and why is the AdditionalDiscountable class not inheriting osv.Model or osv.AbstractModel? You could use the example of mail_thread available in MAIL module and use osv.AbstractModel.