Merge pull request #3773 from ritzalam/fix-get-recordings-api

- display length of recording in minutes instead of milliseconds

bbb-common-web/src/main/java/org/bigbluebutton/api/domain/RecordingMetadata.java

@@ -195,4 +195,13 @@ public class RecordingMetadata {
   public Boolean hasError() {
     return processingError;
   }
+
+  public Integer calculateDuration() {
+    if ((endTime == null) || (endTime == "") || (startTime == null) || (startTime == "")) return 0;
+
+    int start = (int) Math.ceil((Long.parseLong(startTime)) / 60000.0);
+    int end = (int) Math.ceil((Long.parseLong(endTime)) / 60000.0);
+
+    return end - start;
+  }
 }

bbb-common-web/src/main/java/org/bigbluebutton/api/domain/RecordingMetadataPlayback.java

@@ -47,6 +47,15 @@ public class RecordingMetadataPlayback {
     return duration;
   }
 
+  public Long calculateDuration() {
+    if (duration > 0) {
+      // convert to minutes
+      return duration / 60000;
+    } else {
+      return 0L;
+    }
+  }
+
   public void setExtensions(Extensions extensions) {
     this.extensions = extensions;
   }

bbb-common-web/src/test/resources/include-recording.ftlx

@@ -48,7 +48,11 @@
         <type>${pb.getFormat()}</type>
         <url>${pb.getLink()}</url>
         <processingTime>${pb.getProcessingTime()?c}</processingTime>
-        <length>${pb.getDuration()?c}</length>
+        <#if pb.getDuration() == 0>
+            <length>${r.calculateDuration()?c}</length>
+        <#else>
+            <length>${pb.calculateDuration()?c}</length>
+        </#if>
 
         <#if pb.getExtensions()??>
             <#if pb.getExtensions().getPreview()??>

bigbluebutton-web/web-app/WEB-INF/freemarker/include-recording.ftlx

@@ -48,7 +48,12 @@
     <type>${pb.getFormat()}</type>
     <url>${pb.getLink()}</url>
     <processingTime>${pb.getProcessingTime()?c}</processingTime>
-    <length>${pb.getDuration()?c}</length>
+
+    <#if pb.getDuration() == 0>
+        <length>${r.calculateDuration()?c}</length>
+    <#else>
+        <length>${pb.calculateDuration()?c}</length>
+    </#if>
 
     <#if pb.getExtensions()??>
     <#if pb.getExtensions().getPreview()??>