Changeset 9755 for trunk/gli/src/org/greenstone/gatherer/gui/ExportAsPrompt.java

Timestamp:

2005-04-27T11:17:29+12:00 (19 years ago)

Author:

mdewsnip

Message:

Replaced all the nonsense where string arrays were created for running perl scripts with array lists. This removes a lot of duplicated code where the same things were added in different cases, and the silly (and error-prone) "new String[11]"s.

File:

• trunk/gli/src/org/greenstone/gatherer/gui/ExportAsPrompt.java (modified) (1 diff)

trunk/gli/src/org/greenstone/gatherer/gui/ExportAsPrompt.java

@@ -278 +278 @@
     cd_title = title_field.getText();
     cd_title = cd_title.trim();
-    cd_title = cd_title.replaceAll("\"","");
-    int num_args = num_collections + 9;
-    boolean has_title = false;
-    if (!cd_title.equals("")) {
-        has_title = true;
-    }
-    String args[];
-    int next_arg = 0;
-    if(Utility.isWindows()) {
-        args = new String[num_args+2];
-        args[0] = Configuration.perl_path;
-        args[1] = "-S";
-        next_arg = 2;
-    } else {
-        args = new String[num_args];
+    cd_title = cd_title.replaceAll("\"", "");
+
+    String export_type = (String) saveas_combobox.getSelectedItem();
+
+    // Generate the export.pl command
+    ArrayList command_parts_list = new ArrayList();
+    if (Utility.isWindows() && (!Gatherer.isGsdlRemote)) {
+        command_parts_list.add(Configuration.perl_path);
+        command_parts_list.add("-S");
+    }
+    command_parts_list.add(Configuration.getScriptPath() + "export.pl");
+    command_parts_list.add("-gli");
+    command_parts_list.add("-language");
+    command_parts_list.add(Configuration.getLanguage());
+    command_parts_list.add("-removeold");
+    command_parts_list.add("-saveas");
+    command_parts_list.add(export_type);
+    command_parts_list.add("-exportdir");
+    command_parts_list.add(Configuration.gsdl_path + "tmp" + File.separator);
+
+    if (cd_title.equals("")) {
+        command_parts_list.add("exported_" + export_type);
+    }
+    else {
+        command_parts_list.add("exported_" + cd_title.replaceAll("\\s", ""));
     }
     
-    String export_type = (String)saveas_combobox.getSelectedItem();
-
-    args[next_arg] = Configuration.getScriptPath() + "export.pl";
-    next_arg++;
-    args[next_arg] = "-gli"; next_arg++; 
-    args[next_arg] = "-language"; next_arg++;
-    args[next_arg] = Configuration.getLanguage(); next_arg++;
-    args[next_arg] = "-removeold"; next_arg++;
-    args[next_arg] = "-saveas"; next_arg++;
-    args[next_arg] = export_type; next_arg++;   
-
-    args[next_arg] = "-exportdir"; next_arg++;
-    args[next_arg] = Configuration.gsdl_path+"tmp"+File.separator;
-    if(!has_title) {
-        args[next_arg] += "exported_" + export_type;
-    } else {
-        args[next_arg] += "exported_"+cd_title.replaceAll("\\s","");
-    }
-    next_arg++;
-
-    
-    for (int i=0; i<num_collections;i++,next_arg++) {
-        args[next_arg] = ((BasicCollectionConfiguration)selected_collections.get(i)).getShortName();
+    for (int i = 0; i < num_collections; i++) {
+        command_parts_list.add(((BasicCollectionConfiguration) selected_collections.get(i)).getShortName());
     }
 
     DebugStream.print("export command = ");
-    for (int i=0; i<args.length; i++) {
-        DebugStream.print(args[i]+" ");
-        System.err.print(args[i]+" ");
+    for (int i = 0; i < command_parts_list.size(); i++) {
+        DebugStream.print(command_parts_list.get(i) + " ");
+        System.err.print(command_parts_list.get(i) + " ");
     }
     DebugStream.println("");
     System.err.println("");
 
-    GShell process = new GShell(args, GShell.EXPORTAS, 3, this, null, GShell.GSHELL_EXPORTAS);
+    // Run the export.pl command
+    String[] command_parts = (String[]) command_parts_list.toArray(new String[0]);
+    GShell process = new GShell(command_parts, GShell.EXPORTAS, 3, this, null, GShell.GSHELL_EXPORTAS);
     process.start();
     //process.run();