add asset management modules

Indeed.

This is a bug coming from the standard odoo ‘account_asset’ module (I checked and still not fixed in V8) and probably never remarked/reported because this method is usually called via the workflow. 

I have corrected this and submitted the fix.

 

Regards,

Luc

 

 <http://www.noviat.com/> www.noviat.com

Rusatiralaan 1, 1083 Brussel

+32 2 808 86 38

 



 

From: ecino [mailto:notifications@github.com] 
Sent: vrijdag 31 oktober 2014 16:26
To: OCA/account-financial-tools
Cc: luc-demeyer
Subject: Re: [account-financial-tools] add asset management modules (#51)

 

In account_asset_management/account_asset_invoice.py:

> +#    GNU Affero General Public License for more details.
> +#
> +#    You should have received a copy of the GNU Affero General Public License
> +#    along with this program. If not, see <http://www.gnu.org/licenses/>.
> +#
> +##############################################################################
> +
> +from openerp.osv import fields, orm
> +import logging
> +_logger = logging.getLogger(__name__)
> +
> +
> +class account_invoice(orm.Model):
> +    _inherit = 'account.invoice'
> +
> +    def action_number(self, cr, uid, ids, *args):

This is not the correct way of defining the function. You should replace it by :
def action_number(self, cr, uid, ids, context=None):
Or
def action_number(self, cr, uid, ids, **kwargs):

—
Reply to this email directly or view it on GitHub <https://github.com/OCA/account-financial-tools/pull/51/files#r19672864> .  <https://github.com/notifications/beacon/AHWZQXIQoLVJTln9BnAQB0A4xcj4QB-Xks5nI6GBgaJpZM4CpKrQ.gif>