This is the technical support forum for WPML - the multilingual WordPress plugin.
Everyone can read, but only WPML clients can post here. WPML team is replying on the forum 6 days per week, 22 hours per day.
This topic is split from https://wpml.org/forums/forum/english-support/
Sun | Mon | Tue | Wed | Thu | Fri | Sat |
---|---|---|---|---|---|---|
- | 9:00 – 14:00 | 9:00 – 14:00 | 9:00 – 14:00 | 9:00 – 14:00 | 9:00 – 14:00 | - |
- | 15:00 – 18:00 | 15:00 – 18:00 | 15:00 – 18:00 | 15:00 – 18:00 | 15:00 – 18:00 | - |
Supporter timezone: Asia/Dhaka (GMT+06:00)
Tagged: Bug, Known issue, WCML
Related documentation:
This topic contains 38 replies, has 2 voices.
Last updated by Prosenjit Barman 11 months, 2 weeks ago.
Assisted by: Prosenjit Barman.
Author | Posts |
---|---|
January 18, 2024 at 9:39 am #15198961 | |
guy@media-vision.co.il |
Hi Prosenjit, I can assist you with server access. There are two available options: 1. You provide me with your IP address. 2. Simply share your email, and I will send you a link that will grant you access when clicked. Using email is the most convenient and efficient method because you can also share the email with your colleagues, allowing them to gain access easily. Please note that turning off the firewall is not a viable option. I've taken the precaution of restoring the files and the database to address any potential issues- the site is loading properly now. Additionally, I have reapplied the fix provided in this forum post: https://wpml.org/forums/topic/woo-subscriptions-price-mistake-1-parameter-is-taking-wrong-currency-in-2-pages/ Best regards, |
January 18, 2024 at 4:55 pm #15201522 | |
guy@media-vision.co.il |
Prosenjit, Are you still here? |
January 18, 2024 at 5:32 pm #15201668 | |
guy@media-vision.co.il |
Done! Please check your inbox. |
January 18, 2024 at 6:54 pm #15201850 | |
Prosenjit Barman Supporter
Languages: English (English ) Timezone: Asia/Dhaka (GMT+06:00) |
Hello Guy, After making a small modification to the code to address the pricing issue, it now seems that the prices are correctly reflected on the Cart page. For a more detailed view, please refer to this screenshot [hidden link;. It's important to mention that there were no changes made to the language and other settings while implementing this fix. What I've fixed in the code: An important backslash (\) before "WC_Product_Subscription" is missing(Please check the 'if' condition on 2nd line). This backslash is essential for correctly referencing the "WC_Product_Subscription" class from the global namespace. When I added that, the price started to appear correctly. Please let me know if that helps to solve the issue or if you need further assistance in this matter. I will be happy to help. Once again, thank you so much for you kind cooperation in this case! Best regards, |
January 18, 2024 at 7:58 pm #15202066 | |
guy@media-vision.co.il |
Hi Prosenjit, I hope this message finds you well. I wanted to reach out to clarify and confirm my understanding of the recent changes you've implemented. Could you please confirm if the only modification you made was to the code in the 'MulticurrencyHooks.php' file? Additionally, I would like to know if there were any other changes made to any other files, or any adjustments to the settings in the WordPress admin panel. Your attention to these details is greatly appreciated, as it helps ensure everything is in perfect order. Looking forward to your response. Best regards, |
January 19, 2024 at 6:05 am #15202913 | |
Prosenjit Barman Supporter
Languages: English (English ) Timezone: Asia/Dhaka (GMT+06:00) |
Hello Guy, I understand your concern and want to assure you that there were no modifications made to the WPML/WCML settings or in any other areas. The sole action I performed was the addition of the corrected code into the 'MulticurrencyHooks.php' file, and that's the full extent of it. All the settings have remained exactly as they were before. Please let me know if that helps to solve the issue or if there is anything else you need help with. I will be happy to assist. Best regards, |
January 23, 2024 at 6:56 am #15214164 | |
guy@media-vision.co.il |
Hi Prosenjit, Testing. looks like it works but maybe it cause a different bug. i need to do more testing before we can conclude. will update soon. |
January 23, 2024 at 9:36 am #15215257 | |
guy@media-vision.co.il |
Now, the switching works well for USD, but it doesn't work for ILS (ILS was fine previously). If I log in as a Hebrew client (user test he) who has already purchased a subscription in ILS and go to my account to click on the upgrade link here: hidden link it gives me the options to upgrade, but the "Gap payment" is showing as zero. I'm trying to upgrade from 390 to 590 can be Zero gap payment. |
January 23, 2024 at 11:51 am #15216233 | |
guy@media-vision.co.il |
*can not be Zero gap payment. |
January 23, 2024 at 12:18 pm #15216366 | |
guy@media-vision.co.il |
The Image Showing the code change you did and the bug that created in ILS *We just wanted the page to take the currency exactly as the wpml wcml settings:* ******************************* The code fix should make sure it works like the settings. see image of settings again. Simple setting. |
January 25, 2024 at 7:42 am #15224664 | |
Prosenjit Barman Supporter
Languages: English (English ) Timezone: Asia/Dhaka (GMT+06:00) |
Hi There, I've conducted tests in my development environment and successfully replicated the scenario you're experiencing. Given the unique nature of your workflow, the solution might not be simple. Any fix we implement mustn't disrupt the standard operational flow. We need to balance addressing your specific needs with maintaining the overall system integrity. I hope you can understand. As soon as I have more information on this, I will surely inform you. Best regards, |
January 29, 2024 at 6:51 am #15236238 | |
guy@media-vision.co.il |
Hi Prosenjit, Thanks for providing solution that almost works. Please do 1 more thing for me. Since the fix solve the problem for USD in English website. Just ad a condition for the fix to work only on English site. I need you to add 1 Quick condition so the fix will only work in English version. Ad this condition so the fix will only fix when it is English. Kind Regards |
January 29, 2024 at 11:17 am #15237594 | |
Prosenjit Barman Supporter
Languages: English (English ) Timezone: Asia/Dhaka (GMT+06:00) |
Hello Guy, I want to assure you that I have already forwarded your message to our dev team. Our team is fully aware of the situation and is currently in the process of analyzing the best possible approach to address this issue. As of the latest update, it's important to note that the solution involves a critical piece of logic. We are proceeding with caution to ensure that the fix we implement does not inadvertently introduce any new issues. Your suggestion to add a condition so that the fix applies only to the English version of the site, or specifically when the site's currency is set to USD, is indeed valuable. I truly appreciate your input and the proactive approach you've taken to suggest a potential solution. I am personally exploring this further and will keep you informed about any viable solutions I identify. Please rest assured that we are doing everything within our power to swiftly address this issue, without compromising the integrity of your site's functionality. I will be in touch with updates as soon as we have made progress. Best regards, |
January 30, 2024 at 10:55 am #15242783 | |
guy@media-vision.co.il |
Hi Prosenjit, Thanks for providing a detailed explanation, i really understand what you say and it give me confidence that you are with us. The most important thing is that you understand that we are not testing/exploring options. we are dealing with live situation for company that does subscriptions with WPML and WCML for years. and succesfuly. Adding a currency looked naturaly with wcml and the company was not aware of the bug until subscribers complained. so you can call this situation a crisis. The USD (customers) used to keep their plan and not switching. For years ILS clients switching subscriptions succesfuly (usualy client get bigger plan when they want to have biger subscription plan) - and it works perfect to this very moment. We are dealing with crisis on live site. |
January 31, 2024 at 7:22 am #15247434 | |
Prosenjit Barman Supporter
Languages: English (English ) Timezone: Asia/Dhaka (GMT+06:00) |
Hi Guy, Thank you for your response. I completely understand the urgency and critical nature of the situation. Your detailed explanation provides valuable context, and I assure you that we are fully committed to resolving this issue promptly. Following your recommendation, I have explored and integrated a particular condition for the English language and USD currency into the existing solution. After implementing the code, I noticed that the calculations on the cart page began to function accurately across different languages. Please review this screenshot[hidden link; taken after finalizing a purchase in USD, and another screenshot[hidden link; after completing the transaction in ILS currency. For your information, the code has been added to the 'MulticurrencyHooks' file and no other changes have been made. If you use any other currencies than USD and ILS or add more currency, that solution will not work. If you want to test, please make a purchase from a new account and check the functionality. If everything works as expected, you can use this code as a temporary solution until we finalize a permanent resolution. Please let me know the update. Rest assured, we are here to support you throughout this process and maintain a thorough understanding of the situation until a swift and satisfactory solution is in place. Best regards, |
The topic ‘[Closed] Split: Woo subscriptions wrong price’ is closed to new replies.