--- loncom/interface/lonsource.pm	2007/08/28 22:46:32	1.20
+++ loncom/interface/lonsource.pm	2011/10/25 19:11:52	1.26
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
-# Souce Code handler
+# Source Code handler
 #
-# $Id: lonsource.pm,v 1.20 2007/08/28 22:46:32 albertel Exp $
+# $Id: lonsource.pm,v 1.26 2011/10/25 19:11:52 www Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -50,26 +50,25 @@ sub make_link {
 }
 
 sub stage_2 {
-    my ($r, $filename, $author, $listname) = @_;
-    my ($uname, $udom) = &Apache::loncacc::constructaccess('/~'.$author.'/',$r->dir_config('lonDefDomain'));
-    $r->print(&Apache::loncommon::start_page('Problem source code moving operation'));
-    $r->print("Please enter the directory that you would like the source code to go into, a default has also been 			provided <br />");
-    $r->print("Also note, the path is in reference to the root of your construction space, and new directories will be 			automatically created. <br /><br />");
+    my ($r, $filename, $listname) = @_;
+    my ($author)=($filename=~/\/res\/[^\/]+\/([^\/]+)\//);
+    $r->print(&Apache::loncommon::start_page('Copy Problem Source Code to Construction Space')
+             .&mt('Please enter the directory that you would like the source code to go into.')
+             .'<p>'
+             .&mt('Note: the path is in reference to the root of your construction space,'
+                 .' and new directories will be automatically created.')
+             .'</p>');
     $r->print('<form name="copy" action="/adm/source" target="_parent" method="post">
               <input type="hidden" name="filename" value="'.$filename.'" />
               <input type="hidden" name="listname" value="'.$listname.'" />
               <input type="hidden" name="action" value="copy_stage" />
-              <input type="text" size="50" name="newpath" value="/shared_source/'.$author.'" />&nbsp;
-              <input type="submit" value="Copy" />
+              <input type="text" size="50" name="newpath" value="/'.&mt('shared_source').'/'.$author.'" />&nbsp;
+              <input type="submit" value="'.&mt('Copy').'" />
               </form>');
     return OK;
 }
 
-
-sub copy_stage {
-    my ($r, $filename, $listname, $newpath) = @_;
-
-#Figure out if we are author or co-author
+sub copy_author {
     my $role;
     my $domain;
     my $author_name;
@@ -80,11 +79,25 @@ sub copy_stage {
         $domain = $env{'user.domain'};
         $author_name = $env{'user.name'};
     }
+    return ($role,$author_name,$domain);
+}
+
+
+sub copy_stage {
+    my ($r, $filename, $listname, $newpath) = @_;
+
+#Figure out if we are author or co-author
+    my ($role,$author_name,$domain)=&copy_author();
+
+# Construct path to copy and filter out any possibly nasty stuff
+    my $path_to_new_file = '/home/httpd/html/priv/'.$domain.'/'.$author_name.'/'.$newpath.'/'.$listname;
+    $path_to_new_file=~s/\.\.//g;
+    $path_to_new_file=~s/\~//g;
+    $path_to_new_file=~s/\/+/\//g;
 
-    my $path_to_new_file = '/home/'.$author_name.'/public_html/'.$newpath.'/'.$listname;
 #Just checking again for access as we want to make sure that it is really ok now that we have the real path
 
-    my ($uname,$udom)= &Apache::loncacc::constructaccess($path_to_new_file,$domain);
+    my ($uname,$udom)= &Apache::loncacc::constructaccess($path_to_new_file);
 
     if (!$uname || !$udom) {
 	$r->print(&Apache::loncommon::start_page('Not Allowed'));
@@ -93,29 +106,27 @@ sub copy_stage {
 	return;
     }
 
-
     #allowed
     $r->print(&Apache::loncommon::start_page('Copying Source'));
     my $result = &Apache::loncfile::exists($uname, $udom, $path_to_new_file);
     $r->print($result);
     if(($result) && ($result =~ m|published|) ) {
-	&delete_copy_file($r, $author_name, $newpath, $filename, $path_to_new_file, '1');
+	&delete_copy_file($r, $newpath, $filename, $path_to_new_file, '1');
     } elsif(($result) && ($result =~ m|exists!|)) {
-	&confirm($r, $author_name, $newpath, $filename, $path_to_new_file);
+	&confirm($r, $newpath, $filename, $path_to_new_file);
     } else {
-	&copy_file($r, $author_name, $newpath, $filename, $path_to_new_file);
+	&copy_file($r, $newpath, $filename, $path_to_new_file);
     }
     
     $r->print(&Apache::loncommon::end_page());
 }
 
 sub confirm {
-    my ($r, $author_name, $newpath, $filename, $path_to_new_file) = @_;
+    my ($r, $newpath, $filename, $path_to_new_file) = @_;
     $r->print("<b>Press delete to remove file and replace it with a copy of the source you are viewing</b><br /><br /	>");
     $r->print('<form name="delete_confirm" action="/adm/source" target="_parent" method="post">
               <input type="hidden" name="filename" value="'.$filename.'" />
               <input type="hidden" name="path" value="'.$path_to_new_file.'" />
-              <input type="hidden" name="author" value="'.$author_name.'" />
               <input type="hidden" name="newpath" value="'.$newpath.'" />
               <input type="hidden" name="action" value="delete_confirm" />
 
@@ -124,30 +135,40 @@ sub confirm {
 }
 
 sub delete_copy_file {
-    my ($r, $author_name, $newpath, $filename, $path_to_new_file, $type) = @_;
+    my ($r, $newpath, $filename, $path_to_new_file, $type) = @_;
     if($type eq '1') {
-        $r->print("<b>Cannot delete non-obsolete published file</b><br />Please
-              use the code view in previous window to use shared code<br /><br />");
-        $r->print('<input type="button" value="Close Window" name="close" onClick="window.close()" />');
+        $r->print('<p><span class="LC_warning">'
+                 .&mt('Cannot delete non-obsolete published file.')
+                 .'</span><br />'
+                 .&mt('Please use the code view in previous window to use shared code.')
+                 .'<br /><br />');
+        $r->print('<input type="button" value="Close Window" name="close"'
+                 .' onclick="window.close()" />'
+                 .'</p>');
     } else {
         if(-e $path_to_new_file) {
             unless(unlink($path_to_new_file)) {
-                $r->print('<font color="red">'.&mt('Error').': '.$!.'</font>');
+                $r->print('<p class="LC_error"">'.&mt('Error:').' '.$!.'</p>');
                 return 0;
             }
         } else {
-            $r->print('<p> '.&mt('No such file').'. </p></form>');
+            $r->print('<p class="LC_error">'.&mt('No such file').'</p></form>');
             return 0;
         }
-        &copy_file($r, $author_name, $newpath, $filename, $path_to_new_file);
+        &copy_file($r, $newpath, $filename, $path_to_new_file);
     }
 }
 
 sub copy_file {
-    my ($r, $author_name, $newpath, $filename, $path_to_new_file) = @_;
+    my ($r, $newpath, $filename, $path_to_new_file) = @_;
     $r->print("<b>Creating directories</b>");
-    my $path = '/home/'.$author_name.'/public_html/';
+
+#Figure out if we are author or co-author
+    my ($role,$author_name,$domain)=&copy_author();
+
+    my $path = '/home/httpd/html/priv/'.$domain.'/'.$author_name.'/';
     my @directories = split(/\//,$newpath);
+
     foreach my $now_checking (@directories) {
         if($now_checking ne '') {
             $path = $path.'/'.$now_checking;
@@ -155,11 +176,11 @@ sub copy_file {
 
             else {
                 unless(mkdir($path, 02770)) {
-                    $r->print('<font color="red">'.&mt('Error').': '.$!.'</font>');
+                    $r->print('<p class="LC_error">'.&mt('Error:').' '.$!.'</p>');
                     return 0;
                 }
                 unless(chmod(02770, ($path))) {
-                    $r->print('<font color="red"> '.&mt('Error').': '.$!.'</font>');
+                    $r->print('<p class="LC_error"> '.&mt('Error:').' '.$!.'</p>');
                     return 0;
                 }
             }
@@ -174,7 +195,7 @@ sub copy_file {
         print $fs $file_output;
     }
     $r->print("<br /><br />");
-    $r->print('<input type="button" value="Close Window" name="close" onClick="window.close()">');
+    $r->print('<input type="button" value="Close Window" name="close" onclick="window.close()" />');
     #Some 1.3'ish feature is to include the derivative feature, will go here..'
 }
 
@@ -236,25 +257,18 @@ sub handler {
     $r->send_http_header;
 
     if ($env{'form.action'} eq 'stage2') {
-        my $author = &Apache::lonnet::metadata($filename,'authorspace');
-	#strip the domain of the author name
-	if ($author =~ /:/) {
-	    ($author) = split(/:/,$author);
-	} else {
-	    ($author) = split(/@/,$author);
-	}
-        &stage_2($r, $env{'form.filename'}, $author, $listname);
+        &stage_2($r, $filename,$listname);
     } elsif($env{'form.action'} eq 'copy_stage') {
         &copy_stage($r, $filename,$env{'form.listname'},$env{'form.newpath'});
     } elsif($env{'form.action'} eq 'delete_confirm') {
-        &delete_copy_file($r, $env{'form.author'}, $env{'form.newpath'}, $env{'form.filename'}, $env{'form.path'}, '0');
+        &delete_copy_file($r, $env{'form.newpath'}, $env{'form.filename'}, $env{'form.path'}, '0');
     } else {
 	$r->print('<form name="copy" action="/adm/source" target="_parent" method="post">
-                  <input type="button" value="Close Window" name="close" onClick="window.close()" />
+                  <input type="button" value="'.&mt('Close Window').'" name="close" onclick="window.close()" />
                   <input type="hidden" name="filename" value="'.$filename.'" />
                   <input type="hidden" name="listname" value="'.$listname.'" />
                   <input type="hidden" name="action" value="stage2" />
-                  <input type="submit" value="Copy to CSTR" />
+                  <input type="submit" value="'.&mt('Copy to Construction Space').'" />
                   </form>');
         $r->print('<hr />');
         &print_item($r, $env{'form.filename'});