emartech / 3rd-party-integrations-magento-M2

Please note: this plugin is deprecated. we have a new version of the plugin- please contact emarsys support for more info
MIT License
0 stars 5 forks source link

Duplicates in emarasys, eventhough email is selected as a key. #122

Open eghanin opened 5 years ago

eghanin commented 5 years ago

Issue description: In some cases. the plugin will use a combined key ( email#site#store), even though email was selected as a key. The impact is that multiple recipient will be created with the same email address.

This patch fixes the inconsistency of using Email as a key. How to patch:

  1. copy the attached .patch file to app/code/Emarsys/Emarsys.
  2. unzip the file
  3. using your command line type : git apply customers_subscribers_bugfixes_from_113_to_112.patch File: customers_subscribers_bugfixes_from_113_to_112.patch.zip
haiacom commented 5 years ago

The Patch is causing empty etc/frontend/di.xml which is not an valid xml file so cause Magento failed to run

romastepa commented 5 years ago

The Patch is causing empty etc/frontend/di.xml which is not an valid xml file so cause Magento failed to run

this is git issue - it can not remove file. Plz, just remove etc/frontend/di.xml