Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CPT-127 Make list of attachments accessible again #22

Open
wants to merge 1 commit into
base: jobrad-14.0
Choose a base branch
from

Conversation

jhelfferich
Copy link

When opening the list of attachments, Odoo wants to display the total number of items in the list. This is displayed in the top right such as "1 - 80 of 2264".

In order to display the total number, Odoo fetches all IDs and filters out the ids to which the user does not have access. For the JobRad database with more than 32 million entries this causes either a MemoryError or a timeout.

The suggested solution is to skip the filtering. This means the total number of entries will be wrong. The list can still be navigated without any issues.

When opening the list of attachments, Odoo wants to display the total
number of items in the list. This is displayed in the top right such as
"1 - 80 of 2264".

In order to display the total number, Odoo fetches all IDs and filters
out the ids to which the user does not have access. For the JobRad
database with more than 32 million entries this causes either a
MemoryError or a timeout.

The suggested solution is to skip the filtering. This means the total
number of entries will be wrong. The list can still be navigated without
any issues.
@borna-ng borna-ng force-pushed the jobrad-14.0 branch 5 times, most recently from 1c33238 to 9dc00d6 Compare October 16, 2024 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant