{"action":"created","issue":{"url":"https://api.github.com/repos/odoo/odoo/issues/19886","repository_url":"https://api.github.com/repos/odoo/odoo","labels_url":"https://api.github.com/repos/odoo/odoo/issues/19886/labels{/name}","comments_url":"https://api.github.com/repos/odoo/odoo/issues/19886/comments","events_url":"https://api.github.com/repos/odoo/odoo/issues/19886/events","html_url":"https://github.com/odoo/odoo/issues/19886","id":262826033,"number":19886,"title":"[10.0] Payment acquirer automatic payment reconciliation does not support multiple currencies","user":{"login":"yostashiro","id":7766939,"avatar_url":"https://avatars0.githubusercontent.com/u/7766939?v=4","gravatar_id":"","url":"https://api.github.com/users/yostashiro","html_url":"https://github.com/yostashiro","followers_url":"https://api.github.com/users/yostashiro/followers","following_url":"https://api.github.com/users/yostashiro/following{/other_user}","gists_url":"https://api.github.com/users/yostashiro/gists{/gist_id}","starred_url":"https://api.github.com/users/yostashiro/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yostashiro/subscriptions","organizations_url":"https://api.github.com/users/yostashiro/orgs","repos_url":"https://api.github.com/users/yostashiro/repos","events_url":"https://api.github.com/users/yostashiro/events{/privacy}","received_events_url":"https://api.github.com/users/yostashiro/received_events","type":"User","site_admin":false},"labels":[{"id":620800387,"url":"https://api.github.com/repos/odoo/odoo/labels/Accounting","name":"Accounting","color":"ac0000","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-10-04T15:15:19Z","updated_at":"2017-10-05T08:28:29Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"### Impacted versions:\r\n\r\n10.0 (haven't checked other versions)\r\n\r\n### Steps to reproduce:\r\n\r\n- Install eCommerce app.\r\n- Set default company currency to JPY.\r\n- Set exchange rate.  e.g. 1 EUR = 130 JPY\r\n- Configure Stripe payment acquirer with \"Authorize & capture the amount, confirm the SO and auto-validate the invoice on acquirer confirmation\" for Order Confiration, and Bank (JPY) for Payment Journal.\r\n- Create a pricelist with EUR and assign it to the website.\r\n- Purchase something in online shop and pay (10 EUR) through Stripe.  In Stripe, net amount is recognized as 120 JPY after conversion (i.e. Stripe is linked to a JPY bank account) from 10 EUR and deducting their fee.\r\n\r\n### Current behavior:\r\n\r\n- Payment transaction is created as 10 EUR (which is not incorrect)\r\n- Payment (account.payment) is created as 10 JPY (10 from 10 EUR in invoice and JPY from the Payment Journal in Stripe payment acquirer), and journal entry and reconciliation with invoice are all messed up.\r\n\r\n### Expected behavior:\r\n\r\nThere can be a few different approaches to handle this issue.\r\n\r\n#### Option 1:\r\n\r\n- Payment transaction is created as 120 JPY.\r\n- Payment (account.payment) is created as 120 JPY (based on payment transaction) and reconciled with invoice, posting the variance of 10 JPY (with the company currency invoice amount of 130 JPY) as transaction fee.\r\n\r\n*The handling of the variance mentioned above may technically not be correct depending on how you interpret the variance, and this option requires rather radical design changes.*\r\n\r\n#### Option 2:\r\n\r\n- Allow assigning multiple payment journals to a payment acquirer.\r\n- Payment transaction is created as 10 EUR (no change from the current design).\r\n- Payment (account.payment) is created as 10 EUR (10 from 10 EUR in invoice and EUR from the Payment Journal of the same currency as invoice in Stripe payment acquirer).\r\n\r\n*This option is probably easier but requires setting up multiple journals for a payment acquirer according to available currencies, which may complicate the balance management of the accounts for the payment acquirer.*\r\n\r\n#### Option 3:\r\n\r\n- Payment transaction is created as 10 EUR (no change from the current design).\r\n- Payment (account.payment) is created as 10 EUR (just apply the same amount and currency as the invoice regardless of the used journal).\r\n\r\n*Not sure if this will cause some side effects as I believe Odoo does not expect inconsistent currencies between payment and assigned journal.* \r\n\r\n#### Option 4:\r\n\r\nGive up on handling multi currencies with a payment acquirer with the setting of \"Authorize & capture the amount, confirm the SO and auto-validate the invoice on acquirer confirmation\", and show error when customer tries to select a payment acquirer whose journal currency is inconsistent with the order currency.   Document this limitation in related user docs.\r\n\r\n### Video/Screenshot link (optional):\r\n"},"comment":{"url":"https://api.github.com/repos/odoo/odoo/issues/comments/334395765","html_url":"https://github.com/odoo/odoo/issues/19886#issuecomment-334395765","issue_url":"https://api.github.com/repos/odoo/odoo/issues/19886","id":334395765,"user":{"login":"Yenthe666","id":6352350,"avatar_url":"https://avatars3.githubusercontent.com/u/6352350?v=4","gravatar_id":"","url":"https://api.github.com/users/Yenthe666","html_url":"https://github.com/Yenthe666","followers_url":"https://api.github.com/users/Yenthe666/followers","following_url":"https://api.github.com/users/Yenthe666/following{/other_user}","gists_url":"https://api.github.com/users/Yenthe666/gists{/gist_id}","starred_url":"https://api.github.com/users/Yenthe666/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Yenthe666/subscriptions","organizations_url":"https://api.github.com/users/Yenthe666/orgs","repos_url":"https://api.github.com/users/Yenthe666/repos","events_url":"https://api.github.com/users/Yenthe666/events{/privacy}","received_events_url":"https://api.github.com/users/Yenthe666/received_events","type":"User","site_admin":false},"created_at":"2017-10-05T08:28:29Z","updated_at":"2017-10-05T08:28:29Z","author_association":"CONTRIBUTOR","body":"Hmm that looks off indeed. @qdp-odoo and @nim-odoo could one of you look at this?"}}
{"actor":{"display_login":"Yenthe666"}}