update the same data fields as we do for abandoned cart email

Enabling the following core magento functionality:

System > Sales > Dashboard > Aggregated data (beta)

Causes our Dashboard under Marketing automation not to load. Even though the feature in Magento is in Beta we should look into why this happens and resolve it.

_exportQuoteForWebsite
_exportQuoteForWebsiteInSingle

Sync setting will always show the data "Not enabled in account" for :

• Order Sync
• Quote Sync
• Wishlist Sync
• Review Sync
• Catalog Sync

We have to clean the importer table using the cleaner cronjob ddg_automation_cleaner.

This will include all imports older then a month.

According to Magento's system.log, the following error is produced
ERR (3): Warning: include(Dotdigitalgroup/Email/Helper/Review.php): failed to open stream: No such file or directory in /routeToMagentoWebSite/lib/Varien/Autoload.php on line 94

ERR (3): Warning: include(): Failed opening 'Dotdigitalgroup/Email/Helper/Review.php' for inclusion (include_path='/routeToMagentoWebSite/app/code/local:/routeToMagentoWebSite/app/code/community:/routeToMagentoWebSite/app/code/core:/routeToMagentoWebSite/lib:.:/opt/alt/php56/usr/share/pear:/opt/alt/php56/usr/share/php') in /routeToMagentoWebSite/lib/Varien/Autoload.php on line 94

I get error 500 after attempting to create new transactional emails or edit existing ones in the Magento admin System > Transactional Emails
This was working in v5.0.4.
Looking at the logs, this error gets thrown...
exception 'Mage_Core_Exception' with message 'Invalid block type: ddg_automation/system_email_template_edit' in /chroot/home/pexwhole/prelaunch.pexwholesale.co.uk/html/app/Mage.php:595

In configuration this selector box has a --Default Option-- row but others don't

There are certain fields which should be (and are marked as) available on the store view scope, but the section and/or group are marked as only show for website and show for default. If the parent is hidden, the child won't be seen. This is especially important with abandoned carts (<connector_lost_baskets>) as we were unable to select different abandoned cart trigger emails for our multilingual site.

Notice: Trying to get property of non-object
Dotdigitalgroup/Email/Model/Sync/Contact/Bulk.php on line 56

Contact identifier missing.
Remove loading model in loop.

Customers section config in Abandoned Carts config tab can be set on store view level but Abandoned Carts tab is only set to be visible on website level which make it impossible for Customers section config to be set on store view level.

Region api endpoint for OAUTH not matching the account info endpoint:
https://r1-api.dotmailer.com

This occurs in version 5.1.0. Is this a false positive?

This is making alterations to improve 2 things, adding priority to contacts syncing as this is the most valuable data to a merchant. And allowing multiple bulk syncs to occur at the same time to improve performance with large or multi website instances.

'main_table.column_name' not found error.
missing cases from _evaluate function.

When using ANY condition and creating multiline same attribute conditions a notice error is thrown which in turn skips the conditions on the collection

Do not archive CSV file if it is failed so it can be reset. Move archiveCSV to _checkImportStatus()

On a fresh install into a dotmailer account that already contains a suppression list can sometimes result in the email_importer being filled with subscriber_updated.

Reason being when a contact sync is processed we receive a fault file from dotmailer of contacts who are suppressed. We then update the core newsletter subscriber table that the contact is unsubscribed. As a result we trigger the subscriber save after event, which we observe resulting in a row being added to email_importer. These imports then fail as the contact is suppressed.

The observer on this event needs to be updated to ignore subscriber updates on unsubscribed contacts to stop this from occurring.

This is the warning:

Warning: chdir(): No such file or directory (errno 2) in /opt/magento/htdocs/lib/Varien/Io/File.php on line 497

The warning could be avoided with following change

Dotdigitalgroup_Email_Block_Basket

Instead of loading model in a loop, you can get product from already loaded $quoteItems. You can load all needed information this way:
$product = $quoteItem->getProduct();
$productUrl = $product->getProductUrl();

Stop logging single sync logs.

You should change this:

modman clone https://github.com/dotagency/dotmailer_magento.git

to this:

modman clone https://github.com/dotmailer/dotmailer-magento-extension.git

This event is not fired when a customer's account is created during the checkout process. As a result the observer for new customer enrolment automation is not fired and enrollment does not happen.

We should extend Mage_Sales_Model_Service_Quote and rewrite submitOrder function to fire our event or listen to sales_model_service_quote_submit_success , sales_model_service_quote_submit_before, sales_model_service_quote_submit_after events and check if quote is filled with new user data for registration and do our thing then.

