Skip to content

Commit 14953bb

Browse files
committed
rename button and small refactor of configureNavBar
1 parent d3c1cfa commit 14953bb

1 file changed

Lines changed: 5 additions & 10 deletions

File tree

WordPress/Classes/ViewRelated/Stats/SiteStatsDashboardViewController.swift

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ class SiteStatsDashboardViewController: UIViewController {
6969
private var periodTableViewController = SiteStatsPeriodTableViewController.loadFromStoryboard()
7070
private var pageViewController: UIPageViewController?
7171

72-
@objc lazy var addStatsButton: UIBarButtonItem = {
72+
@objc lazy var manageInsightsButton: UIBarButtonItem = {
7373
let button = UIBarButtonItem(
7474
image: .gridicon(.cog),
7575
style: .plain,
@@ -83,21 +83,16 @@ class SiteStatsDashboardViewController: UIViewController {
8383

8484
override func viewDidLoad() {
8585
super.viewDidLoad()
86-
configureNavBar(StatsPeriodType.insights)
86+
configureNavBar()
8787
setupFilterBar()
8888
restoreSelectedDateFromUserDefaults()
8989
restoreSelectedPeriodFromUserDefaults()
9090
addWillEnterForegroundObserver()
9191
view.accessibilityIdentifier = "stats-dashboard"
9292
}
9393

94-
func configureNavBar(_ statsPeriodType: StatsPeriodType) {
95-
if statsPeriodType == StatsPeriodType.insights {
96-
parent?.navigationItem.rightBarButtonItem = addStatsButton
97-
}
98-
else {
99-
parent?.navigationItem.rightBarButtonItem = nil
100-
}
94+
func configureNavBar() {
95+
parent?.navigationItem.rightBarButtonItem = currentSelectedPeriod == .insights ? manageInsightsButton : nil
10196
}
10297

10398
@objc func addStatsCardButtonTapped() {
@@ -170,7 +165,7 @@ private extension SiteStatsDashboardViewController {
170165
@objc func selectedFilterDidChange(_ filterBar: FilterTabBar) {
171166
currentSelectedPeriod = StatsPeriodType(rawValue: filterBar.selectedIndex) ?? StatsPeriodType.insights
172167

173-
configureNavBar(currentSelectedPeriod)
168+
configureNavBar()
174169
}
175170

176171
}

0 commit comments

Comments
 (0)