11.0 Migration partner_location_auto_create #443
Conversation
…he parent location of the company as default location
|
Another improvement: |
| 'license': 'AGPL-3', | ||
| 'complexity': 'normal', | ||
| 'images': [], | ||
| 'website': 'http://www.savoirfairelinux.com', |
There was a problem hiding this comment.
Website should be https://github.com/OCA/stock-logistics-warehouse
| 'test': [], | ||
| 'auto_install': False, | ||
| 'installable': True, | ||
| } |
| # © 2018 brain-tec AG (Kumar Aberer <kumar.aberer@braintec-group.com>) | ||
| # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). | ||
|
|
||
| from odoo import api, SUPERUSER_ID |
There was a problem hiding this comment.
Not sure if this is needed anymore. We have a sudo now.
There was a problem hiding this comment.
It's needed to create a new Environment for the installation hook. Sudo does not help here
| @@ -0,0 +1,26 @@ | |||
| <?xml version="1.0"?> | |||
| <openerp> | |||
|
|
|||
| <button name="%(stock.action_location_form)d" | ||
| position="after"> | ||
| <group> | ||
|
|
| <openerp> | ||
| <data> | ||
|
|
||
| <record id="view_partner_form" model="ir.ui.view" > |
|
|
||
|
|
||
|
|
||
| </data> |
There was a problem hiding this comment.
Remove unnecessary space from xml files.
| ############################################################################## | ||
| # © 2015 Savoir-faire Linux | ||
| # © 2018 brain-tec AG (Kumar Aberer <kumar.aberer@braintec-group.com>) | ||
| # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). | ||
|
|
||
| from openerp import models, fields, api |
There was a problem hiding this comment.
openerp => odoo
| _inherit = 'res.config.settings' | ||
|
|
||
| default_customer_location = fields.Many2one( | ||
| related='company_id.default_customer_location') |
| default_customer_location = fields.Many2one( | ||
| related='company_id.default_customer_location') | ||
| default_supplier_location = fields.Many2one( | ||
| related='company_id.default_supplier_location') |
| ############################################################################## | ||
| # © 2015 Savoir-faire Linux | ||
| # © 2018 brain-tec AG (Kumar Aberer <kumar.aberer@braintec-group.com>) | ||
| # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). | ||
|
|
||
| from openerp import models, fields, api, _ |
There was a problem hiding this comment.
openerp => odoo
| location.unlink() | ||
| for partner in self: | ||
| for location in partner.location_ids: | ||
| moves = move_obj.search([ |
There was a problem hiding this comment.
If you have locations with huge moves, this could be a bottleneck.
I would have used search_count() instead or search with a limit of 1 before the loop and create a dict with result grouping by location id and then iterating.
| @@ -0,0 +1,26 @@ | |||
| <?xml version="1.0"?> | |||
| <openerp> | |||
|
@MeetKD Can you update your review ? |
|
@ghost due to inactivity, this will be closed soon. |
|
Closing this due to inactivity and user has been deleted. |
Migration of partner_location_auto_create
two improvement have been made:
https://github.com/BT-kaberer/stock-logistics-warehouse/blob/11.0-mig-partner_location_auto_create/partner_location_auto_create/hooks.py#L9
#343