From ebc291956c8c9c24a8585f1a5d67e7f2fc617452 Mon Sep 17 00:00:00 2001
From: Norihiro Kamae <norihiro@nagater.net>
Date: Mon, 11 Apr 2022 22:21:03 +0900
Subject: [PATCH] UI: Set automatic file splitting time in minutes

The original PR was made with time specified in seconds because it was
useful to debug the behavior. For production, assuming most users want
to specify 10 minutes or more, the time should be specified in minutes.
---
 UI/forms/OBSBasicSettings.ui     | 8 ++++----
 UI/window-basic-main-outputs.cpp | 2 +-
 UI/window-basic-main.cpp         | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/UI/forms/OBSBasicSettings.ui b/UI/forms/OBSBasicSettings.ui
index 6e6ab61cf..c6bf8146f 100644
--- a/UI/forms/OBSBasicSettings.ui
+++ b/UI/forms/OBSBasicSettings.ui
@@ -2461,16 +2461,16 @@
                             <item row="8" column="1">
                              <widget class="QSpinBox" name="advOutSplitFileTime">
                               <property name="suffix">
-                               <string> s</string>
+                               <string> min</string>
                               </property>
                               <property name="minimum">
-                               <number>5</number>
+                               <number>1</number>
                               </property>
                               <property name="maximum">
-                               <number>21600</number>
+                               <number>360</number>
                               </property>
                               <property name="value">
-                               <number>900</number>
+                               <number>15</number>
                               </property>
                              </widget>
                             </item>
diff --git a/UI/window-basic-main-outputs.cpp b/UI/window-basic-main-outputs.cpp
index 253d8d320..d411065f7 100644
--- a/UI/window-basic-main-outputs.cpp
+++ b/UI/window-basic-main-outputs.cpp
@@ -1910,7 +1910,7 @@ bool AdvancedOutput::StartRecording()
 			obs_data_set_bool(settings, "allow_overwrite",
 					  overwriteIfExists);
 			obs_data_set_int(settings, "max_time_sec",
-					 splitFileTime);
+					 splitFileTime * 60);
 			obs_data_set_int(settings, "max_size_mb",
 					 splitFileSize);
 			obs_data_set_bool(settings, "reset_timestamps",
diff --git a/UI/window-basic-main.cpp b/UI/window-basic-main.cpp
index ddb566c88..af263298b 100644
--- a/UI/window-basic-main.cpp
+++ b/UI/window-basic-main.cpp
@@ -1424,7 +1424,7 @@ bool OBSBasic::InitBasicConfigDefaults()
 	config_set_default_uint(basicConfig, "AdvOut", "Track5Bitrate", 160);
 	config_set_default_uint(basicConfig, "AdvOut", "Track6Bitrate", 160);
 
-	config_set_default_uint(basicConfig, "AdvOut", "RecSplitFileTime", 900);
+	config_set_default_uint(basicConfig, "AdvOut", "RecSplitFileTime", 15);
 	config_set_default_uint(basicConfig, "AdvOut", "RecSplitFileSize",
 				2048);
 	config_set_default_bool(basicConfig, "AdvOut",