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
Missmatch Portugal ZIP Code Matching #34271
Comments
Hi @mklooss. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. Please, add a comment to assign the issue:
🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, please join the Community Contributions Triage session to discuss the appropriate ticket. 🎥 You can find the recording of the previous Community Contributions Triage on the Magento Youtube Channel ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
Simple Workaround / Patch for just PT --- a/Model/ResourceModel/Calculation.php 2021-10-07 13:14:39.696529241 +0200
+++ b/Model/ResourceModel/Calculation.php 2021-10-07 13:15:13.316749197 +0200
@@ -353,6 +353,12 @@
$zipTo = $matches[2];
}
}
+ if ($countryId == 'PT')
+ {
+ $postcodeIsRange = false;
+ $zipFrom = NULL;
+ $zipTo = NULL;
+ }
if ($postcodeIsNumeric || $postcodeIsRange) {
$selectClone = clone $select;
|
Hi @engcom-Delta. Thank you for working on this issue.
|
Hi @mklooss , |
@engcom-Delta just take a look at the list on wikipedia: "Postal regions (first digit of postal code)" -> "9. (brown) - Madeira Islands and Azores" and also at the Image on the right side of the article: https://en.wikipedia.org/wiki/Postal_codes_in_Portugal#/media/File:Postal_Codes_of_Portugal.png Example: https://en.wikipedia.org/wiki/Ribeira_Grande,_Azores |
✅ Jira issue https://jira.corp.magento.com/browse/AC-1925 is successfully created for this GitHub issue. |
✅ Confirmed by @engcom-Delta. Thank you for verifying the issue. |
@mklooss @engcom-Hotel |
Hi @engcom-Alfa. Thank you for working on this issue.
|
Hi @engcom-Hotel. Thank you for working on this issue.
|
Hi @mklooss, Please find attached proofs for reference and updated issue description. Issue description: irrespective of zip code & tax rule , tax gets calculated only default is 23% which is zip code set '* '. |
Missmatch Portugal ZIP Code Matching - ( #14220 )
Issue description: irrespective of zip code & tax rate set , tax gets calculated only default is 23% which is zip code set '* '.
the Issues will be produced by this method: https://github.com/magento/magento2/blob/2.4/app/code/Magento/Tax/Model/ResourceModel/Calculation.php#L345-L355
Preconditions (*)
Steps to reproduce (*)
Expected result (*)
Actual result (*)
Additional Info (*)
Please provide Severity assessment for the Issue as Reporter. This information will help during Confirmation and Issue triage processes.
The text was updated successfully, but these errors were encountered: