iosobjective-cuiinterfaceorientationavcapturesessionviewdidlayoutsubviews

Quick 180 rotation of iOS Device results in the camera viewing upside-down


I've implemented the code below to change the orientation of an AVCaptureVideoSession based upon the UIInterfaceOrientation:

- (AVCaptureVideoOrientation)interfaceOrientationToVideoOrientation:(UIInterfaceOrientation)orientation {
    switch (orientation) {
        case UIInterfaceOrientationPortrait:
            return AVCaptureVideoOrientationPortrait;
        case UIInterfaceOrientationPortraitUpsideDown:
            return AVCaptureVideoOrientationPortraitUpsideDown;
        case UIInterfaceOrientationLandscapeLeft:
            return AVCaptureVideoOrientationLandscapeLeft;
        case UIInterfaceOrientationLandscapeRight:
            return AVCaptureVideoOrientationLandscapeRight;
        default:
            break;
    }
    NSLog(@"Warning - Didn't recognise interface orientation (%ld)",(long)orientation);
    return AVCaptureVideoOrientationPortrait;
}

This code works almost perfectly. However, one problem that occurs is that if you quickly rotate the iOS device 180 degrees, the camera will be shown upside down.

Here's what the camera view looks like before the rotation:

enter image description here

And here's what it looks like after the rotation:

enter image description here

Additionally, here is my implementation of viewDidLayoutSubviews:

- (void)viewDidLayoutSubviews {

    [super viewDidLayoutSubviews];

    previewLayer.frame = self.view.bounds;

    self.view.translatesAutoresizingMaskIntoConstraints = NO;
    previewLayer.videoGravity = AVLayerVideoGravityResizeAspectFill;
    [self.view.layer addSublayer:previewLayer];

    if (previewLayer.connection.supportsVideoOrientation) {
        previewLayer.connection.videoOrientation = [self interfaceOrientationToVideoOrientation:[UIApplication sharedApplication].statusBarOrientation];
    }
}

Does anyone have an idea of why the camera view would be upside down when this 180-degree rotation occurs?


Solution

  • I had the same issue before here what solved my issue.

    I declared a global variable:

    @interface YourViewController ()
    {
        ...
        UIDevice *iOSDevice;
    }
    
    ..
    
    @end
    

    Under implementation (.m file) i declared NSNotification observer under -viewWillAppear like:

    @implementation YourViewController
    
    -(void)viewWillAppear:(BOOL)animated
    {
        [[UIDevice currentDevice] beginGeneratingDeviceOrientationNotifications];
    
        [[NSNotificationCenter defaultCenter] addObserver:self  selector:@selector(orientationChanged:) name:UIDeviceOrientationDidChangeNotification  object:[UIDevice currentDevice]];
    
    }
    
    -(void)viewWillDisappear:(BOOL)animated
    {
        [[NSNotificationCenter defaultCenter]removeObserver:self name:UIDeviceOrientationDidChangeNotification object:nil];
    }
    
    - (void)orientationChanged:(NSNotification *)notification
    {
        iOSDevice = notification.object;
    
        previewLayer.connection.videoOrientation = [self interfaceOrientationToVideoOrientation];
    
        //or
    
        [self setAutoVideoConnectionOrientation:YES];
    
    }
    

    I made a function that returns the orientation of the copied current device like:

    Take a loot at my -interfaceOrientationToVideoOrientation method, it converts the iOSDevice.orientation into AVCaptureVideoOrientation same with what you have there..


    (In my case i needed this functions below, but take a look at it might be useful to you for some reason)

    - (AVCaptureVideoOrientation)interfaceOrientationToVideoOrientation
    {
        switch (iOSDevice.orientation) {
    
            case UIInterfaceOrientationPortraitUpsideDown:
    
                return AVCaptureVideoOrientationPortraitUpsideDown;
    
            case UIInterfaceOrientationLandscapeLeft:
    
                return AVCaptureVideoOrientationLandscapeLeft;
    
            case UIInterfaceOrientationLandscapeRight:
    
                return AVCaptureVideoOrientationLandscapeRight;
    
            default:
            case UIDeviceOrientationPortrait:
                return AVCaptureVideoOrientationPortrait;
        }
    }
    
    - (AVCaptureConnection *)setAutoVideoConnectionOrientation:(BOOL)status
    {
        AVCaptureConnection *videoConnection = nil;
    
        //This is for me
        //for (AVCaptureConnection *connection in stillImageOutput.connections) {
    
        //This might be for you
        for (AVCaptureConnection *connection in previewLayer.connection) {
    
            for (AVCaptureInputPort *port in [connection inputPorts])
            {
                if ([[port mediaType] isEqual:AVMediaTypeVideo])
                {
                    videoConnection = connection;
    
                    if (status == YES)
                    {
                        [videoConnection setVideoOrientation:[self interfaceOrientationToVideoOrientation]];
                    }
                    else
                    {
                        [videoConnection setVideoOrientation:AVCaptureVideoOrientationPortrait];
                    }
                }
                if (videoConnection)
                {
                    break;
                }
            }
        }
        return videoConnection;
    }
    

    Edit

    For default orientation: You need to check the "current" orientation.

    - (void)viewDidLoad
    {
        [super viewDidLoad];
    
        // assuming you finish setting the `previewLayer`
    
        ..
        // after all of that code, when the view is ready for displaying
        // if you implemented this approach the best thing to do is:
    
        // set this 
        iOSDevice = [UIDevice currentDevice];
    
        // then call 
        previewLayer.connection.videoOrientation = [self interfaceOrientationToVideoOrientation];
        // to update the `previewLayer.connection.videoOrientation ` for default orientation        
    
    }
    

    Note:

    Using NSNotificationCenter let the rotation of the device to be settled first before triggered..

    Hope this have help you..