From 453e9482134c894ba4247950656a94909eaf3229 Mon Sep 17 00:00:00 2001
From: Andrew DeMaria <lostonamountain@gmail.com>
Date: Thu, 25 May 2017 18:15:37 -0600
Subject: [PATCH] Created new transcoding for mp4 and use it for videos

Signed-off-by: Andrew DeMaria <lostonamountain@gmail.com>
---
 .../controller/VideoPlayerController.java     |  2 +-
 .../player/service/TranscodingService.java    | 17 ++++-
 .../resources/liquibase/6.3/changelog.xml     |  6 ++
 .../resources/liquibase/6.3/mediaelement.xml  | 18 +++++
 .../main/resources/liquibase/db-changelog.xml |  1 +
 .../resources/liquibase/legacy/schema43.xml   | 76 +++++++++----------
 .../resources/liquibase/legacy/schema46.xml   |  6 +-
 7 files changed, 82 insertions(+), 44 deletions(-)
 create mode 100644 libresonic-main/src/main/resources/liquibase/6.3/changelog.xml
 create mode 100644 libresonic-main/src/main/resources/liquibase/6.3/mediaelement.xml

diff --git a/libresonic-main/src/main/java/org/libresonic/player/controller/VideoPlayerController.java b/libresonic-main/src/main/java/org/libresonic/player/controller/VideoPlayerController.java
index a559023a..fe8072af 100644
--- a/libresonic-main/src/main/java/org/libresonic/player/controller/VideoPlayerController.java
+++ b/libresonic-main/src/main/java/org/libresonic/player/controller/VideoPlayerController.java
@@ -68,7 +68,7 @@ public class VideoPlayerController {
         Integer duration = file.getDurationSeconds();
         String playerId = playerService.getPlayer(request, response).getId();
         String url = NetworkService.getBaseUrl(request);
-        String streamUrl = url + "/stream?id=" + file.getId() + "&player=" + playerId;
+        String streamUrl = url + "/stream?id=" + file.getId() + "&player=" + playerId + "&format=mp4";
         String coverArtUrl = url + "/coverArt.view?id=" + file.getId();
 
         map.put("video", file);
diff --git a/libresonic-main/src/main/java/org/libresonic/player/service/TranscodingService.java b/libresonic-main/src/main/java/org/libresonic/player/service/TranscodingService.java
index af659b18..39858d56 100644
--- a/libresonic-main/src/main/java/org/libresonic/player/service/TranscodingService.java
+++ b/libresonic-main/src/main/java/org/libresonic/player/service/TranscodingService.java
@@ -22,7 +22,7 @@ package org.libresonic.player.service;
 import org.apache.commons.io.FileUtils;
 import org.apache.commons.io.FilenameUtils;
 import org.apache.commons.io.filefilter.PrefixFileFilter;
-import org.apache.commons.lang.StringUtils;
+import org.apache.commons.lang3.StringUtils;
 import org.libresonic.player.controller.VideoPlayerController;
 import org.libresonic.player.dao.TranscodingDao;
 import org.libresonic.player.domain.*;
@@ -393,7 +393,20 @@ public class TranscodingService {
         List<Transcoding> applicableTranscodings = new LinkedList<Transcoding>();
         String suffix = mediaFile.getFormat();
 
-        for (Transcoding transcoding : getTranscodingsForPlayer(player)) {
+        // This is what I'd like todo, but this will most likely break video transcoding as video transcoding is
+        // never expected to be null
+//        if(StringUtils.equalsIgnoreCase(preferredTargetFormat, suffix)) {
+//            LOG.debug("Target formats are the same, returning no transcoding");
+//            return null;
+//        }
+
+        List<Transcoding> transcodingsForPlayer = getTranscodingsForPlayer(player);
+        for (Transcoding transcoding : transcodingsForPlayer) {
+            // special case for now as video must have a transcoding
+            if(mediaFile.isVideo() && StringUtils.equalsIgnoreCase(preferredTargetFormat, transcoding.getTargetFormat())) {
+                LOG.debug("Detected source to target format match for video");
+                return transcoding;
+            }
             for (String sourceFormat : transcoding.getSourceFormatsAsArray()) {
                 if (sourceFormat.equalsIgnoreCase(suffix)) {
                     if (isTranscodingInstalled(transcoding)) {
diff --git a/libresonic-main/src/main/resources/liquibase/6.3/changelog.xml b/libresonic-main/src/main/resources/liquibase/6.3/changelog.xml
new file mode 100644
index 00000000..7e9ed807
--- /dev/null
+++ b/libresonic-main/src/main/resources/liquibase/6.3/changelog.xml
@@ -0,0 +1,6 @@
+<databaseChangeLog
+        xmlns="http://www.liquibase.org/xml/ns/dbchangelog"
+        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+        xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-3.1.xsd">
+    <include file="mediaelement.xml" relativeToChangelogFile="true"/>
+</databaseChangeLog>
\ No newline at end of file
diff --git a/libresonic-main/src/main/resources/liquibase/6.3/mediaelement.xml b/libresonic-main/src/main/resources/liquibase/6.3/mediaelement.xml
new file mode 100644
index 00000000..ceb968c9
--- /dev/null
+++ b/libresonic-main/src/main/resources/liquibase/6.3/mediaelement.xml
@@ -0,0 +1,18 @@
+<databaseChangeLog
+        xmlns="http://www.liquibase.org/xml/ns/dbchangelog"
+        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+        xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-3.1.xsd">
+    <changeSet id="mediaelement_001" author="muff1nman">
+        <insert tableName="transcoding2">
+            <column name="name" value="mp4/h264 video" />
+            <column name="source_formats" value="avi flv mpg mpeg m4v mkv mov wmv ogv divx m2ts" />
+            <column name="target_format" value="mp4" />
+            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f mp4 -vcodec libx264 -preset superfast -threads 0 -movflags frag_keyframe+empty_moov -"/>
+        </insert>
+        <rollback>
+            <delete tableName="transcoding2" >
+                <where>name = 'mp4/h264 video'</where>
+            </delete>
+        </rollback>
+    </changeSet>
+</databaseChangeLog>
diff --git a/libresonic-main/src/main/resources/liquibase/db-changelog.xml b/libresonic-main/src/main/resources/liquibase/db-changelog.xml
index 2af15cef..052fb226 100644
--- a/libresonic-main/src/main/resources/liquibase/db-changelog.xml
+++ b/libresonic-main/src/main/resources/liquibase/db-changelog.xml
@@ -9,4 +9,5 @@
     <property name="varchar_type" value="varchar" />
     <include file="legacy/legacy-changelog.xml" relativeToChangelogFile="true"/>
     <include file="6.2/changelog.xml" relativeToChangelogFile="true"/>
+    <include file="6.3/changelog.xml" relativeToChangelogFile="true"/>
 </databaseChangeLog>
\ No newline at end of file
diff --git a/libresonic-main/src/main/resources/liquibase/legacy/schema43.xml b/libresonic-main/src/main/resources/liquibase/legacy/schema43.xml
index ff7441f1..eadf978a 100644
--- a/libresonic-main/src/main/resources/liquibase/legacy/schema43.xml
+++ b/libresonic-main/src/main/resources/liquibase/legacy/schema43.xml
@@ -23,97 +23,97 @@
             <column name="version" valueNumeric="17" />
         </insert>
         <delete tableName="transcoding">
-            <where>target_format = 'mp4' and source_format in ('avi', 'mpg', 'mpeg', 'mp4', 'm4v', 'mkv', 'mov', 'wmv', 'ogv')</where>
+            <where>target_format = 'flv' and source_format in ('avi', 'mpg', 'mpeg', 'mp4', 'm4v', 'mkv', 'mov', 'wmv', 'ogv')</where>
         </delete>
         <insert tableName="transcoding">
-            <column name="name" value="avi > mp4"/>
+            <column name="name" value="avi > flv"/>
             <column name="source_format" value="avi" />
-            <column name="target_format" value="mp4" />
-            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f mp4 -vcodec libx264 -preset superfast -threads 0 -movflags frag_keyframe+empty_moov -" />
+            <column name="target_format" value="flv" />
+            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f flv -" />
             <column name="enabled" valueBoolean="true" />
         </insert>
         <sql>
-            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'avi > mp4'
+            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'avi > flv'
         </sql>
         <insert tableName="transcoding">
-            <column name="name" value="mpg > mp4"/>
+            <column name="name" value="mpg > flv"/>
             <column name="source_format" value="mpg" />
-            <column name="target_format" value="mp4" />
-            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f mp4 -vcodec libx264 -preset superfast -threads 0 -movflags frag_keyframe+empty_moov -" />
+            <column name="target_format" value="flv" />
+            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f flv -" />
             <column name="enabled" valueBoolean="true" />
         </insert>
         <sql>
-            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'mpg > mp4'
+            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'mpg > flv'
         </sql>
         <insert tableName="transcoding">
-            <column name="name" value="mpeg > mp4"/>
+            <column name="name" value="mpeg > flv"/>
             <column name="source_format" value="mpeg" />
-            <column name="target_format" value="mp4" />
-            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f mp4 -vcodec libx264 -preset superfast -threads 0 -movflags frag_keyframe+empty_moov -" />
+            <column name="target_format" value="flv" />
+            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f flv -" />
             <column name="enabled" valueBoolean="true" />
         </insert>
         <sql>
-            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'mpeg > mp4'
+            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'mpeg > flv'
         </sql>
         <insert tableName="transcoding">
-            <column name="name" value="flv > mp4"/>
-            <column name="source_format" value="flv" />
-            <column name="target_format" value="mp4" />
-            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f mp4 -vcodec libx264 -preset superfast -threads 0 -movflags frag_keyframe+empty_moov -" />
+            <column name="name" value="mp4 > flv"/>
+            <column name="source_format" value="mp4" />
+            <column name="target_format" value="flv" />
+            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f flv -" />
             <column name="enabled" valueBoolean="true" />
         </insert>
         <sql>
-            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'flv > mp4'
+            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'mp4 > flv'
         </sql>
         <insert tableName="transcoding">
-            <column name="name" value="m4v > mp4"/>
+            <column name="name" value="m4v > flv"/>
             <column name="source_format" value="m4v" />
-            <column name="target_format" value="mp4" />
-            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f mp4 -vcodec libx264 -preset superfast -threads 0 -movflags frag_keyframe+empty_moov -" />
+            <column name="target_format" value="flv" />
+            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f flv -" />
             <column name="enabled" valueBoolean="true" />
         </insert>
         <sql>
-            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'm4v > mp4'
+            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'm4v > flv'
         </sql>
         <insert tableName="transcoding">
-            <column name="name" value="mkv > mp4"/>
+            <column name="name" value="mkv > flv"/>
             <column name="source_format" value="mkv" />
-            <column name="target_format" value="mp4" />
-            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f mp4 -vcodec libx264 -preset superfast -threads 0 -movflags frag_keyframe+empty_moov -" />
+            <column name="target_format" value="flv" />
+            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f flv -" />
             <column name="enabled" valueBoolean="true" />
         </insert>
         <sql>
-            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'mkv > mp4'
+            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'mkv > flv'
         </sql>
         <insert tableName="transcoding">
-            <column name="name" value="mov > mp4"/>
+            <column name="name" value="mov > flv"/>
             <column name="source_format" value="mov" />
-            <column name="target_format" value="mp4" />
-            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f mp4 -vcodec libx264 -preset superfast -threads 0 -movflags frag_keyframe+empty_moov -" />
+            <column name="target_format" value="flv" />
+            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f flv -" />
             <column name="enabled" valueBoolean="true" />
         </insert>
         <sql>
-            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'mov > mp4'
+            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'mov > flv'
         </sql>
         <insert tableName="transcoding">
-            <column name="name" value="wmv > mp4"/>
+            <column name="name" value="wmv > flv"/>
             <column name="source_format" value="wmv" />
-            <column name="target_format" value="mp4" />
-            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f mp4 -vcodec libx264 -preset superfast -threads 0 -movflags frag_keyframe+empty_moov -" />
+            <column name="target_format" value="flv" />
+            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f flv -" />
             <column name="enabled" valueBoolean="true" />
         </insert>
         <sql>
-            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'wmv > mp4'
+            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'wmv > flv'
         </sql>
         <insert tableName="transcoding">
-            <column name="name" value="ogv > mp4"/>
+            <column name="name" value="ogv > flv"/>
             <column name="source_format" value="ogv" />
-            <column name="target_format" value="mp4" />
-            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f mp4 -vcodec libx264 -preset superfast -threads 0 -movflags frag_keyframe+empty_moov -" />
+            <column name="target_format" value="flv" />
+            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f flv -" />
             <column name="enabled" valueBoolean="true" />
         </insert>
         <sql>
-            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'ogv > mp4'
+            insert into player_transcoding(player_id, transcoding_id) select p.id as player_id, t.id as transaction_id from player p, transcoding t where t.name = 'ogv > flv'
         </sql>
         <rollback/>
     </changeSet>
diff --git a/libresonic-main/src/main/resources/liquibase/legacy/schema46.xml b/libresonic-main/src/main/resources/liquibase/legacy/schema46.xml
index 54d02119..fd009415 100644
--- a/libresonic-main/src/main/resources/liquibase/legacy/schema46.xml
+++ b/libresonic-main/src/main/resources/liquibase/legacy/schema46.xml
@@ -47,10 +47,10 @@
             <column name="step1" value="ffmpeg -i %s -ab %bk -v 0 -f mp3 -"/>
         </insert>
         <insert tableName="transcoding2">
-            <column name="name" value="mp4/h264 video" />
-            <column name="source_formats" value="avi flv mpg mpeg m4v mkv mov wmv ogv divx m2ts" />
+            <column name="name" value="flv/h264 video" />
+            <column name="source_formats" value="avi mpg mpeg mp4 m4v mkv mov wmv ogv divx m2ts" />
             <column name="target_format" value="flv" />
-            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f mp4 -vcodec libx264 -preset superfast -threads 0 -movflags frag_keyframe+empty_moov -"/>
+            <column name="step1" value="ffmpeg -ss %o -i %s -async 1 -b %bk -s %wx%h -ar 44100 -ac 2 -v 0 -f flv -vcodec libx264 -preset superfast -threads 0 -"/>
         </insert>
         <rollback>
             <dropTable tableName="transcoding2" />