Skip to content

Commit 337a5ab

Browse files
committed
Liste membres autos : On ne tente pas de rajouter les unsubscribed en subscribed
Les unsubscribed sont des personnes qui ont demandé à être désinscrites. vu que maintenant les personnes plus à jour de cotisation sont archivées. On ne cherche pas à ajouter ces personnes à la liste des subscribed.
1 parent 551825a commit 337a5ab

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

sources/AppBundle/Mailchimp/Mailchimp.php

+10
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,16 @@ public function getAllSubscribedMembersAddresses($list)
4949
return $this->callMembersAddresses($list, 'subscribed');
5050
}
5151

52+
/**
53+
* @param string $list
54+
*
55+
* @return array
56+
*/
57+
public function getAllUnSubscribedMembersAddresses($list)
58+
{
59+
return $this->callMembersAddresses($list, 'unsubscribed');
60+
}
61+
5262
/**
5363
* @param string $list
5464
*

sources/AppBundle/Mailchimp/MailchimpMembersAutoListSynchronizer.php

+5-1
Original file line numberDiff line numberDiff line change
@@ -45,11 +45,15 @@ public function __construct(\AppBundle\Mailchimp\Mailchimp $mailchimp, UserRepos
4545
public function synchronize()
4646
{
4747
$subscribedEmailsOnMailchimp = array_map('strtolower', $this->mailchimp->getAllSubscribedMembersAddresses($this->listId));
48+
$unSubscribedEmailsOnMailchimp = array_map('strtolower', $this->mailchimp->getAllUnSubscribedMembersAddresses($this->listId));
4849
$cleanedEmailsOnMailchimp = array_map('strtolower', $this->mailchimp->getAllCleaneddMembersAddresses($this->listId));
4950
$subscribedEmailsOnWebsite = array_map('strtolower', $this->getSubscribedEmailsOnWebsite());
5051

5152
$addressesToArchive = array_diff($subscribedEmailsOnMailchimp, $subscribedEmailsOnWebsite);
52-
$addressesToSubscribe = array_diff($subscribedEmailsOnWebsite, $subscribedEmailsOnMailchimp);
53+
// Vu qu'on archive les personnes qui ne sont plus à jour de cotisation, les adresses unsubscribed sont seulemnt les personnes
54+
// qui ont optout. On ne peux techniquement pas les ajouter et fonctionnellelement il faudrait fournir les infos sur leur optin
55+
// on ne cherche donc pas à ajouter de nouveaux ces personnes dans les subscribers
56+
$addressesToSubscribe = array_diff($subscribedEmailsOnWebsite, $subscribedEmailsOnMailchimp, $unSubscribedEmailsOnMailchimp);
5357

5458
// Les adresses cleaned sont par exemple des hard bounces : on ne peux pas les passer en subscribred dans mailchimp
5559
// Il peuvent tout de même être des membres à jour de cotisation, on va ici éviter des erreurs lors de la synchro en les ignornant

0 commit comments

Comments
 (0)