WIP docs: Add upgrade guide for plugin authors for 8.x

CordovaLib/CordovaLib.docc/CordovaLib.md

@@ -56,6 +56,9 @@ For more information about Apache Cordova, visit [https://cordova.apache.org](ht
 - ``CDVConfigParser``
 - ``CDVTimer``
 
+### Upgrade Guides
+- <doc:upgrading-8>
+
 ### Deprecated
 
 - ``IsAtLeastiOSVersion``

CordovaLib/CordovaLib.docc/upgrading-8.md

@@ -0,0 +1,242 @@
+# Upgrading Plugins to Cordova iOS 8.x
+<!--
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+# 
+# http://www.apache.org/licenses/LICENSE-2.0
+# 
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+#  KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+#
+-->
+
+A guide for plugin authors to understand the API changes in Cordova iOS 8.
+
+Cordova iOS 8 introduces some significant changes to the exposed API for plugin authors and framework consumers. Many of these changes have been made to better align the framework with modern iOS development practices, such as adopting the `UIScene` APIs and fixing conflicts with SwiftUI projects, as well as work to improve the API experience for apps that consume Cordova as a framework (sometimes called the "platform centric workflow"). In all cases, great care has been taken to try to minimize the risk of breakage to existing 3rd party plugins.
+
+Many plugins will notice new deprecation warnings when built against Cordova iOS 8, rather than outright breaking changes. This document aims to explain the changes, the rationale behind the changes, and offer sample code for plugin authors to ensure their plugin code is compatible with future versions of Cordova iOS.
+
+In cases where different behavior is required for different Cordova versions, the ``CORDOVA_VERSION_MIN_REQUIRED`` macro can be used in Objective-C code to test the current Cordova version:
+
+```objc
+#if defined(__CORDOVA_8_0_0) && CORDOVA_VERSION_MIN_REQUIRED >= __CORDOVA_8_0_0
+    // Code for Cordova iOS 8 goes here
+#else
+    // Code for older versions goes here
+#endif
+```
+
+## Major Breaking Changes
+### Minimum iOS version update
+The Cordova iOS framework has increased the minimum supported iOS version from 11.0 to 13.0.
+
+The minimum supported Xcode version for the Cordova tooling and generated template app is now Xcode 15.
+
+### Change to the generated app project naming
+
+The generated app template is now consistently named "App", including "App.xcodeproj", and "App.xcworkspace". The Xcode build target for the app is also named "App". If you are expecting the name of the Xcode project or target to match the name specified in config.xml, it will now fail to find the project.
+
+Use the cordova-ios nodeJS API to retrieve the Xcode project path and other project details:
+
+```js
+// Old code
+const projectRoot = context.opts.projectRoot;
+const platformPath = path.join(projectRoot, 'platforms', 'ios');
+
+const config = getConfigParser(context, path.join(projectRoot, 'config.xml'));
+const projectName = config.name();
+
+const pbxprojPath = path.join(platformPath, `${projectName}.xcodeproj`, 'project.pbxproj');
+const xcodeProject = xcode.project(pbxprojPath);
+```
+
+```js
+// New code
+const projectRoot = context.opts.projectRoot;
+const platformPath = path.join(projectRoot, 'platforms', 'ios');
+
+const cordova_ios = require('cordova-ios');
+const iosProject = new cordova_ios('ios', platformPath);
+
+const xcodeProject = xcode.project(iosProject.locations.pbxproj);
+```
+
+This updated pattern is backwards compatible with existing versions of Cordova iOS 5.0.0 and newer.
+
+Moving to a consistent name for the Xcode project and target resolves a number of issues around dynamic file lookups and Unicode compatibility with other tools. It was actually never safe to use the `name` from config.xml directly, because even in previous versions of Cordova iOS the project name could be normalized to remove Unicode characters.
+
+### CDVAppDelegate window deprecation
+
+The generated app template now uses the `UIScene` API to support multiple windows, which means that it's no longer a safe assumption that an app has only a single window.
+
+The ``CDVAppDelegate/window`` property of ``CDVAppDelegate`` is deprecated as a result.  This property will always have a `nil` value.
+
+```objc
+// Old code
+// Include example here of how to get the root window
+```
+
+```objc
+// New code
+// Include example here of how to get the root window
+```
+
+Because apps can choose whether to adopt `UIScene` or not, it's important to include fallback support.
+
+### CDVAppDelegate viewController deprecation
+
+The ``CDVAppDelegate/viewController`` property of ``CDVAppDelegate`` is deprecated, and may return `nil` if a `CDVViewController` has not yet been initialized.
+
+Plugins should prefer accessing the view controller using their ``CDVPlugin/viewController`` property (which is now typed as ``CDVViewController``).
+
+### UIView scrollView property deprecation
+
+The `scrollView` property added as a global category extension to `UIView` is now deprecated in Objective C code and **removed entirely in Swift code**. This is to prevent conflicts with other Swift classes that extend `UIView` and have their own `scrollView` properties.
+
+You can still access the `scrollView` property of the web view by dynamically invoking the method:
+
+```objc
+// Old code
+UIScrollView *scroller = self.webView.scrollView;
+```
+
+```objc
+// New code
+#import <objc/message.h>
+
+UIScrollView *scroller;
+SEL scrollViewSelector = NSSelectorFromString(@"scrollView");
+
+if ([self.webView respondsToSelector:scrollViewSelector]) {
+    scroller = ((id (*)(id, SEL))objc_msgSend)(self.webView, scrollViewSelector);
+}
+```
+
+This updated code is compatible with existing versions of Cordova iOS.
+
+## Other Major Changes
+### Deprecating AppDelegate category extensions
+
+Please extend the ``CDVAppDelegate`` base class instead:
+
+```objc
+// Old code
+#import "AppDelegate.h"
+
+@interface AppDelegate (myplugin)
+    // Added extension methods here
+@end
+```
+
+```objc
+// New code
+#import <Cordova/CDVAppDelegate.h>
+
+@interface CDVAppDelegate (myplugin)
+    // Added extension methods here
+@end
+```
+
+It was never a completely safe assumption that an app using Cordova would include a class named `AppDelegate` that was a subclass of `CDVAppDelegate` due to the ability to embed CordovaLib in an existing iOS app project as a library. If your plugin needs to add behaviour to the app delegate, it should do so to the ``CDVAppDelegate`` base class.
+
+The updated code is backwards compatible with several existing Cordova iOS versions.
+
+### Deprecating MainViewController category extensions
+
+Please extend the ``CDVViewController`` base class instead:
+
+```objc
+// Old code
+#import "MainViewController.h"
+
+@interface MainViewController (myplugin)
+    // Added extension methods here
+@end
+```
+
+```objc
+// New code
+#import <Cordova/CDVViewController.h>
+
+@interface CDVViewController (myplugin)
+    // Added extension methods here
+@end
+```
+
+It was never a completely safe assumption that an app using Cordova would include a class named `MainViewController` that was a subclass of `CDVViewController` due to the ability to embed CordovaLib in an existing iOS app project as a library. If your plugin needs to add behaviour to the Cordova view controller, it should do so to the ``CDVViewController`` base class.
+
+This updated code is backwards compatible with several existing Cordova iOS versions.
+
+### Deprecating CDVCommandStatus constants in Swift
+
+For plugins written in Swift, the old `CDVCommandStatus_*` constants are deprecated in favour of the enum based aliases:
+
+```swift
+// Old code
+self.commandDelegate.send(CDVPluginResult(status: CDVCommandStatus_OK), callbackId: command.callbackId);
+```
+
+```swift
+// New code
+self.commandDelegate.send(CDVPluginResult(status: .ok), callbackId: command.callbackId);
+```
+
+These aliases were introduced in and are backwards compatible with all existing versions since Cordova iOS 5.0.0. See ``CDVCommandStatus`` for a list of the enum values.
+
+## Public API Removals
+The following classes were previously exposed as part of the Cordova iOS public API, but were only used as internal implementation details. To better establish the public/private API boundary within Cordova iOS, they have been removed from the public API in Cordova iOS 8.
+
+* `CDVAllowList`
+* `CDVURLSchemeHandler`
+
+## Other Changes
+* ``CDVConfigParser``
+  * Added a ``CDVConfigParser/parseConfigFile:`` class method.
+
+  * Added a ``CDVConfigParser/parseConfigFile:withDelegate:`` class method.  
+
+* ``CDVPlugin``
+  * The ``CDVPlugin/viewController`` property is now typed as ``CDVViewController``.  
+    Previously this was typed as the more generic `UIViewController`.
+
+  * Plugin classes that intend to override WebKit scheme handling should implement the ``CDVPluginSchemeHandler`` protocol to ensure compliance with the required methods.
+
+  * The ``CDVPluginHandleOpenURLWithAppSourceAndAnnotationNotification`` notification is now deprecated.  
+    The existing ``CDVPluginHandleOpenURLNotification`` notification now includes the source and annotation in its `userInfo` dictionary.
+
+* ``CDVViewController``
+  * The ``CDVViewController/settings`` property is now typed as ``CDVSettingsDictionary``.  
+    This property provides access to config.xml preferences, and now uses the new `CDVSettingsDictionary` class that behaves like an `NSDictionary` without relying on category extensions to add methods globally.
+
+  * The ``CDVViewController/wwwFolderName`` property is deprecated.  
+    This property has been renamed to ``CDVViewController/webContentFolderName``.
+
+  * The ``CDVViewController/appURLScheme`` property is deprecated.  
+    This property was not used internally by Cordova iOS and should not be used by plugins.
+
+  * The ``CDVViewController/pluginsMap`` and ``CDVViewController/pluginObjects`` properties are deprecated.  
+    These were internal implementation details that should not have been exposed.
+
+  * The ``CDVViewController/configParser`` property is deprecated due to not being used.
+
+  * Added an ``CDVViewController/enumerablePlugins`` property that can safely be iterated to loop over all loaded plugins.
+
+  * The ``CDVViewController/parseSettingsWithParser:`` method is deprecated.  
+    Use the ``CDVConfigParser/parseConfigFile:withDelegate:`` class method on ``CDVConfigParser`` instead.
+
+* ``CDVScreenOrientationDelegate``
+  * This protocol is now deprecated and no longer used.
+
+* ``CDVCommandDelegate``
+  * The ``CDVCommandDelegate/urlTransformer`` property is deprecated.  
+    This property was never used, and does not need to be a required part of the protocol.

CordovaLib/include/Cordova/CDVAppDelegate.h

@@ -34,7 +34,7 @@ NS_ASSUME_NONNULL_BEGIN
  behavior to the app by building on system events such as URL handling, push
  notification registration, and deep linking.
 
- See ``UIApplicationDelegate`` for more details about app delegates.
+ See `UIApplicationDelegate` for more details about app delegates.
  */
 @interface CDVAppDelegate : UIResponder <UIApplicationDelegate>
 

CordovaLib/include/Cordova/CDVConfigParser.h

@@ -31,7 +31,7 @@ NS_ASSUME_NONNULL_BEGIN
  Use ``parseConfigFile:`` to load data from a file path.
 
  Use ``parseConfigFile:withDelegate:`` if you need to intercept the XML parsing
- and handle the data yourself with an ``NSXMLParserDelegate``.
+ and handle the data yourself with an `NSXMLParserDelegate`.
  */
 @interface CDVConfigParser : NSObject <NSXMLParserDelegate>