Hi, I'm having the following error when running the subscriber sync manually...
a:5:{i:0;s:295:"SQLSTATE[42S02]: Base table or view not found: 1146 Table 'pexwhole_magento_staging.newsletter_subscriber' doesn't exist, query was: UPDATE p3x_email_contactAScLEFT JOINnewsletter_subscriberASsON c.customer_id = s.customer_id SETc.subscriber_status=s.subscriber_status";i:1;s:2709:"#0 /chroot/home/pexwhole/staging.pexwholesale.co.uk/html/lib/Varien/Db/Statement/Pdo/Mysql.php(110): Zend_Db_Statement_Pdo->_execute(Array)
It appears that it's looking up a default table name, whereas my installation uses prefixes... p3x_

I am getting this error when running on PHP 7.0.2.
Model/Apiconnector/Client.php::postAddressBookContactsImport

curl_setopt($ch, CURLOPT_SAFE_UPLOAD, false);

This function does not supports in PHP 7.

Since installing the extension a few days ago, I've seen over 40 of these error messages in the var/log/exception.log file.

If you need more information in order to fix this issue, please let me know.

Exception 'Zend_Db_Statement_Exception' with message 'SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '))' at line 1' and code '42000' in /var/www/example.com/htdocs/lib/Zend/Db/Statement/Pdo.php:264
Stack trace:

#0 /var/www/example.com/htdocs/lib/Varien/Db/Statement/Pdo/Mysql.php(110): Zend_Db_Statement_Pdo->_execute( Array(2) )
   #    Array
   #    (
   #        [0] => 1
   #        [1] => "2016-04-14 21:15:38"
   #    )
#1 /var/www/example.com/htdocs/app/code/core/Zend/Db/Statement.php(291): Varien_Db_Statement_Pdo_Mysql->_execute( Array(2) )
   #    Array
   #    (
   #        [0] => 1
   #        [1] => "2016-04-14 21:15:38"
   #    )
#2 /var/www/example.com/htdocs/lib/Zend/Db/Adapter/Abstract.php(479): Zend_Db_Statement->execute( Array(2) )
   #    Array
   #    (
   #        [0] => 1
   #        [1] => "2016-04-14 21:15:38"
   #    )
#3 /var/www/example.com/htdocs/lib/Zend/Db/Adapter/Pdo/Abstract.php(238): Zend_Db_Adapter_Abstract->query( "UPDATE `email_quote` SET `imported` = ?, `updated_at` = ?, `modified` = null WHERE (quote_id IN ())", Array(2) )
   #    Array
   #    (
   #        [0] => 1
   #        [1] => "2016-04-14 21:15:38"
   #    )
#4 /var/www/example.com/htdocs/lib/Varien/Db/Adapter/Pdo/Mysql.php(428): Zend_Db_Adapter_Pdo_Abstract->query( "UPDATE `email_quote` SET `imported` = ?, `updated_at` = ?, `modified` = null WHERE (quote_id IN ())", Array(2) )
   #    Array
   #    (
   #        [0] => 1
   #        [1] => "2016-04-14 21:15:38"
   #    )
#5 /var/www/example.com/htdocs/lib/Zend/Db/Adapter/Abstract.php(632): Varien_Db_Adapter_Pdo_Mysql->query( "UPDATE `email_quote` SET `imported` = ?, `updated_at` = ?, `modified` = null WHERE (quote_id IN ())", Array(2) )
   #    Array
   #    (
   #        [0] => 1
   #        [1] => "2016-04-14 21:15:38"
   #    )
#6 /var/www/example.com/htdocs/.modman/dotmailer-magento-extension/code/Dotdigitalgroup/Email/Model/Resource/Quote.php(64): Zend_Db_Adapter_Abstract->update( "email_quote", Array(3), "quote_id IN ()" )
   #    Array
   #    (
   #        [imported] => 1
   #        [updated_at] => "2016-04-14 21:15:38"
   #        [modified] => Object( Zend_Db_Expr )
   #    )
#7 /var/www/example.com/htdocs/.modman/dotmailer-magento-extension/code/Dotdigitalgroup/Email/Model/Quote.php(214): Dotdigitalgroup_Email_Model_Resource_Quote->setImported( Array() )
   #    Array()
