--- loncom/interface/portfolio.pm 2004/09/10 20:48:33 1.45 +++ loncom/interface/portfolio.pm 2004/09/10 21:33:48 1.46 @@ -379,28 +379,30 @@ sub handler { 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 '/'){ + 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); - + $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); - $r->print("\n\n"); - return OK; + # 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); + $r->print("\n\n"); + return OK; } } 1;