Back In Stock Notifications

[conor]

Hi,

I have had occasional errors with this module because the customer managed to get some invalid data or no data in...

That should be impossible with a correctly installed version of the module. It checks the format of the e-mail address so only valid e-mail addresses can be stored.

The only situation in which this could possibly happen is if the customer subscribed to the notification has deleted their account, in which case after the notification is sent out the e-mail to them should fail and the record be deleted.

All the best..

Conor
ceon

[nagelkruid]

Hi,
The only situation in which this could possibly happen is if the customer subscribed to the notification has deleted their account, in which case after the notification is sent out the e-mail to them should fail and the record be deleted.

All the best..

Conor
ceon

Hi Conor,

I think that's exactly what happened as the name that was left is one that i use for testing a lot and that is about the only account that gets deleted once in a while.

However, the record doesn't get deleted, it results in the error described previously. No problem tough, shouldn't happen in any "normal" situation.

Thanks!
Jeroen

[Gerjan]

Hi Conner, I have exactly the same error. All mail functions work fine, except sending the actuall notification that the product is back in stock I have a complete new/fresh zencart installation.

I will try a fresh install tonight and let you know.

[torvista]

In reference to the above post#475

I found that the solution for full language dependence in email templates was by replacing...

What EXACTLY was not working?

I have duplicated the templates into the /es folder below the /email root and in my testing all emails are sent in the same language as that selected at the moment of use, correctly.

(The only changes I have made have been to replace the hard-coded "Dear" and "Link" with defines, otherwise it has worked correctly.)

[jgreene]

Hopefully useful -

We briefly used "Product Notifications" before realizing "Back In Stock Notifications" existed.

After installation we wanted to add the 2,500 customers who signed up for under Product Notifications into the BISN table. Here is the SQL I used- hopefully it is of use to anyone else needing to do the same. (no warrantees, guarantees, etc).

Note that we use zen_ before table names- remove that if you don't. It is assumed that all Product Notification customers have a customer_id (login) since they aren't allowed to signup otherwise...


INSERT into `zen_back_in_stock_notification_subscriptions`
(
`product_id`,
`customer_id`,
`name`,
`email_address`,
`date_subscribed`
)
SELECT `products_id`, pn.`customers_id`, CONCAT(`customers_firstname`,' ',`customers_lastname`), `customers_email_address`, `date_added`
FROM `zen_products_notifications` pn, `zen_customers` c
where pn.customers_id = c.customers_id


If there is anything wrong with my SQL, or doing this was a Bad Thing to have done, do please let me know!

edit: Just realized you don't actually need the email for registered customers- it is null for rows with a customer_id, so you could delete that part...

[conor]

Hi,

If there is anything wrong with my SQL, or doing this was a Bad Thing to have done, do please let me know!

No, that's a good idea, and it looks like it (has) will work just fine.

Working on a small update to the module today, hopefully release it soon, so look out for that.

All the best...

Conor
ceon