The message should be adjusted on role grant upon purchase. If the role is set to never expire, the message appears as:
"counselor-bob has been granted the assessment access role. It will expire on 12/31/1969 - 17:00."
Perhaps a conditional should be put in there to make a more appropriate message for non-expiring roles.
The code in question is line 1204 of uc_roles.module, in function uc_roles_grant.
Suggested change:
Index: /ubercart/uc_roles/uc_roles.module
===================================================================
diff -u -r1.2 uc_roles.module
--- ubercart/uc_roles/uc_roles.module 29 Oct 2008 18:42:06 -0000 1.2
+++ ubercart/uc_roles/uc_roles.module 31 Oct 2008 21:11:28 -0000
@@ -1195,13 +1195,14 @@
// If the role expires, keep a record.
if (!is_null($timestamp)) {
db_query("INSERT INTO {uc_roles_expirations} (uid, rid, expiration) VALUES (%d, %d, %d)", $user->uid, $rid, $timestamp);
+ $expiration_message = t('It will expire on %date.', array('%date' => format_date($timestamp, 'small')));
}
// Flush visible menu items, since our permissions could've changed.
_uc_roles_flush_menu_cache($user);
$role_name = db_result(db_query('SELECT name FROM {role} WHERE rid = %d', $rid));
- drupal_set_message(t('%user has been granted the %role role. It will expire on %date.', array('%user' => $user->name, '%role' => $role_name, '%date' => format_date($timestamp, 'small'))));
+ drupal_set_message(t('%user has been granted the %role role.', array('%user' => $user->name, '%role' => $role_name, '%date' => format_date($timestamp, 'small'))).' '.$expiration_message);
}
/**
