[17.0][OU-ADD] delivery: Migration to 17.0 #4637
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| # Copyright 2024 Viindoo Technology Joint Stock Company (Viindoo) | ||
| # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). | ||
|
|
||
| from openupgradelib import openupgrade | ||
|
|
||
|
|
||
| @openupgrade.migrate() | ||
| def migrate(env, version): | ||
| openupgrade.load_data(env, "delivery", "17.0.1.0/noupdate_changes.xml") |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,76 @@ | ||
| from openupgradelib import openupgrade | ||
|
|
||
| _new_fields = [ | ||
| ( | ||
| "carrier_id", # Field name | ||
| "stock.move.line", # Model name | ||
| "stock_move_line", # Table name | ||
| "many2one", # Odoo Field type (in lower case) | ||
| False, # [Optional] SQL type (if custom fields) | ||
| "delivery", # Module name | ||
| False, # [Optional] Default value | ||
| ) | ||
| ] | ||
|
|
||
| _xmlids_renames = [ | ||
| ( | ||
| "delivery.act_delivery_trackers_url", | ||
| "stock_delivery.act_delivery_trackers_url", | ||
| ), | ||
| ( | ||
| "delivery.access_choose_delivery_package", | ||
| "stock_delivery.access_choose_delivery_package", | ||
| ), | ||
| ( | ||
| "delivery.access_delivery_carrier_stock_user", | ||
| "stock_delivery.access_delivery_carrier_stock_user", | ||
| ), | ||
| ( | ||
| "delivery.access_delivery_carrier_stock_manager", | ||
| "stock_delivery.access_delivery_carrier_stock_manager", | ||
| ), | ||
| ( | ||
| "delivery.access_delivery_price_rule_stock_manager", | ||
| "stock_delivery.access_delivery_price_rule_stock_manager", | ||
| ), | ||
| ( | ||
| "delivery.access_delivery_zip_prefix_stock_manager", | ||
| "stock_delivery.access_delivery_zip_prefix_stock_manager", | ||
| ), | ||
| ( | ||
| "delivery.menu_action_delivery_carrier_form", | ||
| "stock_delivery.menu_action_delivery_carrier_form", | ||
| ), | ||
| ( | ||
| "delivery.menu_delivery_zip_prefix", | ||
| "stock_delivery.menu_delivery_zip_prefix", | ||
| ), | ||
| ] | ||
|
|
||
|
|
||
| def _delete_sql_constraints(env): | ||
| # Delete constraints to recreate it | ||
| openupgrade.delete_sql_constraint_safely( | ||
| env, "delivery", "delivery_carrier", "margin_not_under_100_percent" | ||
| ) | ||
|
|
||
|
|
||
| def _fill_stock_move_line_carrier_id(env): | ||
| """Field `carrier_id` on stock.move.line is now a stored field.""" | ||
| openupgrade.logged_query( | ||
| env.cr, | ||
| """ | ||
| UPDATE stock_move_line sml | ||
| SET carrier_id = sp.carrier_id | ||
| FROM stock_picking sp | ||
| WHERE sml.picking_id = sp.id; | ||
| """, | ||
| ) | ||
|
|
||
|
|
||
| @openupgrade.migrate() | ||
| def migrate(env, version): | ||
| openupgrade.rename_xmlids(env.cr, _xmlids_renames) | ||
| openupgrade.add_fields(env, _new_fields) | ||
| _delete_sql_constraints(env) | ||
| _fill_stock_move_line_carrier_id(env) | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,95 @@ | ||
| ---Models in module 'delivery'--- | ||
| model choose.delivery.package (moved to stock_delivery) [column['model_type']] | ||
| # DONE: in migration script of module stock_delivery | ||
|
Contributor
There was a problem hiding this comment. no migration script will be executed, as stock_delivery is new module. let's install here explicitly this module.
Member
There was a problem hiding this comment. Migration scripts are also executed in newly installed modules thanks to the patches in
Contributor
There was a problem hiding this comment. I didn't know, nice. But unneeded in this case. |
||
|
|
||
| ---Fields in module 'delivery'--- | ||
| delivery / delivery.carrier / fixed_margin (float) : NEW | ||
| delivery / delivery.carrier / invoice_policy (selection) : selection_keys is now '['estimated']' ('['estimated', 'real']') | ||
| # NOTHING TO DO: | ||
|
|
||
| delivery / sale.order / shipping_weight (float) : NEW hasdefault: compute | ||
| # NOTHING TO DO: new features | ||
|
|
||
| delivery / stock.move.line / carrier_id (many2one) : is now stored | ||
| # DONE: pre-migration: Add new column & set value for it | ||
|
|
||
| delivery / stock.move.line / carrier_name (char) : DEL | ||
| # NOTHING TO DO: | ||
|
|
||
| delivery / product.product / country_of_origin (many2one) : module is now 'stock_delivery' ('delivery') | ||
| delivery / product.product / hs_code (char) : module is now 'stock_delivery' ('delivery') | ||
| delivery / product.template / country_of_origin (many2one) : module is now 'stock_delivery' ('delivery') | ||
| delivery / product.template / hs_code (char) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.move / weight (float) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.move.line / carrier_id (many2one) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.move.line / destination_country_code (char): module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.move.line / sale_price (float) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.package.type / package_carrier_type (selection): module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.package.type / shipper_package_code (char) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / carrier_id (many2one) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / carrier_price (float) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / carrier_tracking_ref (char) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / carrier_tracking_url (char) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / delivery_type (selection) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / destination_country_code (char): module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / is_return_picking (boolean) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / package_ids (many2many) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / return_label_ids (one2many) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / shipping_weight (float) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / weight (float) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / weight_bulk (float) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.picking / weight_uom_name (char) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.quant.package / shipping_weight (float) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.quant.package / weight (float) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.quant.package / weight_is_kg (boolean) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.quant.package / weight_uom_name (char) : module is now 'stock_delivery' ('delivery') | ||
| delivery / stock.quant.package / weight_uom_rounding (float) : module is now 'stock_delivery' ('delivery') | ||
| # NOTHING TO DO: Handled by ORM | ||
|
|
||
| ---XML records in module 'delivery'--- | ||
| DEL ir.actions.act_window: delivery.act_delivery_trackers_url [renamed to stock_delivery module] | ||
| # DONE: pre-migration: moved to stock_delivery | ||
|
|
||
| NEW ir.model.access: delivery.access_delivery_price_rule_sale_manager | ||
| NEW ir.model.access: delivery.access_delivery_zip_prefix_sale_manager | ||
| DEL ir.model.access: delivery.access_choose_delivery_carrier_salesman | ||
| # NOTHING TO DO | ||
|
|
||
| DEL ir.model.access: delivery.access_choose_delivery_package [renamed to stock_delivery module] | ||
| DEL ir.model.access: delivery.access_delivery_carrier_stock_manager [renamed to stock_delivery module] | ||
| DEL ir.model.access: delivery.access_delivery_carrier_stock_user [renamed to stock_delivery module] | ||
| DEL ir.model.access: delivery.access_delivery_price_rule_stock_manager [renamed to stock_delivery module] | ||
| DEL ir.model.access: delivery.access_delivery_zip_prefix_stock_manager [renamed to stock_delivery module] | ||
| # DONE: pre-migration: moved to stock_delivery | ||
|
|
||
| ir.model.constraint: delivery.constraint_delivery_carrier_margin_not_under_100_percent (changed definition: is now 'check(margin >= -1)' ('check(margin >= -100)')) | ||
| # DONE pre-migration: safely delete constraint to recreate it | ||
|
|
||
| DEL ir.ui.menu: delivery.menu_action_delivery_carrier_form [renamed to stock_delivery module] | ||
| DEL ir.ui.menu: delivery.menu_delivery_zip_prefix [renamed to stock_delivery module] | ||
| # DONE: pre-migration: moved to stock_delivery | ||
|
|
||
| DEL ir.ui.view: delivery.choose_delivery_package_view_form | ||
| DEL ir.ui.view: delivery.delivery_stock_report_delivery_no_package_section_line | ||
| DEL ir.ui.view: delivery.delivery_tracking_url_warning_form | ||
| DEL ir.ui.view: delivery.label_package_template_view_delivery | ||
| DEL ir.ui.view: delivery.product_template_hs_code | ||
| DEL ir.ui.view: delivery.report_delivery_document2 | ||
| DEL ir.ui.view: delivery.report_package_barcode_delivery | ||
| DEL ir.ui.view: delivery.report_package_barcode_small_delivery | ||
| DEL ir.ui.view: delivery.report_shipping2 | ||
| DEL ir.ui.view: delivery.sale_order_portal_content_inherit_sale_stock_inherit_website_sale_delivery | ||
| DEL ir.ui.view: delivery.stock_move_line_view_search_delivery | ||
| DEL ir.ui.view: delivery.stock_package_type_form_delivery | ||
| DEL ir.ui.view: delivery.stock_package_type_tree_delivery | ||
| DEL ir.ui.view: delivery.stock_report_delivery_aggregated_move_lines_inherit_delivery | ||
| DEL ir.ui.view: delivery.stock_report_delivery_has_serial_move_line_inherit_delivery | ||
| DEL ir.ui.view: delivery.stock_report_delivery_package_section_line_inherit_delivery | ||
| DEL ir.ui.view: delivery.view_move_line_tree_detailed_delivery | ||
| DEL ir.ui.view: delivery.view_picking_type_form_delivery | ||
| DEL ir.ui.view: delivery.view_picking_withcarrier_out_form | ||
| DEL ir.ui.view: delivery.view_picking_withweight_internal_move_form | ||
| DEL ir.ui.view: delivery.view_quant_package_weight_form | ||
| DEL ir.ui.view: delivery.view_stock_rule_form_delivery | ||
| DEL ir.ui.view: delivery.vpicktree_view_tree | ||
| # NOTHING TO DO | ||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no need, as you will fill the carrier_id by sql. so, better to create column by sql in this case
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No. Why create it by SQL if we have an API for that?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At least, the field should be added in
stock_deliveryinstead :S I prefer let ORM do the work, to avoid messing anything.