--- loncom/interface/portfolio.pm	2006/11/01 02:03:16	1.167
+++ loncom/interface/portfolio.pm	2006/11/01 16:12:35	1.169
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # portfolio browser
 #
-# $Id: portfolio.pm,v 1.167 2006/11/01 02:03:16 rezaferry Exp $
+# $Id: portfolio.pm,v 1.169 2006/11/01 16:12:35 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -95,12 +95,12 @@ sub display_common {
 					 'createdir_label' => 
 					 'Create subdirectory in current directory:');
         my $escuri = &HTML::Entities::encode($r->uri,'&<>"');
-		my $help_fileupload = &Apache::loncommon::help_open_topic('Portfolio AddFiles');
-		my $help_createdir = &Apache::loncommon::help_open_topic('Portfolio CreateDirectory');
+	my $help_fileupload = &Apache::loncommon::help_open_topic('Portfolio AddFiles');
+	my $help_createdir = &Apache::loncommon::help_open_topic('Portfolio CreateDirectory');
 		
-		# TO BE DEPRECATED: This line should be deleted once Portfolio uses breadcrumbs
-		$r->print(&Apache::loncommon::help_open_topic('Portfolio About', 'Help on the portfolio'));
-		# END LINE TO BE DEPRECATED
+	# FIXME: This line should be deleted once Portfolio uses breadcrumbs
+	$r->print(&Apache::loncommon::help_open_topic('Portfolio About', 'Help on the portfolio'));
+
 		
         $r->print(<<"TABLE"); 
 <table id="LC_portfolio_actions">
@@ -162,7 +162,7 @@ TABLE
         }
     }
     $r->print('</span>');
-	$r->print(&Apache::loncommon::help_open_topic('Portfolio ChangeDirectory'));
+    $r->print(&Apache::loncommon::help_open_topic('Portfolio ChangeDirectory'));
     &Apache::lonhtmlcommon::store_recent($namespace,$current_path,$current_path);
     $r->print('<br /><form method="post" action="'.$url.'?mode='.$env{"form.mode"}.'&amp;fieldname='.$env{"form.fieldname"}.&group_args());
     $r->print('">'.
@@ -267,13 +267,14 @@ sub display_directory {
             '<tr><th>Select</th><th>&nbsp;</th><th>&nbsp;</th><th>Name</th><th>Size</th><th>Last Modified</th></tr>');
     } else {
         $r->print('<form method="post" action="'.$url.'">');
-		$r->print(&Apache::loncommon::help_open_topic('Portfolio FileList', 'Using the portfolio file list'));
+	$r->print(&Apache::loncommon::help_open_topic('Portfolio FileList',
+						      'Using the portfolio file list'));
         $r->print('<table id="LC_browser">'.
-            '<tr><th colspan="2">Actions</th><th>&nbsp;</th><th>&nbsp;</th><th>Name'.
-			&Apache::loncommon::help_open_topic('Portfolio OpenFile').
-			'</th><th>Size</th><th>Last Modified</th><th>Current Access Status'.
-			&Apache::loncommon::help_open_topic('Portfolio ShareFile').		
-			'</th></tr>');
+		  '<tr><th colspan="2">Actions</th><th>&nbsp;</th><th>&nbsp;</th><th>Name'.
+		  &Apache::loncommon::help_open_topic('Portfolio OpenFile').
+		  '</th><th>Size</th><th>Last Modified</th><th>Current Access Status'.
+		  &Apache::loncommon::help_open_topic('Portfolio ShareFile').
+		  '</th></tr>');
     }
     $r->print("\n".&group_form_data()."\n");
 
@@ -450,7 +451,7 @@ sub display_directory {
         if ($can_delete) {
             $r->print('
         <input type="submit" name="doit" value="Delete Checked Files" />'.
-		&Apache::loncommon::help_open_topic('Portfolio DeleteFile').'
+	&Apache::loncommon::help_open_topic('Portfolio DeleteFile').'
         <input type="hidden" name="action" value="delete" />
         <input type="hidden" name="currentpath" value="'.$current_path.'" />
         </form>'
@@ -537,7 +538,7 @@ sub delete {
     my @files=&Apache::loncommon::get_env_multiple('form.selectfile');
     my ($uname,$udom) = &get_name_dom();
     if (&Apache::lonnet::is_locked($file_name,$udom,$uname) eq 'true') {
-        $r->print ("The file is locked and cannot be deleted.<br />");
+        $r->print("The file is locked and cannot be deleted.<br />");
         $r->print(&done('Back',$url));
     } else {
         if (scalar(@files)) {
@@ -648,7 +649,7 @@ sub rename {
     my ($uname,$udom) = &get_name_dom();
     $file_name = &prepend_group($file_name);
     if (&Apache::lonnet::is_locked($file_name,$udom,$uname) eq 'true') {
-        $r->print ("The file is locked and cannot be renamed.<br />");
+        $r->print("The file is locked and cannot be renamed.<br />");
         $r->print(&done(undef,$url));
     } else {
         &open_form($r,$url);
@@ -765,9 +766,9 @@ sub display_access {
     if ($can_setacl) {
         &open_form($r,$url);
         $r->print($header.$info);
-		$r->print ('<br/>'.&Apache::loncommon::help_open_topic('Portfolio ShareFile SetAccess', 'Help on setting up share access'));
-		$r->print (&Apache::loncommon::help_open_topic('Portfolio ShareFile ChangeSetting', 'Help on changing settings'));
-		$r->print (&Apache::loncommon::help_open_topic('Portfolio ShareFile StopAccess', 'Help on removing share access'));
+	$r->print('<br />'.&Apache::loncommon::help_open_topic('Portfolio ShareFile SetAccess', 'Help on setting up share access'));
+	$r->print(&Apache::loncommon::help_open_topic('Portfolio ShareFile ChangeSetting', 'Help on changing settings'));
+	$r->print(&Apache::loncommon::help_open_topic('Portfolio ShareFile StopAccess', 'Help on removing share access'));
         &access_setting_table($r,$access_controls{$file_name});
         my $button_text = {
                         'continue' => &mt('Proceed'),