The EU VAT App label on checkout misses the VAT Rate
By Tassos Marinos on Tuesday, 12 January 2021
Posted in General Issues
Likes 0
Views 635
Votes 0
You're stating in the v4.2.4 changelog that the issue I've reported here https://stackideas.com/forums/the-eu-vat-app-label-on-checkout-misses-the-vat-rate has been resolved but that's not true.

I am very disappointed. Are you guys even test your code?

The EU-VAT label now says: EU-VAT Tax added ( %s )

The %s variable doesn't get replaced because you're not using the JText::sprintf() method.

Moreover, even if you use the sprintf() method the label will be EU-VAT Tax added ( 24 ) which is wrong as it misses the percentage % sign. It should become: EU-VAT Tax added (24%).

More time wasted.
Hey Tassos,

I am really sorry for the inconvenience caused to you.
We will check with a testing team where it missed. We will fix it at your site as well.
Please update us with your site details so we will fix it at your site as well.

Thank you for understanding!
Manisha Ranawat
·
Tuesday, 12 January 2021 10:51
·
0 Likes
·
0 Votes
·
0 Comments
·
Hey Tassos,

To fix this issue at your site do the following changes at your site.

1. Find the attached file and replace it with the below-mentioned file path.
root\plugins\payplans\euvat\app\euvat.php

2. Go to file path : root\language\en-GB\en-GB.com_payplans.ini, near line no. 2164, find below mentioned language string.


COM_PAYPLANS_APP_EU_VAT_MODIFIER_MESSAGE="EU-VAT Tax added ( %s )"


Replace with


COM_PAYPLANS_APP_EU_VAT_MODIFIER_MESSAGE="EU-VAT Tax added"


Do the changes and check this issue again.
Update us if the still issue persists.
Manisha Ranawat
·
Tuesday, 12 January 2021 17:27
·
0 Likes
·
0 Votes
·
0 Comments
·
It works. Thank you. You may to use JText::sprintf() instead. It's much clearer.
Tassos Marinos
·
Wednesday, 13 January 2021 17:58
·
0 Likes
·
0 Votes
·
0 Comments
·
Hi there,

You are most welcome. Glad to hear that your issue has been resolved now.

Just for your information, I have locked and marked this thread as resolved to avoid confusions in the future. Please start a new thread if you have any other issue in the future so it will be easier for us to manage your inquiries.

Thanks for understanding.
Manisha Ranawat
·
Wednesday, 13 January 2021 18:02
·
0 Likes
·
0 Votes
·
0 Comments
·
View Full Post