Bug #656370 “Upload image and import orders fail” : Bugs : Magento OpenERP Connector

Revision history for this message

ntrubert-cobweb (ntrubert) wrote on 2010-10-08:

#1

Which magento release are you using?
Which branch of magentoerpconnect module ?
Which setting do you have on Magento Shops -> Order Generation?

My setting works with:
Tax not including
Shipping Policy : Shipping & Manual Invoice
Invoice Generation Policy : Draft
Packing Policy : Packing Delivery
Invoice on : Odrered Quantities
Picking Generation Policy : Draft

Revision history for this message

ntrubert-cobweb (ntrubert) wrote on 2010-10-08:

#2

The first error is an answer from Magento Api Fault: <Fault 104: 'Cannot create image.'>, I do not use image synchronization yet but may be you have check Magento Setting.

Revision history for this message

Mutunga Mutua (jeffmuts) wrote on 2010-10-08:

#3

Thanks for the reply

I changed my magentoerpconnect to the ~openlabs-akretion-consortium/magentoerpconnect/improved_discount_n_address : viewing /i18n for revision 306

Now I am getting the following error.

My shop settings are like yours Nico74

Tax not including
Shipping Policy : Shipping & Manual Invoice
Invoice Generation Policy : Draft
Packing Policy : Packing Delivery
Invoice on : Odrered Quantities
Picking Generation Policy : Draft

Traceback (most recent call last):
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/netsvc.py", line 256, in dispatch
    result = LocalService(service_name)(method, *params)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/netsvc.py", line 76, in __call__
    return getattr(self, method)(*params)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/service/web_services.py", line 577, in execute
    res = service.execute(db, uid, object, method, *args)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py", line 58, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py", line 119, in execute
    res = pool.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py", line 111, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/base_sale_multichannels/sale.py", line 216, in import_orders
    self.import_shop_orders(cr, uid, shop, defaults, ctx)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/sale.py", line 113, in import_shop_orders
    'one_by_one': True,
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/magerp_osv.py", line 236, in mage_import_base
    data = []
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/base_external_referentials/external_osv.py", line 167, in ext_import
    vals = self.oevals_from_extdata(cr, uid, external_referential_id, each_row, for_key_field, mapping_lines, defaults, context)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/sale.py", line 305, in oevals_from_extdata
    lines_vals = []
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/sale.py", line 245, in get_order_addresses
    data_record['billing_address'].update(self.get_mage_customer_address_id(data_record['billing_address']))
AttributeError: 'NoneType' object has no attribute 'translate'

What do I need to do about this error.

Regards

Mutunga

Thanks for the reply

I changed my magentoerpconnect to the ~openlabs-akretion-consortium/magentoerpconnect/improved_discount_n_address : viewing /i18n for revision 306

Now  I am getting the following error.

My shop settings are like yours Nico74

Tax not including
Shipping Policy : Shipping & Manual Invoice
Invoice Generation Policy : Draft
Packing Policy : Packing Delivery
Invoice on : Odrered Quantities
Picking Generation Policy : Draft

Traceback (most recent call last):
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/netsvc.py", line 256, in dispatch
    result = LocalService(service_name)(method, *params)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/netsvc.py", line 76, in __call__
    return getattr(self, method)(*params)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/service/web_services.py", line 577, in execute
    res = service.execute(db, uid, object, method, *args)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py", line 58, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py", line 119, in execute
    res = pool.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py", line 111, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/base_sale_multichannels/sale.py", line 216, in import_orders
    self.import_shop_orders(cr, uid, shop, defaults, ctx)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/sale.py", line 113, in import_shop_orders
    'one_by_one': True,
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/magerp_osv.py", line 236, in mage_import_base
    data = []
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/base_external_referentials/external_osv.py", line 167, in ext_import
    vals = self.oevals_from_extdata(cr, uid, external_referential_id, each_row, for_key_field, mapping_lines, defaults, context)
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/sale.py", line 305, in oevals_from_extdata
    lines_vals = []
  File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/sale.py", line 245, in get_order_addresses
    data_record['billing_address'].update(self.get_mage_customer_address_id(data_record['billing_address']))
AttributeError: 'NoneType' object has no attribute 'translate'

What do I need to do about this error.

Regards

Mutunga

Revision history for this message

Mutunga Mutua (jeffmuts) wrote on 2010-10-08:

#4

Dear Nico74

You were right about the image error. It was about permissions on my magento media folder. I just changed the permissions to 777 and that solved it.

Tell me this. I used the launchpad improved address and discount magentoerpconnect. I get an error which i guess can be fixed by what you were trying to explain here on Question #119102:

I'm using this branch now with some modification :

