Skip to content

Commit

Permalink
Merged PR 34471: Fix code error
Browse files Browse the repository at this point in the history
## What's being changed

This fixes a regression in this PR: https://dev.azure.com/dotdigital/ec/_git/dd-wordpress-signup-form/pullrequest/34393.

## Why it's being changed

Some code was changed but accidentally re-ordered.

## How to review / test this change

- Re-run the sign up and resubscribe tests

Related work items: #162849
  • Loading branch information
sta1r authored and simon-letch committed Mar 2, 2022
1 parent 35571fd commit 6936935
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions DotMailerConnect.php
Original file line number Diff line number Diff line change
Expand Up @@ -121,11 +121,11 @@ function reSubscribeContact( $email, $addressBookId, $datafields = [] ) {
}

}

private function createOrResubscribeContact( $addressBookId, $contact )
{
$result = $addressBookId == -1 ? $this->resources->PostContacts( $contact ) : $this->resources->PostAddressBookContacts( $addressBookId ,$contact );
if ( $contact instanceof DataTypes\ApiContact ) {
$result = $addressBookId == -1 ? $this->resources->PostContacts( $contact ) : $this->resources->PostAddressBookContacts( $addressBookId ,$contact );
} else {
$result = $addressBookId == -1 ? $this->resources->PostContactsResubscribe( $contact ) : $this->resources->PostAddressBookContactsResubscribe( $addressBookId ,$contact );
}
Expand Down

0 comments on commit 6936935

Please sign in to comment.