Skip to content

Commit e44432e

Browse files
committed
Fix memory leaks on viewing a post from a notification
1 parent 78bcc3b commit e44432e

4 files changed

Lines changed: 6 additions & 6 deletions

File tree

WordPress/Classes/ViewRelated/Reader/Comments/ReaderCommentsFollowPresenter.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ class ReaderCommentsFollowPresenter: NSObject {
1515

1616
private let post: ReaderPost
1717
private weak var delegate: ReaderCommentsFollowPresenterDelegate?
18-
private let presentingViewController: UIViewController
18+
private unowned let presentingViewController: UIViewController
1919
private let followCommentsService: FollowCommentsService?
2020

2121
// MARK: - Initialization

WordPress/Classes/ViewRelated/Reader/Detail/Views/ReaderDetailCommentsTableViewDelegate.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ class ReaderDetailCommentsTableViewDelegate: NSObject, UITableViewDataSource, UI
77

88
private(set) var totalComments = 0
99
private var post: ReaderPost?
10-
private var presentingViewController: UIViewController?
10+
private weak var presentingViewController: UIViewController?
1111
private weak var buttonDelegate: BorderedButtonTableViewCellDelegate?
1212
private(set) var headerView: ReaderDetailCommentsHeader?
1313
var followButtonTappedClosure: (() ->Void)?

WordPress/Classes/ViewRelated/Reader/Detail/Views/ReaderDetailHeaderView.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import UIKit
22
import AutomatticTracks
33

4-
protocol ReaderDetailHeaderViewDelegate {
4+
protocol ReaderDetailHeaderViewDelegate: AnyObject {
55
func didTapBlogName()
66
func didTapMenuButton(_ sender: UIView)
77
func didTapHeaderAvatar()
@@ -43,7 +43,7 @@ class ReaderDetailHeaderView: UIStackView, NibLoadable {
4343

4444
/// Any interaction with the header is sent to the delegate
4545
///
46-
var delegate: ReaderDetailHeaderViewDelegate?
46+
weak var delegate: ReaderDetailHeaderViewDelegate?
4747

4848
func configure(for post: ReaderPost) {
4949
self.post = post

WordPress/Classes/ViewRelated/Reader/Detail/Views/ReaderDetailLikesView.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import UIKit
22

3-
protocol ReaderDetailLikesViewDelegate {
3+
protocol ReaderDetailLikesViewDelegate: AnyObject {
44
func didTapLikesView()
55
}
66

@@ -13,7 +13,7 @@ class ReaderDetailLikesView: UIView, NibLoadable {
1313
@IBOutlet private weak var selfAvatarImageView: CircularImageView!
1414

1515
static let maxAvatarsDisplayed = 5
16-
var delegate: ReaderDetailLikesViewDelegate?
16+
weak var delegate: ReaderDetailLikesViewDelegate?
1717

1818
/// Stores the number of total likes _without_ adding the like from self.
1919
private var totalLikes: Int = 0

0 commit comments

Comments
 (0)