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

Correction of the accounting number for foreign customers / suppliers #29387

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

EpixFr
Copy link
Contributor

@EpixFr EpixFr commented Apr 18, 2024

Instructions

These changes correct accounting numbers for exporting customer and suppliers invoices.
We take the accounting number of the invoice line and not the product.

Correction numéro de compte comptable pour les lignes de factures liées aux clients hors pays d'origine (UE, Export)
Fixed accounting account number for supplier invoice lines outside the original country (EU, export) // Correction numéro de compte comptable pour les lignes de factures fournisseurs hors pays d'origine (UE, Export)
@@ -542,8 +543,10 @@ public function __construct($db)
'fd.rowid'=>'LineId', 'fd.description'=>"LineDescription",
'fd.subprice'=>"LineUnitPrice", 'fd.tva_tx'=>"LineVATRate", 'fd.qty'=>"LineQty", 'fd.total_ht'=>"LineTotalHT", 'fd.total_tva'=>"LineTotalVAT",
'fd.total_ttc'=>"LineTotalTTC", 'fd.buy_price_ht'=>'BuyingPrice', 'fd.date_start'=>"DateStart", 'fd.date_end'=>"DateEnd", 'fd.special_code'=>'SpecialCode',
'fd.product_type'=>"TypeOfLineServiceOrProduct", 'fd.fk_product'=>'ProductId', 'p.ref'=>'ProductRef', 'p.label'=>'ProductLabel',
$alias_product_perentity . '.accountancy_code_sell'=>'ProductAccountancySellCode',
Copy link
Member

@eldy eldy Apr 18, 2024

Choose a reason for hiding this comment

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

Why removing this information from export ?
Some use cases need it.

@eldy eldy added the Discussion Some questions or discussions are opened and wait answers of author or other people to be processed label Apr 18, 2024
@eldy eldy added the PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do) label May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Some questions or discussions are opened and wait answers of author or other people to be processed PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants