--- loncom/interface/lonpdfupload.pm	2012/09/14 11:24:06	1.22
+++ loncom/interface/lonpdfupload.pm	2015/06/09 21:22:57	1.25
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # PDF Form Upload Handler
 #
-# $Id: lonpdfupload.pm,v 1.22 2012/09/14 11:24:06 goltermann Exp $
+# $Id: lonpdfupload.pm,v 1.25 2015/06/09 21:22:57 damieng Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -122,6 +122,7 @@ sub handler() {
 sub get_javascripts() {
     
     my $message = &mt('Please choose a PDF-File.');
+    &js_escape(\$message);
 
     # simple test if the upload ends with ".pdf"
     # it's only for giving a message to the user
@@ -183,7 +184,7 @@ sub processPDF {
         &grade_pdf(@pdfdata);
     } else {
         $result .= '<p class="LC_error">'
-                  .&mt("Can't find any valid PDF formfields.")
+                  .&mt("Can't find any valid PDF form fields.")
                   .'</p>';
     }
 }
@@ -193,12 +194,12 @@ sub get_pdf_data() {
     my $pdf = CAM::PDF->new($env{'form.file'});
 
     if($pdf) {
-        my @formFields = $pdf->getFormFieldList(); #get names of formfields
+        my @formFields = $pdf->getFormFieldList(); #get names of form fields
 
         foreach my $field (@formFields) {
-            my $dict = $pdf->getFormFieldDict($pdf->getFormField($field)); # get formfield dictonary
+            my $dict = $pdf->getFormFieldDict($pdf->getFormField($field)); # get form field dictonary
 
-            # this is necessary because CAM::PDF has a problem with formfieldnames which include a
+            # this is necessary because CAM::PDF has a problem with form fieldnames which include a
             # dot in fieldnames. So a fieldname like "i.am.aFormfield" will offer three fieldnames
             # "i", "i.am" and "i.am.aFormfield". The fragmentary names keep no values and will be ignored.
             if($dict->{'V'}) {
@@ -380,7 +381,7 @@ sub parse_grade_answer {
                        'DRAFT' => &mt('Copy saved but not submitted.'),
      );
 
-    foreach my $key (keys %answerhash) {
+    foreach my $key (keys(%answerhash)) {
         if($shortcut eq $key) {
             return $answerhash{$shortcut};
         }  
@@ -394,12 +395,12 @@ sub dumpenv  {
     my $r = shift;
 
     $r->print ("<br />-------------------<br />");
-    foreach my $key (sort (keys %env)) {
+    foreach my $key (sort(keys(%env))) {
         $r->print ("<br />$key -> $env{$key}");
     }
     $r->print ("<br />-------------------<br />");
     $r->print ("<br />-------------------<br />");
-    foreach my $key (sort (keys %ENV)) {
+    foreach my $key (sort(keys(%ENV))) {
         $r->print ("<br />$key -> $ENV{$key}");
     }
     $r->print ("<br />-------------------<br />");