← Back to team overview

c2c-oerpscenario team mailing list archive

[Bug 827746] [NEW] gnucash addon uses wrong logging approach

 

Public bug reported:

the gnucash addon uses parameters that don't work with
netsvc.Logger.notifyChannel, three parameters (self, message, object)
instead of four (self,name,level,msg).  And beyond that, the source code
for netsvc.Logger.notifyChannel basically says "don't use me anymore" so
even with the right number of parameters this would be incorrect.  From
the commit log it looks as though at least one OpenERP developer, ksa,
has been in the file before (and indeed made the notifyChannel change,
previously it was a print statement) so I figured I'd post this rather
than chasing the original dev.

** Affects: openobject-addons
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.
https://bugs.launchpad.net/bugs/827746

Title:
  gnucash addon uses wrong logging approach

Status in OpenERP Modules (addons):
  New

Bug description:
  the gnucash addon uses parameters that don't work with
  netsvc.Logger.notifyChannel, three parameters (self, message, object)
  instead of four (self,name,level,msg).  And beyond that, the source
  code for netsvc.Logger.notifyChannel basically says "don't use me
  anymore" so even with the right number of parameters this would be
  incorrect.  From the commit log it looks as though at least one
  OpenERP developer, ksa, has been in the file before (and indeed made
  the notifyChannel change, previously it was a print statement) so I
  figured I'd post this rather than chasing the original dev.

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-addons/+bug/827746/+subscriptions


Follow ups

References