WebhookNotification object payload parsed incorrectly

[soundsgoodsofar]

I'm using version 1.6.3 of the SDK

The constructor of WebhookNotification.php appears to parse the object payload incorrectly for a number of webhooks.

It assumes that any webhook type starting with 'USER' will have an object of type User. This appears to be incorrect for webhooks for different payment method events.

For example, for a webhook of type USERS.VENMO_ACCOUNTS.CREATED, I get an object with the following structure:

‌‌print_r($properties['object'], true);
‌Array
(
    [token] => trm-REDACTED
    [type] => VENMO_ACCOUNT
    [status] => ACTIVATED
    [createdOn] => 2021-05-28T15:36:53
    [transferMethodCountry] => US
    [transferMethodCurrency] => USD
    [accountId] => REDACTED
    [userToken] => usr-REDACTED
)

[soundsgoodsofar]

I haven't verified the payload for all of these webhooks yet (just venmo), but I believe this would be the fix:

Change this section in WebhookNotification.php from:

} else if (strpos($properties['type'], 'USERS') === 0) {
    $this->object = new User($properties['object']);
}

to this:

} else if (strpos($properties['type'], 'USERS.BANK_ACCOUNTS') === 0) {
    $this->object = new BankAccount($properties['object']);
} else if (strpos($properties['type'], 'USERS.PAYPAL_ACCOUNTS') === 0) {
    $this->object = new PaypalAccount($properties['object']);
} else if (strpos($properties['type'], 'USERS.VENMO_ACCOUNTS') === 0) {
    $this->object = new VenmoAccount($properties['object']);
} else if (strpos($properties['type'], 'USERS') === 0) {
    $this->object = new User($properties['object']);
}

Looks like of those, BankAccounts is already parsed correctly actually.