--- loncom/interface/portfolio.pm	2008/12/10 21:46:58	1.202
+++ loncom/interface/portfolio.pm	2009/01/03 19:28:53	1.205
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # portfolio browser
 #
-# $Id: portfolio.pm,v 1.202 2008/12/10 21:46:58 kaisler Exp $
+# $Id: portfolio.pm,v 1.205 2009/01/03 19:28:53 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -138,7 +138,7 @@ END
                  .'<form method="post" action="'.$escuri.'">'
                  .'<fieldset>'
                  .'<legend>'.$lt{'createdir_label'}.'</legend>'
-                 .'<input name="newdir" type="input" />'.$groupitem
+                 .'<input name="newdir" type="text" />'.$groupitem
                  .'<input type="hidden" name="currentpath" value="'.$current_path.'" />'
                  .'<input type="hidden" name="action" value="'.$env{"form.action"}.'" />'
                  .'<input type="hidden" name="fieldname" value="'.$env{"form.fieldname"}.'" />'
@@ -694,7 +694,7 @@ sub rename {
     } else {
         &open_form($r,$url);
         $r->print('<p>'.&mt('Rename [_1] to [_2]?', &display_file()
-                  , '<input name="filenewname" type="input" size="50" />').'</p>');
+                  , '<input name="filenewname" type="text" size="50" />').'</p>');
         &close_form($r,$url);
     }
 }
@@ -2245,26 +2245,30 @@ sub handler {
     &Apache::loncommon::content_type($r,'text/html');
     $r->send_http_header;
     # Give the LON-CAPA page header
-	my @brcrum = [{href=>"/adm/portfolio",text=>"Portfolio Manager"}];
+    my $brcrum = [{href=>"/adm/portfolio",text=>"Portfolio Manager"}];
+
     if ($env{"form.mode"} eq 'selectfile'){
         $r->print(&Apache::loncommon::start_page($title,undef,
 						 {'only_body' => 1}));
     } elsif ($env{'form.action'} eq 'rolepicker') {
         $r->print(&Apache::loncommon::start_page('New role-based condition',undef,
                                                  {'no_nav_bar'  => 1, }));
+    } elsif ($caller eq 'coursegrp_portfolio') {
+        $r->print(&Apache::loncommon::start_page($title));
     } else {
-        $r->print(&Apache::loncommon::start_page($title,undef,{bread_crumbs => @brcrum}));
+        $r->print(&Apache::loncommon::start_page($title,undef,
+                                                 {'bread_crumbs' => $brcrum}));
+        if (!&Apache::lonnet::usertools_access($uname,$udom,'portfolio')) {
+            $r->print('<h2>'.&mt('No user portfolio available') .'</h2>'.
+                      &mt('This is a result of one of the following:').'<ul>'.
+                      '<li>'.&mt('The administrator of this domain has disabled portfolio functionality for this specific user.').'</li>'.
+                      '<li>'.&mt('The domain has been configured to disable, by default, portfolio functionality for all users in the domain.').'</li>'.
+                      '</ul>');
+            $r->print(&Apache::loncommon::end_page());
+            return OK;
+        }
     }
     $r->rflush();
-    if (!&Apache::lonnet::usertools_access($uname,$udom,'portfolio')) {
-        $r->print('<h2>'.&mt('No user portfolio available') .'</h2>'.
-                  &mt('This is a result of one of the following:').'<ul>'.
-                  '<li>'.&mt('The administrator of this domain has disabled portfolio functionality for this specific user.').'</li>'.
-                  '<li>'.&mt('The domain has been configured to disable, by default, portfolio functionality for all users in the domain.').'</li>'.
-                  '</ul>');
-         $r->print(&Apache::loncommon::end_page());
-         return OK;
-    }
     my ($blocked,$blocktext) = 
         &Apache::loncommon::blocking_status('port',$uname,$udom);
     if ($blocked) {