Skip to content

[17.0][IMP] base_import_pdf_by_template: Prevent time_format if not set - #1374

Merged
OCA-git-bot merged 2 commits into
OCA:17.0from
Tecnativa:17.0-fix-base_import_pdf_by_template-TT63873
Jul 28, 2026
Merged

[17.0][IMP] base_import_pdf_by_template: Prevent time_format if not set#1374
OCA-git-bot merged 2 commits into
OCA:17.0from
Tecnativa:17.0-fix-base_import_pdf_by_template-TT63873

Conversation

@victoralmau

Copy link
Copy Markdown
Member

FWP from 15.0: #1156

Prevent time_format if not set

time_format = self.time_format.replace("*", "%")
AttributeError: 'bool' object has no attribute 'replace'

Please @pedrobaeza can ou review it?

@Tecnativa TT56270

time_format = self.time_format.replace("*", "%")
AttributeError: 'bool' object has no attribute 'replace'

TT56270
@OCA-git-bot OCA-git-bot added series:17.0 mod:base_import_pdf_by_template Module base_import_pdf_by_template labels Jul 28, 2026
****po-python-parse-printf****
base_business_document_import/i18n/de.po:61 Translation string couldn't be parsed correctly using str%variables ValueError("unsupported format character 'w' (0x77) at index 48") - [po-python-parse-printf]
@OCA-git-bot OCA-git-bot added the mod:base_business_document_import Module base_business_document_import label Jul 28, 2026
@pedrobaeza pedrobaeza added this to the 17.0 milestone Jul 28, 2026

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fast-tracking as both bugfixes are clear:

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-1374-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit e116308 into OCA:17.0 Jul 28, 2026
6 of 7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@pedrobaeza
pedrobaeza deleted the 17.0-fix-base_import_pdf_by_template-TT63873 branch July 28, 2026 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged 🎉 mod:base_business_document_import Module base_business_document_import mod:base_import_pdf_by_template Module base_import_pdf_by_template series:17.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants