Video recording does not work

Question

Video recording does not work

AlexanderLukashin opened this issue 4 months ago · 13 comments

AlexanderLukashin commented 4 months ago

After the user finishes recording the video, null is returned instead of the path.

Unity version: 2022.3.29f
Platform: Android
Device: Redmi 10 (Android 13, MIUI 14.0.2), Xiaomi 14 (Android 14, HyperOS 1.0.6.0)

Link to the log file

Answer 1 · 2024-05-20T09:16:57.000Z

May I see your code?

Answer 2 · 2024-05-20T09:21:14.000Z

Here is the method for processing pressing the start recording button

        private void RecordVideo(ClickEvent eventArgs)
        {
            _ = NativeCamera.RecordVideo(Record);

            void Record(string path)
            {
                AddFiles(_data.TaskMedia, path).Forget();
            }
        }

Answer 3 · 2024-05-20T13:02:48.000Z

Hmm, it looks good to me. We need to figure out whether this is a device-specific issue or Unity version-specific issue. Do you have older Unity versions (2021 or older) that you can test the example code on?

Answer 4 · 2024-05-21T06:45:49.000Z

I tried to build in Unity 2021.3.38f, but the path is still returned null. Link to the log file

Answer 5 · 2024-05-21T06:54:48.000Z

This bug is also reproduced on the Galaxy tab 8 tablet (Android 14, OneUI 6.0)

Answer 6 · 2024-05-21T09:10:44.000Z

I've created the APK inside this zip archive on 2021.3.29f1 using Mono (faster build) and it works well on my Galaxy S8. Could you give it a try? Source code:

NativeCamera.Permission permission = NativeCamera.RecordVideo( ( path ) =>
{
	if( path != null )
	{
		// Play the recorded video
		Handheld.PlayFullScreenMovie( "file://" + path );
	}
} );

Answer 7 · 2024-05-21T10:08:46.000Z

It doesn't work again, here is a link to the archive with videos and logs

Answer 8 · 2024-05-21T10:32:03.000Z

Thank you, it's evidently a device-specific issue. I'll investigate it.

Answer 9 · 2024-05-22T08:13:37.000Z

There is no such error in the plugin version 1.4.2, maybe it will help somehow

Answer 10 · 2024-05-29T08:20:35.000Z

I've been having a busy schedule so I couldn't take a look at it yet.

Answer 11 · 2024-06-20T12:06:50.000Z

I've now added a few extra logs to the sections of the code that were added in v1.4.3. Reading those logs could prove useful. Could you replace NativeCamera.aar with the one inside this zip archive and post all logcat logs that have the log tag "Unity"?

PS. The issue doesn't occur on an Android 14 Galaxy S23 device either.

Answer 12 · 2024-06-21T07:31:36.000Z

Xiaomi-22011119UY-Android-13_2024-06-21_110356.zip

Answer 13 · 2024-06-22T09:42:41.000Z

OK I think I figured it out. Thank you for providing the logs. The issue is, the video is saved to path X (that we provide to the camera) but Android wraps that path inside a Uri, let's call it Y. Normally, the native GetPathFromURI function can convert Y to X but for unknown reasons, this failed on your Xiaomi device. The fallback to this scenario is, copying the file from Y to a filepath that we provide (this behaviour was added in v1.4.3). The thing I overlooked is, I was passing back X to that filepath. So, NativeCamera was trying to copy the video from X back to X.

I've resolved this issue in two steps:

If conversion from Y to X fails, I also check if there is a non-empty file existing at X (behaviour in v1.4.2). If so, then it means that the video was recorded to X successfully and we can simply return X
In the worst case, if the "copy file" fallback is reached, I copy the file to a different path than X to avoid collisions

Could you give this new aar file a shot and verify that the issue is resolved? I'd also appreciate if you could send the Unity-tagged logs one last time to make sure that nothing goes wrong.