magespecialist / m2-MSP_CashOnDelivery

Cash on delivery module for Magento 2
52 stars 53 forks source link

Question about order email #37

Closed giorg closed 7 years ago

giorg commented 7 years ago

Hi,

customer asks me to remove from new order email the line "Cash On Delivery Tax", and the title "Pagamento in contrassegno" (because it's shown twice), is this possible?

Thanks

phoenix128 commented 7 years ago

Hello @giorg , actually it seems it was an error during email rendering.

Please upgrade to version 1.2.4, ve just fixed it.

giorg commented 7 years ago

Hi,

thanks for your answer. I installed it through composer, how to upgrade? Thanks a lot

phoenix128 commented 7 years ago

Try with:

composer require msp/cashondelivery:1.2.4
php bin/magento setup:upgrade
giorg commented 7 years ago

I get this:

`composer require msp/cashondelivery:1.2.4 Do not run Composer as root/super user! See https://getcomposer.org/root for details ./composer.json has been updated Loading composer repositories with package information Updating dependencies (including require-dev) Your requirements could not be resolved to an installable set of packages.

Problem 1

Installation failed, reverting ./composer.json to its original content.`

any hint?

phoenix128 commented 7 years ago

Hello @giorg , try it now, packagist was out of date.

giorg commented 7 years ago

hi,

ok now the installation went fine, but when I try to complete order, I get an ajax error and cannot go on, maybe this screenshot helps?

screen shot 2017-09-12 at 14 52 33

phoenix128 commented 7 years ago

That is the request, see if you getsomething in response tab or if you get something in your error log.

giorg commented 7 years ago

this the response:


Fatal error: Uncaught Error: Call to a member function getData() on null in /var/www/coronelli /vendor/magento/module-payment/Block/Info.php:87 Stack trace:

0 /var/www/coronelli/vendor/magento/module-payment/view/frontend/templates/info/default.phtml(13): Magento

\Payment\Block\Info->getSpecificInformation()

1 /var/www/coronelli/vendor/magento/framework/View/TemplateEngine/Php.php(59): include('/var/www/corone

...')

2 /var/www/coronelli/vendor/magento/framework/View/Element/Template.php(255): Magento\Framework\View

\TemplateEngine\Php->render(Object(MSP\CashOnDelivery\Block\Info\CashOnDelivery), '/var/www/corone ...', Array)

3 /var/www/coronelli/vendor/magento/framework/View/Element/Template.php(279): Magento\Framework\View

\Element\Template->fetchView('/var/www/corone...')

4 /var/www/coronelli/vendor/magento/framework/View/Element/AbstractBlock.php(659): Magento\Framework

\View\Element\Template->_toHtml()

5 /var/www/coronelli/vendor/magento/module-payment/Helper/Data.php(215): Magento\Framework\View\Element

\AbstractBlock- in /var/www/coronelli/vendor/magento/module-payment/Block/Info.php on line <b

87
{"messages":{"error":[{"code":500,"message":"Server internal error. See details in report api\/1493178814345" }]}}

phoenix128 commented 7 years ago

Our fault, sorry for this. Upgrade to 1.2.5, it should work.

giorg commented 7 years ago

no problem, you guys are great, thanks a lot!