https://code.launchpad.net/~openlabs-akretion-consortium/magentoerpconnect/improved_discount_n_address
Go feature in it (check if address is already registered)

And also I disabled auto invoice generation.

My issue is solved and I'm working on magento 1.4.1.1

Exactly how do you fix the address issue

I get the error:

File "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/sale.py", line 254, in get_order_addresses
#inv_res = partner_address_obj.find_or_create(cr, uid, data_record['billing_address'], external_referential_id, address_default, context)
NameError: global name 'address_default' is not defined

Kindly assist me ASAP

Thank you

Regards

Mutunga Mutua

Revision history for this message

ntrubert-cobweb (ntrubert) wrote on 2010-10-08:

#5

1°) According to your first error taxvat need to be set or change the line in sale.py file like this:

        # Adds vat number (country code+magento vat) if base_vat module is installed and Magento sends customer_taxvat
        cr.execute('select * from ir_module_module where name=%s and state=%s', ('base_vat','installed'))
- if cr.fetchone() and 'customer_taxvat' in data_record:
+ if cr.fetchone() and ('customer_taxvat' and 'customer_taxvat'!="None") in data_record:
            allchars = string.maketrans('', '')
            delchars = ''.join([c for c in allchars if c not in string.letters + string.digits])

2°) For the second one I remember that changed the get_order_addresses method

Could you put the status of your get_order_addresses method
Do you have "address_default" on some call like :

billing_default.update({'email' : data_record.get('customer_email', False)})
inv_res = partner_address_obj.find_or_create(cr, uid, data_record['billing_address'], external_referential_id, address_default, context)

if yes you need to add to lines on get_order_addresses method:

    def get_order_addresses(self, cr, uid, res, external_referential_id, data_record, key_field, mapping_lines, defaults, context):
        partner_obj = self.pool.get('res.partner')
        partner_address_obj = self.pool.get('res.partner.address')
        del(data_record['billing_address']['parent_id'])
        if 'parent_id' in data_record['shipping_address']:
            del(data_record['shipping_address']['parent_id'])

        #Magento uses to create same addresses over and over, try to detect if customer already have such an address (Magento won't tell it!)
        #We also create new addresses for each command here, passing a custom magento_id key in the following is what
        #avoid the base_external_referentials framework to try to update existing partner addresses
        data_record['billing_address'].update(self.get_mage_customer_address_id(data_record['billing_address']))
        if 'address_type' in data_record['shipping_address']:
            data_record['shipping_address'].update(self.get_mage_customer_address_id(data_record['shipping_address']))
        shipping_default = {}
        billing_default = {}

+ address_default = {}

        if res.get('partner_id', False):
            shipping_default = {'partner_id': res.get('partner_id', False)}
        billing_default = shipping_default.copy()

+ address_default = billing_default

        billing_default.update({'email' : data_record.get('customer_email', False)})
        inv_res = partner_address_obj.find_or_create(cr, uid, data_record['billing_address'], external_referential_id, address_default, context)
        if 'address_type' in data_record['shipping_address']:

Hope that can helpfull for you!

1°) According to your first error taxvat need to be set or change the line in sale.py file like this:

# Adds vat number (country code+magento vat) if base_vat module is installed and Magento sends customer_taxvat
        cr.execute('select * from ir_module_module where name=%s and state=%s', ('base_vat','installed'))
-        if cr.fetchone() and 'customer_taxvat' in data_record:
+       if cr.fetchone() and ('customer_taxvat' and 'customer_taxvat'!="None") in data_record:
            allchars = string.maketrans('', '')
            delchars = ''.join([c for c in allchars if c not in string.letters + string.digits])

2°) For the second one I remember that changed the get_order_addresses method

Could you put the status of your get_order_addresses method 
Do you have "address_default" on some call like :

billing_default.update({'email' : data_record.get('customer_email', False)})
        inv_res = partner_address_obj.find_or_create(cr, uid, data_record['billing_address'], external_referential_id, address_default, context)

if yes you need to add to lines on get_order_addresses method:

def get_order_addresses(self, cr, uid, res, external_referential_id, data_record, key_field, mapping_lines, defaults, context):
        partner_obj = self.pool.get('res.partner')
        partner_address_obj = self.pool.get('res.partner.address')
        del(data_record['billing_address']['parent_id'])
        if 'parent_id' in data_record['shipping_address']:
            del(data_record['shipping_address']['parent_id'])

#Magento uses to create same addresses over and over, try to detect if customer already have such an address (Magento won't tell it!)
        #We also create new addresses for each command here, passing a custom magento_id key in the following is what
        #avoid the base_external_referentials framework to try to update existing partner addresses
        data_record['billing_address'].update(self.get_mage_customer_address_id(data_record['billing_address']))
        if 'address_type' in data_record['shipping_address']:
            data_record['shipping_address'].update(self.get_mage_customer_address_id(data_record['shipping_address']))
        shipping_default = {}
        billing_default = {}

+       address_default = {}

if res.get('partner_id', False):
            shipping_default = {'partner_id': res.get('partner_id', False)}
        billing_default = shipping_default.copy()

+       address_default = billing_default

billing_default.update({'email' : data_record.get('customer_email', False)})
        inv_res = partner_address_obj.find_or_create(cr, uid, data_record['billing_address'], external_referential_id, address_default, context)
        if 'address_type' in data_record['shipping_address']:

Hope that can helpfull for you!

Revision history for this message

Gabriel Ruiz (gruiz) wrote on 2010-10-21:

#6

Hi

After adding the two lines that Nico74 suggested, I now get this error:

Environment Information :
System : Linux-2.6.32-21-generic-i686-with-Ubuntu-10.04-lucid
OS Name : posix
Distributor ID: Ubuntu
Description: Ubuntu 10.04 LTS
Release: 10.04
Codename: lucid
Operating System Release : 2.6.32-21-generic
Operating System Version : #32-Ubuntu SMP Fri Apr 16 08:10:02 UTC 2010
Operating System Architecture : 32bit
Operating System Locale : es_ES.UTF8
Python Version : 2.6.5
OpenERP-Client Version : 5.0.14
Last revision No. & ID :Bazaar Package not Found !Traceback (most recent call last):
  File "/home/opentia/openerp-server-5.0.14/bin/netsvc.py", line 256, in dispatch
    result = LocalService(service_name)(method, *params)
  File "/home/opentia/openerp-server-5.0.14/bin/netsvc.py", line 76, in __call__
    return getattr(self, method)(*params)
  File "/home/opentia/openerp-server-5.0.14/bin/service/web_services.py", line 577, in execute
    res = service.execute(db, uid, object, method, *args)
  File "/home/opentia/openerp-server-5.0.14/bin/osv/osv.py", line 58, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/home/opentia/openerp-server-5.0.14/bin/osv/osv.py", line 119, in execute
    res = pool.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/home/opentia/openerp-server-5.0.14/bin/osv/osv.py", line 111, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/base_sale_multichannels/sale.py", line 216, in import_orders
    self.import_shop_orders(cr, uid, shop, defaults, ctx)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/magentoerpconnect/sale.py", line 114, in import_shop_orders
    'ids_or_filter':ids_or_filter
  File "/home/opentia/openerp-server-5.0.14/bin/addons/magentoerpconnect/magerp_osv.py", line 255, in mage_import_base
    rec_result = self.ext_import(cr, uid, rec_data, external_referential_id, defaults, context)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/base_external_referentials/external_osv.py", line 167, in ext_import
    vals = self.oevals_from_extdata(cr, uid, external_referential_id, each_row, for_key_field, mapping_lines, defaults, context)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/magentoerpconnect/sale.py", line 368, in oevals_from_extdata
    res = self.get_order_addresses(cr, uid, res, external_referential_id, data_record, key_field, mapping_lines, defaults, context)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/magentoerpconnect/sale.py", line 256, in get_order_addresses
    inv_res = partner_address_obj.find_or_create(cr, uid, data_record['billing_address'], external_referential_id, address_default, context)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/magentoerpconnect/partner.py", line 77, in find_or_create
    logger = netsvc.Logger()
NameError: global name 'netsvc' is not defined

I'm using connector branch ~openlabs-akretion-consortium/magentoerpconnect/improved_discount_n_address

Any help?
Thanks!

Hi

After adding the two lines that Nico74 suggested, I now get this error:

