LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Packages Projects Releases 238 Wiki Activity

Fix argv.length check in makeSingleInstance callback

Browse source 
Also correct length of argv: with Electron, there's just the
executable, not interpreter + filename.
This commit is contained in:
Alex Liebowitz 2017-05-10 21:27:57 -04:00
parent 856aa28db2
commit 2b6528ca3e
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
app/main.js
View file

@ -178,19 +178,19 @@ function quitNow() {
}
const isSecondaryInstance = app.makeSingleInstance((argv) => {
if (process.argv.length < 3) {
if (argv.length < 2) {
return;
}
if (!win) {
openUri = argv[2];
openUri = argv[1];
} else {
if (win.isMinimized()) {
win.restore();
win.focus();
}
win.webContents.send('open-uri-requested', argv[2]);
win.webContents.send('open-uri-requested', argv[1]);
}
});
@ -347,11 +347,11 @@ if (process.platform == 'darwin') {
win.webContents.send('open-uri-requested', uri);
}
});
} else if (process.argv.length >= 3) {
} else if (process.argv.length >= 2) {
// No open-url event on Win, but we can still handle URIs provided at launch time
if (!win) {
openUri = process.argv[2];
openUri = process.argv[1];
} else {
win.webContents.send('open-uri-requested', process.argv[2]);
win.webContents.send('open-uri-requested', process.argv[1]);
}
}