--- loncom/interface/portfolio.pm 2004/09/08 17:53:37 1.38
+++ loncom/interface/portfolio.pm 2004/09/10 20:14:45 1.43
@@ -38,127 +38,6 @@ sub make_anchor {
return $anchor;
}
-# returns html with <br /> separated contents of the directory
-# returns a <strong>currentFile</strong> (bolds the selected file/dir)
-sub display_directory_old {
- my ($currentPath, $currentFile, @dirList,) = @_;
- my $displayOut='';
- my $filename;
- my $upPath;
- if ($currentPath ne '/') {
- $displayOut = 'Listing of '.$currentPath.'<br /><hr />';
- # provides the "up one directory level" function
- # it means shortening the currentpath to the parent directory
- $currentPath =~ m:(^/.*)(/.*/$):;
- if ($1 ne '/') {
- $upPath = $1.'/';
- } else {
- $upPath = $1;
- }
-
- # $displayOut = $displayOut.'<a href="/adm/portfolio?selectfile='.$upPath.'¤tpath='.$upPath.'">..</a><br />';
- } else {
- # $displayOut = $displayOut.'at root '.$currentPath.'<br />';
- }
- foreach my $line (@dirList) {
- #$strip holds directory/file name
- #$dom
- my ($filename,$dom,undef,$testdir,undef,undef,undef,undef,undef,undef,undef,undef,undef,undef,$obs,undef)=split(/\&/,$line,16);
- if (($filename ne '.') && ($filename ne '..')) {
- if ($testdir =~ m:^1:) {
- # handle directories different from files
- if ($filename eq $currentFile) {
- #checks to bold the selected file
- $displayOut.= '<strong>'.(&make_anchor($filename.'/', $filename.'/').'</strong><br />'."\n");
- } else {
- $displayOut.= (&make_anchor($filename.'/', $filename.'/').'<br />'."\n");
- }
- } else {
- if ($filename eq $currentFile) {
- #checks to bold the selected file
- $displayOut.='<strong>'.(&make_anchor($filename, $currentPath).'</strong><br />'."\n");
- } else {
- $displayOut.=(&make_anchor($filename, $currentPath).'<br />'."\n");
- }
- }
-
- }
-
- }
- return $displayOut;
-}
-
-sub displayActions {
- # returns html to offer user appropriate actions depending on selected file/directory
- my $displayOut;
- my ($currentPath, $currentFile, $isEmpty) = @_;
-# $displayOut = 'here are actions for '.$currentFile;
- if ($currentFile =~ m:/$:) {
- # if the selected file is a directory, these are the options
- # offer the chance to delete the directory only if it is empty
- if ($isEmpty && ($currentPath ne '/')) {
- $displayOut = $displayOut.'<form method="POST">
- <input type="hidden" name="selectfile"
- value="'.$currentFile.'" />
- <input type="hidden" name="fileaction" value="delete" />
- <input type="hidden" name="currentpath" value="'.$currentPath.'" />
- <center>
- <input type="submit"
- value="Delete '.$currentFile.'" />
- </center>
- </form>';
- }
- if ($currentPath ne '/') {
- $displayOut = $displayOut.'<hr />
- <form method="POST">
- <input type="hidden" name="selectfile"
- value="'.$currentFile.'" />
- <input type="hidden" name="fileaction" value="rename" />
- <input type="hidden" name="currentpath" value="'.$currentPath.'" />
- <input type="input" name="filenewname" value="Type new name here" />
- <input type="submit"
- value="Rename '.$currentFile.'" />
- </form>';
- }
- } else { #action options offered for files
- $displayOut = $displayOut.'<form method="POST">';
- $displayOut = $displayOut.'<input type="hidden" name="selectfile"';
- $displayOut = $displayOut.'value="'.$currentFile;
- $displayOut = $displayOut.'" /><input type="hidden" name="fileaction" value="delete" />
- <input type="hidden" name="currentpath" value="'.$currentPath.'" />
- <center>
- <input type="submit"
- value="Delete '.$currentFile.'" />
- </center>
- </form>';
-
- $displayOut = $displayOut.'<hr />
- <form method="POST">
- <input type="hidden" name="selectfile"
- value="'.$currentFile.'" />
- <input type="hidden" name="fileaction" value="rename" />
- <input type="hidden" name="currentpath" value="'.$currentPath.'" />
- <input type="input" name="filenewname" value="Type new name here" />
- <input type="submit"
- value="Rename '.$currentFile.'" />
- </form>
- <hr />';
- }
- $displayOut = $displayOut.'<hr />Add a file to '.$currentPath;
- # file upload form
- $displayOut = $displayOut.'<form method="post" enctype="multipart/form-data">';
- $displayOut = $displayOut.'<input name="uploaddoc" type="file" />'.
- '<input type="hidden" name="currentpath" value="'.$currentPath.'" />'.
- '<input type="submit" name="storeupl" value="Upload" />'.
- '</form><hr />';
- $displayOut = $displayOut.'<form method="POST">
- <input name="subdir" type="text" />
- <input type="submit" value="Create Subdirectory" />
- </form>
- ';
- return $displayOut;
-}
-
my $dirptr=16384;
sub display_directory {
my ($r,$current_path,$is_empty,$dir_list)=@_;
@@ -190,6 +69,7 @@ sub display_directory {
}
$r->print('</font>');
&Apache::lonhtmlcommon::store_recent('portfolio',$current_path,$current_path);
+ #Fixme -- remove from recent those paths that are deleted
$r->print('<br /><form method=post action="/adm/portfolio">'.
&Apache::lonhtmlcommon::select_recent('portfolio','currentpath',
'this.form.submit();'));
@@ -207,7 +87,7 @@ sub display_directory {
}
$r->print('<table border="0" cellspacing="2" cellpadding="2">'.
'<tr><th>Actions</th><th> </th><th>Name</th><th>Size</th><th>Last Modified</th></tr>');
- my $href_location="/uploaded/$ENV{'user.domain'}/$ENV{'user.name'}/portfolio/$current_path/";
+ my $href_location="/uploaded/$ENV{'user.domain'}/$ENV{'user.name'}/portfolio$current_path";
foreach my $line (sort
{
my ($afile)=split('&',$a,2);
@@ -226,20 +106,20 @@ sub display_directory {
} else {
$r->print('<tr bgcolor="#CCCCFF">');
$r->print('<td>
-<form method="post" action="/adm/portfolio">
-<select name="action">
- <option value=""></option>
- <option value="delete">'.&mt("Delete").'</option>
- <option value="rename">'.&mt("Rename").'</option>
-</select>
-<input type="submit" name="doit" value="Go" />
-<input type="hidden" name="selectfile" value="'.$filename.'" />
-<input type="hidden" name="currentpath" value="'.$current_path.'" />
-</form>
-</td>');
+ <form method="post" action="/adm/portfolio">
+ <select name="action">
+ <option value=""></option>
+ <option value="delete">'.&mt("Delete").'</option>
+ <option value="rename">'.&mt("Rename").'</option>
+ </select>
+ <input type="submit" name="doit" value="Go" />
+ <input type="hidden" name="selectfile" value="'.$filename.'" />
+ <input type="hidden" name="currentpath" value="'.$current_path.'" />
+ </form>
+ </td>');
$r->print('<td><img src="'.$iconpath.'unknown.gif"></td>');
$r->print('<td><a href="'.$href_location.$filename.'">'.
- $filename.'</a></td>');
+ $filename.'</a></td>');
$r->print('<td>'.$size.'</td>');
$r->print('<td>'.&Apache::lonlocal::locallocaltime($mtime).'</td>');
$r->print('</tr>');
@@ -331,6 +211,9 @@ sub delete_dir_confirmed {
$r->print('<font color="red"> An error occured (dir) ('.$result.
') while trying to delete '.$directory_name.'</font><br />');
} else {
+ # now remove from recent
+# $r->print('<br /> removing '.$directory_name.'<br /');
+ &Apache::lonhtmlcommon::remove_recent('portfolio',[$directory_name.'/']);
my @dirs = split m!/!, $directory_name;
# $directory_name =~ m/^(\/*\/)(\/*.)$/;
@@ -467,14 +350,10 @@ sub handler {
&Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
['selectfile','currentpath',
'currentfile']);
- # make sure there is a portfolio directory off /userfiles
- # Fixme -- Do this in a better way -- offload it to the LOND routine
- my @dir_list=&Apache::lonnet::dirlist('/',
- $ENV{'user.domain'},
- $ENV{'user.name'},$portfolio_root);
- if ($dir_list[0] eq 'no_such_dir'){
- &Apache::lonnet::mkdiruserfile($ENV{'user.name'},
- $ENV{'user.domain'},'portfolio');
+ if (($ENV{'form.storeupl'} eq 'Upload') & (!$ENV{'form.uploaddoc.filename'})){
+ $r->print('<font color="red"> No file was selected to upload.'.
+ 'To upload a file, click <strong>Browse...</strong>'.
+ ', select a file, then click <strong>Upload</strong>,</font>');
}
if ($ENV{'form.uploaddoc.filename'}) {
&upload($r);
@@ -497,10 +376,26 @@ sub handler {
if ($ENV{'form.currentpath'}) {
$current_path = $ENV{'form.currentpath'};
}
- @dir_list=&Apache::lonnet::dirlist($current_path,
+ my @dir_list=&Apache::lonnet::dirlist($current_path,
$ENV{'user.domain'},
$ENV{'user.name'},$portfolio_root);
-
+ if (@dir_list[0] eq 'no_such_dir'){
+ # two main reasons for this: 1) never been here, so directory structure not created
+ # 2) back-button navigation after deleting a directory
+ if ($current_path eq '/'){
+ &Apache::lonnet::mkdiruserfile($ENV{'user.name'},
+ $ENV{'user.domain'},'portfolio');
+ }else{ # some directory that snuck in
+ # get rid of the directory from the recent pulldown, just in case
+ &Apache::lonhtmlcommon::remove_recent('portfolio',[$current_path]);
+ $current_path = '/'; # force it back to the root
+ }
+ # now grab the directory list again, for the first time
+ @dir_list=&Apache::lonnet::dirlist($current_path,
+ $ENV{'user.domain'},
+ $ENV{'user.name'},$portfolio_root);
+
+ }
# need to know if directory is empty so it can be removed if desired
my $is_empty=(@dir_list == 2);
&display_directory($r,$current_path,$is_empty,\@dir_list);