Use external propTypes file

Reduces duplication and avoids linting issues

src/ReactPlayer.js

@@ -1,21 +1,11 @@
-import React, { Component, PropTypes } from 'react'
+import React, { Component } from 'react'
 import 'array.prototype.find'
 
+import propTypes from './propTypes'
 import players from './players'
 
 export default class MediaPlayer extends Component {
-  static propTypes = {
-    url: PropTypes.string,
-    playing: PropTypes.bool,
-    volume: PropTypes.number,
-    width: PropTypes.oneOfType([ PropTypes.string, PropTypes.number ]),
-    height: PropTypes.oneOfType([ PropTypes.string, PropTypes.number ]),
-    onPlay: PropTypes.func,
-    onPause: PropTypes.func,
-    onBuffer: PropTypes.func,
-    onEnded: PropTypes.func,
-    onError: PropTypes.func
-  }
+  static propTypes = propTypes
   static defaultProps = {
     volume: 0.8,
     width: 640,

src/players/Base.js

@@ -1,20 +1,11 @@
-import { Component, PropTypes } from 'react'
+import { Component } from 'react'
+
+import propTypes from '../propTypes'
 
 const UPDATE_FREQUENCY = 500
 
 export default class Base extends Component {
-  static propTypes = {
-    url: PropTypes.string,
-    playing: PropTypes.bool,
-    volume: PropTypes.number,
-    width: PropTypes.oneOfType([ PropTypes.string, PropTypes.number ]),
-    height: PropTypes.oneOfType([ PropTypes.string, PropTypes.number ]),
-    onPlay: PropTypes.func,
-    onPause: PropTypes.func,
-    onBuffer: PropTypes.func,
-    onEnded: PropTypes.func,
-    onError: PropTypes.func
-  }
+  static propTypes = propTypes
   static defaultProps = {
     onProgress: function () {}
   }

src/players/SoundCloud.js

@@ -1,6 +1,7 @@
 import React from 'react'
 import loadScript from 'load-script'
 
+import propTypes from '../propTypes'
 import Base from './Base'
 
 const CLIENT_ID = 'e8b6f84fbcad14c301ca1355cae1dea2'
@@ -10,7 +11,7 @@ const RESOLVE_URL = '//api.soundcloud.com/resolve.json'
 const MATCH_URL = /^https?:\/\/(soundcloud.com|snd.sc)\/(.*)$/
 
 export default class SoundCloud extends Base {
-  static propTypes = Base.propTypes // HACK: Prevent lint error
+  static propTypes = propTypes
   static canPlay (url) {
     return MATCH_URL.test(url)
   }

src/players/Vimeo.js

@@ -1,13 +1,14 @@
 import React from 'react'
 
+import propTypes from '../propTypes'
 import Base from './Base'
 
 const IFRAME_SRC = 'https://player.vimeo.com/video/'
 const MATCH_URL = /https?:\/\/(?:www\.|player\.)?vimeo.com\/(?:channels\/(?:\w+\/)?|groups\/([^\/]*)\/videos\/|album\/(\d+)\/video\/|video\/|)(\d+)(?:$|\/|\?)/
 const MATCH_MESSAGE_ORIGIN = /^https?:\/\/player.vimeo.com/
 
 export default class Vimeo extends Base {
-  static propTypes = Base.propTypes // HACK: Prevent lint error
+  static propTypes = propTypes
   static canPlay (url) {
     return MATCH_URL.test(url)
   }

src/players/YouTube.js

@@ -1,6 +1,7 @@
 import React from 'react'
 import loadScript from 'load-script'
 
+import propTypes from '../propTypes'
 import Base from './Base'
 
 const SDK_URL = '//www.youtube.com/iframe_api'
@@ -9,7 +10,7 @@ const MATCH_URL = /^(?:https?:\/\/)?(?:www\.)?(?:youtu\.be\/|youtube\.com\/(?:em
 const PLAYER_ID = 'youtube-player'
 
 export default class YouTube extends Base {
-  static propTypes = Base.propTypes // HACK: Prevent lint error
+  static propTypes = propTypes
   static canPlay (url) {
     return MATCH_URL.test(url)
   }

src/propTypes.js

@@ -0,0 +1,14 @@
+import { PropTypes } from 'react'
+
+export default {
+  url: PropTypes.string,
+  playing: PropTypes.bool,
+  volume: PropTypes.number,
+  width: PropTypes.oneOfType([ PropTypes.string, PropTypes.number ]),
+  height: PropTypes.oneOfType([ PropTypes.string, PropTypes.number ]),
+  onPlay: PropTypes.func,
+  onPause: PropTypes.func,
+  onBuffer: PropTypes.func,
+  onEnded: PropTypes.func,
+  onError: PropTypes.func
+}