Просмотр исходного кода

Merge pull request #859 from sharelatex/ho-mailchimp-change-email

always set status_if_new
Timothée Alby 8 лет назад
Родитель
Сommit
107e8ce633
1 измененных файлов с 3 добавлено и 2 удалено
  1. 3 2
      services/web/app/coffee/Features/Newsletter/NewsletterManager.coffee

+ 3 - 2
services/web/app/coffee/Features/Newsletter/NewsletterManager.coffee

@@ -59,16 +59,17 @@ hashEmail = (email)->
 
 
 buildOptions = (user, is_subscribed)->
 buildOptions = (user, is_subscribed)->
 	subscriber_hash = hashEmail(user.email)
 	subscriber_hash = hashEmail(user.email)
+	status = if is_subscribed then "subscribed" else "unsubscribed"
 	opts =
 	opts =
 		method: "PUT"
 		method: "PUT"
 		path: "/lists/#{Settings.mailchimp?.list_id}/members/#{subscriber_hash}"
 		path: "/lists/#{Settings.mailchimp?.list_id}/members/#{subscriber_hash}"
 		body:
 		body:
 			email_address:user.email
 			email_address:user.email
+			status_if_new: status
 				
 				
-	status = if is_subscribed then "subscribed" else "unsubscribed"
+	#only set status if we explictly want to set it
 	if is_subscribed?
 	if is_subscribed?
 		opts.body.status = status
 		opts.body.status = status
-		opts.body.status_if_new = status
 
 
 	if user._id?
 	if user._id?
 		opts.body.merge_fields = 
 		opts.body.merge_fields =