Skip to content
This repository was archived by the owner on Sep 28, 2025. It is now read-only.

Commit af0b0aa

Browse files
refactor(src): simplify logic to bound custom events
1 parent ae048b5 commit af0b0aa

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

src/hls.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -193,14 +193,14 @@ export default class HlsjsPlayback extends HTML5Video {
193193
this.customListeners.forEach(item => {
194194
const requestedEventName = item.eventName
195195
const typeOfListener = item.once ? 'once': 'on'
196-
HLSJS.Events[`${requestedEventName}`] && this._hls[`${typeOfListener}`](HLSJS.Events[`${requestedEventName}`], item.callback)
196+
requestedEventName && this._hls[`${typeOfListener}`](requestedEventName, item.callback)
197197
})
198198
}
199199

200200
unbindCustomListeners() {
201201
this.customListeners.forEach(item => {
202202
const requestedEventName = item.eventName
203-
HLSJS.Events[`${requestedEventName}`] && this._hls.off(HLSJS.Events[`${requestedEventName}`], item.callback)
203+
requestedEventName && this._hls.off(requestedEventName, item.callback)
204204
})
205205
}
206206

src/hls.test.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ describe('HlsjsPlayback', () => {
2222
const playback = new HlsjsPlayback({
2323
src: 'http://clappr.io/foo.m3u8',
2424
hlsPlayback: {
25-
customListeners: [{ eventName: 'MEDIA_ATTACHING', callback: cb }]
25+
customListeners: [{ eventName: 'hlsMediaAttaching', callback: cb }]
2626
}
2727
})
2828
expect(playback.customListeners).toEqual(playback.options.hlsPlayback.customListeners)
@@ -268,7 +268,7 @@ describe('HlsjsPlayback', () => {
268268
const playback = new HlsjsPlayback({
269269
src: 'http://clappr.io/foo.m3u8',
270270
hlsPlayback: {
271-
customListeners: [{ eventName: 'MEDIA_ATTACHING', callback: cb }]
271+
customListeners: [{ eventName: HLSJS.Events.MEDIA_ATTACHING, callback: cb }]
272272
}
273273
})
274274
playback._setup()
@@ -305,7 +305,7 @@ describe('HlsjsPlayback', () => {
305305
const playback = new HlsjsPlayback({
306306
src: 'http://clappr.io/foo.m3u8',
307307
hlsPlayback: {
308-
customListeners: [{ eventName: 'MEDIA_ATTACHING', callback: cb, once: true }]
308+
customListeners: [{ eventName: HLSJS.Events.MEDIA_ATTACHING, callback: cb, once: true }]
309309
}
310310
})
311311
playback._setup()
@@ -324,7 +324,7 @@ describe('HlsjsPlayback', () => {
324324
const playback = new HlsjsPlayback({
325325
src: 'http://clappr.io/foo.m3u8',
326326
hlsPlayback: {
327-
customListeners: [{ eventName: 'FRAG_LOADED', callback: cb }]
327+
customListeners: [{ eventName: 'hlsFragLoaded', callback: cb }]
328328
}
329329
})
330330
playback._setup()

0 commit comments

Comments
 (0)