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

[IMP] convert_field_to_html: be ablet to avoid spamming query for each row #304

Merged

Conversation

MiquelRForgeFlow
Copy link
Contributor

In v15, this method is used several times and some times in tables that usually have many rows.

@pedrobaeza
Copy link
Member

Uhm, the operation is not the same right now...

You can add a verbose=True keyword argument and don't log if verbose=False.

@MiquelRForgeFlow
Copy link
Contributor Author

Uhm, the operation is not the same right now...

What do you mean?

@pedrobaeza
Copy link
Member

You are not processing each content and pass plaintext2html over it.

@MiquelRForgeFlow
Copy link
Contributor Author

MiquelRForgeFlow commented Sep 12, 2022

ah, shit. right! 🤦‍♂️

@MiquelRForgeFlow MiquelRForgeFlow force-pushed the master-imp-convert_field_to_html branch from ea402a8 to 7cdcfa7 Compare September 12, 2022 10:27
@MiquelRForgeFlow MiquelRForgeFlow force-pushed the master-imp-convert_field_to_html branch from 7cdcfa7 to 24efb18 Compare September 12, 2022 10:35
@MiquelRForgeFlow MiquelRForgeFlow force-pushed the master-imp-convert_field_to_html branch 2 times, most recently from cfaff9a to 6eb5ca8 Compare September 12, 2022 12:00
@MiquelRForgeFlow MiquelRForgeFlow force-pushed the master-imp-convert_field_to_html branch from 6eb5ca8 to f936c98 Compare September 12, 2022 12:32
@MiquelRForgeFlow MiquelRForgeFlow changed the title [IMP] convert_field_to_html: avoid spamming query for each row [IMP] convert_field_to_html: be ablet to avoid spamming query for each row Sep 12, 2022
@MiquelRForgeFlow MiquelRForgeFlow merged commit f97bf37 into OCA:master Sep 12, 2022
@MiquelRForgeFlow MiquelRForgeFlow deleted the master-imp-convert_field_to_html branch September 12, 2022 13:02
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.

2 participants