Whats wrong with this query? Fresh eyes needed ;)

[Ajeh]

Note: this line ...

return $points['customers_shopping_points']; 

Might result in a better answer if you use:

return $points->fields['customers_shopping_points']; 

[Reesy]

Thanks to Ajeh for the pointer...
In the end my cola power cotton wool fueled head decided to check how another file does this and finally came up with the following ...

    if (zen_not_null(POINTS_AUTO_EXPIRES)){
    global $db;
      $points_query = "SELECT customers_shopping_points FROM " . TABLE_CUSTOMERS . " WHERE customers_id = '" . (int)$id . "' AND customers_points_expires > CURDATE() LIMIT 1";
    } else {
      $points_query = "SELECT customers_shopping_points FROM " . TABLE_CUSTOMERS . " WHERE customers_id = '" . (int)$id . "' LIMIT 1";
    }
    $points = $db->Execute("$points_query");

    return $points->fields['customers_shopping_points'];
  } 

[Ajeh]

NOTE: if this is in a function ...

The standard way to do this is to have:

function my_function_name() {
  global $whoever, $whoever2, $whoever_etc;
// code stuff here
} 

[Reesy]

Thank you Linda!