Merge pull request #18575 from wordpress-mobile/fix/reader-interests-view-crash

hassaanelgarem · web-flow · commit 57410443f6ea · 2022-05-12T15:12:47.000+02:00
Empty Stats: Fix crash during Reader Discover tour
diff --git a/WordPress/Classes/Utility/Animator.swift b/WordPress/Classes/Utility/Animator.swift
@@ -58,7 +58,7 @@ class Animator: NSObject {
             preamble?()
         }
 
-        UIView.animate(withDuration: duration, delay: 0, options: .curveEaseOut, animations: animations) { [unowned self] _ in
+        UIView.animate(withDuration: duration, delay: 0, options: .curveEaseOut, animations: animations) { _ in
             self.animationsInProgress -= 1
 
             if self.animationsInProgress == 0 {
diff --git a/WordPress/Classes/ViewRelated/Cells/ExpandableCell.swift b/WordPress/Classes/ViewRelated/Cells/ExpandableCell.swift
@@ -38,7 +38,7 @@ class ExpandableCell: WPReusableTableViewCell {
                 alpha = 0
             }
 
-            UIView.animate(withDuration: 0.2) { [unowned self] in
+            UIView.animate(withDuration: 0.2) {
                 self.chevronImageView?.transform = transform
                 self.expandableTextView?.alpha = alpha
             }
diff --git a/WordPress/Classes/ViewRelated/Reader/ReaderCardsStreamViewController.swift b/WordPress/Classes/ViewRelated/Reader/ReaderCardsStreamViewController.swift
@@ -265,7 +265,7 @@ private extension ReaderCardsStreamViewController {
 
         UIView.animate(withDuration: 0.2, animations: {
             self.selectInterestsViewController.view.alpha = 0
-        }) { [unowned self] _ in
+        }) { _ in
             self.selectInterestsViewController.remove()
             self.selectInterestsViewController.view.alpha = 1
         }

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ class Animator: NSObject {`
`58`	`58`	`preamble?()`
`59`	`59`	`}`
`60`	`60`
`61`		`- UIView.animate(withDuration: duration, delay: 0, options: .curveEaseOut, animations: animations) { [unowned self] _ in`
	`61`	`+ UIView.animate(withDuration: duration, delay: 0, options: .curveEaseOut, animations: animations) { _ in`
`62`	`62`	`self.animationsInProgress -= 1`
`63`	`63`
`64`	`64`	`if self.animationsInProgress == 0 {`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ class ExpandableCell: WPReusableTableViewCell {`
`38`	`38`	`alpha = 0`
`39`	`39`	`}`
`40`	`40`
`41`		`- UIView.animate(withDuration: 0.2) { [unowned self] in`
	`41`	`+ UIView.animate(withDuration: 0.2) {`
`42`	`42`	`self.chevronImageView?.transform = transform`
`43`	`43`	`self.expandableTextView?.alpha = alpha`
`44`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -265,7 +265,7 @@ private extension ReaderCardsStreamViewController {`
`265`	`265`
`266`	`266`	`UIView.animate(withDuration: 0.2, animations: {`
`267`	`267`	`self.selectInterestsViewController.view.alpha = 0`
`268`		`- }) { [unowned self] _ in`
	`268`	`+ }) { _ in`
`269`	`269`	`self.selectInterestsViewController.remove()`
`270`	`270`	`self.selectInterestsViewController.view.alpha = 1`
`271`	`271`	`}`