postnl / postnl-magento1-End-of-life

This is the official Magento 1 extension of PostNL.
12 stars 9 forks source link

{[13804] Insurance value is invalid} #32

Closed edwinknol-mediact closed 6 years ago

edwinknol-mediact commented 6 years ago

Submitting issues trough Github

Please follow the guide below


Make sure you are using the latest version: https://tig.nl/postnl-magento-extensies/

Issues with outdated version will be rejected.


What is the purpose of your issue?


Description of your issue, suggested solution and other information

We're getting the following error when clicking on the "PostNL - Print Verzendlabel" button:

Er is een fout opgetreden tijdens het verwerken van uw verzoek:Validation failed for shipment: 3STOTA646783173, Messages: 3STOTA646783173: {[13804] Insurance value is invalid}

Explanation of your issue in arbitrary form goes here. Please make sure the [description is worded well enough to be understood]. Provide as much context and examples as possible. If work on your issue requires account credentials please provide them or explain how one can obtain them.

TIG supportdesk

On Github we will respond in English even when the question was asked in Dutch.

tig-dennisvanderhammen commented 6 years ago

Hi @edwinknol-mediact,

I'm assuming that you are trying to confirm a shipment with insurance (extra cover).

In the database, there is a field named "extra_cover_amount" in the tig_postnl_shipment. Could you tell me what the value of this of this field in the shipment you are trying to confirm?

edwinknol-mediact commented 6 years ago

Hi @tig-dennisvanderhammen

I've checked the shipment in the database. The value of "extra_cover_amount" is NULL. When is set a value by hand in this field, i'm able to print the shipping label.

tig-dennisvanderhammen commented 6 years ago

@edwinknol-mediact, The servers of PostNL don't allow a value of 0 or lower to be insured. We recently did change the behaviour how the extra cover amount works.

In the past we would take the shipmentBaseGrandTotal and ceiled it to the nearest 500 (for example, when the total is 1150 euro, the extra cover amount would be 1500) with the minimum being 500. A change in the PostNL service allows this to be any number above zero. Now we simply return the shipmentBaseGrandTotal.

If you're certain the shipmentBaseGrandTotal is also above 0, you can check the code based around the method getDefaultExtraCoverAmount() in TIG/PostNL/Model/Core/Shipment.php.

If you can't figure out a solution, could you send us an e-mail at servicedesk@tig.nl with access to a staging environment where you can reproduce this issue?

tig-jaspersmits commented 6 years ago

Hi @edwinknol-mediact ,

I'm not 100% sure, but it seems you didn't mail us about this issue. Correct? Did you figure out a solution or do you still want us to take a look at it?

Kind regards,

Jasper TIG

mediact-pipelines commented 6 years ago

Hi @tig-jaspersmits

Sorry for not replying anymore. We've fixed the issue, so this bugreport can be set closed.

Edwin

tig-jaspersmits commented 6 years ago

Hi Edwin,

No problem! Good to hear that you found a solution. Hereby i will close this bugreport.

Kind regards,

Jasper TIG