--- loncom/interface/lonsource.pm	2007/08/28 22:23:22	1.19
+++ loncom/interface/lonsource.pm	2009/06/09 13:50:03	1.23
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
-# Souce Code handler
+# Source Code handler
 #
-# $Id: lonsource.pm,v 1.19 2007/08/28 22:23:22 albertel Exp $
+# $Id: lonsource.pm,v 1.23 2009/06/09 13:50:03 bisitz Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -51,19 +51,19 @@ sub make_link {
 
 sub stage_2 {
     my ($r, $filename, $author, $listname) = @_;
-    $filename = $filename;
-    &Apache::loncommon::content_type($r,'text/html');
     my ($uname, $udom) = &Apache::loncacc::constructaccess('/~'.$author.'/',$r->dir_config('lonDefDomain'));
-    $r->send_http_header;
-    $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 />");
+    $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;
 }
@@ -71,11 +71,11 @@ sub stage_2 {
 
 sub copy_stage {
     my ($r, $filename, $listname, $newpath) = @_;
+
+#Figure out if we are author or co-author
     my $role;
     my $domain;
     my $author_name;
-#Figure out if we are author or co-author
-
     if ($env{'request.role'} =~ m{^ca\.}) {
         ($role, $domain, $author_name) = split(/\//,$env{'request.role'});
     } else {
@@ -88,24 +88,28 @@ sub copy_stage {
 #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);
-    unless (($uname) && ($udom)) {
-        return HTTP_NOT_ACCEPTABLE;
+
+    if (!$uname || !$udom) {
+	$r->print(&Apache::loncommon::start_page('Not Allowed'));
+	$r->print(&mt('Not allowed to create file [_1]', $path_to_new_file));
+	$r->print(&Apache::loncommon::end_page());
+	return;
     }
-    &Apache::loncommon::content_type($r,'text/html');
-    $r->send_http_header;
+
+
+    #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, $author_name, $newpath, $filename, $path_to_new_file, '1');
     } elsif(($result) && ($result =~ m|exists!|)) {
-        &confirm($r, $author_name, $newpath, $filename, $path_to_new_file);
+	&confirm($r, $author_name, $newpath, $filename, $path_to_new_file);
     } else {
-        &copy_file($r, $author_name, $newpath, $filename, $path_to_new_file);
+	&copy_file($r, $author_name, $newpath, $filename, $path_to_new_file);
     }
-
-    return OK;
-
+    
+    $r->print(&Apache::loncommon::end_page());
 }
 
 sub confirm {
@@ -125,17 +129,22 @@ sub confirm {
 sub delete_copy_file {
     my ($r, $author_name, $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);
@@ -154,11 +163,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;
                 }
             }
@@ -173,7 +182,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..'
 }
 
@@ -230,25 +239,30 @@ sub handler {
         $env{'user.error.msg'}="$filename:bre:1:1:Access to resource denied";
         return HTTP_NOT_ACCEPTABLE;
     } 
+
+    &Apache::loncommon::content_type($r,'text/html');
+    $r->send_http_header;
+
     if ($env{'form.action'} eq 'stage2') {
         my $author = &Apache::lonnet::metadata($filename,'authorspace');
-        ($author) = split('@',$author); #strip the domain of the author name
+	#strip the domain of the author name
+	if ($author =~ /:/) {
+	    ($author) = split(/:/,$author);
+	} else {
+	    ($author) = split(/@/,$author);
+	}
         &stage_2($r, $env{'form.filename'}, $author, $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') {
-        &Apache::loncommon::content_type($r,'text/html');
-        $r->send_http_header;
         &delete_copy_file($r, $env{'form.author'}, $env{'form.newpath'}, $env{'form.filename'}, $env{'form.path'}, '0');
     } else {
-        &Apache::loncommon::content_type($r,'text/html');
-        $r->send_http_header;
-        $r->print('<form name="copy" action="/adm/source" target="_parent" method="post">
-                  <input type="button" value="Close Window" name="close" onClick="window.close()" />
+	$r->print('<form name="copy" action="/adm/source" target="_parent" method="post">
+                  <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'});