From e0401de1a335e667f21b62bcfeb253d94aedabd0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tim=20Daubensch=C3=BCtz?= Date: Wed, 9 Mar 2016 16:54:11 +0100 Subject: [PATCH] Add async tiff thumbnail encoding support - Fixes #171 --- .../ascribe_detail/media_container.js | 77 +++++++++++++------ js/components/ascribe_media/media_player.js | 65 +++++++++++----- js/constants/application_constants.js | 4 +- sass/ascribe_media_player.scss | 10 +++ 4 files changed, 113 insertions(+), 43 deletions(-) diff --git a/js/components/ascribe_detail/media_container.js b/js/components/ascribe_detail/media_container.js index 5511fdb4..45ea81a2 100644 --- a/js/components/ascribe_detail/media_container.js +++ b/js/components/ascribe_detail/media_container.js @@ -15,6 +15,8 @@ import CollapsibleButton from './../ascribe_collapsible/collapsible_button'; import AclProxy from '../acl_proxy'; +import AppConstants from '../../constants/application_constants'; + import { getLangText } from '../../utils/lang_utils'; import { extractFileExtensionFromString } from '../../utils/file_utils'; @@ -23,7 +25,7 @@ const EMBED_IFRAME_HEIGHT = { video: 315, audio: 62 }; -const ENCODE_UPDATE_TIME = 5000; + let MediaContainer = React.createClass({ propTypes: { @@ -41,21 +43,18 @@ let MediaContainer = React.createClass({ componentDidMount() { const { content: { digital_work: digitalWork }, refreshObject } = this.props; + const { timerId } = this.state; - if (digitalWork) { - const isEncoding = digitalWork.isEncoding; - - if (digitalWork.mime === 'video' && typeof isEncoding === 'number' && isEncoding !== 100 && !this.state.timerId) { - this.setState({ - timerId: window.setInterval(refreshObject, ENCODE_UPDATE_TIME) - }); - } + if (digitalWork && this.isDigitalWorkEncoding() && !timerId) { + this.setState({ timerId: window.setInterval(refreshObject, AppConstants.encodeUpdateThreshold) }); } }, componentWillUpdate() { - if (this.props.content.digital_work.isEncoding === 100) { - window.clearInterval(this.state.timerId); + const { timerId } = this.state; + + if (!this.isDigitalWorkEncoding() && timerId) { + window.clearInterval(timerId); } }, @@ -63,13 +62,35 @@ let MediaContainer = React.createClass({ window.clearInterval(this.state.timerId); }, - render() { - const { content, currentUser } = this.props; - // Pieces and editions are joined to the user by a foreign key in the database, so - // the information in content will be updated if a user updates their username. - // We also force uniqueness of usernames, so this check is safe to dtermine if the - // content was registered by the current user. - const didUserRegisterContent = currentUser && (currentUser.username === content.user_registered); + /* + * A digital work is encoding if: + * + * - it's either a video that has `isEncoding` < 100 + * - of a .tif|.tiff file that has not yet been converted + * to a .jpeg thumbnail + */ + isDigitalWorkEncoding() { + const { + content: { + digital_work: digitalWork, + thumbnail + } } = this.props; + + return ( + ( + digitalWork.mime === 'video' && + typeof digitalWork.isEncoding === 'number' && + digitalWork.isEncoding !== 100 + ) + || + ( + digitalWork.mime === 'image' && + (this.getFileExtensionFromUrl(thumbnail.url) === 'tif' || this.getFileExtensionFromUrl(thumbnail.url) === 'tiff') + ) + ); + }, + + getFileExtensionFromUrl(url) { // We want to show the file's extension as a label of the download button. // We can however not only use `extractFileExtensionFromString` on the url for that @@ -78,8 +99,18 @@ let MediaContainer = React.createClass({ // domain: e.g. '.net/live/'. // Therefore, we extract the file's name (last part of url, separated with a slash) // and try to extract the file extension from there. - const fileName = content.digital_work.url.split('/').pop(); - const fileExtension = extractFileExtensionFromString(fileName); + const fileName = url.split('/').pop(); + return extractFileExtensionFromString(fileName); + }, + + render() { + const { content, currentUser } = this.props; + + // Pieces and editions are joined to the user by a foreign key in the database, so + // the information in content will be updated if a user updates their username. + // We also force uniqueness of usernames, so this check is safe to dtermine if the + // content was registered by the current user. + const didUserRegisterContent = currentUser && (currentUser.username === content.user_registered); let thumbnail = content.thumbnail.thumbnail_sizes && content.thumbnail.thumbnail_sizes['600x600'] ? content.thumbnail.thumbnail_sizes['600x600'] : content.thumbnail.url_safe; @@ -117,7 +148,8 @@ let MediaContainer = React.createClass({
@@ -127,7 +159,6 @@ let MediaContainer = React.createClass({ - + fileExtension={this.getFileExtensionFromUrl(content.digital_work.url)} /> {embed}

diff --git a/js/components/ascribe_media/media_player.js b/js/components/ascribe_media/media_player.js index 1552b44c..7811be78 100644 --- a/js/components/ascribe_media/media_player.js +++ b/js/components/ascribe_media/media_player.js @@ -6,9 +6,12 @@ import Q from 'q'; import Panel from 'react-bootstrap/lib/Panel'; import ProgressBar from 'react-bootstrap/lib/ProgressBar'; +import AscribeSpinner from '../ascribe_spinner'; + import AppConstants from '../../constants/application_constants'; import { escapeHTML } from '../../utils/general_utils'; +import { getLangText } from '../../utils/lang_utils'; import { InjectInHeadUtils } from '../../utils/inject_utils'; /** @@ -31,18 +34,18 @@ let Other = React.createClass({ render() { let filename = this.props.url.split('/').pop(); let tokens = filename.split('.'); - let preview; + let thumbnail; if (tokens.length > 1) { - preview = '.' + tokens.pop(); + thumbnail = '.' + tokens.pop(); } else { - preview = 'file'; + thumbnail = 'file'; } return (

- {preview} + {thumbnail}

); @@ -52,11 +55,11 @@ let Other = React.createClass({ let Image = React.createClass({ propTypes: { url: React.PropTypes.string, - preview: React.PropTypes.string.isRequired + thumbnail: React.PropTypes.string.isRequired }, componentDidMount() { - if(this.props.url) { + if (this.props.url) { InjectInHeadUtils.inject(AppConstants.jquery.sdkUrl) .then(() => Q.all([ @@ -67,15 +70,15 @@ let Image = React.createClass({ }, render() { - const { url, preview } = this.props; + const { url, thumbnail } = this.props; - if(url) { + if (url) { return ( - + ); } else { return ( - + ); } } @@ -130,7 +133,7 @@ let Video = React.createClass({ */ propTypes: { - preview: React.PropTypes.string.isRequired, + thumbnail: React.PropTypes.string.isRequired, url: React.PropTypes.string.isRequired, extraData: React.PropTypes.array.isRequired, encodingStatus: React.PropTypes.number @@ -170,7 +173,7 @@ let Video = React.createClass({ prepareVideoHTML() { let sources = this.props.extraData.map((data) => ''); let html = [ - '']; @@ -184,7 +187,7 @@ let Video = React.createClass({ ); } else { return ( - + ); } } @@ -200,20 +203,31 @@ let resourceMap = { let MediaPlayer = React.createClass({ propTypes: { mimetype: React.PropTypes.oneOf(['image', 'video', 'audio', 'pdf', 'other']).isRequired, - preview: React.PropTypes.string.isRequired, + thumbnail: React.PropTypes.string.isRequired, + thumbnailFileExtension: React.PropTypes.string, url: React.PropTypes.string.isRequired, extraData: React.PropTypes.array, - encodingStatus: React.PropTypes.number + encodingStatus: React.PropTypes.number, + }, + + isVideoEncoding() { + const { mimetype, encodingStatus } = this.props; + return mimetype === 'video' && encodingStatus !== undefined && encodingStatus !== 100; + }, + + isImageEncoding() { + const { mimetype, thumbnailFileExtension } = this.props; + return mimetype === 'image' && (thumbnailFileExtension === 'tif' || thumbnailFileExtension === 'tiff'); }, render() { const { mimetype, - preview, + thumbnail, url, extraData, encodingStatus } = this.props; - if (mimetype === 'video' && encodingStatus !== undefined && encodingStatus !== 100) { + if (this.isVideoEncoding()) { return (

@@ -225,10 +239,23 @@ let MediaPlayer = React.createClass({ className="ascribe-progress-bar" />

); + } else if (this.isImageEncoding()) { + return ( +
+ + + {getLangText('We successfully received your image and it is now being encoded.')} +
+ {getLangText('We will be refreshing this page as soon as encoding has finished.')} +
+ {getLangText('(You may close this page)')} + +
+ ); } else { let Component = resourceMap[mimetype] || Other; let componentProps = { - preview, + thumbnail, url, extraData, encodingStatus @@ -242,7 +269,7 @@ let MediaPlayer = React.createClass({ // // If this is the case, we disable shmui by deleting the original `url` prop and replace // the assigned component to `Image`. - if(!decodeURIComponent(preview).match(/https:\/\/.*\/media\/thumbnails\/ascribe_spiral.png/) && + if (!decodeURIComponent(thumbnail).match(/https:\/\/.*\/media\/thumbnails\/ascribe_spiral.png/) && Component === Other) { Component = resourceMap.image; delete componentProps.url; diff --git a/js/constants/application_constants.js b/js/constants/application_constants.js index 897926a3..042e336e 100644 --- a/js/constants/application_constants.js +++ b/js/constants/application_constants.js @@ -92,13 +92,15 @@ const constants = { 'searchThreshold': 500, + 'encodeUpdateThreshold': 5000, + 'supportedThumbnailFileFormats': [ 'x-sgi-movie', 'x-msvideo', 'quicktime', 'mpeg', 'png', 'jpeg', 'gif', 'ogg', 'oga', 'ogv', 'ogx', 'wmv', 'wma', 'flv', '3gpp2', '3p2', '3pg', 'png', 'jpg', 'jpeg', 'gif', '264', '3g', '3g2', '3gp', '3gp2', '3gpp', 'mp4', 'm4a', 'm4v', 'f4v', 'f4a', 'm4b', 'm4r', 'f4b', 'mov', 'quicktime', 'webm', 'x264', 'mpeg', 'mpeg4', 'mpg4', 'bmp', 'eps', 'jp2', 'j2k', 'jpm', - 'mj2' + 'mj2', 'tif', 'tiff' ], // in case of whitelabel customization, we store stuff here diff --git a/sass/ascribe_media_player.scss b/sass/ascribe_media_player.scss index 25d4aa64..681adae1 100644 --- a/sass/ascribe_media_player.scss +++ b/sass/ascribe_media_player.scss @@ -115,5 +115,15 @@ .vjs-default-skin .vjs-control-bar { background-color: rgba(0,0,0,.7); } + + &.encoding-image { + text-align: center; + margin: 2em 0 2em 0; + + em { + display: block; + margin-top: 1em; + } + } }