Skip to content

Commit 99ae28e

Browse files
author
马赐崇
committed
fix ssr build bug
1 parent c523686 commit 99ae28e

6 files changed

+68
-35
lines changed

dist/vue-video-player.js

+41-15
Large diffs are not rendered by default.

dist/vue-video-player.min.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/vue-video-player.min.js.map

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "vue-video-player",
3-
"version": "4.0.0",
3+
"version": "4.0.2",
44
"description": "Video Player component for Vue",
55
"main": "index.js",
66
"scripts": {

src/player.vue

+6-4
Original file line numberDiff line numberDiff line change
@@ -108,10 +108,12 @@
108108
if (videoOptions.plugins) {
109109
delete videoOptions.plugins.__ob__
110110
}
111+
111112
this.player = videojs(this.$el.children[0], videoOptions, function() {
112113
113114
// player readied
114-
self.$emit('ready', self.player);
115+
var _this = this
116+
self.$emit('ready', self.player)
115117
116118
// events
117119
['loadeddata',
@@ -121,11 +123,11 @@
121123
'pause',
122124
'waiting',
123125
'playing',
124-
'ended'].forEach(event => {
125-
this.on(event, function() {
126+
'ended'].forEach(function(event) {
127+
_this.on(event, function() {
126128
emitPlayerState(event, true)
127129
})
128-
});
130+
})
129131
130132
this.on('timeupdate', function() {
131133
emitPlayerState('timeupdate', this.currentTime())

ssr.js

+18-13
Original file line numberDiff line numberDiff line change
@@ -93,20 +93,25 @@ var videoPlayer = {
9393
player = _this[instanceName] = videojs(el.children[0], options, function() {
9494

9595
// player ready
96-
emitPlayerState('ready');
96+
var self = this
97+
emitPlayerState('ready')
9798

98-
['loadeddata',
99-
'canplay',
100-
'canplaythrough',
101-
'play',
102-
'pause',
103-
'waiting',
104-
'playing',
105-
'ended'].forEach(event => {
106-
this.on(event, function() {
107-
emitPlayerState(event, true)
108-
})
109-
});
99+
// events
100+
var events = ['loadeddata',
101+
'canplay',
102+
'canplaythrough',
103+
'play',
104+
'pause',
105+
'waiting',
106+
'playing',
107+
'ended']
108+
for (var i = 0; i < events.length; i++) {
109+
(function(event) {
110+
self.on(event, function() {
111+
emitPlayerState(event, true)
112+
})
113+
})(events[i])
114+
}
110115

111116
this.on('timeupdate', function() {
112117
emitPlayerState('timeupdate', this.currentTime())

0 commit comments

Comments
 (0)