Environment Information : 
System : Linux-2.6.32-21-generic-i686-with-Ubuntu-10.04-lucid
OS Name : posix
Distributor ID:	Ubuntu
Description:	Ubuntu 10.04 LTS
Release:	10.04
Codename:	lucid
Operating System Release : 2.6.32-21-generic
Operating System Version : #32-Ubuntu SMP Fri Apr 16 08:10:02 UTC 2010
Operating System Architecture : 32bit
Operating System Locale : es_ES.UTF8
Python Version : 2.6.5
OpenERP-Client Version : 5.0.14
Last revision No. & ID :Bazaar Package not Found !Traceback (most recent call last):
  File "/home/opentia/openerp-server-5.0.14/bin/netsvc.py", line 256, in dispatch
    result = LocalService(service_name)(method, *params)
  File "/home/opentia/openerp-server-5.0.14/bin/netsvc.py", line 76, in __call__
    return getattr(self, method)(*params)
  File "/home/opentia/openerp-server-5.0.14/bin/service/web_services.py", line 577, in execute
    res = service.execute(db, uid, object, method, *args)
  File "/home/opentia/openerp-server-5.0.14/bin/osv/osv.py", line 58, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/home/opentia/openerp-server-5.0.14/bin/osv/osv.py", line 119, in execute
    res = pool.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/home/opentia/openerp-server-5.0.14/bin/osv/osv.py", line 111, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/base_sale_multichannels/sale.py", line 216, in import_orders
    self.import_shop_orders(cr, uid, shop, defaults, ctx)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/magentoerpconnect/sale.py", line 114, in import_shop_orders
    'ids_or_filter':ids_or_filter
  File "/home/opentia/openerp-server-5.0.14/bin/addons/magentoerpconnect/magerp_osv.py", line 255, in mage_import_base
    rec_result = self.ext_import(cr, uid, rec_data, external_referential_id, defaults, context)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/base_external_referentials/external_osv.py", line 167, in ext_import
    vals = self.oevals_from_extdata(cr, uid, external_referential_id, each_row, for_key_field, mapping_lines, defaults, context)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/magentoerpconnect/sale.py", line 368, in oevals_from_extdata
    res = self.get_order_addresses(cr, uid, res, external_referential_id, data_record, key_field, mapping_lines, defaults, context)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/magentoerpconnect/sale.py", line 256, in get_order_addresses
    inv_res = partner_address_obj.find_or_create(cr, uid, data_record['billing_address'], external_referential_id, address_default, context)
  File "/home/opentia/openerp-server-5.0.14/bin/addons/magentoerpconnect/partner.py", line 77, in find_or_create
    logger = netsvc.Logger()
NameError: global name 'netsvc' is not defined

I'm using connector branch ~openlabs-akretion-consortium/magentoerpconnect/improved_discount_n_address

Any help?
Thanks!

Revision history for this message

ntrubert-cobweb (ntrubert) wrote on 2010-10-21:

#7

You need to add "Import " on patner.py file

# Copyright (C) 2009 Sharoon Thomas, Raphaël Valyi #
# #
#This program is free software: you can redistribute it and/or modify #
#it under the terms of the GNU General Public License as published by #
#the Free Software Foundation, either version 3 of the License, or #
#(at your option) any later version. #
# #
#This program is distributed in the hope that it will be useful, #
#but WITHOUT ANY WARRANTY; without even the implied warranty of #
#MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the #
#GNU General Public License for more details. #
# #
#You should have received a copy of the GNU General Public License #
#along with this program. If not, see <http://www.gnu.org/licenses/>. #
#########################################################################

from osv import osv, fields
import magerp_osv
# add line
import netsvc

Revision history for this message

Gabriel Ruiz (gruiz) wrote on 2010-10-21:

#8

Yes, you were right, thanks

But now I get another error:

  File "/openerp/openerp-server-5.0.12/bin/netsvc.py", line 247, in dispatch
    result = LocalService(service_name)(method, *params)
  File "/openerp/openerp-server-5.0.12/bin/netsvc.py", line 76, in __call__
    return getattr(self, method)(*params)
  File "/openerp/openerp-server-5.0.12/bin/service/web_services.py", line 577, in execute
    res = service.execute(db, uid, object, method, *args)
  File "/openerp/openerp-server-5.0.12/bin/osv/osv.py", line 58, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/openerp/openerp-server-5.0.12/bin/osv/osv.py", line 119, in execute
    res = pool.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/openerp/openerp-server-5.0.12/bin/osv/osv.py", line 111, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/openerp/openerp-server-5.0.12/bin/addons/base_sale_multichannels/sale.py", line 216, in import_orders
    self.import_shop_orders(cr, uid, shop, defaults, ctx)
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/sale.py", line 115, in import_shop_orders
    'ids_or_filter':ids_or_filter
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/magerp_osv.py", line 255, in mage_import_base
    rec_result = self.ext_import(cr, uid, rec_data, external_referential_id, defaults, context)
  File "/openerp/openerp-server-5.0.12/bin/addons/base_external_referentials/external_osv.py", line 167, in ext_import
    vals = self.oevals_from_extdata(cr, uid, external_referential_id, each_row, for_key_field, mapping_lines, defaults, context)
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/sale.py", line 369, in oevals_from_extdata
    res = self.get_order_addresses(cr, uid, res, external_referential_id, data_record, key_field, mapping_lines, defaults, context)
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/sale.py", line 257, in get_order_addresses
    inv_res = partner_address_obj.find_or_create(cr, uid, data_record['billing_address'], external_referential_id, address_default, context)
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/partner.py", line 81, in find_or_create
    oe_address = self.mage_to_oe_address(cr, uid, magento_address, address_default)
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/partner.py", line 73, in mage_to_oe_address
    oe_address['partner_id'] = address_default['partner_id']
