Skip to content

[IMP] stock_storage_type: Don't retrieve too much values#1166

Open
rousseldenis wants to merge 1 commit into
OCA:16.0from
acsone:16.0-imp-stock-storage-type-perf-dro
Open

[IMP] stock_storage_type: Don't retrieve too much values#1166
rousseldenis wants to merge 1 commit into
OCA:16.0from
acsone:16.0-imp-stock-storage-type-perf-dro

Conversation

@rousseldenis

Copy link
Copy Markdown

Don't retrieve unneeded values for locations that should not compute 'location_will_contain_product_ids'.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @jbaudoux,
some modules you are maintaining are being modified, check this out!

@OCA-git-bot OCA-git-bot added series:16.0 mod:stock_storage_type Module stock_storage_type labels May 15, 2026
@rousseldenis rousseldenis added enhancement New feature or request needs review and removed series:16.0 mod:stock_storage_type Module stock_storage_type labels May 15, 2026
# computed (in 'else'), field value for records of 'self' will be retrieved.
# Like 'quant_ids' for locations like 'Customers'
prefetch_ids = [
rec.id for rec in self if rec._should_compute_will_contain_product_ids()

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.

Do not call it twice

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Do not call it twice

Fixed

@lmignon lmignon force-pushed the 16.0-imp-stock-storage-type-perf-dro branch from 412802a to cdb3244 Compare June 9, 2026 12:27
@OCA-git-bot OCA-git-bot added series:16.0 mod:stock_storage_type Module stock_storage_type labels Jun 9, 2026
Don't retrieve unneeded values for locations that should not compute
'location_will_contain_product_ids'.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request mod:stock_storage_type Module stock_storage_type needs review series:16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants