Changeset 30620

Timestamp:

2016-07-25T14:44:07+12:00 (8 years ago)

Author:

ak19

Message:

The %s in the perlmodule strings was being replaced with the entire contents of the buffer when it was written out, when generating excel spreadsheet xml. So use fwrite instead of printf, as it doesn't do special formatting

File:

• main/trunk/greenstone2/runtime-src/src/recpt/gtiaction.cpp (modified) (2 diffs)

main/trunk/greenstone2/runtime-src/src/recpt/gtiaction.cpp

@@ -663 +663 @@
 
     // don't actually need to add toplevel gsdlhome to classpath in the following:
-
+    
     gti_arguments = "get-first-n-chunks-requiring-work " + target_language_code + " " + translation_file_key + " " + "10000" + " | " + java + " -cp " + gsdlhome + path_separator + gsdlhome + "/bin/java/ApplyXSLT.jar org.nzdl.gsdl.ApplyXSLT -c -t " + gsdlhome + "/bin/script/gti-generate-excel-xml.xsl";
 
@@ -681 +681 @@
   }
 
-  // Write the Excel spreadsheet data to the browser
+    // Write the Excel spreadsheet data to the browser
   char* gti_response_xml_text_cstr = gti_response_xml_text.getcstr();
-  printf(gti_response_xml_text_cstr);
+  // use fwrite instead of printf as there are %s in the strings
+  fwrite(gti_response_xml_text_cstr, 1, strlen(gti_response_xml_text_cstr), stdout);
   delete[] gti_response_xml_text_cstr;