Skip to content

Commit

Permalink
Merge pull request #1868 from mailchimp/Issue1866-2.3.6-p1
Browse files Browse the repository at this point in the history
closes #1866 for magento 2.3.6-p1
  • Loading branch information
gonzaloebiz authored Feb 23, 2024
2 parents 409f97d + c6d2fc0 commit 29db1f9
Show file tree
Hide file tree
Showing 7 changed files with 82 additions and 5 deletions.
1 change: 1 addition & 0 deletions Helper/Data.php
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ class Data extends \Magento\Framework\App\Helper\AbstractHelper
const XML_MAGENTO_MAIL = 'mailchimp/general/magentoemail';
const XML_SEND_PROMO = 'mailchimp/ecommerce/send_promo';
const XML_INCLUDING_TAXES = 'mailchimp/ecommerce/including_taxes';
const XML_CAMPAIGN_ACTION = 'mailchimp/ecommerce/campaign_action';
const XML_POPUP_FORM = 'mailchimp/general/popup_form';
const XML_POPUP_URL = 'mailchimp/general/popup_url';
const XML_CLEAN_ERROR_MONTHS = 'mailchimp/ecommerce/clean_errors_months';
Expand Down
46 changes: 42 additions & 4 deletions Model/Api/Order.php
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,7 @@ class Order
protected $_counter;

protected $_batchId;
protected $modifiedOrder = false;

