[WCiOS17] Update .onchange() usage to iOS17 API v2 (#16080)

iamgabrielma · web-flow · commit df47d3296afb · 2025-09-08T13:25:55.000+07:00
diff --git a/RELEASE-NOTES.txt b/RELEASE-NOTES.txt
@@ -3,7 +3,7 @@
 
 23.3
 -----
-
+- [internal] Address deprecated view modifiers usage following iOS17 API updates [https://github.com/woocommerce/woocommerce-ios/pull/16080]
 
 23.2
 -----
@@ -18,7 +18,6 @@
 - [***] Increased app's minimum deployment target to iOS17 [https://github.com/woocommerce/woocommerce-ios/pull/16003]
 - [*] Jetpack setup: Native experience for the connection step [https://github.com/woocommerce/woocommerce-ios/pull/15983]
 - [*] Payments: Updated the In-Person Payments `Learn More` redirection to display the correct page based on the selected payment provider [https://github.com/woocommerce/woocommerce-ios/pull/15998]
-- [internal] Address deprecated view modifiers usage following iOS17 API updates [https://github.com/woocommerce/woocommerce-ios/pull/16002]
 - [*] Add "POS" label in Most recent orders in My store dashboard [https://github.com/woocommerce/woocommerce-ios/pull/16006]
 - [*] Order details: Always show shipping labels section if order is eligible for label creation. [https://github.com/woocommerce/woocommerce-ios/pull/16010]
 - [*] Order details: Remove print buttons from shipment details [https://github.com/woocommerce/woocommerce-ios/pull/16012]
diff --git a/WooCommerce/Classes/Authentication/Jetpack Setup/Native Jetpack Setup/JetpackSetupView.swift b/WooCommerce/Classes/Authentication/Jetpack Setup/Native Jetpack Setup/JetpackSetupView.swift
@@ -288,7 +288,7 @@ struct JetpackSetupView: View {
             .renderedIf(viewModel.setupFailed)
         })
         .padding()
-        .onChange(of: viewModel.shouldPresentWebView) { shouldPresent in
+        .onChange(of: viewModel.shouldPresentWebView) { _, shouldPresent in
             if shouldPresent {
                 webViewPresentationHandler()
             }
diff --git a/WooCommerce/Classes/View Modifiers/View+Measurements.swift b/WooCommerce/Classes/View Modifiers/View+Measurements.swift
@@ -14,7 +14,7 @@ extension View {
                     .onAppear {
                         callback(proxy.size.height)
                     }
-                    .onChange(of: proxy.size.height) { newHeight in
+                    .onChange(of: proxy.size.height) { _, newHeight in
                         callback(newHeight)
                     }
             }
@@ -32,7 +32,7 @@ extension View {
                     .onAppear {
                         callback(proxy.size.width)
                     }
-                    .onChange(of: proxy.size.width) { newWidth in
+                    .onChange(of: proxy.size.width) { _, newWidth in
                         callback(newWidth)
                     }
             }
@@ -48,10 +48,10 @@ extension View {
                     .onAppear {
                         callback(proxy.frame(in: .global))
                     }
-                    .onChange(of: proxy.size.height) { newHeight in
+                    .onChange(of: proxy.size.height) { _, newHeight in
                         callback(proxy.frame(in: .global))
                     }
-                    .onChange(of: proxy.frame(in: .global)) { newHeight in
+                    .onChange(of: proxy.frame(in: .global)) { _, newHeight in
                         callback(proxy.frame(in: .global))
                     }
             }
diff --git a/WooCommerce/Classes/ViewRelated/Authentication/AccountCreationForm.swift b/WooCommerce/Classes/ViewRelated/Authentication/AccountCreationForm.swift
@@ -125,7 +125,7 @@ struct AccountCreationForm: View {
             }
             .background(Color(uiColor: .systemBackground))
         }
-        .onChange(of: viewModel.shouldTransitionToPasswordField) { shouldTransition in
+        .onChange(of: viewModel.shouldTransitionToPasswordField) { _, shouldTransition in
             if shouldTransition {
                 withAnimation {
                     viewModel.transitionToPasswordField()
diff --git a/WooCommerce/Classes/ViewRelated/Blaze/AdDestination/BlazeAddParameterView.swift b/WooCommerce/Classes/ViewRelated/Blaze/AdDestination/BlazeAddParameterView.swift
@@ -21,7 +21,7 @@ struct BlazeAddParameterView: View {
                             .frame(width: Layout.keyWidth, alignment: .leading)
                         TextField(Localization.keyLabel, text: $viewModel.key)
                     }
-                    .onChange(of: viewModel.key) { _ in
+                    .onChange(of: viewModel.key) { _, _ in
                         viewModel.validateInputs()
                     }
 
@@ -31,7 +31,7 @@ struct BlazeAddParameterView: View {
                             .frame(width: Layout.keyWidth, alignment: .leading)
                         TextField(Localization.valueLabel, text: $viewModel.value)
                     }
-                    .onChange(of: viewModel.value) { _ in
+                    .onChange(of: viewModel.value) { _, _ in
                         viewModel.validateInputs()
                     }
                 } footer: {
diff --git a/WooCommerce/Classes/ViewRelated/Blaze/CampaignCreation/BlazeCampaignCreationForm.swift b/WooCommerce/Classes/ViewRelated/Blaze/CampaignCreation/BlazeCampaignCreationForm.swift
@@ -222,7 +222,7 @@ struct BlazeCampaignCreationForm: View {
                 isShowingCampaignObjectivePicker = false
             }
         }
-        .onChange(of: viewModel.error) { newValue in
+        .onChange(of: viewModel.error) { _, newValue in
             isShowingAISuggestionsErrorAlert = newValue == .failedToLoadAISuggestions
         }
         .alert(Localization.AISuggestionsErrorAlert.fetchingAISuggestions, isPresented: $isShowingAISuggestionsErrorAlert) {
diff --git a/WooCommerce/Classes/ViewRelated/Orders/Order Details/Shipping Labels/WooShipping Create Shipping Labels/WooShipping Package and Rate Selection/WooAddCustomPackageView.swift b/WooCommerce/Classes/ViewRelated/Orders/Order Details/Shipping Labels/WooShipping Create Shipping Labels/WooShipping Package and Rate Selection/WooAddCustomPackageView.swift
@@ -149,10 +149,10 @@ struct WooAddCustomPackageView: View {
                     .padding(.horizontal)
                     .frame(minHeight: geometry.size.height)
                     .frame(width: geometry.size.width)
-                    .onChange(of: viewModel.showSaveTemplate) { newValue in
+                    .onChange(of: viewModel.showSaveTemplate) { _, newValue in
                         packageTemplateNameFieldFocused = newValue
                     }
-                    .onChange(of: packageTemplateNameFieldFocused) { focused in
+                    .onChange(of: packageTemplateNameFieldFocused) { _, focused in
                         if focused {
                             // More info about why small delay is added:
                             // - https://github.com/woocommerce/woocommerce-ios/pull/14086#discussion_r1806036901
diff --git a/WooCommerce/Classes/ViewRelated/Orders/Order Details/Shipping Labels/WooShipping Create Shipping Labels/WooShippingAddresses/WooShippingEditAddressView.swift b/WooCommerce/Classes/ViewRelated/Orders/Order Details/Shipping Labels/WooShipping Create Shipping Labels/WooShippingAddresses/WooShippingEditAddressView.swift
@@ -85,7 +85,7 @@ struct WooShippingEditAddressView: View {
                 }
             }
             .padding()
-            .onChange(of: focusedField) { newField in
+            .onChange(of: focusedField) { _, newField in
                 if let previousFocusedField {
                     viewModel.validate(previousFocusedField)
                 }
@@ -262,7 +262,7 @@ struct WooShippingEditAddressView: View {
                 TextField(Localization.title(for: field.type), text: $field.value, prompt: Text(field.required ? "" : Localization.optional))
                     .keyboardType(keyboardType)
                     .focused($focused, equals: field.type)
-                    .onChange(of: field.value) { _ in
+                    .onChange(of: field.value) {
                         field.clearError()
                     }
                     .padding()
diff --git a/WooCommerce/Classes/ViewRelated/Products/Add Product/AddProductWithAI/Preview/ProductDetailPreviewView.swift b/WooCommerce/Classes/ViewRelated/Products/Add Product/AddProductWithAI/Preview/ProductDetailPreviewView.swift
@@ -134,7 +134,7 @@ struct ProductDetailPreviewView: View {
             .onDisappear {
                 viewModel.onViewDisappear()
             }
-            .onChange(of: viewModel.errorState) { newValue in
+            .onChange(of: viewModel.errorState) { _, newValue in
                 isShowingErrorAlert = newValue != .none
             }
             .alert(viewModel.errorState.errorMessage, isPresented: $isShowingErrorAlert) {
diff --git a/WooCommerce/Classes/ViewRelated/Products/Add Product/AddProductWithAI/StartingInfo/ProductCreationAIStartingInfoView.swift b/WooCommerce/Classes/ViewRelated/Products/Add Product/AddProductWithAI/StartingInfo/ProductCreationAIStartingInfoView.swift
@@ -102,11 +102,11 @@ private extension ProductCreationAIStartingInfoView {
             .padding(insets: Layout.messageContentInsets)
             .focused($editorIsFocused)
             // Scrolls to the "TextField" view with a smooth animation while typing.
-            .onChange(of: viewModel.features) { _ in
+            .onChange(of: viewModel.features) { _, _ in
                 scrollToTextField(using: proxy)
             }
             // Scrolls to the "TextField" view with a smooth animation when the editor is focused in a small screen.
-            .onChange(of: editorIsFocused) { isFocused in
+            .onChange(of: editorIsFocused) { _, isFocused in
                 if isFocused {
                     scrollToTextField(using: proxy)
                 }
diff --git a/WooCommerce/Classes/ViewRelated/Products/Add Product/AddProductWithAI/StartingInfo/PromptProgressBar/ProductCreationAIPromptProgressBar.swift b/WooCommerce/Classes/ViewRelated/Products/Add Product/AddProductWithAI/StartingInfo/PromptProgressBar/ProductCreationAIPromptProgressBar.swift
@@ -29,9 +29,9 @@ struct ProductCreationAIPromptProgressBar: View {
         .onAppear(perform: {
             viewModel.updateText(to: text)
         })
-        .onChange(of: text, perform: { newText in
+        .onChange(of: text) { _, newText in
             viewModel.updateText(to: newText)
-        })
+        }
     }
 
 }
diff --git a/WooCommerce/Classes/ViewRelated/ReusableViews/SwiftUI Components/FormattableAmountTextField.swift b/WooCommerce/Classes/ViewRelated/ReusableViews/SwiftUI Components/FormattableAmountTextField.swift
@@ -18,7 +18,9 @@ struct FormattableAmountTextField: View {
         ZStack(alignment: .center) {
             // Hidden input text field
             TextField("", text: $viewModel.textFieldAmountText)
-                .onChange(of: viewModel.textFieldAmountText, perform: viewModel.updateAmount)
+                .onChange(of: viewModel.textFieldAmountText) { _, newValue in
+                    viewModel.updateAmount(newValue)
+                }
                 .focused()
                 .focused($focusAmountInput)
                 .keyboardType(viewModel.allowNegativeNumber ? .numbersAndPunctuation : .decimalPad)
diff --git a/WooCommerce/Classes/ViewRelated/ReusableViews/SwiftUI Components/TopTabView.swift b/WooCommerce/Classes/ViewRelated/ReusableViews/SwiftUI Components/TopTabView.swift
@@ -172,7 +172,7 @@ struct TopTabView<Content: View>: View {
                                                     scrollFocusTab(in: scrollViewProxy, at: index)
                                                 }
                                             }
-                                            .onChange(of: geometry.size) { newSize in
+                                            .onChange(of: geometry.size) { _, newSize in
                                                 /// Support dynamic type size change
                                                 if index < tabWidths.count {
                                                     tabWidths[index] = newSize.width
@@ -194,11 +194,11 @@ struct TopTabView<Content: View>: View {
                                     .offset(x: underlineOffset),
                                 alignment: .bottomLeading
                             )
-                            .onChange(of: selectedTab, perform: { newSelectedTab in
+                            .onChange(of: selectedTab) { _, newSelectedTab in
                                 withAnimation {
                                     selectTab(in: scrollViewProxy, at: newSelectedTab)
                                 }
-                            })
+                            }
                             .coordinateSpace(name: Constants.tabsHorizontalStackNameSpace)
                         }
                         .padding(.horizontal, tabsContainerHorizontalPadding)
@@ -235,7 +235,7 @@ struct TopTabView<Content: View>: View {
                                 .onAppear {
                                     contentSize = contentGeometry.size
                                 }
-                                .onChange(of: contentGeometry.size) { newSize in
+                                .onChange(of: contentGeometry.size) { _, newSize in
                                     contentSize = newSize
                                 }
                         })

Original file line number	Diff line number	Diff line change
`@@ -288,7 +288,7 @@ struct JetpackSetupView: View {`
`288`	`288`	`.renderedIf(viewModel.setupFailed)`
`289`	`289`	`})`
`290`	`290`	`.padding()`
`291`		`- .onChange(of: viewModel.shouldPresentWebView) { shouldPresent in`
	`291`	`+ .onChange(of: viewModel.shouldPresentWebView) { _, shouldPresent in`
`292`	`292`	`if shouldPresent {`
`293`	`293`	`webViewPresentationHandler()`
`294`	`294`	`}`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ extension View {`
`14`	`14`	`.onAppear {`
`15`	`15`	`callback(proxy.size.height)`
`16`	`16`	`}`
`17`		`- .onChange(of: proxy.size.height) { newHeight in`
	`17`	`+ .onChange(of: proxy.size.height) { _, newHeight in`
`18`	`18`	`callback(newHeight)`
`19`	`19`	`}`
`20`	`20`	`}`
`@@ -32,7 +32,7 @@ extension View {`
`32`	`32`	`.onAppear {`
`33`	`33`	`callback(proxy.size.width)`
`34`	`34`	`}`
`35`		`- .onChange(of: proxy.size.width) { newWidth in`
	`35`	`+ .onChange(of: proxy.size.width) { _, newWidth in`
`36`	`36`	`callback(newWidth)`
`37`	`37`	`}`
`38`	`38`	`}`
`@@ -48,10 +48,10 @@ extension View {`
`48`	`48`	`.onAppear {`
`49`	`49`	`callback(proxy.frame(in: .global))`
`50`	`50`	`}`
`51`		`- .onChange(of: proxy.size.height) { newHeight in`
	`51`	`+ .onChange(of: proxy.size.height) { _, newHeight in`
`52`	`52`	`callback(proxy.frame(in: .global))`
`53`	`53`	`}`
`54`		`- .onChange(of: proxy.frame(in: .global)) { newHeight in`
	`54`	`+ .onChange(of: proxy.frame(in: .global)) { _, newHeight in`
`55`	`55`	`callback(proxy.frame(in: .global))`
`56`	`56`	`}`
`57`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,7 @@ struct AccountCreationForm: View {`
`125`	`125`	`}`
`126`	`126`	`.background(Color(uiColor: .systemBackground))`
`127`	`127`	`}`
`128`		`- .onChange(of: viewModel.shouldTransitionToPasswordField) { shouldTransition in`
	`128`	`+ .onChange(of: viewModel.shouldTransitionToPasswordField) { _, shouldTransition in`
`129`	`129`	`if shouldTransition {`
`130`	`130`	`withAnimation {`
`131`	`131`	`viewModel.transitionToPasswordField()`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ struct BlazeAddParameterView: View {`
`21`	`21`	`.frame(width: Layout.keyWidth, alignment: .leading)`
`22`	`22`	`TextField(Localization.keyLabel, text: $viewModel.key)`
`23`	`23`	`}`
`24`		`- .onChange(of: viewModel.key) { _ in`
	`24`	`+ .onChange(of: viewModel.key) { _, _ in`
`25`	`25`	`viewModel.validateInputs()`
`26`	`26`	`}`
`27`	`27`
`@@ -31,7 +31,7 @@ struct BlazeAddParameterView: View {`
`31`	`31`	`.frame(width: Layout.keyWidth, alignment: .leading)`
`32`	`32`	`TextField(Localization.valueLabel, text: $viewModel.value)`
`33`	`33`	`}`
`34`		`- .onChange(of: viewModel.value) { _ in`
	`34`	`+ .onChange(of: viewModel.value) { _, _ in`
`35`	`35`	`viewModel.validateInputs()`
`36`	`36`	`}`
`37`	`37`	`} footer: {`
Original file line number	Diff line number	Diff line change
`@@ -222,7 +222,7 @@ struct BlazeCampaignCreationForm: View {`
`222`	`222`	`isShowingCampaignObjectivePicker = false`
`223`	`223`	`}`
`224`	`224`	`}`
`225`		`- .onChange(of: viewModel.error) { newValue in`
	`225`	`+ .onChange(of: viewModel.error) { _, newValue in`
`226`	`226`	`isShowingAISuggestionsErrorAlert = newValue == .failedToLoadAISuggestions`
`227`	`227`	`}`
`228`	`228`	`.alert(Localization.AISuggestionsErrorAlert.fetchingAISuggestions, isPresented: $isShowingAISuggestionsErrorAlert) {`
Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ struct WooShippingEditAddressView: View {`
`85`	`85`	`}`
`86`	`86`	`}`
`87`	`87`	`.padding()`
`88`		`- .onChange(of: focusedField) { newField in`
	`88`	`+ .onChange(of: focusedField) { _, newField in`
`89`	`89`	`if let previousFocusedField {`
`90`	`90`	`viewModel.validate(previousFocusedField)`
`91`	`91`	`}`
`@@ -262,7 +262,7 @@ struct WooShippingEditAddressView: View {`
`262`	`262`	`TextField(Localization.title(for: field.type), text: $field.value, prompt: Text(field.required ? "" : Localization.optional))`
`263`	`263`	`.keyboardType(keyboardType)`
`264`	`264`	`.focused($focused, equals: field.type)`
`265`		`- .onChange(of: field.value) { _ in`
	`265`	`+ .onChange(of: field.value) {`
`266`	`266`	`field.clearError()`
`267`	`267`	`}`
`268`	`268`	`.padding()`
Original file line number	Diff line number	Diff line change
`@@ -134,7 +134,7 @@ struct ProductDetailPreviewView: View {`
`134`	`134`	`.onDisappear {`
`135`	`135`	`viewModel.onViewDisappear()`
`136`	`136`	`}`
`137`		`- .onChange(of: viewModel.errorState) { newValue in`
	`137`	`+ .onChange(of: viewModel.errorState) { _, newValue in`
`138`	`138`	`isShowingErrorAlert = newValue != .none`
`139`	`139`	`}`
`140`	`140`	`.alert(viewModel.errorState.errorMessage, isPresented: $isShowingErrorAlert) {`
Original file line number	Diff line number	Diff line change
`@@ -102,11 +102,11 @@ private extension ProductCreationAIStartingInfoView {`
`102`	`102`	`.padding(insets: Layout.messageContentInsets)`
`103`	`103`	`.focused($editorIsFocused)`
`104`	`104`	`// Scrolls to the "TextField" view with a smooth animation while typing.`
`105`		`- .onChange(of: viewModel.features) { _ in`
	`105`	`+ .onChange(of: viewModel.features) { _, _ in`
`106`	`106`	`scrollToTextField(using: proxy)`
`107`	`107`	`}`
`108`	`108`	`// Scrolls to the "TextField" view with a smooth animation when the editor is focused in a small screen.`
`109`		`- .onChange(of: editorIsFocused) { isFocused in`
	`109`	`+ .onChange(of: editorIsFocused) { _, isFocused in`
`110`	`110`	`if isFocused {`
`111`	`111`	`scrollToTextField(using: proxy)`
`112`	`112`	`}`