| Multiple Bugs on v1.0.15 | |||
|---|---|---|---|
| harshit.jain | Posted: 2012-06-16 | ||
Joined: 2012-06-16
Posts: 2
|
I am using MageMonkey v1.0.15 Found many bugs: 1. When you subscribe customers via MassAction dropdown in customer grid (admin panel), the customer name sent is 'Guest'. This is because in function _mergeVars the customer isn't loaded because the customer isn't logged in nor the current_customer mage registry is set. To resolve this you need to put: if (!$customer->getId() && $object->getCustomerId()) {
$customer = Mage::getModel('customer/customer');
$customer->load($object->getCustomerId());
}before if (is_object($object))
in Observer.php <br /><br />
2. I have a multi-store setup. When customers are created from the admin panel and assigned to a website, the customer_entity table only logs the website_id and puts store_id as 0. This also carries forward to the newsletter_subscriber table where the store_id is again put as 0. This totally screws up the extension as the extension relies on the store_id field to sync to the correct list. To resolve this:<br />
a. I have built a very small extension that assigns the website's default store id when saving customers (if it already does not have a store id). This ensures that all customers have a store_id. This store_id automatically carries forward to newsletter_subscriber. Maybe this can be incorporated in the MageMonkey extension?<br />
b. You need to modify line in handleSubscriber() function in Observer.php to:
$listId = Mage::helper('monkey')->getDefaultList(($subscriber->getMcStoreId() ? $subscriber->getMcStoreId() : ($subscriber->getStoreId() ? $subscriber->getStoreId() : Mage::app()->getStore()->getId())));
If you do not want to do this then in system.xml you show allow configurations at website level and then send websiteId in handleSubscriber() function in Observer.php:
$listId = Mage::helper('monkey')->getDefaultList...
and modify function getDefaultList() function in Data.php to pick up configuration based on websiteId.
|
||
| Quote | |||
