🚀 go-pugleaf

> If anyone can knock a hole in the above script, I'd be very interested in hearing about it.
>   

What a fun  challenge! ;)

Besides the unnecessary case (which you mentioned already):

1) AFAIK, every object has a valid parent, even the app itself has
itself as a parent -- itself. There's no way to get through the loop a
second time with "null".

I would do a while(theObj) without any conditions because once you made
it in, you can't get out without either returning null or getting your
page. I'd just add a case of Page and return theObj and at the end of
the function return null

2) Also, "if (theObj == null)" is probably not technically correct. "if
(theObj == undefined)" is more accurate (since the only way to get there
is with an undefined text frame). (Although I'd just use if(!theObj) for
the sake of brevity...)

3) It does not deal with references to Footnotes (although it will work 
if given a reference to the text inside a footnote).

4) Your function does not deal with notes. It will not find the page a 
note is on, and will error if given a reference to text inside a note.

5) You can make the function infinitely more useful by adding a second
argument which could accept any InDesign object which can contain
objects (like Spread, MasterSpread, TextFrame, etc.) I usually want to
find the location of the last text frame on overset text, so I prefer to
use (a variation of) jxswm's version of the function (which finds the
last text frame).


-- 
Harbs
http://www.in-tools.com