KeyError: 'partner_id'

Do I have to change somethhing else?
Sorry for these issues but I got this branch because it was suppose to be mature. Is there another branch that doesn't contain these bugs?
Or it is a problem related with my configuration.

Thansk a lot.

Yes, you were right, thanks

But now I get another error:

File "/openerp/openerp-server-5.0.12/bin/netsvc.py", line 247, in dispatch
    result = LocalService(service_name)(method, *params)
  File "/openerp/openerp-server-5.0.12/bin/netsvc.py", line 76, in __call__
    return getattr(self, method)(*params)
  File "/openerp/openerp-server-5.0.12/bin/service/web_services.py", line 577, in execute
    res = service.execute(db, uid, object, method, *args)
  File "/openerp/openerp-server-5.0.12/bin/osv/osv.py", line 58, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/openerp/openerp-server-5.0.12/bin/osv/osv.py", line 119, in execute
    res = pool.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/openerp/openerp-server-5.0.12/bin/osv/osv.py", line 111, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/openerp/openerp-server-5.0.12/bin/addons/base_sale_multichannels/sale.py", line 216, in import_orders
    self.import_shop_orders(cr, uid, shop, defaults, ctx)
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/sale.py", line 115, in import_shop_orders
    'ids_or_filter':ids_or_filter
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/magerp_osv.py", line 255, in mage_import_base
    rec_result = self.ext_import(cr, uid, rec_data, external_referential_id, defaults, context)
  File "/openerp/openerp-server-5.0.12/bin/addons/base_external_referentials/external_osv.py", line 167, in ext_import
    vals = self.oevals_from_extdata(cr, uid, external_referential_id, each_row, for_key_field, mapping_lines, defaults, context)
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/sale.py", line 369, in oevals_from_extdata
    res = self.get_order_addresses(cr, uid, res, external_referential_id, data_record, key_field, mapping_lines, defaults, context)
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/sale.py", line 257, in get_order_addresses
    inv_res = partner_address_obj.find_or_create(cr, uid, data_record['billing_address'], external_referential_id, address_default, context)
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/partner.py", line 81, in find_or_create
    oe_address = self.mage_to_oe_address(cr, uid, magento_address, address_default)
  File "/openerp/openerp-server-5.0.12/bin/addons/improved_discount_n_address/partner.py", line 73, in mage_to_oe_address
    oe_address['partner_id'] = address_default['partner_id']
KeyError: 'partner_id'

Do I have to change somethhing else?
Sorry for these issues but I got this branch because it was suppose to be mature. Is there another branch that doesn't contain these bugs?
Or it is a problem related with my configuration.

Thansk a lot.

Revision history for this message

ntrubert-cobweb (ntrubert) wrote on 2010-10-22:

#9

I got this branch for the same reason "mature" status. So the partner management is better for sure.
When I will get more time I propose some modification to avoid these issue.

For now, could check your file "sale.py" I added 2 lines before calling "find_or_create" function?

    def get_order_addresses(self, cr, uid, res, external_referential_id, data_record, key_field, mapping_lines, defaults, context):
        partner_obj = self.pool.get('res.partner')
        partner_address_obj = self.pool.get('res.partner.address')
        del(data_record['billing_address']['parent_id'])
        if 'parent_id' in data_record['shipping_address']:
            del(data_record['shipping_address']['parent_id'])

        #Magento uses to create same addresses over and over, try to detect if customer already have such an address (Magento won't tell it!)
        #We also create new addresses for each command here, passing a custom magento_id key in the following is what
        #avoid the base_external_referentials framework to try to update existing partner addresses
        data_record['billing_address'].update(self.get_mage_customer_address_id(data_record['billing_address']))
        if 'address_type' in data_record['shipping_address']:
            data_record['shipping_address'].update(self.get_mage_customer_address_id(data_record['shipping_address']))
        shipping_default = {}
        billing_default = {}

+ # add by Nico74
+ address_default = {}

        if res.get('partner_id', False):
            shipping_default = {'partner_id': res.get('partner_id', False)}
        billing_default = shipping_default.copy()

+ # add by Nico74
+ address_default = billing_default

        billing_default.update({'email' : data_record.get('customer_email', False)})
        inv_res = partner_address_obj.find_or_create(cr, uid, data_record['billing_address'], external_referential_id, address_default, context)
        if 'address_type' in data_record['shipping_address']:
            ship_res = partner_address_obj.find_or_create(cr, uid, data_record['shipping_address'], external_referential_id, address_default, context)

