[ASTPPCOM-1319] Error transferring balance Created: 19/Nov/22 Updated: 23/Dec/22 Resolved: 23/Dec/22 |
|
| Status: | Closed |
| Project: | ASTPP Community |
| Component/s: | None |
| Affects Version/s: | v6.0 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Highest |
| Reporter: | Danilo Borges | Assignee: | Kinjal Prajapati |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | 10h 35m | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Hello, see the error when the client tries to transfer credit between accounts in version 6 community in the Portuguese language, in the English language it works normally.
Error Number: 1136 Column count doesn’t match value count at row 1 INSERT INTO invoice_details (accountid, description, created_date, invoiceid, reseller_id, is_tax, order_item_id, payment_id, before_balance, product_category, charge_type, after_balance, base_currency, exchange_rate, account_currency, debit, credit) VALUES (‘2’, ‘Fund has been transfered from 4723285745 '(' 51294181 ')'’, ‘2022-11-15 21:04:02’, 8, ‘0’, 0, 0, 8, ‘697.60000’, ‘3’, ‘FUNDTRANSFER’, 687,6, ‘BRL’, ‘3.875’, ‘BRL’, 10, ‘0.00’) Filename: /opt/ASTPP/web_interface/astpp/libraries/astpp/invoice.php Line Number: 199 This path does not exist at this address:
Thanks |
||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Reproducibility: | always | ||||||||
| Category: | Functional | ||||||||
| Sprint: | 2022-ENT-DEV-V70-W49 | ||||||||
| Edition: | Community | ||||||||
| Description |
|
Hello, see the error when the client tries to transfer credit between accounts in version 6 community in the Portuguese and Spanish language, in the English language it works normally.
Error Number: 1136 Column count doesn’t match value count at row 1 INSERT INTO invoice_details (accountid, description, created_date, invoiceid, reseller_id, is_tax, order_item_id, payment_id, before_balance, product_category, charge_type, after_balance, base_currency, exchange_rate, account_currency, debit, credit) VALUES (‘2’, ‘Fund has been transfered from 4723285745 '(' 51294181 ')'’, ‘2022-11-15 21:04:02’, 8, ‘0’, 0, 0, 8, ‘697.60000’, ‘3’, ‘FUNDTRANSFER’, 687,6, ‘BRL’, ‘3.875’, ‘BRL’, 10, ‘0.00’) Filename: /opt/ASTPP/web_interface/astpp/libraries/astpp/invoice.php Line Number: 199 This path does not exist at this address:
Thanks |
| Comments |
| Comment by Bhavyata Raval (Inactive) [ 28/Nov/22 ] |
|
verified in latest community v6, working as expected. |
| Comment by Danilo Borges [ 28/Nov/22 ] |
|
Did you test in Spanish and Portuguese language? Here it only worked in English, but my customers don't use English! Please access the test client area in Spanish and Portuguese and take the test. Thanks |
| Comment by Danilo Borges [ 06/Dec/22 ] |
|
Friends, any resolution? Thanks and regards |
| Comment by Kinjal Prajapati [ 07/Dec/22 ] |
|
Verified with the specified Portuguese and Spanish languages. It's working for both languages. |
| Comment by Danilo Borges [ 07/Dec/22 ] |
|
Impossible!!! Have you tested it on version 6? Because the error is visible, see the attached image, is some column missing in mysql, or did you add it manually? Did you test it in the customer area in Portuguese and Spanish? |
| Comment by Danilo Borges [ 08/Dec/22 ] |
|
Hello, here is the attached video(screen-recording.webm) of the error in question with clean installation of astpp 6. Thanks |
| Comment by Alpesh Kaklottar [ 08/Dec/22 ] |
|
Thanks for the attention Danilo Borges We are able to reproduce this issue. Fund transfer is giving error when customer account balance is in decimal value. Kinjal Prajapati check attached video. Customer, admin and base currency is BRL. |
| Comment by Sanket Thakkar (Inactive) [ 08/Dec/22 ] |
|
This is Fixed. PR Details are as follows. Alpesh Kaklottar can you please verify and let us know if anything. |
| Comment by Danilo Borges [ 08/Dec/22 ] |
|
Hello Sanket Thakkar, the problem is not fully resolved. *Assuming I have a balance of 84,408 and I try to transfer 1.65 it says I have no balance. |
| Comment by Alpesh Kaklottar [ 09/Dec/22 ] |
|
Dear Danilo Borges Thank for the more clarification. Please wait for a while, internal verification is going on. We will let you know once it is ready for your verification. |
| Comment by Bhavyata Raval (Inactive) [ 12/Dec/22 ] |
|
Kinjal Prajapati , when we set BRL currency (in admin, both customer, base currency) then try to transfer 1.65 amount, more than 300 amount transferred with Portuguese language, expected result: if there is no need to currency conversion then it should be transferred same amount because we set same currency. please check recent attachment and try to reproduce. |
| Comment by Bhavyata Raval (Inactive) [ 12/Dec/22 ] |
|
Kinjal Prajapati as we discussed to do not change base currency, set base currency as USD, ADMIN has BRL, both customer have BRL, 1. now try to transfer with english language : got error as " you need to enter minimum 0.1905 for fund transfer" : so it should be resolved for validation. 2. try to transfer with portuguese language : got error as " no sufficient fund " : where customer has sufficient fund 3. try to transfer with spanish language : fund transfer successfully. please do needful as mentioned above. let me know if anything. |
| Comment by Danilo Borges [ 12/Dec/22 ] |
|
Hi, After updating the 2 files via github request #677 I tested leaving the base currency in USD, and admin and user in BRL, and see that it transfers the wrong values: user: 4727985745 balance: 250.71 BRL user: 54004181 balance: 552.12 BRL
when transferring 25.01 from 4727985745 the balance is: 188.71 and the correct one would be: 225.70
after the transfer the balance of 54004181 was in: 614.12 and the correct one would be: 577.13
And leaving the base in BRL it says that the balance is insufficient!
Thanks and Regards
|
| Comment by Alpesh Kaklottar [ 15/Dec/22 ] |
|
Following issues are occurring when base currency(USD) and customer's currency(BRL) are different.
Check attached videos. |
| Comment by Alpesh Kaklottar [ 15/Dec/22 ] |
|
Can you please share below details
|
| Comment by Danilo Borges [ 15/Dec/22 ] |
|
Hi, let's start from the beginning, I just gave the git command below to get the most current version: Configuration>>>Settings: Configuration>>>Languages>>>Languages Accounts>>>Admins: Accounts>>>Customers>>>Customers: In this configuration above, when I enter the user, he already enters the Portuguese language (default language), and I try to make a transfer in the amount of 26.00, he does it correctly and without errors. In this same configuration above, when I enter the user, he already enters the Portuguese language (default language), I go to the footer and change it to "English" any transfer works, both integer value 26.00 and decimal value 26.71 without any error!
Thanks and Regards |
| Comment by Danilo Borges [ 22/Dec/22 ] |
|
Friends, I just confirmed that the problem was solved, but for that I had to copy the files from https://github.com/iNextrix/ASTPP/pull/678/files if I just do a git it fetches the files without the changes what did you guys do, will we always have to make these changes manually or does it go to the github master? |
| Comment by Alpesh Kaklottar [ 23/Dec/22 ] |
|
Danilo Borges Thanks for the confirmation, Definitely changes will merge into Master of ASTPP v6.0. |