Ok, try this. I've changed the loadObjectList to just loadResult, assuming that they only have one subscription?


if ($user->id > 0 ) {
  $db = & JFactory::getDBO();

  $query = "SELEC" . "T b.expiry_date
    from #__cbsubs_subscriptions as b
    left join #__comprofiler as a on a.user_id = b.user_id
    where b.user_id=" . (int)$user->id;

  $db->setQuery( $query );
  $result = $db->loadResult(); // prob in form 2010-08-03 00:41:21

  $target_ts = strtotime($result);
  $days_to_go = (int) (($target_ts - time()) / (24*60*60));

  if ($days_to_go < 30) {
    session_start();
    $instance_name = "cbsubs_expiring";
    if (!isset($_SESSION[$instance_name]) ||
      $_SESSION[$instance_name] == false) {

      $_SESSION[$instance_name] = true;
      $target_date = date("d/m/Y",$target_ts);
      $message = "Your membership expires on $target_date! Click here to renew."; // add a hyperlink
      echo $message;
    }
  }
}


In this case we're just displaying some text in a module, rather than MetaMod pulling in another module. But you could still pull in another module by returning a module id after displaying the message.

I haven't tested this but it looks like you can code PHP, so at least this will get you started.

Best regards,
Stephen

Yes, when you use loadResult() then it will be a string. The original example returned an array. In the example I am assuming that there's only one subscription, so a single returned string is what I would expect.

Cheers,
Stephen