I got this branch for the same reason "mature" status. So the partner management is better for sure.
When I will get more time I propose some modification to avoid these issue.

For now, could check your file "sale.py" I added 2 lines before calling "find_or_create" function?
 
    def get_order_addresses(self, cr, uid, res, external_referential_id, data_record, key_field, mapping_lines, defaults, context):
        partner_obj = self.pool.get('res.partner')
        partner_address_obj = self.pool.get('res.partner.address')
        del(data_record['billing_address']['parent_id'])
        if 'parent_id' in data_record['shipping_address']:
            del(data_record['shipping_address']['parent_id'])

#Magento uses to create same addresses over and over, try to detect if customer already have such an address (Magento won't tell it!)
        #We also create new addresses for each command here, passing a custom magento_id key in the following is what
        #avoid the base_external_referentials framework to try to update existing partner addresses
        data_record['billing_address'].update(self.get_mage_customer_address_id(data_record['billing_address']))
        if 'address_type' in data_record['shipping_address']:
            data_record['shipping_address'].update(self.get_mage_customer_address_id(data_record['shipping_address']))
        shipping_default = {}
        billing_default = {}

+       # add by Nico74
+        address_default = {}

if res.get('partner_id', False):
            shipping_default = {'partner_id': res.get('partner_id', False)}
        billing_default = shipping_default.copy()

+        # add by Nico74
+        address_default = billing_default

billing_default.update({'email' : data_record.get('customer_email', False)})
        inv_res = partner_address_obj.find_or_create(cr, uid, data_record['billing_address'], external_referential_id, address_default, context)
        if 'address_type' in data_record['shipping_address']:
            ship_res = partner_address_obj.find_or_create(cr, uid, data_record['shipping_address'], external_referential_id, address_default, context)

Revision history for this message

Sébastien BEAU - http://www.akretion.com (sebastien.beau) wrote on 2010-10-24: Re: [Bug 656370] Re: Upload image and import orders fail

#10

Download full text (9.1 KiB)

Hi everybody
Please don't used the
lp:~openlabs-akretion-consortium/magentoerpconnect/improved_discount_n_address<https://code.launchpad.net/%7Eopenlabs-akretion-consortium/magentoerpconnect/improved_discount_n_address>,
indeed is tag as "mature" because it is ready for a merge, but this don't
mean that is a "mature" version of the connector. (Use it only if you are
interested by the "discount_n_address")

For information, there is* no* "mature" branch of magentoerpconnect, you
need to use the developpement branch ! !
V5 :
https://code.launchpad.net/~openlabs-akretion-consortium/magentoerpconnect/magentoerpconnect
V6 :
https://code.launchpad.net/~openlabs-akretion-consortium/magentoerpconnect/magentoerpconnect-v6

Try with the last code.

Have a nice day.

2010/10/22 Nico74 <email address hidden>

