🚀 go-pugleaf

Michael,

You can optimise two things in your code. First, avoid referring to collections (such as myTable.rows and myTable.columns) repeatedly. Instead, create arrays from collections. Earlier this week this was discussed here: Harbs, "[JS] :: [CSX] - PageItem Stacking Order" #15, 6 Jan 2009 11:23 am </webx?14@@.3bbf275d.59b767cf/14> . see especially the reference to Kris Coppetier's article in Harbs's post, who explains that each time you refer to a collection, InDesign rebuilds it -- so you can imagine that rebuilding a big table such as yours causes the script to run for hours.

In your case, instead of referring to myTable.rows.length-1, do this:

myRows = myTable.rows.everyItem().getElements();
myRows_length = myRows.length-1;
for (var f = myRows; . . .

Secondly, you needn't refer to myRows[f].columns[...].cells[...]: myRows[...].cells[...] will do too. So your script coulkd look like this:

myRows = myTable.rows.everyItem().getElements(); 
myRows_length = myRows.length-1; 
for (var f = myRows_length; f >= 0; f--) 
   myRows[f].cells[3].merge (myRows[f].cells[4]);

This will probably speed things up.

Peter