expo-screen-orientation 6.0.3 → 6.0.4

package/CHANGELOG.md

@@ -10,6 +10,13 @@
 
 ### 💡 Others
 
+## 6.0.4 — 2023-07-23
+
+### 🐛 Bug fixes
+
+- [iOS] Fix event emitter sending events with no registered listeners. ([#23462](https://github.com/expo/expo/pull/23462) by [@behenate](https://github.com/behenate))
+- [iOS] Fix config plugin deleting the orientations key from `Info.plist` when the initial orientation value is set to `DEFAULT`. ([#23637](https://github.com/expo/expo/pull/23637) by [@behenate](https://github.com/behenate))
+
 ## 6.0.3 — 2023-07-12
 
 ### 🐛 Bug fixes

package/android/build.gradle

@@ -3,7 +3,7 @@ apply plugin: 'kotlin-android'
 apply plugin: 'maven-publish'
 
 group = 'host.exp.exponent'
-version = '6.0.3'
+version = '6.0.4'
 
 buildscript {
   def expoModulesCorePlugin = new File(project(":expo-modules-core").projectDir.absolutePath, "ExpoModulesCorePlugin.gradle")
@@ -67,7 +67,7 @@ android {
     minSdkVersion safeExtGet("minSdkVersion", 21)
     targetSdkVersion safeExtGet("targetSdkVersion", 33)
     versionCode 7
-    versionName '6.0.3'
+    versionName '6.0.4'
   }
   lintOptions {
     abortOnError false

package/ios/ScreenOrientationModule.swift

@@ -4,7 +4,7 @@ public class ScreenOrientationModule: Module, ScreenOrientationController {
   static let didUpdateDimensionsEvent = "expoDidUpdateDimensions"
 
   let screenOrientationRegistry = ScreenOrientationRegistry.shared
-  var eventEmitter: EXEventEmitterService?
+  var shouldEmitEvents = false
 
   public func definition() -> ModuleDefinition {
     Name("ExpoScreenOrientation")
@@ -82,12 +82,20 @@ public class ScreenOrientationModule: Module, ScreenOrientationController {
     OnDestroy {
       screenOrientationRegistry.unregisterController(self)
     }
+
+    OnStartObserving {
+      shouldEmitEvents = true
+    }
+
+    OnStopObserving {
+      shouldEmitEvents = false
+    }
   }
 
   // MARK: - ScreenOrientationController
 
   public func screenOrientationDidChange(_ orientation: UIInterfaceOrientation) {
-    guard let currentTraitCollection = screenOrientationRegistry.currentTraitCollection else {
+    guard let currentTraitCollection = screenOrientationRegistry.currentTraitCollection, shouldEmitEvents else {
       return
     }
 

package/ios/ScreenOrientationRegistry.swift

@@ -79,7 +79,8 @@ public class ScreenOrientationRegistry: NSObject, UIApplicationDelegate {
   /**
    Rotates the view to currentScreenOrientation or default orientation from the orientationMask.
    */
-  func enforceDesiredDeviceOrientation(withOrientationMask orientationMask: UIInterfaceOrientationMask) {
+  @objc
+  public func enforceDesiredDeviceOrientation(withOrientationMask orientationMask: UIInterfaceOrientationMask) {
     var newOrientation = orientationMask.defaultOrientation()
 
     if orientationMask.contains(currentScreenOrientation) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expo-screen-orientation",
-  "version": "6.0.3",
+  "version": "6.0.4",
   "description": "Expo universal module for managing device's screen orientation",
   "main": "build/ScreenOrientation.js",
   "types": "build/ScreenOrientation.d.ts",
@@ -41,5 +41,5 @@
   "peerDependencies": {
     "expo": "*"
   },
-  "gitHead": "8fdc53c90c52242a80ea511ee3073d9ab950bc68"
+  "gitHead": "c0d646e9295094bca877513e500d3c9f2e990c42"
 }

package/plugin/build/withScreenOrientation.d.ts

@@ -15,7 +15,6 @@ type OrientationMasks = keyof typeof OrientationLock;
 interface ExpoConfigWithInitialOrientation extends ExpoConfig {
     initialOrientation?: OrientationMasks;
 }
-export declare function getInitialOrientation(config: Pick<ExpoConfigWithInitialOrientation, 'initialOrientation'>): OrientationMasks;
 export declare function setInitialOrientation(config: Pick<ExpoConfigWithInitialOrientation, 'initialOrientation'>, infoPlist: InfoPlist): InfoPlist;
 declare const _default: ConfigPlugin<void | {
     initialOrientation?: "DEFAULT" | "ALL" | "PORTRAIT" | "PORTRAIT_UP" | "PORTRAIT_DOWN" | "LANDSCAPE" | "LANDSCAPE_LEFT" | "LANDSCAPE_RIGHT" | undefined;

package/plugin/build/withScreenOrientation.js

@@ -3,7 +3,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.setInitialOrientation = exports.getInitialOrientation = exports.INITIAL_ORIENTATION_KEY = void 0;
+exports.setInitialOrientation = exports.INITIAL_ORIENTATION_KEY = void 0;
 const assert_1 = __importDefault(require("assert"));
 const config_plugins_1 = require("expo/config-plugins");
 const pkg = require('expo-screen-orientation/package.json');
@@ -19,7 +19,7 @@ const OrientationLock = {
     LANDSCAPE_LEFT: 'UIInterfaceOrientationMaskLandscapeLeft',
     LANDSCAPE_RIGHT: 'UIInterfaceOrientationMaskLandscapeRight',
 };
-const withScreenOrientationViewController = (config, { initialOrientation = 'DEFAULT' } = {}) => {
+const withScreenOrientationViewController = (config, { initialOrientation } = {}) => {
     config = (0, config_plugins_1.withInfoPlist)(config, (config) => {
         const extendedConfig = {
             ...config,
@@ -30,19 +30,14 @@ const withScreenOrientationViewController = (config, { initialOrientation = 'DEF
     });
     return config;
 };
-function getInitialOrientation(config) {
-    return config.initialOrientation ?? 'DEFAULT';
-}
-exports.getInitialOrientation = getInitialOrientation;
 function setInitialOrientation(config, infoPlist) {
-    const initialOrientation = getInitialOrientation(config);
-    (0, assert_1.default)(initialOrientation in OrientationLock, `Invalid initial orientation "${initialOrientation}" expected one of: ${Object.keys(OrientationLock).join(', ')}`);
-    if (initialOrientation === 'DEFAULT') {
+    const initialOrientation = config.initialOrientation;
+    if (!initialOrientation) {
         delete infoPlist[exports.INITIAL_ORIENTATION_KEY];
+        return infoPlist;
     }
-    else {
-        infoPlist[exports.INITIAL_ORIENTATION_KEY] = OrientationLock[initialOrientation];
-    }
+    (0, assert_1.default)(initialOrientation in OrientationLock, `Invalid initial orientation "${initialOrientation}" expected one of: ${Object.keys(OrientationLock).join(', ')}`);
+    infoPlist[exports.INITIAL_ORIENTATION_KEY] = OrientationLock[initialOrientation];
     return infoPlist;
 }
 exports.setInitialOrientation = setInitialOrientation;

package/plugin/src/withScreenOrientation.ts

@@ -29,7 +29,7 @@ const withScreenOrientationViewController: ConfigPlugin<
   {
     initialOrientation?: keyof typeof OrientationLock;
   } | void
-> = (config, { initialOrientation = 'DEFAULT' } = {}) => {
+> = (config, { initialOrientation } = {}) => {
   config = withInfoPlist(config, (config) => {
     const extendedConfig = {
       ...config,
@@ -41,17 +41,16 @@ const withScreenOrientationViewController: ConfigPlugin<
   return config;
 };
 
-export function getInitialOrientation(
-  config: Pick<ExpoConfigWithInitialOrientation, 'initialOrientation'>
-): OrientationMasks {
-  return config.initialOrientation ?? 'DEFAULT';
-}
-
 export function setInitialOrientation(
   config: Pick<ExpoConfigWithInitialOrientation, 'initialOrientation'>,
   infoPlist: InfoPlist
 ): InfoPlist {
-  const initialOrientation = getInitialOrientation(config);
+  const initialOrientation = config.initialOrientation;
+
+  if (!initialOrientation) {
+    delete infoPlist[INITIAL_ORIENTATION_KEY];
+    return infoPlist;
+  }
 
   assert(
     initialOrientation in OrientationLock,
@@ -60,11 +59,8 @@ export function setInitialOrientation(
     ).join(', ')}`
   );
 
-  if (initialOrientation === 'DEFAULT') {
-    delete infoPlist[INITIAL_ORIENTATION_KEY];
-  } else {
-    infoPlist[INITIAL_ORIENTATION_KEY] = OrientationLock[initialOrientation];
-  }
+  infoPlist[INITIAL_ORIENTATION_KEY] = OrientationLock[initialOrientation];
+
   return infoPlist;
 }