> I got this branch for the same reason "mature" status. So the partner
> management is better for sure.
> When I will get more time I propose some modification to avoid these issue.
>
> For now, could check your file "sale.py" I added 2 lines before calling
> "find_or_create" function?
>
> def get_order_addresses(self, cr, uid, res, external_referential_id,
> data_record, key_field, mapping_lines, defaults, context):
> partner_obj = self.pool.get('res.partner')
> partner_address_obj = self.pool.get('res.partner.address')
> del(data_record['billing_address']['parent_id'])
> if 'parent_id' in data_record['shipping_address']:
> del(data_record['shipping_address']['parent_id'])
>
> #Magento uses to create same addresses over and over, try to detect
> if customer already have such an address (Magento won't tell it!)
> #We also create new addresses for each command here, passing a
> custom magento_id key in the following is what
> #avoid the base_external_referentials framework to try to update
> existing partner addresses
>
> data_record['billing_address'].update(self.get_mage_customer_address_id(data_record['billing_address']))
> if 'address_type' in data_record['shipping_address']:
>
> data_record['shipping_address'].update(self.get_mage_customer_address_id(data_record['shipping_address']))
> shipping_default = {}
> billing_default = {}
>
> + # add by Nico74
> + address_default = {}
>
> if res.get('partner_id', False):
> shipping_default = {'partner_id': res.get('partner_id', False)}
> billing_default = shipping_default.copy()
>
> + # add by Nico74
> + address_default = billing_default
>
> billing_default.update({'email' : data_record.get('customer_email',
> False)})
> inv_res = partner_address_obj.find_or_create(cr, uid,
> data_record['billing_address'], external_referential_id, address_default,
> context)
> if 'address_type' in data_record['shipping_address']:
> ship_res = partner_address_obj.find_or_create(cr, uid,
> data_record['shipping_address'], external_referential_id, address_default,
> context)
>
> --
> Upload image and import orders fail
> https://bugs.launchpad.net/bugs/656370
> You received this bug notification because you are subscribed to Magento
> Open ERP Con...

Hi everybody
Please don't used the
lp:~openlabs-akretion-consortium/magentoerpconnect/improved_discount_n_address<https://code.launchpad.net/%7Eopenlabs-akretion-consortium/magentoerpconnect/improved_discount_n_address>,
indeed is tag as "mature" because it is ready for a merge, but this don't
mean that is a "mature" version of the connector. (Use it only if you are
interested by the "discount_n_address")

For information, there is* no* "mature" branch of magentoerpconnect, you
need to use the developpement branch ! !
V5 :
https://code.launchpad.net/~openlabs-akretion-consortium/magentoerpconnect/magentoerpconnect
V6 :
https://code.launchpad.net/~openlabs-akretion-consortium/magentoerpconnect/magentoerpconnect-v6

Try with the last code.

Have a nice day.

2010/10/22 Nico74 <656370@bugs.launchpad.net>

> I got this branch for the same reason "mature" status. So the partner
> management is better for sure.
> When I will get more time I propose some modification to avoid these issue.
>
> For now, could check your file "sale.py" I added 2 lines before calling
> "find_or_create" function?
>
>    def get_order_addresses(self, cr, uid, res, external_referential_id,
> data_record, key_field, mapping_lines, defaults, context):
>        partner_obj = self.pool.get('res.partner')
>        partner_address_obj = self.pool.get('res.partner.address')
>        del(data_record['billing_address']['parent_id'])
>        if 'parent_id' in data_record['shipping_address']:
>            del(data_record['shipping_address']['parent_id'])
>
>        #Magento uses to create same addresses over and over, try to detect
> if customer already have such an address (Magento won't tell it!)
>        #We also create new addresses for each command here, passing a
> custom magento_id key in the following is what
>        #avoid the base_external_referentials framework to try to update
> existing partner addresses
>
>  data_record['billing_address'].update(self.get_mage_customer_address_id(data_record['billing_address']))
>        if 'address_type' in data_record['shipping_address']:
>
>  data_record['shipping_address'].update(self.get_mage_customer_address_id(data_record['shipping_address']))
>        shipping_default = {}
>        billing_default = {}
>
> +       # add by Nico74
> +        address_default = {}
>
>        if res.get('partner_id', False):
>            shipping_default = {'partner_id': res.get('partner_id', False)}
>        billing_default = shipping_default.copy()
>
> +        # add by Nico74
> +        address_default = billing_default
>
>        billing_default.update({'email' : data_record.get('customer_email',
> False)})
>         inv_res = partner_address_obj.find_or_create(cr, uid,
> data_record['billing_address'], external_referential_id, address_default,
> context)
>         if 'address_type' in data_record['shipping_address']:
>             ship_res = partner_address_obj.find_or_create(cr, uid,
> data_record['shipping_address'], external_referential_id, address_default,
> context)
>
> --
> Upload image and import orders fail
> https://bugs.launchpad.net/bugs/656370
> You received this bug notification because you are subscribed to Magento
> Open ERP Connector.
>
> Status in Magento Open ERP Connector: New
>
> Bug description:
> Dear all
>
> My Name is Mutunga
>
> I have installed OpenERP version 5 and Magento. I have also installed and
> configured my magentoerpconnect with my magento shop setting and followed
> the instructions.
>
> I am able to synchronize referential settings and import product categories
> , attributes, attribute sets etc. I am also able to export catalog from the
> magento shop actions.
>
> However when I try Update images i get the following error
>
> Environment Information :
> System : Windows-XP-5.1.2600-SP2
> OS Name : nt
> Operating System Release : XP
> Operating System Version : 5.1.2600
> Operating System Architecture : 32bit
> Operating System Locale : en_US.cp1252
> Python Version : 2.5.2
> OpenERP-Client Version : 5.0.12
> Last revision No. & ID :Bazaar Package not Found !Traceback (most recent
> call last):
>  File "/usr/local/lib/python2.6/dist-packages/openerp-server/netsvc.py",
> line 256, in dispatch
>    result = LocalService(service_name)(method, *params)
>  File "/usr/local/lib/python2.6/dist-packages/openerp-server/netsvc.py",
> line 76, in __call__
>    return getattr(self, method)(*params)
>  File
> "/usr/local/lib/python2.6/dist-packages/openerp-server/service/web_services.py",
> line 577, in execute
>    res = service.execute(db, uid, object, method, *args)
>  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py",
> line 58, in wrapper
>    return f(self, dbname, *args, **kwargs)
>  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py",
> line 119, in execute
>    res = pool.execute_cr(cr, uid, obj, method, *args, **kw)
>  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py",
> line 111, in execute_cr
>    return getattr(object, method)(cr, uid, *args, **kw)
>  File
> "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/sale.py",
> line 64, in export_images
>    res = self.pool.get('product.images').update_remote_images(cr, uid,
> recent_changed_images, ctx)
>  File
> "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/product_images.py",
> line 96, in update_remote_images
>    'mime':each.filename and mimetypes.guess_type(each.filename)[0] or
> 'image/jpeg',
>  File
> "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/magerp_osv.py",
> line 53, in call
>    res = self.ser.call(self.session, method, arguments)
>  File "/usr/lib/python2.6/xmlrpclib.py", line 1199, in __call__
>    return self.__send(self.__name, args)
>  File "/usr/lib/python2.6/xmlrpclib.py", line 1489, in __request
>    verbose=self.__verbose
>  File "/usr/lib/python2.6/xmlrpclib.py", line 1253, in request
>    return self._parse_response(h.getfile(), sock)
>  File "/usr/lib/python2.6/xmlrpclib.py", line 1392, in _parse_response
>    return u.close()
>  File "/usr/lib/python2.6/xmlrpclib.py", line 838, in close
>    raise Fault(**self._stack[0])
> Fault: <Fault 104: 'Cannot create image.'>
>
>
> --------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
> Also when I try to import products i get the following error
>
>
> Environment Information :
> System : Windows-XP-5.1.2600-SP2
> OS Name : nt
> Operating System Release : XP
> Operating System Version : 5.1.2600
> Operating System Architecture : 32bit
> Operating System Locale : en_US.cp1252
> Python Version : 2.5.2
> OpenERP-Client Version : 5.0.12
> Last revision No. & ID :Bazaar Package not Found !Traceback (most recent
> call last):
>  File "/usr/local/lib/python2.6/dist-packages/openerp-server/netsvc.py",
> line 256, in dispatch
>    result = LocalService(service_name)(method, *params)
>  File "/usr/local/lib/python2.6/dist-packages/openerp-server/netsvc.py",
> line 76, in __call__
>    return getattr(self, method)(*params)
>  File
> "/usr/local/lib/python2.6/dist-packages/openerp-server/service/web_services.py",
> line 577, in execute
>    res = service.execute(db, uid, object, method, *args)
>  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py",
> line 58, in wrapper
>    return f(self, dbname, *args, **kwargs)
>  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py",
> line 119, in execute
>    res = pool.execute_cr(cr, uid, obj, method, *args, **kw)
>  File "/usr/local/lib/python2.6/dist-packages/openerp-server/osv/osv.py",
> line 111, in execute_cr
>    return getattr(object, method)(cr, uid, *args, **kw)
>  File
> "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/base_sale_multichannels/sale.py",
> line 216, in import_orders
>    self.import_shop_orders(cr, uid, shop, defaults, ctx)
>  File
> "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/sale.py",
> line 113, in import_shop_orders
>    'ids_or_filter':ids_or_filter
>  File
> "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/magerp_osv.py",
> line 236, in mage_import_base
>    rec_result = self.ext_import(cr, uid, rec_data, external_referential_id,
> defaults, context)
>  File
> "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/base_external_referentials/external_osv.py",
> line 167, in ext_import
>    vals = self.oevals_from_extdata(cr, uid, external_referential_id,
> each_row, for_key_field, mapping_lines, defaults, context)
>  File
> "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/sale.py",
> line 305, in oevals_from_extdata
>    res = self.get_order_addresses(cr, uid, res, external_referential_id,
> data_record, key_field, mapping_lines, defaults, context)
>  File
> "/usr/local/lib/python2.6/dist-packages/openerp-server/addons/magentoerpconnect/sale.py",
> line 245, in get_order_addresses
>    vat = data_record['customer_taxvat'].translate(allchars,
> delchars).upper()
> AttributeError: 'NoneType' object has no attribute 'translate'
>
>
> Can anyone help.
>
>
>

--

BEAU Sébastien
www.akretion.com

Revision history for this message

ag2888 (q-contact-gdaudit-com) wrote on 2010-11-13:

#11

Error du to name of picture or in link of your pict !

Beware !! When you add a picture, add a NAME !!

Sincerely

Kyle Waid (midwest) on 2011-06-28

Changed in magentoerpconnect:
status:	New → Invalid

Magento OpenERP Connector

Upload image and import orders fail

Bug Description

Other bug subscribers

Related questions

Remote bug watches