Electron event fires multiple times

Question

I have one page, that load new elements via jquery. I have page that have event listener like

ipcRenderer.send('getlist');
ipcRenderer.once('return:list', function (e, l, wn) {
    console.log('a');
    for(let i = 0; i < l.length; i++) {
        $('.a').append('<div>'+l[i]+wn[i]'</div>');
    }
    $('.a').append('<div>End</div>');
});

Code that send data to page:

ipcMain.on('getlist', function (e) {
    wn = [];
    cfg['a'].forEach(s => {
        wn.push(yaml.readSync('charcfg.yaml')['name']);
    });
    mainWindow.webContents.send('return:list', cfg['a'], wn);
});

And it's works perfectly until I load another page, and again load this page. This event fires multiple times. Each time I come back to this page, more times it goes.

I tried to use

ipcRenderer.once('someevent', function(e, l){...});

It's works only ONE TIME. On third reload it's starts do listener multiple times again.

Code with this script loads with page!

Sorry for my english.Output

Function that loads page via jquery:

const htmlContent = $('.content');
function setContent(s) {
    $('.content').animate({
        opacity: 0
    }, 200, function () {
        setTimeout(function () {
            htmlContent.load(s + '.html');
            $('.content').animate({
                opacity: 1
            }, 200);
        }, 100);
    });
}

What exactly is happening in your handlers that happens multiple times? More specific context would be helpful. — obermillerk, Jun 16 '18 at 20:23
It's dynamically adds content on page, using data that receives from event — SoundOfTheSky, Jun 16 '18 at 20:26
It'd be better if you edit your post so I can see it formatted, easier to interpret. But What exactly is happening multiple times? The append or the whole handler? — obermillerk, Jun 16 '18 at 20:28
Okay, I edited my post. Multiple times appends text that recieves from event. — SoundOfTheSky, Jun 16 '18 at 20:30
So is there a list involved like your initial comment or is it just a static append? — obermillerk, Jun 16 '18 at 20:32
List. There foreach() to add all elements that variable l contains. — SoundOfTheSky, Jun 16 '18 at 20:33
So that should be what your edit shows, it's very different. I would check your main side of the ipc. I suspect you're not resetting the list that you're sending, so it just grows every time. — obermillerk, Jun 16 '18 at 20:34
No, this list resetting and sending only needed data. There only problem with listener that calls multiple times. — SoundOfTheSky, Jun 16 '18 at 20:38
If you could post your full listener and maybe some example of what it's showing or a console.log it'd be much easier to understand what you're saying. — obermillerk, Jun 16 '18 at 20:39
Yes, I added output in the end of post. Each time i open and close this window new 'a' adds. it's like 5,6,7,8 'a'. — SoundOfTheSky, Jun 16 '18 at 20:47
Is that renderer code only being run once per page or is it in some other function that can be run multiple times? — obermillerk, Jun 16 '18 at 20:51
Also, maybe try using ipcRenderer.removeAllListeners on page unload, might help I don't know. — obermillerk, Jun 16 '18 at 20:52
That code runs each time i load document to page with this code. I load html to another div via jquery — SoundOfTheSky, Jun 16 '18 at 20:53
I tried to use removeAllListener but it's same as i do .once() — SoundOfTheSky, Jun 16 '18 at 20:54
So wait, is this renderer code not in it's own page loaded by electron but in something that you load with jquery? — obermillerk, Jun 16 '18 at 20:55
Yes, this is only one page. I load some.html with this code each time via jquery and don't reload it. Script on some.html everytime do thing as i load page. — SoundOfTheSky, Jun 16 '18 at 20:57
So theres page page1.html that loads page2.html (which contains this code) via jquery, or page1.html itself has this code in it? — obermillerk, Jun 16 '18 at 20:59
Yes, first thing. page1.html that loads page2.html (which contains this code) — SoundOfTheSky, Jun 16 '18 at 21:00
Did you try using removeAllListeners at the beginning of that setContent function? — obermillerk, Jun 16 '18 at 21:07
No, i'll do that tomorrow. Thanks for your support. I will write here tomorrow, no matter it works or not. Good night — SoundOfTheSky, Jun 16 '18 at 21:11
With a clear head after sleep, I figured out the problem. Actually there was TWO listeners that go twice. It's ipcRenderer.on(replaced to .once) and jqury.on('click) that was binded to button that loads page. Fixed this with jquery.unbind().on('click'). — SoundOfTheSky, Jun 17 '18 at 08:32
I'm facing the same problem but in different scenario.If anybody could help. [Ticket](https://stackoverflow.com/questions/63906065/electron-js-event-is-firing-multiple-times?noredirect=1#comment113006699_63906065) — Abhishek Matta, Sep 16 '20 at 17:51

score 4 · Answer 1 · edited Jun 17 '18 at 18:44

4

Actually there was TWO listeners that go twice.

It's ipcRenderer.on (replaced to .once) and jquery.on('click') that was bound to button that loads page. Fixed this with jquery.unbind().on('click').

Thanks for your help, obermillerk.

edited Jun 17 '18 at 18:44

pushkin

9,575
15
51
95

answered Jun 17 '18 at 08:34

SoundOfTheSky

99
1
10

Just wanted to add that in my case I've included `renderer.js` in my html manually and it was already getting included by `main.js` so that's why it fired twice for me. – Miro Sep 29 '21 at 19:14

Electron event fires multiple times

1 Answers1

Linked