Super_Invoice - Check customer orders count (background check)

[sbbemn]

Hello all,

I am close but no cigar on checking the customers orders and if greater than 1, then no action.

Trouble I am having is getting the history of orders. Here is the code I have so far...

       $customers_orders = $db->Execute("select o.orders_id, o.date_purchased
                                          from " . TABLE_ORDERS . " o
                                          where customers_id = '" . (int)$order_check->customers_id . "'");
       $no_history = ($customers_orders->recordCount() = 1); 

I am getting " Fatal error: Can't use method return value in write context " referring to the last line above. Any guidance would be appreciated!

[Ajeh]

recordCount() should be RecordCount() and = should be ==

But what are you really trying to test? If there is only 1 Order? Or if there is at least 1 Order?

[sbbemn]

Thanks - silly little things overlooked! I will work on that right now.

Basically checking to see if it is the customer's 1st (and only) order. That way we know they are a 1st time buyer and flag them if the order is over $x.

[sbbemn]

Works great! Thanks Ajeh
here is what I did to check customer's orders and if only 1 order and 1st time buyer...

       $customers_orders = $db->Execute("select o.orders_id, o.date_purchased
                                          from " . TABLE_ORDERS . " o
                                          where customers_id = '" . (int)$order_check->customers_id . "'");
       $no_history = ($customers_orders->RecordCount() == 1); 

Then on the super_invoice I used $no_history to flag the customer. Use any method or special heading of your choice.

[Ajeh]

Thanks for the update that this is working for you and the explanation of its use for you ...

[sbbemn]

I came up with another feature though that is of use - order age.

If the customer orders list is newer than 35 days. This is within credit card billing cycle for fraud and chargebacks.

I tested this on a dev site - seemed to work. Hard to test completely on a dev site based on the others parameter checks I have.

       $customers_orders = $db->Execute("select o.orders_id, o.date_purchased
                                          from " . TABLE_ORDERS . " o
                                          where customers_id = '" . (int)$order_check->customers_id . "'");
       
       $days35fromnow = strtotime("+35 days", strtotime("<date-from-db>"));
       $no_history = ($customers_orders->date_purchased >= $days35fromnow); 

[Ajeh]

At a glance ... you have an error:

$no_history = ($customers_orders->date_purchased >= $days35fromnow);

This should be:

$no_history = ($customers_orders->fields['date_purchased'] >= $days35fromnow);

[sbbemn]

It seems to only grab the current order date_purchased rather than the whole potential list of order based on customer_id

Shouldn't this code select ALL o.orders_id from the given customer?

where customers_id = '" . (int)$order_check->customers_id . "'" 

This should grab the customer_id based on the order, but not the date_purchased from that order.

[Ajeh]

You grab all orders but then only are getting data from the first one of whatever is grabbed ...

I am not sure how you are using the results, but if you are trying to list something per order, you might do the SELECT with the current order_id and customer_id ...

[sbbemn]

Well, come to find out the "look at all orders within 35 days" is the best way to go in our case, but isn't working correctly with the code above.
What is important is if their list of orders spans MORE than 35 days. Having just more than one order doesn't cut it.

So what I was trying to do is select all orders based on the customer ID with the ID taken from the current orderID from the invoice. Then take date_purchased of all orders produced to see if any order is older than 35 days old from now. #4 below.

If no, then take action

Here's the whole thing and what we use it for for anyone's use....
Check the following and if it meets requirements, display wording at the top of the invoice. We use this one among many other personal features not listed...
1. Checks match of billing/shipping address line 1
2. If order is over a $ threshold of choice
3. Order is not a certain payment module
4. Customer meets requirements of no_history (orders are greater than 35 days old)

 $customers_orders = $db->Execute("select o.orders_id, o.date_purchased
                                          from " . TABLE_ORDERS . " o
                                          where customers_id = '" . (int)$order_check->customers_id . "'");
       $days35fromnow = strtotime("+35 days", strtotime("<date-from-db>"));
       $no_history = ($customers_orders->fields['date_purchased'] < $days35fromnow); 

Further down to display wording on the invoice...

<?php
$headings_array = array();

if (($order_check->fields['delivery_street_address'] != $order_check->fields['billing_street_address'] and $order_check->fields['delivery_street_address'] != '' && 
    $order_check->fields['order_total'] > 100 && 
    array_search($order_check->fields['payment_module_code'], array('cc')) == FALSE &&
    $no_history)) {
        $headings_array[] = "YOUR WORDING HERE";
}

for ($i=0; $i < sizeof($headings_array); $i++) {
  echo $headings_array[$i];
  if($i+1 < sizeof($headings_array)) {
    echo "<br />";
  }
}
?>