Skip to content

[16.0][FWP][ADD] shopfloor_reception_refund_return#1060

Merged
OCA-git-bot merged 5 commits into
OCA:16.0from
mt-software-de:16-add-shopfloor_reception_refund_return
Jun 24, 2025
Merged

[16.0][FWP][ADD] shopfloor_reception_refund_return#1060
OCA-git-bot merged 5 commits into
OCA:16.0from
mt-software-de:16-add-shopfloor_reception_refund_return

Conversation

@mt-software-de

@mt-software-de mt-software-de commented Jun 22, 2025

Copy link
Copy Markdown

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @sebalix, @guewen, @mmequignon, @JuMiSanAr, @simahawk,
some modules you are maintaining are being modified, check this out!

@jbaudoux jbaudoux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to change shopfloor_reception version in manifest and put that version for the pre-migration

@mt-software-de
mt-software-de force-pushed the 16-add-shopfloor_reception_refund_return branch from 9a659ca to c6ac389 Compare June 23, 2025 07:09
@mt-software-de

Copy link
Copy Markdown
Author

You need to change shopfloor_reception version in manifest and put that version for the pre-migration

yes of course. thx

@jbaudoux

Copy link
Copy Markdown
Contributor

@mt-software-de pre-commit is red

@mt-software-de
mt-software-de force-pushed the 16-add-shopfloor_reception_refund_return branch from c6ac389 to bd4c1f6 Compare June 23, 2025 08:57
@mt-software-de

Copy link
Copy Markdown
Author

@mt-software-de pre-commit is red

done

@jbaudoux

Copy link
Copy Markdown
Contributor

@mt-software-de you need to split one module per commit

@mt-software-de
mt-software-de force-pushed the 16-add-shopfloor_reception_refund_return branch from bd4c1f6 to a007507 Compare June 23, 2025 11:56
@mt-software-de

Copy link
Copy Markdown
Author

@mt-software-de you need to split one module per commit

done

@jbaudoux

Copy link
Copy Markdown
Contributor

@mt-software-de you need to split one module per commit

done

@mt-software-de it is not yet right, one commit is still on 2 modules

@mt-software-de

mt-software-de commented Jun 24, 2025

Copy link
Copy Markdown
Author

@mt-software-de you need to split one module per commit

done

@mt-software-de it is not yet right, one commit is still on 2 modules

That are the forward ported ones from #870
I could fix them via #1059 but therefore #870 needs to be closed.

@jbaudoux

Copy link
Copy Markdown
Contributor

@mt-software-de it is not yet right, one commit is still on 2 modules

That are the forward ported ones from #870 I could fix them via #1059 but therefore #870 needs to be closed.

yes let's do that otherwise it makes it complicate to include in v18 migration

@mt-software-de

Copy link
Copy Markdown
Author

@mt-software-de it is not yet right, one commit is still on 2 modules

That are the forward ported ones from #870 I could fix them via #1059 but therefore #870 needs to be closed.

yes let's do that otherwise it makes it complicate to include in v18 migration

OK everything is ready.
#870 can be closed
and is replaced #1059

@jbaudoux

Copy link
Copy Markdown
Contributor

yes let's do that otherwise it makes it complicate to include in v18 migration

OK everything is ready. #870 can be closed and is replaced #1059

the pre migration script is there twice and you need also to increase the version of shopfloor as it's all in the same PR

@mt-software-de
mt-software-de force-pushed the 16-add-shopfloor_reception_refund_return branch from f95d541 to 4f9986d Compare June 24, 2025 14:14
@mt-software-de

Copy link
Copy Markdown
Author

@mt-software-de it is not yet right, one commit is still on 2 modules

That are the forward ported ones from #870 I could fix them via #1059 but therefore #870 needs to be closed.

yes let's do that otherwise it makes it complicate to include in v18 migration

OK everything is ready. #870 can be closed and is replaced #1059

Yes of course. Thx. Done.

@jbaudoux

Copy link
Copy Markdown
Contributor

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-1060-by-jbaudoux-bump-nobump, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit fc62e3f into OCA:16.0 Jun 24, 2025
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at e26cf83. Thanks a lot for contributing to OCA. ❤️

@giarve

giarve commented Jun 24, 2025

Copy link
Copy Markdown

Hi.

Got this running module upgrade:

