--- loncom/interface/lonaboutme.pm	2006/08/02 19:29:35	1.44
+++ loncom/interface/lonaboutme.pm	2006/09/13 21:37:29	1.54
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # "About Me" Personal Information
 #
-# $Id: lonaboutme.pm,v 1.44 2006/08/02 19:29:35 albertel Exp $
+# $Id: lonaboutme.pm,v 1.54 2006/09/13 21:37:29 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -37,6 +37,7 @@ use Apache::lonfeedback;
 use Apache::lonrss();
 use Apache::lonlocal;
 use Apache::lonmsgdisplay();
+use HTML::Entities();
 
 sub handler {
     my $r = shift;
@@ -48,7 +49,7 @@ sub handler {
     if ($target eq 'tex') {
 	$r->print(&Apache::lonprintout::print_latex_header($env{'form.latex_type'}));
     }
-    my (undef,undef,$cdom,$cnum)=split(/\//,$r->uri);
+    my (undef,undef,$cdom,$cnum,undef,$action)=split(/\//,$r->uri);
 # Is this even a user?
     if (&Apache::lonnet::homeserver($cnum,$cdom) eq 'no_host') {
 	&Apache::loncommon::simple_error_page($r,'No info',
@@ -62,18 +63,14 @@ sub handler {
        'ccc_webreferences' => 'Web References');
 
 # ------------------------------------------------------------ Get Query String
-    &Apache::loncommon::get_unprocessed_cgi
-        ($ENV{'QUERY_STRING'},['forceedit','forcestudent','register','action']);
+    &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
+					    ['forceedit','forcestudent',
+					     'register']);
 
 # ----------------------------------------------- Available Portfolio file display 
-    if (($target ne 'tex') && ($env{'form.action'} eq 'portfolio')) {
-        my %lt = &Apache::lonlocal::texthash (
-                  withoutpass => 'passphrase not required',
-                  withpass    => 'passphrase protected',
-                  both        => 'all access types ',
-        );
+    if (($target ne 'tex') && ($action eq 'portfolio')) {
         &display_portfolio_header($r);
-        &display_portfolio_files($r,\%lt);
+        &display_portfolio_files($r);
         $r->print(&Apache::loncommon::end_page());
         return OK;
     }
@@ -93,7 +90,7 @@ sub handler {
 	my $start_page = 
 	    &Apache::loncommon::start_page("Personal Information",undef,
 					   {'function'       => $forcestudent,
-					    'domain'         => $cdom,
+ 					    'domain'         => $cdom,
 					    'force_register' =>
 						            $forceregister,});
 	$r->print($start_page);
@@ -210,7 +207,7 @@ sub handler {
 	      }
               if ($allowed) {
                  $r->print('<br /><textarea cols="80" rows="6" name="'.$_.'">'.
-			   $syllabus{$_}.
+			   &HTML::Entities::encode($syllabus{$_},'"&<>').
            '</textarea><input type="submit" name="storesyl" value="'.
 			   &mt('Store').'" />');
 	      }
@@ -219,7 +216,7 @@ sub handler {
        if ($allowed) {
 	   $r->print('</form>');
        }
-       if ($target ne 'tex') {$r->print('</p>');} else {$r->print('\\\\');}
+       if ($target ne 'tex') {$r->print('<br />');} else {$r->print('\\\\');}
     } else {
        $r->print('<p>'.&mt('No personal information provided').'.</p>');
     }
@@ -274,19 +271,23 @@ sub print_portfiles_link {
     my ($r) = @_;
     my ($cdom,$cnum,$name) = &aboutme_info($r);
     my $filecounts = &portfolio_files($r,'showlink');
-    my $query_string = &build_query_string({'action' => 'portfolio',});
+    my $query_string = &build_query_string();
     my $output;
     if ($filecounts->{'both'} > 0) {
         $output = &mt('<h3>Viewable portfolio files</h3>');
-        $output .= '<a href="/adm/'.$cdom.'/'.$cnum.'/aboutme'.
+        $output .= '<a href="/adm/'.$cdom.'/'.$cnum.'/aboutme/portfolio'.
                    $query_string.'">'.&mt('Display file listing').
                    '</a><br /><br />';
-        $output .= &mt('A total of [_1] portfolio files owned by [_2] are available for viewing.',$filecounts->{'both'},$name).'<ul>';
+        if ($filecounts->{'both'} == 1) {
+            $output .= &mt('One portfolio file owned by [_2] is available.',$filecounts->{'both'},$name).'<ul>';
+        } else {
+            $output .= &mt('A total of [_1] portfolio files owned by [_2] are available.',$filecounts->{'both'},$name).'<ul>';
+        }
         if ($filecounts->{'withoutpass'}) {
-            $output .= '<li>'.&mt('[_1] files are either publicly accessible, or directly available to you because of your current role, domain and/or username.',$filecounts->{'withoutpass'}).'</li>';   
+	    $output .= '<li>'.&mt('[quant,_1,file is,files are] publicly accessible.',$filecounts->{'withoutpass'}).'</li>';
         }
         if ($filecounts->{'withpass'}) {
-            $output .= '<li>'.&mt('[_1] files will be available after you have entered a passphrase set for each specific file.',$filecounts->{'withpass'}).'</li>';
+	    $output .= '<li>'.&mt('[quant,_1,file requires,files require] a passphrase for access.',$filecounts->{'withpass'}).'</li>';
         }
         $output .= '</ul>';
     }
@@ -298,9 +299,9 @@ sub build_query_string {
     my ($new_items) = @_;
     my $query_string;
     my @formelements = ('register'); 
-    my $nonew = 1;
+    my $new = 0;
     if (ref($new_items) eq 'HASH') {
-        $nonew = 0;
+        $new = 1;
         if (!defined($new_items->{'forceedit'}) && 
             !defined($new_items->{'forcestudent'})) {
             push(@formelements,('forceedit','forcestudent'));
@@ -310,12 +311,12 @@ sub build_query_string {
     }
     foreach my $element (@formelements) {
         if (exists($env{'form.'.$element})) {
-            if (($nonew) || (!defined($new_items->{$element}))) {
+            if ((!$new) || (!defined($new_items->{$element}))) {
                 $query_string .= '&amp;'.$element.'='.$env{'form.'.$element};
             }
         }
     }
-    if (!$nonew) {
+    if ($new) {
         foreach my $key (keys(%{$new_items})) {
             $query_string .= '&amp;'.$key.'='.$new_items->{$key};
         }
@@ -331,52 +332,61 @@ sub display_portfolio_header {
     &Apache::lonhtmlcommon::clear_breadcrumbs();
     my $forcestudent='';
     if ($env{'form.forcestudent'}) { $forcestudent='student'; };
-    my $output =
+    my $output = 
         &Apache::loncommon::start_page('Viewable portfolio files',undef,
                                             {'function' => $forcestudent, 
                                              'domain'   => $cdom,});
-    &Apache::lonhtmlcommon::add_breadcrumb
-        ({href=>"/adm/$cdom/$cnum/aboutme".$query_string,
-          text=>"Personal information - $name",
-          title=>"Go to personal information page for $name"},
-         {href=>"/adm/$cdom/$cnum/aboutme?action=portfolio",
-          text=>"Viewable files - $name",
-          title=>"Viewable portfolio files for $name"}
-);
-    $output .= &Apache::lonhtmlcommon::breadcrumbs(&mt('Viewable portfolio files.'));
+    if (!($env{'user.name'} eq 'public' && $env{'user.domain'} eq 'public')) {
+        &Apache::lonhtmlcommon::add_breadcrumb
+            ({href=>"/adm/$cdom/$cnum/aboutme".$query_string,
+              text=>"Personal information - $name",
+              title=>"Go to personal information page for $name"},
+             {href=>"/adm/$cdom/$cnum/aboutme/portfolio",
+              text=>"Viewable files - $name",
+              title=>"Viewable portfolio files for $name"}
+        );
+        $output .= &Apache::lonhtmlcommon::breadcrumbs(&mt('Viewable portfolio files.'));
+    }
     $output .= '<h3>'.&mt('Portfolio files for [_1]',$name).'</h3>';
     $r->print($output);
     return;
 }
 
 sub display_portfolio_files {
-    my ($r,$lt) = @_;
-    my ($cdom,$cnum,$name) = aboutme_info($r);
+    my ($r) = @_;
+    my ($cdom,$cnum,$name) = &aboutme_info($r);
+    my %lt = ( withoutpass => 'passphrase not required',
+	       withpass    => 'passphrase protected',
+	       both        => 'all access types ',);
+    %lt = &Apache::lonlocal::texthash(%lt);
+
     my $portaccess = 'withoutpass';
     if (exists($env{'form.portaccess'})) {
         $portaccess = $env{'form.portaccess'};
     }
-    my $output = '<form name="displaystatus" method="post">'.
-                 &mt('File access type: ').'<select name="portaccess">';
+
+    my $output = '<form action="'.&HTML::Entities::encode($r->uri,'<>&"')
+	.'" name="displaystatus" method="post">'.
+	&mt('File access type: ').'<select name="portaccess">';
     foreach my $type ('withoutpass','withpass','both') {
         $output .= '<option value="'.$type.'" ';
         if ($portaccess eq $type) {
             $output .= 'selected="selected"';
         }
-        $output .= '>'.$lt->{$type}.'</option>';
+        $output .= '>'.$lt{$type}.'</option>';
     }
     $output .= '</select>'."\n".
                '<input type="submit" name="portaccessbutton" value="'.
-               &mt('Update display').'">'.
-               '<input type="hidden" name="action" value="'.
-               $env{'form.action'}.'"/>';
+               &mt('Update display').'" />';
     $output .= '</form><br /><br />';
     $r->print($output);
-    my $filecounts = &portfolio_files($r,'listfiles',$lt);
-    my $query_string = &build_query_string();
-    $r->print('<br /><br /><a href="/adm/'.$cdom.'/'.$cnum.
-               '/aboutme'.$query_string.'">'.
-               &mt('Information about [_1]',$name).'</a>');
+    my $filecounts = &portfolio_files($r,'listfiles',\%lt);
+    if (!($env{'user.name'} eq 'public' && $env{'user.domain'} eq 'public')) {
+        my $query_string = &build_query_string();
+        $r->print('<br /><br /><a href="/adm/'.$cdom.'/'.$cnum.
+                  '/aboutme'.$query_string.'">'.
+                  &mt('Information about [_1]',$name).'</a>');
+    }
     return;
 }
 
@@ -445,11 +455,8 @@ sub portfolio_files {
     if ($mode eq 'listfiles') {
         my $output;
         if (keys(%allfileshash) > 0) {
-            my $depth = 0;
-            my $count = 0;
-            $output = '<table border="0" cellpadding="2" cellspacing="0">';
-            ($count,$output) = &parse_directory($r,$output,$depth,
-                                               \%allfileshash,$count);
+            $output = &portfolio_table_start();
+            $output .= &parse_directory($r,0,\%allfileshash,'');
             $output .= '</table>';
         } else {
             my $access_text;
@@ -463,39 +470,76 @@ sub portfolio_files {
     return $filecounts;
 }
 
+{ 
+    my $count=0;
+    sub portfolio_table_start {
+	$count=0;
+	return '<table class="LC_aboutme_port">';
+    }
+    sub portfolio_row_start {
+	$count++;
+	my $class = ($count%2)?'LC_odd_row'
+	                      :'LC_even_row';
+	return '<tr class="'.$class.'">';
+    }
+}
+
 sub parse_directory {
-    my ($r,$output,$depth,$currhash,$count) = @_;
+    my ($r,$depth,$currhash,$path) = @_;
     my ($cdom,$cnum,$name) = &aboutme_info($r);
-    $depth ++;
-    my @backgroundColors = ("#FFFFFF", "#F6F6F6");
-    my $indent = '<img src="'.&Apache::loncommon::lonhttpdurl("/adm/lonIcons/whitespace1.gif").'" width="25" height="1" alt="&nbsp;&nbsp;" border="0" />';
-    $output .= $indent;
+    $depth++;
+    my $output;
+
+    my $portfolio_root = &Apache::portfolio::get_portfolio_root($cdom,$cnum);
+    my %dirlist = map {
+	    ((split('&',$_,2))[0],1)
+	} &Apache::lonnet::dirlist($path,$cdom,$cnum,$portfolio_root);
     foreach my $item (sort(keys(%{$currhash}))) {
-        $count ++;
-        my $backgroundcolor = $backgroundColors[$count%2]; 
-        $output .= '<tr bgcolor="'.$backgroundcolor.'">';
-        $output .= '<td>';
-        for (my $i=0; $i<$depth; $i++) {
-            $output .= $indent;
-        }
+        $output .= &portfolio_row_start();
+        $output .= '<td style="padding-left: '.($depth*25).'px">';
         if (ref($currhash->{$item}) eq 'HASH') {
-            my $title = $item;
-            $title=~s/\"/\&quot;/g;
-            $output .= '<img src="'.&Apache::loncommon::lonhttpdurl("/adm/lonIcons/navmap.folder.open.gif").'" alt="'.&mt('Folder').' '.$title.' border="0" />&nbsp;'.$title;
-            ($count,$output) = &parse_directory($r,$output,$depth,
-                                                $currhash->{$item},$count);
+            my $title=&HTML::Entities::encode($item,'<>&"');
+            $output .= '<img src="'.&Apache::loncommon::lonhttpdurl("/adm/lonIcons/navmap.folder.open.gif").'" alt="'.&mt('Folder').' '.$title.'" class="LC_icon" />&nbsp;'.$title;
+	    $output .= '</td><td></td></tr>';
+            $output .= &parse_directory($r,$depth,$currhash->{$item},
+					$path.'/'.$item);
         } else {
+	   
+
+	    my $file_name; 
+	    if ($currhash->{$item} =~ m|/([^/]+)$|) {
+		$file_name = $1;
+	    } else {
+		$file_name = $currhash->{$item};
+	    }
+	    my $have_meta = exists($dirlist{$file_name.'.meta'});
+	    my $url = '/uploaded/'.$cdom.'/'.$cnum.'/portfolio/'.
+		$currhash->{$item};
+
             my $showname;
-            if ($currhash->{$item} =~ m|/([^/]+)$|) {
-                $showname = $1;
-            } else {
-                $showname = $currhash->{$item};
-            }
-            $output .= '<img alt="" src="'.&Apache::loncommon::icon($currhash->{$item}).'" />&nbsp;<a href="/uploaded/'.$cdom.'/'.$cnum.'/portfolio/'.$currhash->{$item}.'">'.$showname.'</a>';
+	    if ($have_meta) {
+		$showname = &Apache::lonnet::metadata($url,'title');
+	    }
+	    if ($showname eq '') {
+		$showname = $file_name;
+	    } else {
+		$showname = $file_name.' ('.$showname.')';
+	    }
+
+            $showname=&HTML::Entities::encode($showname,'<>&"');
+            $output .= '<a href="'.$url.'">'.
+		'<img alt="" src="'.&Apache::loncommon::icon($currhash->{$item}).'" class="LC_icon" />'.
+		'&nbsp;'.$showname.'</a>';
+	    $output.='</td><td>';
+	    if ($have_meta) {
+		$output.= '<a href="'.$url.'.meta"><img alt="'.&mt('Catalog Information').'" src="'.
+		&Apache::loncommon::lonhttpdurl('/res/adm/pages/catalog.gif').
+		'" class="LC_icon" /></a>';
+	    }
+	    $output .= '</td></tr>';
         }
-        $output .= '</td></tr>';
     }
-    return ($count,$output);
+    return $output;
 }
 
 1;