Syncing from upstream OCA/bank-statement-import (19.0)#748
Merged
Conversation
[FIX] Mostly currency-related fixes [IMP] sample_statement_en test: add dates on description Co-authored-by: Marco Colombo <marco.colombo@gmail.com> [UPD] Update account_statement_import_txt_xlsx.pot
Translated using Weblate (Italian) Currently translated at 64.3% (56 of 87 strings) Translation: bank-statement-import-14.0/bank-statement-import-14.0-account_statement_import_txt_xlsx Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-14-0/bank-statement-import-14-0-account_statement_import_txt_xlsx/it/ Translated using Weblate (Italian) Currently translated at 73.5% (64 of 87 strings) Translation: bank-statement-import-14.0/bank-statement-import-14.0-account_statement_import_txt_xlsx Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-14-0/bank-statement-import-14-0-account_statement_import_txt_xlsx/it/ Translated using Weblate (Italian) Currently translated at 98.8% (86 of 87 strings) Translation: bank-statement-import-14.0/bank-statement-import-14.0-account_statement_import_txt_xlsx Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-14-0/bank-statement-import-14-0-account_statement_import_txt_xlsx/it/
Currently translated at 100.0% (87 of 87 strings) Translation: bank-statement-import-14.0/bank-statement-import-14.0-account_statement_import_txt_xlsx Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-14-0/bank-statement-import-14-0-account_statement_import_txt_xlsx/es_AR/ account_statement_import_txt_xlsx 14.0.1.0.1
[UPD] Update account_statement_import_txt_xlsx.pot [UPD] README.rst
Currently translated at 100.0% (87 of 87 strings) Translation: bank-statement-import-15.0/bank-statement-import-15.0-account_statement_import_txt_xlsx Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-15-0/bank-statement-import-15-0-account_statement_import_txt_xlsx/it/
…r IN and OUT [IMP] account_statement_import_txt_xlsx: wizard: amount2_reverse boolean field, with tests
… to column numbers, and concatenation A new field 'File does not contain header line' is added in the Statement Sheet Mappings. If you set to True, then you can map the columns by indicating in each field of the 'Columns' section the column number in the file. We also allow to concatenate multiple columns in the file to a single column to a single field of the statement line. You have to indicate the names of the columns separated by comma.
…ot provided in the import file [UPD] Update account_statement_import_txt_xlsx.pot account_statement_import_txt_xlsx 15.0.2.0.0 Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: bank-statement-import-15.0/bank-statement-import-15.0-account_statement_import_txt_xlsx Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-15-0/bank-statement-import-15-0-account_statement_import_txt_xlsx/
…ping account_statement_import_txt_xlsx 15.0.2.0.1
If there is a problem parsing the file we are not showing the error to the user, instead a generic message telling something is wrong with the type of file is shown, and we only showing the error if we are running unit tests. In this case we think is good to show the error to the user, this way they can check what is wrong and fix it in the sheet mapping. * We are only showing the title of the error as a HINT, we think that is a good start to undestand what needs to change in the sheet mapping. * The complete traceback error is shown in the log if we needed for more detail review from the technical teams.
Currently translated at 74.4% (73 of 98 strings) Translation: bank-statement-import-15.0/bank-statement-import-15.0-account_statement_import_txt_xlsx Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-15-0/bank-statement-import-15-0-account_statement_import_txt_xlsx/it/
Currently translated at 98.9% (97 of 98 strings) Translation: bank-statement-import-15.0/bank-statement-import-15.0-account_statement_import_txt_xlsx Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-15-0/bank-statement-import-15-0-account_statement_import_txt_xlsx/es/
- Depending on the installed set of modules, the company currency may be USD or EUR. If the second case, these tests will fail, so we make sure that the company currency is USD for our tests, doing the change by SQL, as there's a Python constraint that prevents it. Not needed in v17 due to: odoo/odoo#107113. account_statement_import_txt_xlsx 15.0.2.0.2 [UPD] Update account_statement_import_txt_xlsx.pot account_statement_import_txt_xlsx 15.0.2.0.3 [UPD] README.rst Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: bank-statement-import-15.0/bank-statement-import-15.0-account_statement_import_txt_xlsx Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-15-0/bank-statement-import-15-0-account_statement_import_txt_xlsx/
Co-authored-by: zaoral
[IMP] add csv meta data management [FIX] exclude footer meta data [IMP] import separated credit/debit column file [FIX] make comptatible with new version of multi_step_wizard module & add migration file [FIX] all not provided value are handled in_parse_decimal method [REF] Remove unnecessary \n Renamed to account_statement_import_sheet_file Co-authored-by: zaoral
Currently translated at 100.0% (95 of 95 strings) Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/es/
…ign_column. In some cases such as in credit card statements the amounts are expressed in the inverse sign. By setting this flag during the upload the amounts will be inverted in sign.
Currently translated at 95.9% (95 of 99 strings) Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/tr/
Currently translated at 100.0% (99 of 99 strings) Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/es/
Currently translated at 58.5% (58 of 99 strings) Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/it/
Currently translated at 98.9% (98 of 99 strings) Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/
Currently translated at 100.0% (100 of 100 strings) Translation: bank-statement-import-16.0/bank-statement-import-16.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-16-0/bank-statement-import-16-0-account_statement_import_sheet_file/it/
Currently translated at 100.0% (99 of 99 strings) Translation: bank-statement-import-17.0/bank-statement-import-17.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-17-0/bank-statement-import-17-0-account_statement_import_sheet_file/it/
Currently translated at 100.0% (99 of 99 strings) Translation: bank-statement-import-17.0/bank-statement-import-17.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-17-0/bank-statement-import-17-0-account_statement_import_sheet_file/zh_CN/
Currently translated at 100.0% (104 of 104 strings) Translation: bank-statement-import-17.0/bank-statement-import-17.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-17-0/bank-statement-import-17-0-account_statement_import_sheet_file/it/
There can be files where the date is only expressed with day and month, but no year. You put in that case a timestamp expression like '%d/%m'. For that, strptime parses the content and return the day and month in the year 1900. This code checks that condition and replace the year by the current one, as clearly 1900 is not a year to post transactions. There's also logic for putting previous year if the current month is lower than the imported one (i.e., current month is January and you are importing December).
…parser header When importing an Excel file with a header and the field header_lines_skip_count is set to its default value (0), the header is retrieved using the -1 index (the last row). complementary to commit 13285ab
…tune the UI Before this commit, the parse_header function was unused. It is now utilized to reduce redundant code and improve maintainability. Additionally, UI adjustments have been made for better usability and clarity.
- Reuse duplicated code: * Journal creation * Wizard creation * etc - Remove obsolete code. - Increase readability of the tests. - Use setUpClass instead of setUp.
Currently translated at 97.1% (101 of 104 strings) Translation: bank-statement-import-18.0/bank-statement-import-18.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-18-0/bank-statement-import-18-0-account_statement_import_sheet_file/tr/
Currently translated at 100.0% (104 of 104 strings) Translation: bank-statement-import-18.0/bank-statement-import-18.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-18-0/bank-statement-import-18-0-account_statement_import_sheet_file/de/
Currently translated at 100.0% (104 of 104 strings) Translation: bank-statement-import-18.0/bank-statement-import-18.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-18-0/bank-statement-import-18-0-account_statement_import_sheet_file/de/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: bank-statement-import-18.0/bank-statement-import-18.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-18-0/bank-statement-import-18-0-account_statement_import_sheet_file/
Currently translated at 100.0% (105 of 105 strings) Translation: bank-statement-import-18.0/bank-statement-import-18.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-18-0/bank-statement-import-18-0-account_statement_import_sheet_file/it/
Currently translated at 96.1% (101 of 105 strings) Translation: bank-statement-import-18.0/bank-statement-import-18.0-account_statement_import_sheet_file Translate-URL: https://translation.odoo-community.org/projects/bank-statement-import-18-0/bank-statement-import-18-0-account_statement_import_sheet_file/tr/
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## 19.0 #748 +/- ##
==========================================
+ Coverage 85.74% 87.72% +1.97%
==========================================
Files 50 59 +9
Lines 1508 2134 +626
Branches 188 269 +81
==========================================
+ Hits 1293 1872 +579
- Misses 145 177 +32
- Partials 70 85 +15 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
bt_gitbot