Merge pull request #128 from bem13/master

Fixed external players not working, updated Readme, added building instructions

Author: thecoder75

README.md

@@ -30,7 +30,6 @@ Its an [Electron](https://electronjs.org) based application for Live.me Social V
 - See and navigate user's followers and followings
 - Watch and download replays
 - Allows downloading of replays using internal downloader or FFMPEG
-- Supports [LAMD](https://github.com/thecoder75/lamd)
 - FFMPEG Hardware-Accelerated video transcoding supported on Linux only!  
   - Fully supports VAAPI on AMD and Intel cards for now.
 
@@ -80,7 +79,7 @@ If you find any bugs or would like to help add features or additional functions,
 * [Tashiketh](https://notabug.org/Tashiketh)
 * [monstergarden](https://github.org/monstergarden)
 * [mustang-sally](https://github.com/mustang-sally)
-* [marcell](https://github.com/bem13)
+* [bem13](https://github.com/bem13)
 
 ### License
 This project is now licensed under the [GNU AGPLv3](LICENSE) License.

docs/building.md

@@ -1,3 +1,35 @@
 ## Building/Compiling LiveMe Pro Tools
 
-*This document will contain directions on how to compile LiveMe Pro Tools from source when we have time to write the steps.*
+## Building on Linux
+First of all, you will need to install Node.js and make sure it works.
+- [Follow the instructions provided here to install it](https://nodejs.org/en/download/package-manager/)
+- [Then follow these instructions to test if it works correctly](https://www.electronjs.org/docs/tutorial/development-environment#setting-up-linux)  
+If both commands return a version number, you should be good to go.
+
+If you've been using LMPT for a while, you probably want to back up your settings, bookmarks, etc. before you build a new version. Copy the following files from `/home/your_username/.config/Electron` to some safe location:  
+- Preferences
+- Settings
+- bookmarks.json
+- downloaded.json
+- errored.json
+- follows.json
+- ignored.json
+- profiles.json
+- queued.json
+- watched.json
+
+1. Clone the repository to some directory:  
+`git clone https://github.com/thecoder75/liveme-pro-tools.git`
+
+2. `cd` into that directory:  
+`cd liveme-pro-tools`
+
+3. Issue the command `npm install` to download the dependencies
+
+4. Issue `npm start` to start the app
+
+That's it!  
+If you want to work on the code, you'll need to close all running instances of the app, issue the command `npm rebuild` and restart the app to see your changes.
+
+## Building on Windows
+TODO

src/electron/index.js

@@ -674,8 +674,8 @@ ipcMain.on('watch-replay', (event, arg) => {
         .then(video => {
 
             console.log('Player Selection: ' + appSettings.get('player.pick'))
-            switch(appSettings.get('player.selection') || 0) {
-                case 99:    // External Player
+            switch(appSettings.get('player.pick') || 0) {
+                case '99':    // External Player
                     let playerPath = appSettings.get('player.path') || ' '
                     exec(playerPath.replace('%url%', LiveMe.pickProperVideoSource(video)))
                     break