News:

Looking for documentation? Take a look on our wiki

Main Menu

PayPal error email notification to over 200 customers

Started by nastal, November 16, 2015, 17:26:36 PM

Previous topic - Next topic

nastal

Our virtuemart store sent following email with error notification to over 200 customer:

Quote
Hello,
An error with your payment method paypal on your store {storename}. The details are logged in the file paypal.1.log.php
. The VirtueMart Team

Most unpleasant thing is what our customers shouldn't receive such emails, because no one of them are store admin or vendor. Error in paypal.1.log.php is about some paypal IPN connection, but it's not so important in this case, as unexpected emails to our customers.

I've checked vmpsplugin.php and sendEmailToVendorAndAdmins function, line 670:

$query = 'SELECT name, email, sendEmail FROM #__users WHERE sendEmail=1';

so I believe, this line is a point, why notification have been sent not only to admins, but also to customers.

I've changed temporarily to
$query = 'SELECT name, email, sendEmail FROM #__users WHERE id=108'; //108 is admin id

Please, check.

Our system --
PHP Version:           5.3.27
Joomla! Version:   Joomla! 3.3.6 Stable [ Ember ] 01-October-2014 02:00 GMT
VM version:               3.0.2

AH

You have completely outdated versions of VM and Joomla

Both are insecure and should be updated.

Your php version is also very old and insecure.

Regards
A

Joomla 4.4.5
php 8.1

nastal

QuoteYou have completely outdated versions of VM and Joomla

Thanks, I know. But I've checked this function in last version of VM, it remains the same.

GJC Web Design

Quote$query = 'SELECT name, email, sendEmail FROM #__users WHERE sendEmail=1';

but in Joomla speak "sendEmail" means send admin emails....  Receive System Emails...
GJC Web Design
VirtueMart and Joomla Developers - php developers https://www.gjcwebdesign.com
VM4 AusPost Shipping Plugin - e-go Shipping Plugin - VM4 Postcode Shipping Plugin - Radius Shipping Plugin - VM4 NZ Post Shipping Plugin - AusPost Estimator
Samport Payment Plugin - EcomMerchant Payment Plugin - ccBill payment Plugin
VM2 Product Lock Extension - VM2 Preconfig Adresses Extension - TaxCloud USA Taxes Plugin - Virtuemart  Product Review Component
https://extensions.joomla.org/profile/profile/details/67210
Contact for any VirtueMart or Joomla development & customisation

nastal

Thanks GJC,
I thought this option about all emails sent by system  :D
Ok, I will update this option in database user table.
Please, close this topic.

Milbo

I wonder how it happens that your users have sendEmail 1
Should I fix your bug, please support the VirtueMart project and become a member
______________________________________
Extensions approved by the core team: http://extensions.virtuemart.net/