=== modified file 'uc_roles/uc_roles.module'
--- uc_roles/uc_roles.module	2009-02-02 18:30:06 +0000
+++ uc_roles/uc_roles.module	2009-02-02 21:11:49 +0000
@@ -809,7 +809,7 @@
     'model'       => $form_state['values']['uc_roles_model'],
     'rid'         => $form_state['values']['uc_roles_role'],
     'duration'    => $form_state['values']['uc_roles_expire_relative_granularity'] != 'never' ? $form_state['values']['uc_roles_expire_relative_duration'] : NULL,
-    'granularity' => $form_state['values']['uc_roles_expire_relative_granularity'] != 'never' ? $form_state['values']['uc_roles_expire_relative_granularity'] : NULL,
+    'granularity' => $form_state['values']['uc_roles_expire_relative_granularity'],
     'by_quantity' => $form_state['values']['uc_roles_by_quantity'],
     'shippable'   => $form_state['values']['uc_roles_shippable'],
 
@@ -825,7 +825,7 @@
   }
   else {
     switch ($product_role['granularity']) {
-      case NULL:
+      case 'never':
         $description .= t('<strong>Expiration:</strong> never<br/>');
         break;
       case 'day':
@@ -1293,7 +1293,7 @@
 function _uc_roles_get_expiration($duration, $granularity, $start_time = NULL) {
 
   // Never expires?
-  if (!$granularity) {
+  if ($granularity == 'never') {
     return NULL;
   }