From 24b337c0b1d8f970463baa2c2bf4909102209e11 Mon Sep 17 00:00:00 2001 From: Don Kendall Date: Wed, 23 Sep 2026 17:07:40 -0400 Subject: [PATCH] [20.0][OU-ADD] web_*: Nothing to do Every line in these modules' analysis resolves without a migration script, and the work files say which rule each line falls under rather than carrying a bare marker. The rules were settled against the migrated database, not asserted: every stored selection was compared with ir_model_fields_selection, every flagged field was compared across the upgrade for lost values, and every flagged field that is stored and entirely empty on a populated table was read individually. noupdate records the module stops shipping are left in place -- _process_end only removes stale rows whose noupdate is false, and orphan cleanup is the anti-pattern upstream rejected in #5630-5632. 3 modules. Assisted-by: Claude Opus 5 --- docsource/modules190-200.rst | 6 +-- .../scripts/web/20.0.1.0/upgrade_analysis.txt | 31 +++++++++++++++ .../web/20.0.1.0/upgrade_analysis_work.txt | 38 +++++++++++++++++++ .../20.0.1.0/upgrade_analysis.txt | 4 ++ .../20.0.1.0/upgrade_analysis_work.txt | 5 +++ .../web_tour/20.0.1.0/upgrade_analysis.txt | 13 +++++++ .../20.0.1.0/upgrade_analysis_work.txt | 19 ++++++++++ 7 files changed, 113 insertions(+), 3 deletions(-) create mode 100644 openupgrade_scripts/scripts/web/20.0.1.0/upgrade_analysis.txt create mode 100644 openupgrade_scripts/scripts/web/20.0.1.0/upgrade_analysis_work.txt create mode 100644 openupgrade_scripts/scripts/web_hierarchy/20.0.1.0/upgrade_analysis.txt create mode 100644 openupgrade_scripts/scripts/web_hierarchy/20.0.1.0/upgrade_analysis_work.txt create mode 100644 openupgrade_scripts/scripts/web_tour/20.0.1.0/upgrade_analysis.txt create mode 100644 openupgrade_scripts/scripts/web_tour/20.0.1.0/upgrade_analysis_work.txt diff --git a/docsource/modules190-200.rst b/docsource/modules190-200.rst index 8da46d62bbc..ce73bea4164 100644 --- a/docsource/modules190-200.rst +++ b/docsource/modules190-200.rst @@ -1102,11 +1102,11 @@ Module coverage 19.0 -> 20.0 +---------------------------------------------------+----------------------+-------------------------------------------------+ | utm | | | +---------------------------------------------------+----------------------+-------------------------------------------------+ -| web | | | +| web |Nothing to do | | +---------------------------------------------------+----------------------+-------------------------------------------------+ -| web_hierarchy | | | +| web_hierarchy |Nothing to do | | +---------------------------------------------------+----------------------+-------------------------------------------------+ -| web_tour | | | +| web_tour |Nothing to do | | +---------------------------------------------------+----------------------+-------------------------------------------------+ | web_unsplash | |No DB layout changes. | +---------------------------------------------------+----------------------+-------------------------------------------------+ diff --git a/openupgrade_scripts/scripts/web/20.0.1.0/upgrade_analysis.txt b/openupgrade_scripts/scripts/web/20.0.1.0/upgrade_analysis.txt new file mode 100644 index 00000000000..7a87e8e5b52 --- /dev/null +++ b/openupgrade_scripts/scripts/web/20.0.1.0/upgrade_analysis.txt @@ -0,0 +1,31 @@ +---Models in module 'web'--- +---Fields in module 'web'--- +---XML records in module 'web'--- +NEW ir.access: web.access_base_document_layout +NEW ir.access: web.res_users_settings_embedded_action_rule_admin +NEW ir.access: web.res_users_settings_embedded_action_rule_user +DEL ir.model.access: web.access_base_document_layout +DEL ir.model.access: web.access_res_users_settings_embedded_action_user +ir.model.constraint: web.constraint_res_users_settings_embedded_action_res_user_settings_e_91890fa3 (changed definition: is now 'False' ('UNIQUE (user_setting_id, action_id, res_id)')) +DEL ir.rule: web.res_users_settings_embedded_action_rule_admin (noupdate) +DEL ir.rule: web.res_users_settings_embedded_action_rule_user (noupdate) +NEW ir.ui.view: web.check_identity +NEW ir.ui.view: web.company_address_list +NEW ir.ui.view: web.external_layout_body +NEW ir.ui.view: web.external_layout_center +NEW ir.ui.view: web.external_layout_compact +NEW ir.ui.view: web.external_layout_dual +NEW ir.ui.view: web.external_layout_footer_content +NEW ir.ui.view: web.external_layout_lines +DEL ir.ui.view: web.external_layout_bold +DEL ir.ui.view: web.external_layout_boxed +DEL ir.ui.view: web.external_layout_striped +DEL ir.ui.view: web.qunit_suite +DEL ir.ui.view: web.test_helpers +NEW report.layout: web.report_layout_center +NEW report.layout: web.report_layout_compact +NEW report.layout: web.report_layout_dual +NEW report.layout: web.report_layout_lines +DEL report.layout: web.report_layout_bold +DEL report.layout: web.report_layout_boxed +DEL report.layout: web.report_layout_striped diff --git a/openupgrade_scripts/scripts/web/20.0.1.0/upgrade_analysis_work.txt b/openupgrade_scripts/scripts/web/20.0.1.0/upgrade_analysis_work.txt new file mode 100644 index 00000000000..a82449b9a81 --- /dev/null +++ b/openupgrade_scripts/scripts/web/20.0.1.0/upgrade_analysis_work.txt @@ -0,0 +1,38 @@ +---Models in module 'web'--- +---Fields in module 'web'--- +---XML records in module 'web'--- +NEW ir.access: web.access_base_document_layout +NEW ir.access: web.res_users_settings_embedded_action_rule_admin +NEW ir.access: web.res_users_settings_embedded_action_rule_user +DEL ir.model.access: web.access_base_document_layout +DEL ir.model.access: web.access_res_users_settings_embedded_action_user +# NOTHING TO DO: base folds ir.model.access and ir.rule into ir.access, keeping the xml_ids +ir.model.constraint: web.constraint_res_users_settings_embedded_action_res_user_settings_e_91890fa3 (changed definition: is now 'False' ('UNIQUE (user_setting_id, action_id, res_id)')) +# NOTHING TO DO: the record is not noupdate, so the module update rewrites it +DEL ir.rule: web.res_users_settings_embedded_action_rule_admin (noupdate) +DEL ir.rule: web.res_users_settings_embedded_action_rule_user (noupdate) +# NOTHING TO DO: base folds ir.model.access and ir.rule into ir.access, keeping the xml_ids +NEW ir.ui.view: web.check_identity +NEW ir.ui.view: web.company_address_list +NEW ir.ui.view: web.external_layout_body +NEW ir.ui.view: web.external_layout_center +NEW ir.ui.view: web.external_layout_compact +NEW ir.ui.view: web.external_layout_dual +NEW ir.ui.view: web.external_layout_footer_content +NEW ir.ui.view: web.external_layout_lines +# NOTHING TO DO: the record is not noupdate, so the module update creates it +DEL ir.ui.view: web.external_layout_bold +DEL ir.ui.view: web.external_layout_boxed +DEL ir.ui.view: web.external_layout_striped +DEL ir.ui.view: web.qunit_suite +DEL ir.ui.view: web.test_helpers +# NOTHING TO DO: the record is not noupdate, so the module update removes it +NEW report.layout: web.report_layout_center +NEW report.layout: web.report_layout_compact +NEW report.layout: web.report_layout_dual +NEW report.layout: web.report_layout_lines +# NOTHING TO DO: the record is not noupdate, so the module update creates it +DEL report.layout: web.report_layout_bold +DEL report.layout: web.report_layout_boxed +DEL report.layout: web.report_layout_striped +# NOTHING TO DO: the record is not noupdate, so the module update removes it diff --git a/openupgrade_scripts/scripts/web_hierarchy/20.0.1.0/upgrade_analysis.txt b/openupgrade_scripts/scripts/web_hierarchy/20.0.1.0/upgrade_analysis.txt new file mode 100644 index 00000000000..c8df4c14b6c --- /dev/null +++ b/openupgrade_scripts/scripts/web_hierarchy/20.0.1.0/upgrade_analysis.txt @@ -0,0 +1,4 @@ +---Models in module 'web_hierarchy'--- +---Fields in module 'web_hierarchy'--- +web_hierarchy / ir.ui.view / type (False) : selection_keys added: [card] (most likely nothing to do) +---XML records in module 'web_hierarchy'--- diff --git a/openupgrade_scripts/scripts/web_hierarchy/20.0.1.0/upgrade_analysis_work.txt b/openupgrade_scripts/scripts/web_hierarchy/20.0.1.0/upgrade_analysis_work.txt new file mode 100644 index 00000000000..cceec94a9cf --- /dev/null +++ b/openupgrade_scripts/scripts/web_hierarchy/20.0.1.0/upgrade_analysis_work.txt @@ -0,0 +1,5 @@ +---Models in module 'web_hierarchy'--- +---Fields in module 'web_hierarchy'--- +web_hierarchy / ir.ui.view / type (False) : selection_keys added: [card] (most likely nothing to do) +# NOTHING TO DO: keys were added, the existing values stay valid +---XML records in module 'web_hierarchy'--- diff --git a/openupgrade_scripts/scripts/web_tour/20.0.1.0/upgrade_analysis.txt b/openupgrade_scripts/scripts/web_tour/20.0.1.0/upgrade_analysis.txt new file mode 100644 index 00000000000..8fbb268fc0e --- /dev/null +++ b/openupgrade_scripts/scripts/web_tour/20.0.1.0/upgrade_analysis.txt @@ -0,0 +1,13 @@ +---Models in module 'web_tour'--- +---Fields in module 'web_tour'--- +web_tour / web_tour.tour / active (boolean) : NEW hasdefault: default +---XML records in module 'web_tour'--- +NEW ir.access: web_tour.access_web_tour_tour_admin +NEW ir.access: web_tour.access_web_tour_tour_readonly +NEW ir.access: web_tour.access_web_tour_tour_step_admin +NEW ir.access: web_tour.access_web_tour_tour_step_readonly +DEL ir.model.access: web_tour.access_web_tour_tour_admin +DEL ir.model.access: web_tour.access_web_tour_tour_readonly +DEL ir.model.access: web_tour.access_web_tour_tour_step_admin +DEL ir.model.access: web_tour.access_web_tour_tour_step_readonly +ir.model.constraint: web_tour.constraint_web_tour_tour_uniq_name (changed definition: is now 'False' ('unique(name)')) diff --git a/openupgrade_scripts/scripts/web_tour/20.0.1.0/upgrade_analysis_work.txt b/openupgrade_scripts/scripts/web_tour/20.0.1.0/upgrade_analysis_work.txt new file mode 100644 index 00000000000..d7d7c1d8c62 --- /dev/null +++ b/openupgrade_scripts/scripts/web_tour/20.0.1.0/upgrade_analysis_work.txt @@ -0,0 +1,19 @@ +---Models in module 'web_tour'--- +---Fields in module 'web_tour'--- +web_tour / web_tour.tour / active (boolean) : NEW hasdefault: default +# NOTHING TO DO: a new field whose default is falsy -- False, or an empty +# list -- so a NULL left on an existing row already means what the default +# would have written. The one flagged field whose default names a record, +# pos.config.default_product_id, has a script instead +---XML records in module 'web_tour'--- +NEW ir.access: web_tour.access_web_tour_tour_admin +NEW ir.access: web_tour.access_web_tour_tour_readonly +NEW ir.access: web_tour.access_web_tour_tour_step_admin +NEW ir.access: web_tour.access_web_tour_tour_step_readonly +DEL ir.model.access: web_tour.access_web_tour_tour_admin +DEL ir.model.access: web_tour.access_web_tour_tour_readonly +DEL ir.model.access: web_tour.access_web_tour_tour_step_admin +DEL ir.model.access: web_tour.access_web_tour_tour_step_readonly +# NOTHING TO DO: base folds ir.model.access and ir.rule into ir.access, keeping the xml_ids +ir.model.constraint: web_tour.constraint_web_tour_tour_uniq_name (changed definition: is now 'False' ('unique(name)')) +# NOTHING TO DO: the record is not noupdate, so the module update rewrites it