Bug in script for differencing docx files

I believe I've found a bug in the diff-doc.js script (located at C:\Program
Files\TortoiseSVN\Diff-Scripts\diff-doc.js on my machine), used to launch
MS Word or OpenOffice as a differencing tool for docx files (among other
file types, I imagine). Today, I started receiving an error dialog telling
me that I must have MS Word or OpenOffice installed when I tried to do a
difference on a docx file. I most definitely have MS Word 2010 installed
properly. I did some digging and believe I traced the error to line 66:

    if(parseInt(word.Version) >= vOffice2013)

I believe the variable vOffice2013 is undefined, as adding the following
line immediately following the definition of vOffice2007 (line 24) fixed my
issue:

var vOffice2013 = 15;

It is interesting to note that there is also no definition for a 2010
version variable (which I believe to be 14), either. Furthermore, it seems
strange to me that the check against vOffice2013 occurs inside the
try/catch block when it appears that other, similar version checking code
is placed outside the try/catch block (see line 130).

I am not certain, but I believe that the change related to this post may be
the source of the bug:
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=2872557
.

Thank you for looking into this.

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=3052877

To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2013-04-09 21:31:03 CEST