Crash ID 316

Opus crashed (Crash ID 316) while using the script AutoOpenPairedFolder (Add-in) by Ixp. Actually the script was running fine till I made little edit to code:

I put // before the line: ' NEWTAB=findexisting' +

in order to open the paired folder in the same tab instead of a new tab.

The problem is partly because the script is reacting to itself in an endless feedback loop if you remove the NEWTAB=findexisting line.

You can fix it by adding the NOSCRIPT argument to the command it runs:

    cmd.RunCommand('Go' +
        ' PATH="' + pairedFolder.path + '"' +
       // ' NEWTAB=findexisting' +
        ' OPENINDUAL NOSCRIPT');
1 Like

Thanks @Leo for pointing out the reason for the crash and giving me the solution. Thanks @lxp for providing us very useful script. The crash occurred due to my fault. I should not have modified the script without fully knowing it.

Don't worry, breaking stuff is the best way to learn. :smiley:

FWIW, I couldn't reproduce the crash when I tried, but it may depend what you click and when after the two file displays are endlessly reacting to each other. (It's also possible some unreleased changes avoid the crash, as we've made some general stability improvements for the next update; not sure though. Once the lister is in that endless loop, it's not much use either way, of course.)

@Jon @Leo, If possible it would be nice if Opus can detects if the script results in an endless feedback loop and instead of crashing, it stops the script and shows an error dialog.

That could be difficult in general, without getting in the way of legitimate scripts.

You should! Bug and error reports make scripts better for everyone. I hope the endless loop didn't cause too much trouble.

Me neither :man_shrugging:

1 Like