-
Notifications
You must be signed in to change notification settings - Fork 15
Issue with extension attributes in plugin/interceptor #93
Comments
@KevinMace - are you still able to reproduce the issue after clearing the Magento cache and performing a DI compile: Are you able to reproduce the issue on a vanilla Magento 2 install using the latest version of the AvaTax extension? If you can reproduce the issue, please provide specific steps for doing so. |
@rsisco Yes, even after clearing the cache, deleting var/generation, var/cache and running the di:compile command the issue was still occurring. The AvaTax version is the one set in this extensions dependency list in the composer.json, which does appear to be their latest release https://github.com/avadev/AvaTax-Calc-SOAP-PHP/releases I'll try with a vanilla Magento 2 install and see how that goes. |
@rsisco I've been doing some testing using a vanilla Magento EE 2.1.10 install, I can confirm that issue does still occur. These are the steps I went through to recreate the issue.
|
@KevinMace - Thank you for the update, I am working on reproducing locally and providing a resolution. |
@KevinMace - We've released a new version of the extension that addresses this issue. Please update the extension and reopen this issue if needed. |
@rsisco Great, thank you. I'll install the new version today and see how we get on. |
Preconditions
Steps to reproduce
Expected result
Actual result
The following error is logged in the apache logs
Got error 'PHP message: PHP Fatal error: Uncaught Error: Call to a member function getTax() on null in /path/to/magento/vendor/classyllama/module-avatax/Plugin/Sales/Total/Quote/CommonTaxCollector.php:43
Which relates to this line of code, the 'getExtensionAttributes' is returning NULL instead of an object.
'extension_attributes' => ['tax' => $rateDataObject->getExtensionAttributes()->getTax()]
The text was updated successfully, but these errors were encountered: