Error 33: Email is required (and E00027)

Merge request for this module submitted. We're just adding the customer->email to the request as an array which works and solves the problem. However, we can also coordinate with the upstream library (commerceguys/authnet) for which I have a pull request to add the data type https://github.com/commerceguys/authnet/issues/39. Since having that merged and a release rolled will likely take some time, I setteld for the "quick and dirty" fix here. If it is merged, we'll change this to :

   use CommerceGuys\AuthNet\DataTypes\Customer;
...
    $transaction_request->addDataType(new Customer([
      'email' => $order->getEmail(),
    ]));

🐛 Error on refund if payment hasn't been settled Needs review should also be updated to include this in the void request.

Email addresses are not required for orders. We shouldn't assume there is one on the order. Based on the code in Entity/Order.php it looks like $order->getEmail() can return an empty string.

Good point. Let me confirm that an empty string would break this request and I'll add a condition if necessary.

I went ahead and added the condition so we're good either way.