--- loncom/interface/loncommon.pm	2006/06/22 13:02:22	1.384
+++ loncom/interface/loncommon.pm	2006/06/22 19:11:13	1.388
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # a pile of common routines
 #
-# $Id: loncommon.pm,v 1.384 2006/06/22 13:02:22 raeburn Exp $
+# $Id: loncommon.pm,v 1.388 2006/06/22 19:11:13 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -3315,6 +3315,40 @@ table.LC_mail_list tr.LC_mail_other {
 table.LC_mail_list tr.LC_mail_other:hover {
   background-color: $mail_other_hover;
 }
+
+table#LC_portfolio_actions {
+  width: auto;
+  background: $pgbg;
+  border: 0px;
+  border-spacing: 2px 2px;
+  padding: 0px;
+  margin: 0px;
+  border-collapse: separate;
+}
+table#LC_portfolio_actions td.LC_label {
+  background: $tabbg;
+  text-align: right;
+}
+table#LC_portfolio_actions td.LC_value {
+  background: $tabbg;
+}
+
+table.LC_brower {
+}
+table.LC_browser tr.LC_browser_file {
+  background: #CCFF88
+}
+table.LC_browser tr.LC_browser_file_locked {
+  background: #FFAA99
+}
+table.LC_browser tr.LC_browser_folder {
+  background: #CCCCFF
+}
+span.LC_current_location {
+  font-size: x-large;
+  background: $pgbg;
+}
+
 END
 }
 
@@ -3999,8 +4033,11 @@ sub get_secgrprole_info {
                   };
     if ($needroles) {
         $rolehash = {'all' => 'all'};
-        my %dumphash = &Apache::lonnet::dump('nohist_userroles',$cdom,$cnum);
-        foreach my $item (keys(%dumphash)) {
+        my %user_roles = &Apache::lonnet::dump('nohist_userroles',$cdom,$cnum);
+	if (&Apache::lonnet::error(%user_roles)) {
+	    undef(%user_roles);
+	}
+        foreach my $item (keys(%user_roles)) {
             my ($role)=split(/\:/,$item,2);
             if ($role eq 'cr') { next; }
             if ($role =~ /^cr/) {