2025-06-24 17:46:34,193 1 ERROR ? click_odoo.env_options: exception
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/click_odoo/env_options.py", line 194, in _invoke
    with self.environment_manager(
  File "/usr/local/lib/python3.10/contextlib.py", line 135, in __enter__
    return next(self.gen)
  File "/usr/local/lib/python3.10/site-packages/click_odoo_contrib/update.py", line 299, in OdooEnvironmentWithUpdate
    _update_db(
  File "/usr/local/lib/python3.10/site-packages/click_odoo_contrib/update.py", line 270, in _update_db
    _update_db_nolock(
  File "/usr/local/lib/python3.10/site-packages/click_odoo_contrib/update.py", line 248, in _update_db_nolock
    odoo.modules.registry.Registry.new(database, update_module=True)
  File "<decorator-gen-16>", line 2, in new
  File "/opt/odoo/custom/src/odoo/odoo/tools/func.py", line 87, in locked
    return func(inst, *args, **kwargs)
  File "/opt/odoo/custom/src/odoo/odoo/modules/registry.py", line 87, in new
    odoo.modules.load_modules(registry, force_demo, status, update_module)
  File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 489, in load_modules
    processed_modules += load_marked_modules(cr, graph,
  File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 374, in load_marked_modules
    loaded, processed = load_module_graph(
  File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 185, in load_module_graph
    migrations.migrate_module(package, 'pre')
  File "/opt/odoo/custom/src/odoo/odoo/modules/migration.py", line 170, in migrate_module
    exec_script(self.cr, installed_version, pyfile, pkg.name, stage, stageformat[stage] % version)
  File "/opt/odoo/custom/src/odoo/odoo/modules/migration.py", line 188, in exec_script
    migrate(cr, installed_version)
  File "/opt/odoo/auto/addons/shopfloor_reception/migrations/16.0.1.2.0/pre-migration.py", line 17, in migrate
    move_fields_to_new_module(cr)
  File "/opt/odoo/auto/addons/shopfloor_reception/migrations/16.0.1.2.0/pre-migration.py", line 11, in move_fields_to_new_module
    openupgrade.update_module_moved_fields(
  File "/usr/local/lib/python3.10/site-packages/openupgradelib/openupgrade.py", line 3285, in update_module_moved_fields
    do_raise("moved_fields %s must be a tuple or list!" % moved_fields)
  File "/usr/local/lib/python3.10/site-packages/openupgradelib/openupgrade.py", line 129, in do_raise
    raise UserError(error)
odoo.exceptions.UserError: moved_fields is_shopfloor_created must be a tuple or list!
Error: moved_fields is_shopfloor_created must be a tuple or list!

@mt-software-de

Copy link
Copy Markdown
Author

Hi.

Got this running module upgrade:

2025-06-24 17:46:34,193 1 ERROR ? click_odoo.env_options: exception
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/click_odoo/env_options.py", line 194, in _invoke
    with self.environment_manager(
  File "/usr/local/lib/python3.10/contextlib.py", line 135, in __enter__
    return next(self.gen)
  File "/usr/local/lib/python3.10/site-packages/click_odoo_contrib/update.py", line 299, in OdooEnvironmentWithUpdate
    _update_db(
  File "/usr/local/lib/python3.10/site-packages/click_odoo_contrib/update.py", line 270, in _update_db
    _update_db_nolock(
  File "/usr/local/lib/python3.10/site-packages/click_odoo_contrib/update.py", line 248, in _update_db_nolock
    odoo.modules.registry.Registry.new(database, update_module=True)
  File "<decorator-gen-16>", line 2, in new
  File "/opt/odoo/custom/src/odoo/odoo/tools/func.py", line 87, in locked
    return func(inst, *args, **kwargs)
  File "/opt/odoo/custom/src/odoo/odoo/modules/registry.py", line 87, in new
    odoo.modules.load_modules(registry, force_demo, status, update_module)
  File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 489, in load_modules
    processed_modules += load_marked_modules(cr, graph,
  File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 374, in load_marked_modules
    loaded, processed = load_module_graph(
  File "/opt/odoo/custom/src/odoo/odoo/modules/loading.py", line 185, in load_module_graph
    migrations.migrate_module(package, 'pre')
  File "/opt/odoo/custom/src/odoo/odoo/modules/migration.py", line 170, in migrate_module
    exec_script(self.cr, installed_version, pyfile, pkg.name, stage, stageformat[stage] % version)
  File "/opt/odoo/custom/src/odoo/odoo/modules/migration.py", line 188, in exec_script
    migrate(cr, installed_version)
  File "/opt/odoo/auto/addons/shopfloor_reception/migrations/16.0.1.2.0/pre-migration.py", line 17, in migrate
    move_fields_to_new_module(cr)
  File "/opt/odoo/auto/addons/shopfloor_reception/migrations/16.0.1.2.0/pre-migration.py", line 11, in move_fields_to_new_module
    openupgrade.update_module_moved_fields(
  File "/usr/local/lib/python3.10/site-packages/openupgradelib/openupgrade.py", line 3285, in update_module_moved_fields
    do_raise("moved_fields %s must be a tuple or list!" % moved_fields)
  File "/usr/local/lib/python3.10/site-packages/openupgradelib/openupgrade.py", line 129, in do_raise
    raise UserError(error)
odoo.exceptions.UserError: moved_fields is_shopfloor_created must be a tuple or list!
Error: moved_fields is_shopfloor_created must be a tuple or list!

I will prepare a fix for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants