pfpayments / prestashop-1.6

PostFinance Checkout integration for PrestaShop 1.6
https://www.postfinance-checkout.ch
Apache License 2.0
0 stars 0 forks source link

PostFinanceCheckoutWebhookModuleFrontController::postProcess throws an Exception with an empty message #8

Closed lmeyer1 closed 1 year ago

lmeyer1 commented 2 years ago

Describe the bug

GET /module/postfinancecheckout/webhook gives an error 500, with the error Property PrestaShopLogger->message is empty image

Looking closer, this is because PostFinanceCheckoutWebhookModuleFrontController::postProcess throws an Exception with an empty message. This is because https://github.com/pfpayments/prestashop-1.6/blob/master/inc/Exception/Incompleteconfig.php does not specify a default message, and no message is defined at invocation neither https://github.com/pfpayments/prestashop-1.6/blob/7c90efc95453ba94a3653ecd0c67b1037c22d093/inc/Helper.php#L45

Expected behavior

Any Exception anywhere in any PHP program is supposed to have a message. This only allows for a straight forward debugging.

To Reproduce

  1. Create a multishop installation of Prestashop
  2. Configure Postfinance Checkout in a shop different from the first.
  3. The webhook will throw an error 500.
  4. Look at the error log on the server.

Software versions (please complete the following information):

theodorspringwallee commented 2 years ago

Dear lmeyer1

Thank you for this discovered BUG.

We will try to reproduce it on our side and give you feedback, if we get further information.

theodorspringwallee commented 1 year ago

Dear lmeyer1

According to our devs, the plugin multi-shop setup for PS 1.6 is a not supported feature.

Therefore, we will close this issue.

lmeyer1 commented 1 year ago

@theodorspringwallee I just cannot understand this.

Please advertise your product honestly. Support honestly and tell if something does not work. Hire developers who are working, and fire those that just say Doesn't work in order to shirk work.

And finally, coordinate marketing, support and development. Don't sell what you won't develop: that is deceiving people.

theodorspringwallee commented 1 year ago

Dear lmeyer1

You definitely raised a good point. We apologize for the inconvenience, for this misinformation on our website. We will adjust this.

Best regards

lmeyer1 commented 1 year ago

@theodorspringwallee Thank you very much for your apologizing. It is soothing after almost a year of failing communication.

I'd like to apologize too for the harsh tone in my past answers.

Unfortunately for your business, we were forced to look for an alternative some months ago. The failure of your technical support and absence of reaction here on github where the main reasons for this decision.