--- uc_donation.module	2008-11-07 14:10:58.000000000 -0600
+++ uc_donation_new.module	2008-11-12 01:13:30.000000000 -0600
@@ -476,15 +476,16 @@ function uc_donation_cart_display($item)
   $element['nid'] = array('#type' => 'value', '#value' => $node->nid);
   $element['module'] = array('#type' => 'value', '#value' => 'uc_donation');
   $element['remove'] = array('#type' => 'checkbox');
-  $op_names = '';
-  if (module_exists('uc_attribute')){
-    $op_names = "<ul class=\"cart-options\">\n";
-    foreach ($item->options as $option){
-      $op_names .= '<li>'. $option['attribute'] .': '. $option['name'] ."</li>\n";
+  
+  $element['options'] = '';
+  if (module_exists('uc_attribute') && is_array($item->options)) {
+    foreach ($item->options as $option) {
+      $op_names[] = t('@attribute: @option', array('@attribute' => $option['attribute'], '@option' => $option['name']));
     }
-    $op_names .= "</ul>\n";
+    $element['options'] = array('#value' => theme('item_list', $op_names, NULL, 'ul', array('class' => 'cart-options')));
   }
-  $element['options'] = array('#value' => $op_names);
+  
+  
   $element['title'] = array(
     '#value' => l($node->title, 'node/'. $node->nid),
   );