Notifier help please

[lat9]

What's it supposed to do? Is there a reason why you're loading your class so early in the start-up? Code point 10 is pretty early.

P.S. Just for grins-and-giggles, try changing changing Array to array in your attach call.

[slobadog]

You are both right, thank you, it was late last night. I have it basically working, it's a way to 'activate' a customers 12 month site membership upon payment. Just a little more help to extract the payment method before allowing the update to occur would be great. Thanks.

<?php
// /includes/classes/observers/class.membersUpdate.php
//
// Observer class to activate members upon payment
//
//    see also /includes/auto_loaders/config.membersUpdate.php

class membersUpdate extends base {

    // CONSTRUCTOR METHOD
    function __construct() {
    $this->attach($this, array('NOTIFY_ORDER_AFTER_SEND_ORDER_EMAIL'));
  }
    
    // UPDATE METHOD
    function update(&$class, $eventID, $paramsArray = array()) {
    	global $db;
    	if (($_SESSION['cart']->in_cart('products_id', '1')) && ($_SESSION['payment'] != 'Cheque/Money Order')) {
        $db->Execute("UPDATE " . TABLE_CUSTOMERS ." SET customers_membership = '1' WHERE customers_id = '" . (int)$_SESSION['customer_id'] . "'");
		}
	}
}

?>

[slobadog]

Got that sorted by using the payment_module_code field instead of payment_method. Now I don't seem to be able to filter by product ID. If I set productID) == FALSE, it works, but the ID is definitely '1'.

<?php
// /includes/classes/observers/class.membersUpdate.php
//
// Observer class to activate members upon payment
//
//    see also /includes/auto_loaders/config.membersUpdate.php

class membersUpdate extends base {
	var $productID = '1';

    // CONSTRUCTOR METHOD
    function __construct() {
    $this->attach($this, array('NOTIFY_ORDER_AFTER_SEND_ORDER_EMAIL'));
  }
    
    // UPDATE METHOD
    function update(&$class, $eventID, $paramsArray = array()) {
    	global $db;
    	if (($_SESSION['cart']->in_cart($this->productID) == TRUE) && ($_SESSION['payment'] != 'moneyorder')) {
        $db->Execute("UPDATE " . TABLE_CUSTOMERS ." SET customers_membership = '1' WHERE customers_id = '" . (int)$_SESSION['customer_id'] . "'");
		}
	}
}

?>

[slobadog]

Anyone got any clues? I've done some experimenting but can't get it to work unless in_cart is set to false, I'm finding it a little strange. There is only one product and it is ID 1. I've looked up the function and used many different types of syntax but still it only works if set to FALSE, as below:

<?php
// /includes/classes/observers/class.membersUpdate.php
//
// Observer class to activate members upon payment
//
//    see also /includes/auto_loaders/config.membersUpdate.php

class membersUpdate extends base {
	var $prod_id = 1;

    // CONSTRUCTOR METHOD
    function __construct() {
    $this->attach($this, array('NOTIFY_ORDER_AFTER_SEND_ORDER_EMAIL'));
  }
    
    // UPDATE METHOD
    function update(&$class, $eventID, $paramsArray = array()) {
    	global $db;
    	if (($_SESSION['cart']->in_cart($prod_id) == FALSE) && ($_SESSION['payment'] != 'moneyorder')) {
			
        $db->Execute("UPDATE " . TABLE_CUSTOMERS ." SET customers_membership = '1' WHERE customers_id = '" . (int)$_SESSION['customer_id'] . "'");
		}
	}
}

?>

[DrByte]

In your last post, $prod_id is null, not declared (because you're using it as a local-scope variable, not the $this-> class variable), and so will never be found by in_cart, thus FALSE is an accurate response.

[slobadog]

In your last post, $prod_id is null, not declared (because you're using it as a local-scope variable, not the $this-> class variable), and so will never be found by in_cart, thus FALSE is an accurate response.

I've tried

if (($_SESSION['cart']->in_cart($this->prod_id) == TRUE)

and even

if (($_SESSION['cart']->in_cart(1) == TRUE)

But that didn't work either. I've tried by placing 51 experimental orders so far, I'm sure it's a simple fix!

[slobadog]

Could it be that the products_id is actually the products_prid as I am using attributes for the different Membership levels? Would working zen_get_prid() into it somewhere fix the problem? Is there another way I could check that the products_id is indeed '1' before updating the customer table?

A little help? Please?

[DrByte]

Could it be that the products_id is actually the products_prid as I am using attributes for the different Membership levels? Would working zen_get_prid() into it somewhere fix the problem? Is there another way I could check that the products_id is indeed '1' before updating the customer table?

A little help? Please?

Ya, you didn't mention the attributes bit before.

Ajeh posted your answer in another thread asking the same question: http://www.zen-cart.com/showthread.p...92#post1184392 :

$_SESSION['cart']->in_cart_check('products_id', $this->prod_id)

[slobadog]

Ah ha! That did the trick, thanks so much. I had seen that thread by Ajeh but wasn't considering the attributes thing myself at the time. You've solved a 3 day mystery for me, best I shout you a cappuccino!