1 Correct answer
Here is a version of the script that adds a progress bar (which apparently doesn't always show on certain versions of Mac). In a clean new document with just 2,500 phone numbers the script runs almost a minute on my (almost new, well-specced) PC (in CC2015 and 2017). The progress bar shows that the script tears off furiously, then slows down after about 500 links, and crawls to the finish. But do try a test on a document with a few dozen numbers.
Peter
...#targetengine phone_numbers;
(function () {
36
Replies
Really cool stuff pkharel!!! 
There just might be a small mistake in the GREP query \(\d\d\d\)\s\d\d\d\d\d\d\d
Space should be replaced by dash \(\d\d\d\)-\d\d\d\d\d\d\d
Additional question: why do you close the hyperlink panel? Is there a specific reason for that?
Thanks for spotting that, Vinny. But you corrected it wrong 
. It should be
\(\d\d\d\)\s\d\d\d-\d\d\d\d
> Additional question: why do you close the hyperlink panel? Is there a specific reason for that?
It speeds up the script because InDesign doesn't have to update the panel.
P.
Peter - I've added the one line above to the script provided six days ago and I keep getting this popup when I attempt to run the revised script. Does it make sense to you?
This is exactly how I have the script with the one line added:
(function () {
var i;
var w;
var cStyle;
var found;
var source;
var destination;
app.scriptPreferences.enableRedraw = true;
try {
app.panels.item('$ID/Hyperlinks').visible = false;
} catch (_) {
}
if (!app.documents[0].characterStyles.item ('Hyperlink').isValid) {
app.documents[0].characterStyles.add ({name: 'Hyperlink', underline: true});
}
cStyle = app.documents[0].characterStyles.item ('Hyperlink');
app.findGrepPreferences = null;
app.findGrepPreferences.findWhat = /\(\d\d\d\)\s\d\d\d-\d\d\d\d/.source;
found = app.documents[0].findGrep();
w = new Window ('palette {text: "Phone URLs"}');
w.pbar = w.add ('progressbar', undefined, 0, found.length);
w.pbar.preferredSize.width = 300;
w.show();
for (i = 0; i < found.length; i++) {
w.pbar.value = i+1;
source = app.documents[0].hyperlinkTextSources.add (found, {appliedCharacterStyle: cStyle});
destination = app.documents[0].hyperlinkURLDestinations.add ({
destinationURL: 'tel:' + found.contents.replace(/\D/g,'')
});
app.documents[0].hyperlinks.add (source, destination);
}
try {
w.close();
} catch (_) {
}
}());
Can this be fixed?
~Stephanie
Darn - I forgot to send the script with the added line. OK, so this is the revised script that I actually used that is giving me that JavaScript error popup above:
(function () {
var i;
var w;
var cStyle;
var found;
var source;
var destination;
app.scriptPreferences.enableRedraw = true;
app.documents[0].hyperlinks.everyItem().remove();
try {
app.panels.item('$ID/Hyperlinks').visible = false;
} catch (_) {
}
if (!app.documents[0].characterStyles.item ('Hyperlink').isValid) {
app.documents[0].characterStyles.add ({name: 'Hyperlink', underline: true});
}
cStyle = app.documents[0].characterStyles.item ('Hyperlink');
app.findGrepPreferences = null;
app.findGrepPreferences.findWhat = /\(\d\d\d\)\s\d\d\d-\d\d\d\d/.source;
found = app.documents[0].findGrep();
w = new Window ('palette {text: "Phone URLs"}');
w.pbar = w.add ('progressbar', undefined, 0, found.length);
w.pbar.preferredSize.width = 300;
w.show();
for (i = 0; i < found.length; i++) {
w.pbar.value = i+1;
source = app.documents[0].hyperlinkTextSources.add (found, {appliedCharacterStyle: cStyle});
destination = app.documents[0].hyperlinkURLDestinations.add ({
destinationURL: 'tel:' + found.contents.replace(/\D/g,'')
});
app.documents[0].hyperlinks.add (source, destination);
}
try {
w.close();
} catch (_) {
}
}());
So then I went ahead and added the first line, #targetengine phone_numbers;, and then got this error message:
Does this make any sense to you? Ideally I would LOVE to have everything run in just one script - would definitely be more convenient!
Thanks so much!
~Stephanie
AdChoices