Merge pull request #16 from Qeepsake/bugfix/change-getduration-signature

fix(getduration-signature): Changes the getDuration signature to getV…
Qeepsake · Dec 23, 2021 · d13446d · d13446d
2 parents 1a42197 + ace4bb7
commit d13446d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/example/src/App.tsx b/example/src/App.tsx
@@ -2,7 +2,7 @@ import * as React from 'react';
 
 import { StyleSheet, View, Text, Button } from 'react-native';
 import {
-  getDuration,
+  getVideoDuration,
   getDimensions,
   getMimeType,
   getTimestamp,
@@ -40,7 +40,7 @@ export default function App() {
       console.log('Results from @qeepsake/react-native-file-utils:');
       console.log('-------------');
 
-      const duration = await getDuration(uri, mediaType);
+      const duration = await getVideoDuration(uri);
       console.log('duration:');
       console.log(duration);
 

diff --git a/src/index.tsx b/src/index.tsx
@@ -24,11 +24,7 @@ const FileUtils = NativeModules.FileUtils
  * @param uri The full on device uri for the media item.
  * @returns Duration of the video in number of seconds.
  */
-export async function getDuration(
-  uri: string,
-  mediaType: 'video' | 'image'
-): Promise<number> {
-  if (mediaType !== 'video') return 0;
+export async function getVideoDuration(uri: string): Promise<number> {
   return FileUtils.getDuration(uri);
 }