Added preview images for non-image files

web/react/components/view_image.jsx

@@ -31,17 +31,14 @@ module.exports = React.createClass({
             return;
         };
 
-        var fileInfo = utils.splitFileLocation(this.props.filenames[id]);
+        var filename = this.props.filenames[id];
-        // This is a temporary patch to fix issue with old files using absolute paths
+
-        if (fileInfo.path.indexOf("/api/v1/files/get") !== -1) {
+        var fileInfo = utils.splitFileLocation(filename);
-            fileInfo.path = fileInfo.path.split("/api/v1/files/get")[1];
+        var fileType = utils.getFileType(fileInfo.ext);
-        }
-        fileInfo.path = utils.getWindowLocationOrigin() + "/api/v1/files/get" + fileInfo.path;
-        var src = fileInfo['path'] + '_preview.jpg';
 
         var self = this;
         var img = new Image();
-        img.load(src,
+        img.load(this.getPreviewImagePath(filename),
             function(){
                 var progress = self.state.progress;
                 progress[id] = img.completedPercentage;
@@ -134,20 +131,17 @@ module.exports = React.createClass({
             bgClass = "black-bg";
         } else if (this.state.viewed) {
             for (var id in this.state.images) {
-                var info = utils.splitFileLocation(this.props.filenames[id]);
+                var filename = this.props.filenames[id];
-                var preview_filename = "";
+                var fileInfo = utils.splitFileLocation(filename);
-
-                // This is a temporary patch to fix issue with old files using absolute paths
-                if (info.path.indexOf("/api/v1/files/get") !== -1) {
-                    info.path = info.path.split("/api/v1/files/get")[1];
-                }
-                info.path = utils.getWindowLocationOrigin() + "/api/v1/files/get" + info.path;
-                preview_filename = info['path'] + '_preview.jpg';
 
                 var imgClass = "hidden";
                 if (this.state.loaded[id] && this.state.imgId == id) imgClass = "";
 
-                img[info['path']] = <a key={info['path']} className={imgClass} href={info.path+"."+info.ext} target="_blank"><img ref="image" src={preview_filename}/></a>;
+                img[fileInfo.path] = (
+                    <a key={fileInfo.path} className={imgClass} href={fileInfo.path+"."+fileInfo.ext} target="_blank">
+                        <img ref="image" src={this.getPreviewImagePath(filename)}/>
+                    </a>
+                );
             }
         }
 
@@ -197,5 +191,22 @@ module.exports = React.createClass({
                 </div>
             </div>
         );
+    },
+    getPreviewImagePath: function(filename) {
+        var fileInfo = utils.splitFileLocation(filename);
+        var fileType = utils.getFileType(fileInfo.ext);
+
+        if (fileType === "image") {
+            // This is a temporary patch to fix issue with old files using absolute paths
+            if (fileInfo.path.indexOf("/api/v1/files/get") !== -1) {
+                fileInfo.path = fileInfo.path.split("/api/v1/files/get")[1];
+            }
+            fileInfo.path = utils.getWindowLocationOrigin() + "/api/v1/files/get" + fileInfo.path;
+
+            return fileInfo.path + '_preview.jpg';
+        } else {
+            // only images have proper previews, so just use a placeholder icon for non-images
+            return utils.getPreviewImagePathForFileType(fileType);
+        }
     }
 });

web/react/utils/utils.jsx

@@ -533,6 +533,19 @@ module.exports.getFileType = function(ext) {
     return "other";
 };
 
+module.exports.getPreviewImagePathForFileType = function(fileType) {
+    fileType = fileType.toLowerCase();
+
+    var icon;
+    if (fileType in Constants.ICON_FROM_TYPE) {
+        icon = Constants.ICON_FROM_TYPE[fileType];
+    } else {
+        icon = Constants.ICON_FROM_TYPE["other"];
+    }
+
+    return "/static/images/icons/" + icon + ".png";
+};
+
 module.exports.getIconClassName = function(fileType) {
     fileType = fileType.toLowerCase();