radiantmediaplayer / rmp-vast

A client-side JavaScript solution to load, parse and display VAST 2, 3 & 4 resources with HTML5 video
https://www.radiantmediaplayer.com/rmp-vast/app/
Other
95 stars 37 forks source link

Failed to fire an event because of console.dir() #51

Closed kamtungc closed 2 years ago

kamtungc commented 2 years ago

In v6.0.0, createApiEvent() in utils.js is below static createApiEvent(event) { if (Array.isArray(event)) { event.forEach(currentEvent => { if (currentEvent) { if (this.debug) { FW.log(currentEvent); } FW.createStdEvent(currentEvent, this.container); } }); } else if (event) { if (this.debug) { FW.log(event); } FW.createStdEvent(event, this.container); } }

In v7.1.0, createApiEvent() in utils.js is below static createApiEvent(event) { if (Array.isArray(event)) { event.forEach(currentEvent => { if (currentEvent) { console.dir(currentEvent); FW.createStdEvent(currentEvent, this.container); } }); } else if (event) { console.dir(event); FW.createStdEvent(event, this.container); } }

In v7.1.0, I got an error "Could not complete the operation due to error 80020101." for an event, such as adfirstquartile.

After I removed both console.dir(currentEvent); and console.dir(event);, I didn't have an error.

radiantmediaplayer commented 2 years ago

I cannot reproduce this issue on our side. Can you provide a code sample that reproduces this issue? What browser and what os are your testing on?

kamtungc commented 2 years ago

I am using UWP WebView.

radiantmediaplayer commented 2 years ago

This should be fixed in 7.2.0