how to use electron latest in electron-dl about electron-dl

[sKY02644]

Any solution to this please?

Facing the same issue

[corwin-of-amber]

@integemjack can you provide a description of the error?

I am trying the following:

ipcMain.on('download-request', async (event, {url}) => {
  console.log('requesting to download', url);
  try {
    const win = BrowserWindow.getFocusedWindow();
    await download(win, url);
  }
  catch (e) { console.error('(download)', e); }
});

and I'm getting this popup:

Uncaught Exception:
TypeError: Cannot read properties of null (reading 'getOwnerBrowserWindow')
at Function.n.fromWebContents (node:electron/js2c/browser_init:33:1573)
at Session.listener (/Users/corwin/var/workspace/Teaching.Web.GR/node_modules/electron-dl/index.js:68:64)
at Session.emit (node:events:406:35)

The exception is not caught by my try .. catch because apparently it occurs in a tasklet.

[Kiyozz]

@integemjack can you provide a description of the error?

I am trying the following:

ipcMain.on('download-request', async (event, {url}) => {
  console.log('requesting to download', url);
  try {
    const win = BrowserWindow.getFocusedWindow();
    await download(win, url);
  }
  catch (e) { console.error('(download)', e); }
});

and I'm getting this popup:

Uncaught Exception:
TypeError: Cannot read properties of null (reading 'getOwnerBrowserWindow')
at Function.n.fromWebContents (node:electron/js2c/browser_init:33:1573)
at Session.listener (/Users/corwin/var/workspace/Teaching.Web.GR/node_modules/electron-dl/index.js:68:64)
at Session.emit (node:events:406:35)

The exception is not caught by my try .. catch because apparently it occurs in a tasklet.

const win is for sure null. That is why you get this error.

[corwin-of-amber]

Thanks for the reply. But that's exactly what the docs say:
https://github.com/sindresorhus/electron-dl#use-it-manually

Which also worked with a previous version of Electron, and this is why I attached my excerpt to this issue.
Why would BrowserWindow.getFocusedWindow return null?

I went ahead and printed it. It is not null.

[Kiyozz]

I use latest electron-dl and Electron in my project. No issue so far.

Sample code

win is injected through function parameter, not with getFocusedWindow

import dl from 'electron-dl'  

void dl.download(this.win, mod.url, {
        filename: mod.name,
        onStarted: item => {
          modToUse = this.updateDownloadInStore(modToUse, {
            localFilePath: item.getSavePath(),
          })
          this.addDownloadInStore(modToUse, item)
          this.setPreviousTime(modToUse)
          this.startInterval(mod.id)
          resolve(modToUse)
        },
        onProgress: () => {
          this.sendIpcEvents.progressModDownload(this.modDownloadStore.get())
        },
        onCompleted: () => {
          modToUse = this.updateDownloadInStore(modToUse, {
            state: DownloadState.finish,
            downloadedSize: mod.size,
          })

          this.sendIpcEvents.finishModDownload(this.modDownloadStore.get())
          this.deletePreviousTime(modToUse)
        },
      })
    })

this.win: BrowserWindow
mod.url: string

[corwin-of-amber]

@Kiyozz thanks, but what is this.win in your case? Is this code part of a class definition (since you are using this)?

[Kiyozz]

Class method

startDownloadOfMod(mod: BaseModDownload): Promise<BaseModDownload> {
    // ...

    return new Promise<BaseModDownload>(resolve => {
      void dl.download(this.win, mod.url, {
        filename: mod.name,
        onStarted: item => {
          modToUse = this.updateDownloadInStore(modToUse, {
            localFilePath: item.getSavePath(),
          })
          this.addDownloadInStore(modToUse, item)
          this.setPreviousTime(modToUse)
          this.startInterval(mod.id)
          resolve(modToUse)
        },
      })
    })
  }

Class constructor

constructor(
    private http: HttpManager,
    private modDownloadStore: ModDownloadStore,
    private win: BrowserWindow,
    private sendIpcEvents: SendIpcEvents,
  ) {}

So in my main file, I create the window. I use a function I called initialize.

await initialize(win)

In initialize, I create the instance and I inject it.

const downloadManager = new DownloadManager(
    httpManager,
    modDownloadStore,
    win,
    sendIpcEvents,
  )

[corwin-of-amber]

Can anyone try this with their code base? (I kind of stopped using electron-dl in my code as a result of this issue; @OfekA-IAI, @sKY02644, perhaps?)