#8 /var/www/example.com/htdocs/.modman/dotmailer-magento-extension/code/Dotdigitalgroup/Email/Model/Quote.php(92): Dotdigitalgroup_Email_Model_Quote->_exportQuoteForWebsiteInSingle( Object( Mage_Core_Model_Website ) )
#9 /var/www/example.com/htdocs/.modman/dotmailer-magento-extension/code/Dotdigitalgroup/Email/Model/Cron.php(100): Dotdigitalgroup_Email_Model_Quote->sync()
#10 /var/www/example.com/htdocs/.modman/dotmailer-magento-extension/code/Dotdigitalgroup/Email/Model/Cron.php(114): Dotdigitalgroup_Email_Model_Cron->quoteSync()
#11 [internal function]: Dotdigitalgroup_Email_Model_Cron->orderAndQuoteSync( Object( Aoe_Scheduler_Model_Schedule ) )
#12 /var/www/example.com/htdocs/.modman/Aoe_Scheduler/app/code/community/Aoe/Scheduler/Model/Schedule.php(207): call_user_func_array( Array(2), Array(1) )
   #    Array
   #    (
   #        [0] => Object( Dotdigitalgroup_Email_Model_Cron )
   #        [1] => "orderAndQuoteSync"
   #    )
   #    Array
   #    (
   #        [0] => Object( Aoe_Scheduler_Model_Schedule )
   #    )
#13 /var/www/example.com/htdocs/.modman/Aoe_Scheduler/app/code/community/Aoe/Scheduler/Model/Schedule.php(593): Aoe_Scheduler_Model_Schedule->runNow( true )
#14 /var/www/example.com/htdocs/.modman/Aoe_Scheduler/app/code/community/Aoe/Scheduler/Model/Observer.php(39): Aoe_Scheduler_Model_Schedule->process()
#15 /var/www/example.com/htdocs/app/code/core/Mage/Core/Model/App.php(1343): Aoe_Scheduler_Model_Observer->dispatch( Object( Varien_Event_Observer ) )
#16 /var/www/example.com/htdocs/app/code/core/Mage/Core/Model/App.php(1322): Mage_Core_Model_App->_callObserverMethod( Object( Aoe_Scheduler_Model_Observer ), "dispatch", Object( Varien_Event_Observer ) )
#17 /var/www/example.com/htdocs/app/Mage.php(448): Mage_Core_Model_App->dispatchEvent( "default", Array(4) )
   #    Array
   #    (
   #        [include_groups] => Array()
   #        [exclude_groups] => Array()
   #        [include_jobs] => Array()
   #        [exclude_jobs] => Array()
   #    )
#18 /var/www/example.com/htdocs/.modman/Aoe_Scheduler/shell/scheduler.php(404): Mage::dispatchEvent( "default", Array(4) )
   #    Array
   #    (
   #        [include_groups] => Array()
   #        [exclude_groups] => Array()
   #        [include_jobs] => Array()
   #        [exclude_jobs] => Array()
   #    )
#19 /var/www/example.com/htdocs/.modman/Aoe_Scheduler/shell/scheduler.php(37): Aoe_Scheduler_Shell_Scheduler->cronAction()
#20 /var/www/example.com/htdocs/.modman/Aoe_Scheduler/shell/scheduler.php(430): Aoe_Scheduler_Shell_Scheduler->run()

If error returned from account info api call then it means credentials are wrong in that case do not throw api endpoint exception. This will let user change credentials.

Automation enrolments can go into different enrolments if there are sent on the same time.

Pending automation are sorted by the status only.

Commented-out code detected, Multi-line function call not indented correctly, Expected 1 space after comma in function call

From options "Products Attributes In Order Sync" date/datetime should be sent as date to DM

send proper error message instead or string "Error". Also check if error

We have currently production issues since the IP Addresses seem to be outdated:

        <ip_restriction>
            <ip_addresses>94.143.111.2, 94.143.111.3</ip_addresses>
        </ip_restriction>

Could you please provide us with the new ip address list?

Customer class set data should contain key as well as value

Import status check for the import without the ID (import_id)

getContactsTransactionalDataImportByImportId - id is null and will fail for GET reqeust

The external dynamic links for both best sellers and most viewed if in a multi-website environment are not filtering by the appropriate website

Since the method uses the undefined order class attribute connector_id the REST API call always fails since ContactIdentifier is mandatory.
Following notice is generated:

Notice: Undefined property: Dotdigitalgroup_Email_Model_Connector_Order::$connector_id in /opt/magento/.modman/dotmailer-magento-extension/code/Dotdigitalgroup/Email/Model/Apiconnector/Client.php on line 670

This could be fixed by following patch. Please verify this - thanks!

Hello!

We have faced with performance issue, tied with using of Magento collection in Dotdigitalgroup_Email_Model_Adminhtml_Observer class.

Code below creates Collection based on big amount of entries that significantly affect overall performance of server CPU. This happens for all unlogged users (case when $customerId is empty).

