Bug fix for PalyPal_Subscription - Patch

The following function when looking back through the versions used the isset() line to prevent the database from being hit with multiple requests for the same result. Since then, the function has been utilized in a loop where the request is changing the ID each time. However, the isset() will now just ignore that fact and return the same result all the time. Clearly, this is going to cause a problem and the module will not work correctly for anyone who uses more than one PayPal Subscription Role.

I have a fix for this below the following code as well. I have never done patches or CVS (this is my first time submitting a fix) so if someone could incorporate this into the versioning that would be great.

PREVIOUS CODE:
function _paypal_subscription_load($id) {
static $result;

if (!isset($result)) {
$result = db_fetch_object(db_query('SELECT pps.*, r.name AS role_name FROM {paypal_subscription} pps LEFT JOIN {role} r ON pps.role=r.rid WHERE pps.subscription_id=%d', $id));
_paypal_debug_var($result, 'Subscription details loaded for id='.$id);
}

return $result;
}

NEW CODE:
function _paypal_subscription_load($id) {
static $result;
static $prevID;

if ($id != $prevID) {
$result = db_fetch_object(db_query('SELECT pps.*, r.name AS role_name FROM {paypal_subscription} pps LEFT JOIN {role} r ON pps.role=r.rid WHERE pps.subscription_id=%d', $id));
$prevID = $id;
_paypal_debug_var($result, 'Subscription details loaded for id='.$id);
}
return $result;
}