Skip to content

Commit 4b16658

Browse files
committed
Break a reference cycle between POSAggregateModel and CardPresentPaymentsOnboardingViewModel
1 parent 434f8fe commit 4b16658

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

WooCommerce/Classes/POS/Presentation/PointOfSaleDashboardView.swift

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -133,8 +133,8 @@ private extension PointOfSaleDashboardView {
133133
}
134134

135135
func paymentsOnboardingView(from onboardingViewModel: CardPresentPaymentsOnboardingViewModel) -> some View {
136-
onboardingViewModel.showSupport = {
137-
posModel.cancelCardPaymentsOnboarding()
136+
onboardingViewModel.showSupport = { [weak posModel] in
137+
posModel?.cancelCardPaymentsOnboarding()
138138
showSupport = true
139139
}
140140
return PointOfSaleCardPresentPaymentOnboardingView(viewModel: .init(onboardingViewModel: onboardingViewModel,
@@ -144,9 +144,6 @@ private extension PointOfSaleDashboardView {
144144
.onAppear {
145145
posModel.trackCardPaymentsOnboardingShown()
146146
}
147-
.onDisappear {
148-
onboardingViewModel.showSupport = nil
149-
}
150147
}
151148
}
152149

0 commit comments

Comments
 (0)