--- loncom/publisher/lonpubdir.pm 2010/02/11 21:49:19 1.114.2.1 +++ loncom/publisher/lonpubdir.pm 2008/12/17 13:04:08 1.117 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # Construction Space Directory Lister # -# $Id: lonpubdir.pm,v 1.114.2.1 2010/02/11 21:49:19 raeburn Exp $ +# $Id: lonpubdir.pm,v 1.117 2008/12/17 13:04:08 harmsja Exp $ # # Copyright Michigan State University Board of Trustees # @@ -62,7 +62,7 @@ sub handler { &Apache::loncacc::constructaccess( $fn,$r->dir_config('lonDefDomain')); unless (($uname) && ($udom)) { - $r->log_reason($uname.':'.$udom. + $r->log_reason($uname.' at '.$udom. ' trying to list directory '.$env{'form.filename'}. ' ('.$fn.') - not authorized', $r->filename); @@ -74,7 +74,7 @@ sub handler { $fn=~s/\/$//; unless ($fn) { - $r->log_reason($env{'user.name'}.':'.$env{'user.domain'}. + $r->log_reason($env{'user.name'}.' at '.$env{'user.domain'}. ' trying to list empty directory', $r->filename); return HTTP_NOT_FOUND; } @@ -105,15 +105,16 @@ sub handler { my $numres = 0; # Start off the directory table. - $r->print('<h3>'.&mt('Directory Contents:').'</h3>'); - $r->print('<table id="LC_browser"><tr>'. - '<th>'.&mt('Type').'</th>'. - '<th>'.&mt('Actions').'</th>'. - '<th>'.&mt('Name').'</th>'. - '<th>'.&mt('Title').'</th>'. - '<th>'.&mt('Status').'</th>'. - '<th>'.&mt('Last Modified'). - '</th></tr>'."\n"); + $r->print(&Apache::loncommon::start_data_table() + .&Apache::loncommon::start_data_table_header_row() + .'<th>'.&mt('Type').'</th>' + .'<th>'.&mt('Actions').'</th>' + .'<th>'.&mt('Name').'</th>' + .'<th>'.&mt('Title').'</th>' + .'<th colspan="2">'.&mt('Status').'</th>' + .'<th>'.&mt('Last Modified').'</th>' + .&Apache::loncommon::end_data_table_header_row() + ); my $filename; my $dirptr=16384; # Mask indicating a directory in stat.cmode. @@ -139,7 +140,9 @@ sub handler { } closedir(DIR); - $r->print('</table>'.&Apache::loncommon::end_page()); + $r->print(&Apache::loncommon::end_data_table() + .&Apache::loncommon::end_page() + ); return OK; } # @@ -156,8 +159,7 @@ sub getEffectiveUrl { # Replace the ~username of the URL with /home/username/public_html # so that we don't have to worry about ~ expansion internally. # - $fn=~s/^https?\:\/\/[^\/]+\///; - + $fn=~s/^http\:\/\/[^\/]+\///; $fn=~s/^\///; $fn=~s{~($LONCAPA::username_re)}{/home/$1/public_html}; @@ -331,7 +333,7 @@ ENDPUBDIRSCRIPT if ((($uname ne $env{'user.name'}) || ($udom ne $env{'user.domain'})) && $env{'environment.remote'} ne 'off') { - $r->print('<h3>'.&mt('Co-Author [_1]',$uname.':'.$udom).'</h3>'); + $r->print('<h3>'.&mt('Co-Author').': '.$uname.' at '.$udom. '</h3>'); } } @@ -374,42 +376,46 @@ sub dircontrols { ); my $mytype = $lt{'type'}; # avoid conflict with " and ' in javascript $r->print(<<END); - <table id="LC_cstr_controls"> - <tr> - <th>$lt{'acti'}</th> - <th>$lt{'updc'}</th> - <th>$lt{'crea'}</th> - </tr> - <tr> - <td> - <form name="curractions" method="post" action=""> - <select name="dirtask" onchange="currdiract(this.form)"> +<div class="LC_columnSection"> + <div> + <form name="curractions" method="post" action=""> + <fieldset> + <legend>$lt{'acti'}</legend> + <select name="dirtask" onchange="currdiract(this.form)"> <option>$lt{'sela'}</option> <option value="publish">$lt{'pubd'}</option> <option value="editmeta">$lt{'edit'}</option> <option value="printdir">$lt{'prnt'}</option> <option value="delete">$lt{'dedr'}</option> - </select> - <input type="hidden" name="filename" value="/~$uname$thisdisfn/" /> - </form> - <form name="publishdir" method="post" action="/adm/publish" target="_parent"> - <input type="hidden" name="pubrec" value="" /> - <input type="hidden" name="filename" value="" /> - </form> - <form name="printdir" method="post" action="/adm/printout" target="_parent"> - <input type="hidden" name="postdata" value="" /> - </form> - </td> - <td> - <form name="upublisher" enctype="multipart/form-data" method="post" action="/adm/upload" target="_parent"> - <input type="hidden" name="filename" value="/~$uname$thisdisfn/" /> - <input type="file" name="upfile" size="20" /> - <input type="button" value="$lt{'uplo'}" onclick="checkUpload(this.form)" /> - </form> - </td> - <td> - <form name="fileaction" method="post" action="/adm/cfile" target="_parent"> - <span style="white-space: nowrap"> + </select> + <input type="hidden" name="filename" value="/~$uname$thisdisfn/" /> + </fieldset> + </form> + <form name="publishdir" method="post" action="/adm/publish" target="_parent"> + <input type="hidden" name="pubrec" value="" /> + <input type="hidden" name="filename" value="" /> + </form> + <form name="printdir" method="post" action="/adm/printout" target="_parent"> + <input type="hidden" name="postdata" value="" /> + </form> + </div> + + <div> + <form name="upublisher" enctype="multipart/form-data" method="post" action="/adm/upload" target="_parent"> + <fieldset> + <legend>$lt{'updc'}</legend> + <input type="hidden" name="filename" value="/~$uname$thisdisfn/" /> + <input type="file" name="upfile" size="20" /> + <input type="button" value="$lt{'uplo'}" onclick="checkUpload(this.form)" /> + </fieldset> + </form> + </div> + + <div> + <form name="fileaction" method="post" action="/adm/cfile" target="_parent"> + <fieldset> + <legend>$lt{'crea'}</legend> + <span class="LC_nobreak"> <input type="hidden" name="filename" value="/~$uname$thisdisfn/" /> <script type="text/javascript"> function validate_go() { @@ -435,10 +441,10 @@ sub dircontrols { <option value="newdir">$lt{'nsub'}:</option> </select> <input type="text" name="newfilename" value="$lt{'type'}" onfocus="if (this.value == '$mytype') this.value=''" /> <input type="button" value="Go" onclick="validate_go();" /> </span> - </form> - </td> - </tr> - </table> + </fieldset> + </form> + </div> +</div> END } @@ -450,7 +456,7 @@ sub pubbuttons { '<input type="submit" value="'.&mt('Publish Directory').'" /></td><td>'. '<input type="button" onclick="window.location='."'/~". $uname.$thisdisfn."/default.meta'".'" value="'. -&mt('Edit Directory Metadata').'" /></td></tr></table></form>'); +&mt('Edit Directory Catalog Information').'" /></td></tr></table></form>'); } sub resourceactions { @@ -567,7 +573,7 @@ sub putdirectory { '<td>'.$actionitem.'</td>'. '<td><span class="LC_filename"><a href="'.&HTML::Entities::encode($here.'/'.$dirname,'<>&"').'/" target="_parent">'. $disfilename.'</a></span></td>'. - '<td colspan="2">'.($kaputt?&Apache::lonhtmlcommon::authorbombs($resdir.'/'.$disfilename.'/'):'').$Apache::lonpublisher::metadatafields{'title'}); + '<td colspan="3">'.($kaputt?&Apache::lonhtmlcommon::authorbombs($resdir.'/'.$disfilename.'/'):'').$Apache::lonpublisher::metadatafields{'title'}); if ($Apache::lonpublisher::metadatafields{'subject'} ne '') { $r->print(' <i>'. $Apache::lonpublisher::metadatafields{'subject'}. @@ -622,17 +628,9 @@ sub putresource { $meta_same = 0; } $publish_button=&mt('Re-publish'); - - my $rights_status = - &mt(&getCopyRightString($targetdir.'/'.$filename)).', '; - - my %lt_SourceRight = &Apache::lonlocal::texthash( - 'open' => 'Source: open', - 'closed' => 'Source: closed', - ); - $rights_status .= - $lt_SourceRight{&getSourceRightString($targetdir.'/'.$filename)}; - + my $rights_status = + &mt(&getCopyRightString($targetdir.'/'.$filename)).' '. + &mt(&getSourceRightString($targetdir.'/'.$filename)); $title = '<a href="/res/'.$targetdir.'/'.$filename. '.meta" target="cat">'. &getTitleString($targetdir.'/'.$filename).'</a>'; @@ -697,7 +695,7 @@ sub putresource { } my $pub_select = ''; &create_pubselect($r,\$pub_select,$udom,$uname,$thisdisfn,$filename,$resdir,$pubstatus,$publish_button,$numres); - $r->print('<tr class="LC_browser_file_'.$pubstatus.'">'. + $r->print(&Apache::loncommon::start_data_table_row(). '<td>'.($filename=~/[\#\~]$/?' ': '<img src="'.&Apache::loncommon::icon($filename).'" alt="" />').'</td>'. '<td>'.$pub_select.'</td>'. @@ -706,9 +704,11 @@ sub putresource { $filename.'</a></span>'.$editlink2.$editlink. '</td>'. '<td>'.$title.'</td>'. - '<td>'.$status.'</td>'. + '<td class="LC_browser_file_'.$pubstatus.'"> </td>'. # Display publication status + '<td>'.$status.'</td>'. '<td>'.&Apache::lonlocal::locallocaltime($cmtime).'</td>'. - "</tr>\n"); + &Apache::loncommon::end_data_table_row() + ); return OK; }