protected function addContactsFromWebsiteSegments($customerId, $segmentIds, $websiteId){
...
$contact = Mage::getModel('ddg_automation/contact')->getCollection()
            ->addFieldToFilter('customer_id', $customerId)
            ->addFieldToFilter('website_id', $websiteId)
            ->getFirstItem();
...

Function addContactsFromWebsiteSegments() had been added at V4.0.0 release.

Best practice from Magento is to divide collection into pages (using method setPageSize()) or to limit number of selected rows.

So we have made changes below to solve performance issue:

$contact = Mage::getModel('ddg_automation/contact')->getCollection()
            ->addFieldToFilter('customer_id', $customerId)
            ->addFieldToFilter('website_id', $websiteId)
            ->setPageSize(1)
            ->getFirstItem();

Please let me know if it may somehow affects plugin's functionality.

Thanks,
Maxim.

Fatal error: Call to a member function getAccountInfo() on boolean in Dotdigitalgroup/Email/Helper/Data.php on line 1070

Above is thrown when api is disabled but installation beacon still tries to get api client and make api call but returned client is a boolean false since it is disabled,

Remove SMS functionallity from the module. This feature can be used in dotmailer itself

For abandoned carts try to load the quote with no store data.

The config value for apiuser pass is obsucre but the value checked is raw, this may show the api auth is invalid and disable.

Check if pass changed and validate according to the config encoding used.

When we create and attempt to import "Single Orders" this error is being displayed on a number of sites. My assumption is we are altering the JSON type/schema between bulk and single inserts.

JsonValueIncompatibleWithSchema. A value named 'custom' of type List already exists in this collection and new values must be of this type. ERROR_TRANSACTIONAL_DATA_VALIDATION_FAILED

Authorization link need to access the account region

https://r1-app.dotmailer.com/OAuth2/authorise.aspx
https://r2-app.dotmailer.com/OAuth2/authorise.aspx

Remove nosto recommendation.

Hello,

Quite minor issue, but is still filling up our logs with PHP warnings.

ERR (3): Warning: strlen() expects parameter 1 to be string, array given

Line 288

protected function _limitLength($value)
{
    if (strlen($value) > 250)
        $value = substr($value, 0, 250);
    return $value;
}

The function does not check if $value is an array.

If you install this extension there is a fatal error as the system.xml is looking for the module named "connector" but cannot find it. The error does not occur In the package available on magento connect!

Steps to reproduce

• Login as customer on frontend and leave review on a product
• Login to admin, find review, change to "Approved" and save review:
  
• You will see the following error in the error log:

2016-04-13T18:29:21+00:00 DEBUG (7): Exception message: Unrecognized method 'getFirstItem()'
Trace: #0 /var/www/html/.modman/dotmailer-magento-extension/code/Dotdigitalgroup/Email/Model/Customer/Observer.php(204): Zend_Db_Select->__call('getFirstItem', Array)
#1 /var/www/html/.modman/dotmailer-magento-extension/code/Dotdigitalgroup/Email/Model/Customer/Observer.php(204): Varien_Db_Select->getFirstItem()
#2 /var/www/html/app/code/core/Mage/Core/Model/App.php(1343): Dotdigitalgroup_Email_Model_Customer_Observer->reviewSaveAfter(Object(Varien_Event_Observer))
#3 /var/www/html/app/code/core/Mage/Core/Model/App.php(1322): Mage_Core_Model_App->_callObserverMethod(Object(Dotdigitalgroup_Email_Model_Customer_Observer), 'reviewSaveAfter', Object(Varien_Event_Observer))
#4 /var/www/html/app/Mage.php(448): Mage_Core_Model_App->dispatchEvent('review_save_aft...', Array)
#5 /var/www/html/app/code/core/Mage/Core/Model/Abstract.php(466): Mage::dispatchEvent('review_save_aft...', Array)
#6 /var/www/html/app/code/core/Mage/Core/Model/Abstract.php(319): Mage_Core_Model_Abstract->_afterSave()
#7 /var/www/html/app/code/core/Mage/Adminhtml/controllers/Catalog/Product/ReviewController.php(130): Mage_Core_Model_Abstract->save()
#8 /var/www/html/app/code/core/Mage/Core/Controller/Varien/Action.php(418): Mage_Adminhtml_Catalog_Product_ReviewController->saveAction()
#9 /var/www/html/app/code/core/Mage/Core/Controller/Varien/Router/Standard.php(254): Mage_Core_Controller_Varien_Action->dispatch('save')
#10 /var/www/html/app/code/core/Mage/Core/Controller/Varien/Front.php(172): Mage_Core_Controller_Varien_Router_Standard->match(Object(Mage_Core_Controller_Request_Http))
#11 /var/www/html/app/code/core/Mage/Core/Model/App.php(354): Mage_Core_Controller_Varien_Front->dispatch()
#12 /var/www/html/app/Mage.php(685): Mage_Core_Model_App->run(Array)
#13 /var/www/html/index.php(132): Mage::run('', 'store')
#14 {main}