/**
* @param \Ebizmarts\MailChimp\Helper\Data $helper
Expand Down Expand Up @@ -161,6 +162,7 @@ protected function _getModifiedOrders($magentoStoreId)
\Ebizmarts\MailChimp\Helper\Data::XML_MAILCHIMP_STORE,
$magentoStoreId
);
$isSynced = $this->_helper->getConfigValue(\Ebizmarts\MailChimp\Helper\Data::XML_PATH_IS_SYNC, $magentoStoreId);
$modifiedOrders = $this->_getCollection();
// select orders for the current Magento store id
$modifiedOrders->addFieldToFilter('store_id', ['eq' => $magentoStoreId]);
Expand Down Expand Up @@ -202,7 +204,11 @@ protected function _getModifiedOrders($magentoStoreId)
}
}

$orderJson = $this->generatePOSTPayload($order, $mailchimpStoreId, $magentoStoreId, true);
$orderJson = $this->generatePOSTPayload($order, $mailchimpStoreId, $magentoStoreId, true, $isSynced);
if ($this->modifiedOrder) {
$order->save();
$this->modifiedOrder = false;
}
if ($orderJson!==false) {
if (!empty($orderJson)) {
$this->_helper->modifyCounter(\Ebizmarts\MailChimp\Helper\Data::ORD_MOD);
Expand Down Expand Up @@ -241,6 +247,7 @@ protected function _getNewOrders($magentoStoreId)
\Ebizmarts\MailChimp\Helper\Data::XML_MAILCHIMP_STORE,
$magentoStoreId
);
$isSynced = $this->_helper->getConfigValue(\Ebizmarts\MailChimp\Helper\Data::XML_PATH_IS_SYNC, $magentoStoreId);
$newOrders = $this->_getCollection();
// select carts for the current Magento store id
$newOrders->addFieldToFilter('store_id', ['eq' => $magentoStoreId]);
Expand Down Expand Up @@ -283,7 +290,11 @@ protected function _getNewOrders($magentoStoreId)
$this->_counter++;
}
}
$orderJson = $this->generatePOSTPayload($order, $mailchimpStoreId, $magentoStoreId);
$orderJson = $this->generatePOSTPayload($order, $mailchimpStoreId, $magentoStoreId,false,$isSynced);
if ($this->modifiedOrder) {
$order->save();
$this->modifiedOrder = false;
}
if ($orderJson!==false) {
if (!empty($orderJson)) {
$this->_helper->modifyCounter(\Ebizmarts\MailChimp\Helper\Data::ORD_NEW);
Expand Down Expand Up @@ -327,12 +338,20 @@ protected function generatePOSTPayload(
\Magento\Sales\Model\Order $order,
$mailchimpStoreId,
$magentoStoreId,
$isModifiedOrder = false
$isModifiedOrder,
$isSynced
) {
$data = [];
$data['id'] = $order->getIncrementId();
if ($order->getMailchimpCampaignId()) {
$data['campaign_id'] = $order->getMailchimpCampaignId();
} elseif ($isSynced) {
if ($campaignId = $this->getCampaign($magentoStoreId, $order->getCustomerEmail())) {
$data['campaign_id'] = $campaignId;
$order->setMailchimpCampaignId($campaignId);
$order->setMailchimpFlag(1);
$this->modifiedOrder = true;
}
}

if ($order->getMailchimpLandingPage()) {
Expand Down Expand Up @@ -737,7 +756,26 @@ protected function _getPromoData(\Magento\Sales\Model\Order $order)
}
return $promo;
}

protected function getCampaign($store, $email)
{
$campaign_id = null;
$api = $this->_helper->getApi($store);
$actions = $this->_helper->getConfigValue(\Ebizmarts\MailChimp\Helper\Data::XML_CAMPAIGN_ACTION, $store);
try {
$activity = $api->lists->members->memberActivity->get($this->_helper->getDefaultList($store), md5($email), null, null, $actions);
if ($activity) {
foreach ($activity['activity'] as $act) {
if (key_exists('action', $act) && key_exists('campaign_id', $act) && $act['campaign_id']) {
$campaign_id = $act['campaign_id'];
break;
}
}
}
} catch (\Mailchimp_Error $e) {
$this->_helper->log("No activity for $email");
}
return $campaign_id;
}
protected function _updateOrder($storeId, $entityId, $sync_delta = null, $sync_error = null, $sync_modified = null)
{
if (!empty($sync_error)) {
Expand Down
16 changes: 16 additions & 0 deletions Model/Config/Source/CampaignAction.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
<?php

namespace Ebizmarts\MailChimp\Model\Config\Source;

class CampaignAction implements \Magento\Framework\Option\ArrayInterface
{
private $actions = [
['value' => 'sent', 'label' => 'Sent'],
['value' => 'open', 'label' => 'Open'],
['value' => 'click', 'label' => 'Click']
];
public function toOptionArray()
{
return $this->actions;
}
}
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
},
"require" : {
"magento/framework": "102.0.6-p1||^102.0.7",
"ebizmarts/mailchimp-lib": ">=3.0.37",
"ebizmarts/mailchimp-lib": ">=3.0.38",
"ext-json": "*"
}
}
10 changes: 10 additions & 0 deletions etc/adminhtml/system.xml
Original file line number Diff line number Diff line change
Expand Up @@ -229,6 +229,16 @@
<field id="*/*/active">1</field>
</depends>
</field>
<field id="campaign_action" translate="label" type="multiselect" sortOrder="155" showInDefault="1" showInWebsite="1" showInStore="1">
<label>Campaign Attribution Action</label>
<source_model>Ebizmarts\MailChimp\Model\Config\Source\CampaignAction</source_model>
<can_be_empty>1</can_be_empty>
<size>3</size>
<depends>
<field id="*/*/active">1</field>
</depends>
<comment>Order attribution based on customer actions</comment>
</field>
<field id="reset_errors_retry" translate="button_label" type="button" sortOrder="160" showInDefault="1" showInWebsite="1" showInStore="1">
<button_label>Reset Errors and retry</button_label>
<frontend_model>Ebizmarts\MailChimp\Block\Adminhtml\System\Config\ResetErrors</frontend_model>
Expand Down
11 changes: 11 additions & 0 deletions etc/config.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
<?xml version="1.0"?>
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:noNamespaceSchemaLocation="urn:magento:module:Magento_Store:etc/config.xsd">
<default>
<mailchimp>
<ecommerce>
<campaign_action>sent,open,click</campaign_action>
</ecommerce>
</mailchimp>
</default>
</config>
1 change: 1 addition & 0 deletions view/adminhtml/web/js/configapikey.js
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ define(
if (ecommerceEnabled == 0 && abandonedCartEnabled == 1) {
self._changeAbandonedCart();
}
$('#mailchimp_ecommerce_campaign_action').attr('size',3);
},
_changeEcommerce: function () {
var self = this;
Expand Down

0 comments on commit 29db1f9

Please sign in to comment.