Replace dirlist with function filtering out images

src/components/Settings.jsx

@@ -80,7 +80,7 @@ const colorSchemes = [< MenuItem value = "random" key = "random" primaryText = {
                     }
                     imageReadyHandler(event) {}
                     updateImages() {
-                        const imagePaths = greeterutil.dirlist(this.props.directory);
+                        const imagePaths = greeterutil.dirimages(this.props.directory);
 
                         // THESE LINKS ARE NSFW -- DO NOT UNCOMMENT
                         // const imagePaths = [

src/components/background/RandomImage.jsx

@@ -8,7 +8,7 @@ class RandomImage extends React.Component {
         if (!this.props.directory)
             return console.error('Images directory prop not given.');
 
-        this.imagePaths = greeterutil.dirlist(this.props.directory);
+        this.imagePaths = greeterutil.dirimages(this.props.directory);
         if (typeof this.imagePaths === 'undefined')
             this.imagePaths = [''];
         }

src/index.js

@@ -41,6 +41,9 @@ if (!lightdm && process.env.NODE_ENV === 'development') {
         window.greeterutil = {
             dirlist: function(dir) {
                 return undefined;
+            },
+            dirimages: function(dir) {
+                return undefined;
             }
         }
     }

src/mock.js

@@ -278,6 +278,16 @@ class GreeterUtil {
         return this._mock_data.dirlist;
     }
 
+    /**
+     * Returns list of images inside directory at `path`.
+     *
+     * @param path
+     * @returns {String[]} List of abs paths for the images found in `path`.
+     */
+    dirimages(path) {
+        return this._mock_data.dirimages;
+    }
+
     /**
      * Escape HTML entities in a string.
      *

src/reducers/settings.jsx

@@ -1,6 +1,24 @@
 const moment = require('moment');
 require('moment-timezone');
 
+function dirimages(dir)
+{
+    let imagesExtensions = ["jpg", "jpeg", "png"];
+    function isImage(path) {
+        let filename = path.substring(path.lastIndexOf("/") + 1);
+        let dotPosition = filename.lastIndexOf(".");
+        if (dotPosition > -1) {
+            let extension = filename.substring(dotPosition + 1);
+            if (imagesExtensions.indexOf(extension) > -1) {
+                return true;
+            }
+        }
+        return false;
+    }
+    return greeterutil.dirlist(dir).filter(isImage);
+}
+greeterutil.dirimages = dirimages;
+
 let initialState = {
     'language': lightdm.languages[0].name,
     'backgroundEngine': 'trianglify',
@@ -16,7 +34,7 @@ let initialState = {
     'scaling': 1,
     'alignment': 1 // 0: left, 1: center, 2: right
 };
-let firstImage = greeterutil.dirlist(initialState['backgroundImagesDirectory']) ? greeterutil.dirlist(initialState['backgroundImagesDirectory'])[0] : false;
+let firstImage = greeterutil.dirimages(initialState['backgroundImagesDirectory']) ? greeterutil.dirimages(initialState['backgroundImagesDirectory'])[0] : false;
 initialState['backgroundImageLocation'] = firstImage || '';
 
 // If hide users is set with lightdm, then clear the default users