search

ebizmarts / magento2-mandrill

Mandrill integration for Magento 2.
https://mandrill.com/
36 stars 36 forks source link

fixed the email subscription check #18

Closed loekderooij closed 7 years ago

loekderooij commented 7 years ago

Function _checkSubscription was commented and isSubscribed function cause a notice because empty array value was addressed.

It would be advised to add unit tests for these methods. If they had existed, this bug would not have been necessary. I wanted to add the tests, but had some troubles running them. My employer would not be happy if I spent hours getting this to run, so I dropped this.

Santiagoebizmarts commented 7 years ago

@LuckyLoek Thanks for your help.