JSON & TrustPilot

[delia]

TrustPilot is a reviews site that requires, either 1) a BCC email of the sale or 2) JSON coding of a similar email.

I have no idea why you can't just add the additional email address in admin but they claim it has to be a BCC, not a separate email copy.

Looking into this I know that what should be done is to use an observer to trigger the addition of a BCC line when an order is placed. That's just not something I've ever stepped into.

This post steps into it but it dates from 2008.##https://www.zen-cart.com/showthread....il-to-Customer

So if what they are saying is true and all we need is an additional BCC added to the email, this actually sounds pretty simple if one knows how to do it. So the question becomes, is it as simple as it now appears?##

If it is, perhaps someone can direct me to examples of this type of thing in perhaps a plugin or give me some solid hints as to how to alter this provided code from 2008:

class sendBCCofAdminEmails extends base {

 /**  constructor method !
   *
   * Attach observer class to the global $zco_notifier and watch for a single notifier event.
   */
  function sendBCCofAdminEmails() {
    $this->attach($this, array('NOTIFY_EMAIL_BEFORE_PROCESS_ATTACHMENTS'));
  }


 /**   Actual Method that does the desired activity
   *
   * Called by observed class when any of the notifiable events occur
   *
   * @param object $class
   * @param string $eventID
   */
  function update(&$class, $eventID, $paramsArray = array()) {
    global $mail, $module;
    if ($paramsArray['module'] == 'direct_email') $mail->AddBCC(STORE_NAME, STORE_OWNER_EMAIL_ADDRESS);
  }
} 

That is adding a bcc if one is sending email through the admin. So what would the triggering code be to add to an order? Or am I off track here?

[carlwhat]

delia,
i will go on the assumption that you can get your observer to load. in v157, the auto loading observer function is pretty cool; but you are running v156 which means no auto loading of your observer.

just in case, here is the link on getting the observer to load:

https://docs.zen-cart.com/dev/code/n...observer-class

after that, this is how i would code it.

<?php

    class sendBCCofAdminEmails extends base
    {
        public function __construct()
        {
            $this->attach($this, [
                'NOTIFY_EMAIL_BEFORE_PROCESS_ATTACHMENTS',
            ]);
        }

        public function update(&$class, $eventID, &$p1, &$p2, &$p3, &$p4, &$p5, &$p6, &$p7, &$p8)
        {
            switch ($eventID) {
                case 'NOTIFY_EMAIL_BEFORE_PROCESS_ATTACHMENTS':
                    if ((strpos($p2->subject, 'Order Confirmation') !== false) && (strpos($p2->subject, 'NEW ORDER') === false)) {
                        $p2->addBCC('trustpilot at whatever.com');
                    }
                    break;
            }
        }
    }

obviously one can change the conditional as necessary. but i'm pretty confident this works.

hope that helps!

[delia]

oh man, that's perfect. But the site is 1.5.6. Is there much difference between the two in this case?

[carlwhat]

create a file named as such:

includes/auto_loaders/config.delia_mods.php

in that file, put:

<?php
    $autoLoadConfig[202][] = [
        'autoType' => 'class',
        'loadFile' => 'observers/sendBCCofAdminEmails.php',
        'classPath' => DIR_WS_CLASSES
    ];

         $autoLoadConfig[202][] = [
        'autoType' => 'classInstantiate',
        'className' => 'sendBCCofAdminEmails',
        'objectName' => 'sendBCCofAdminEmails'
    ];

make sure the previous code is here:

includes/classes/observers/sendBCCofAdminEmails.php

getting stuff to load can be tricky.

best.

[delia]

thank you so much!!

[carlwhat]

thank you so much!!

[delia]

after creating numerous errors, the site is back to working. But no extra bbc email was sent/received. Since there was no errors was the class actually loaded? I assumed so but then realized I actually don't know. Definitely didn't work.

[carlwhat]

after creating numerous errors, the site is back to working. But no extra bbc email was sent/received. Since there was no errors was the class actually loaded? I assumed so but then realized I actually don't know. Definitely didn't work.

yeah, as i stated the loading can be tricky.

i have added a couple of trigger_error statements which should result in debug logs getting generated. we may be getting the observer loaded, but then may not make it inside the conditional if the following is overridden (or if the language is not english):

includes/languages/english/checkout_process.php:9: define('EMAIL_TEXT_SUBJECT', 'Order Confirmation');

the bcc will only work for an order confirmation to the customer.

so try adding the trigger_error statements and see if any debug logs get generated:

<?php

    class sendBCCofAdminEmails extends base
    {
        public function __construct()
        {
            $this->attach($this, [
                'NOTIFY_EMAIL_BEFORE_PROCESS_ATTACHMENTS',
            ]);
        }

        public function update(&$class, $eventID, &$p1, &$p2, &$p3, &$p4, &$p5, &$p6, &$p7, &$p8)
        {
            switch ($eventID) {
                case 'NOTIFY_EMAIL_BEFORE_PROCESS_ATTACHMENTS':
                    trigger_error('the observer got loaded!');
                    if ((strpos($p2->subject, 'Order Confirmation') !== false) && (strpos($p2->subject, 'NEW ORDER') === false)) {
                        trigger_error('we are inside the conditional!');
                        $p2->addBCC('trustpilot at whatever.com');
                    }
                    break;
            }
        }
    }

best.

[delia]

and it says the class got loaded so that question is answered. But still no email.

Now to add in to that, this site will not send copy of order emails to multiple recipients, only to the first one. I saw an old post where DrByte says that's an indication of some sort of corruption? As this is a clean install on an old database, that can't be though so many things have gone wrong with site that I felt obligation to do a clean install and rebuild. I wouldn't think that would create this present problem but just in case, throwing it out there.

[carlwhat]

what about this?

trigger_error('we are inside the conditional!');

does it say we are inside the conditional?

you can now change the first trigger error to this:

trigger_error('Subject: ' . $p2->subject);

if the class is loaded, then we need to get inside the conditional.

the database and the rest of that stuff have nothing to do with existing troubleshooting. either we get inside the conditional or not. if we get in and no bcc gets generated we then need to look at mail function.

best.