Skip to content

Commit 8f88d03

Browse files
authored
Remove redundant available(iOS 14, *) annotations and checks (#19709)
2 parents 4659448 + 525027e commit 8f88d03

19 files changed

+35
-127
lines changed

WordPress/Classes/Extensions/UITextField+WorkaroundContinueIssue.swift

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,7 @@ extension UITextField {
99
/// Once we drop support for iOS 14, we could remove this extension entirely.
1010
///
1111
public class func shouldActivateWorkaroundForBulgarianKeyboardCrash() -> Bool {
12-
if #available(iOS 14.0, *) {
13-
return true
14-
}
15-
16-
return false
12+
return true
1713
}
1814

1915
/// We're swizzling `UITextField.becomeFirstResponder()` so that we can fix an issue with

WordPress/Classes/Services/Stories/WPMediaPicker+MediaPicker.swift

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,7 @@ class WPMediaPickerForKanvas: WPNavigationMediaPickerViewController, MediaPicker
6060
photoPicker.dataSource = WPPHAssetDataSource.sharedInstance()
6161
photoPicker.tabBarItem = UITabBarItem(title: Constants.photosTabBarTitle, image: Constants.photosTabBarIcon, tag: 0)
6262

63-
if #available(iOS 14.0, *),
64-
FeatureFlag.mediaPickerPermissionsNotice.enabled {
63+
if FeatureFlag.mediaPickerPermissionsNotice.enabled {
6564
photoPicker.mediaPicker.registerClass(forCustomHeaderView: DeviceMediaPermissionsHeader.self)
6665
}
6766

@@ -182,8 +181,7 @@ class MediaPickerDelegate: NSObject, WPMediaPickerViewControllerDelegate {
182181
}
183182

184183
func mediaPickerControllerShouldShowCustomHeaderView(_ picker: WPMediaPickerViewController) -> Bool {
185-
guard #available(iOS 14.0, *),
186-
FeatureFlag.mediaPickerPermissionsNotice.enabled,
184+
guard FeatureFlag.mediaPickerPermissionsNotice.enabled,
187185
picker.dataSource is WPPHAssetDataSource else {
188186
return false
189187
}
@@ -192,19 +190,14 @@ class MediaPickerDelegate: NSObject, WPMediaPickerViewControllerDelegate {
192190
}
193191

194192
func mediaPickerControllerReferenceSize(forCustomHeaderView picker: WPMediaPickerViewController) -> CGSize {
195-
guard #available(iOS 14.0, *) else {
196-
return .zero
197-
}
198-
199193
let header = DeviceMediaPermissionsHeader()
200194
header.translatesAutoresizingMaskIntoConstraints = false
201195

202196
return header.referenceSizeInView(picker.view)
203197
}
204198

205199
func mediaPickerController(_ picker: WPMediaPickerViewController, configureCustomHeaderView headerView: UICollectionReusableView) {
206-
guard #available(iOS 14.0, *),
207-
let headerView = headerView as? DeviceMediaPermissionsHeader else {
200+
guard let headerView = headerView as? DeviceMediaPermissionsHeader else {
208201
return
209202
}
210203

WordPress/Classes/Stores/StatsWidgetsStore.swift

Lines changed: 4 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -18,34 +18,22 @@ class StatsWidgetsStore {
1818

1919
if let newTodayData = refreshStats(type: HomeWidgetTodayData.self) {
2020
HomeWidgetTodayData.write(items: newTodayData)
21-
22-
if #available(iOS 14.0, *) {
23-
WidgetCenter.shared.reloadTimelines(ofKind: AppConfiguration.Widget.Stats.todayKind)
24-
}
21+
WidgetCenter.shared.reloadTimelines(ofKind: AppConfiguration.Widget.Stats.todayKind)
2522
}
2623

2724
if let newAllTimeData = refreshStats(type: HomeWidgetAllTimeData.self) {
2825
HomeWidgetAllTimeData.write(items: newAllTimeData)
29-
30-
if #available(iOS 14.0, *) {
31-
WidgetCenter.shared.reloadTimelines(ofKind: AppConfiguration.Widget.Stats.allTimeKind)
32-
}
26+
WidgetCenter.shared.reloadTimelines(ofKind: AppConfiguration.Widget.Stats.allTimeKind)
3327
}
3428

3529
if let newThisWeekData = refreshStats(type: HomeWidgetThisWeekData.self) {
3630
HomeWidgetThisWeekData.write(items: newThisWeekData)
37-
38-
if #available(iOS 14.0, *) {
39-
WidgetCenter.shared.reloadTimelines(ofKind: AppConfiguration.Widget.Stats.thisWeekKind)
40-
}
31+
WidgetCenter.shared.reloadTimelines(ofKind: AppConfiguration.Widget.Stats.thisWeekKind)
4132
}
4233
}
4334

4435
/// Initialize the local cache for widgets, if it does not exist
4536
func initializeStatsWidgetsIfNeeded() {
46-
guard #available(iOS 14.0, *) else {
47-
return
48-
}
4937
if HomeWidgetTodayData.read() == nil {
5038
DDLogInfo("StatsWidgets: Writing initialization data into HomeWidgetTodayData.plist")
5139
HomeWidgetTodayData.write(items: initializeHomeWidgetData(type: HomeWidgetTodayData.self))
@@ -70,8 +58,7 @@ class StatsWidgetsStore {
7058
/// - widgetType: concrete type of the widget
7159
/// - stats: stats to be stored
7260
func storeHomeWidgetData<T: HomeWidgetData>(widgetType: T.Type, stats: Codable) {
73-
guard #available(iOS 14.0, *),
74-
let siteID = SiteStatsInformation.sharedInstance.siteID else {
61+
guard let siteID = SiteStatsInformation.sharedInstance.siteID else {
7562
return
7663
}
7764

@@ -243,9 +230,6 @@ private extension StatsWidgetsStore {
243230
// MARK: - Extract this week data
244231
extension StatsWidgetsStore {
245232
func updateThisWeekHomeWidget(summary: StatsSummaryTimeIntervalData?) {
246-
guard #available(iOS 14.0, *) else {
247-
return
248-
}
249233
switch summary?.period {
250234
case .day:
251235
guard summary?.periodEndDate == StatsDataHelper.currentDateForSite().normalizedDate() else {
@@ -269,10 +253,6 @@ private extension StatsWidgetsStore {
269253
/// Observes WPAccountDefaultWordPressComAccountChanged notification and reloads widget data based on the state of account.
270254
/// The site data is not yet loaded after this notification and widget data cannot be cached for newly signed in account.
271255
func observeAccountChangesForWidgets() {
272-
guard #available(iOS 14.0, *) else {
273-
return
274-
}
275-
276256
NotificationCenter.default.addObserver(forName: .WPAccountDefaultWordPressComAccountChanged,
277257
object: nil,
278258
queue: nil) { notification in
@@ -294,10 +274,6 @@ private extension StatsWidgetsStore {
294274
/// Observes WPSigninDidFinishNotification notification and initializes the widget.
295275
/// The site data is loaded after this notification and widget data can be cached.
296276
func observeAccountSignInForWidgets() {
297-
guard #available(iOS 14.0, *) else {
298-
return
299-
}
300-
301277
NotificationCenter.default.addObserver(forName: NSNotification.Name(rawValue: WordPressAuthenticator.WPSigninDidFinishNotification),
302278
object: nil,
303279
queue: nil) { [weak self] _ in
@@ -307,10 +283,6 @@ private extension StatsWidgetsStore {
307283

308284
/// Observes applicationLaunchCompleted notification and runs migration.
309285
func observeApplicationLaunched() {
310-
guard #available(iOS 14.0, *) else {
311-
return
312-
}
313-
314286
NotificationCenter.default.addObserver(forName: NSNotification.Name.applicationLaunchCompleted,
315287
object: nil,
316288
queue: nil) { [weak self] _ in

WordPress/Classes/Utility/InteractiveNotificationsManager.swift

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -619,11 +619,7 @@ extension InteractiveNotificationsManager: UNUserNotificationCenterDelegate {
619619
if notification.request.content.categoryIdentifier == NoteCategoryDefinition.bloggingReminderWeekly.rawValue
620620
|| notification.request.content.categoryIdentifier == NoteCategoryDefinition.weeklyRoundup.rawValue {
621621

622-
if #available(iOS 14.0, *) {
623-
completionHandler([.banner, .list, .sound])
624-
} else {
625-
completionHandler([.alert, .sound])
626-
}
622+
completionHandler([.banner, .list, .sound])
627623
return
628624
}
629625

WordPress/Classes/ViewRelated/Aztec/ViewControllers/AztecPostViewController.swift

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -636,8 +636,7 @@ class AztecPostViewController: UIViewController, PostEditor {
636636

637637
// Required to work around an issue present in iOS 14 beta 2
638638
// https://github.com/wordpress-mobile/WordPress-iOS/issues/14460
639-
if #available(iOS 14.0, *),
640-
presentedViewController?.view.accessibilityIdentifier == MoreSheetAlert.accessibilityIdentifier {
639+
if presentedViewController?.view.accessibilityIdentifier == MoreSheetAlert.accessibilityIdentifier {
641640
dismiss(animated: true)
642641
}
643642
}
@@ -1326,8 +1325,7 @@ private extension AztecPostViewController {
13261325

13271326
alert.addCancelActionWithTitle(MoreSheetAlert.keepEditingTitle)
13281327

1329-
if #available(iOS 14.0, *),
1330-
let button = navigationBarManager.moreBarButtonItem.customView {
1328+
if let button = navigationBarManager.moreBarButtonItem.customView {
13311329
// Required to work around an issue present in iOS 14 beta 2
13321330
// https://github.com/wordpress-mobile/WordPress-iOS/issues/14460
13331331
alert.popoverPresentationController?.sourceRect = button.convert(button.bounds, to: navigationController?.navigationBar)

WordPress/Classes/ViewRelated/Blog/Site Picker/SiteIconPickerView.swift

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import SwiftUI
22
import UIKit
33

4-
@available(iOS 14.0, *)
54
struct SiteIconPickerView: View {
65
private let initialIcon = Image("blavatar-default")
76

WordPress/Classes/ViewRelated/Blog/Site Picker/SitePickerViewController+SiteIcon.swift

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -92,10 +92,6 @@ extension SitePickerViewController {
9292
}
9393

9494
func showEmojiPicker() {
95-
guard #available(iOS 14.0, *) else {
96-
return
97-
}
98-
9995
var pickerView = SiteIconPickerView()
10096

10197
pickerView.onCompletion = { [weak self] image in

WordPress/Classes/ViewRelated/Blog/Site Picker/SitePickerViewController.swift

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -88,11 +88,7 @@ extension SitePickerViewController: BlogDetailHeaderViewDelegate {
8888
showUpdateSiteIconAlert()
8989
}
9090

91-
if #available(iOS 14.0, *) {
92-
showSiteIconSelectionAlert()
93-
} else {
94-
showUpdateSiteIconAlert()
95-
}
91+
showSiteIconSelectionAlert()
9692
}
9793

9894
func siteIconReceivedDroppedImage(_ image: UIImage?) {

WordPress/Classes/ViewRelated/Gutenberg/GutenbergMediaPickerHelper.swift

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,7 @@ class GutenbergMediaPickerHelper: NSObject {
6868
picker.delegate = self
6969
picker.mediaPicker.registerClass(forReusableCellOverlayViews: DisabledVideoOverlay.self)
7070

71-
if #available(iOS 14.0, *),
72-
FeatureFlag.mediaPickerPermissionsNotice.enabled {
71+
if FeatureFlag.mediaPickerPermissionsNotice.enabled {
7372
picker.mediaPicker.registerClass(forCustomHeaderView: DeviceMediaPermissionsHeader.self)
7473
}
7574

@@ -137,8 +136,7 @@ extension GutenbergMediaPickerHelper: WPMediaPickerViewControllerDelegate {
137136
}
138137

139138
func mediaPickerControllerShouldShowCustomHeaderView(_ picker: WPMediaPickerViewController) -> Bool {
140-
guard #available(iOS 14.0, *),
141-
FeatureFlag.mediaPickerPermissionsNotice.enabled,
139+
guard FeatureFlag.mediaPickerPermissionsNotice.enabled,
142140
picker !== cameraPicker else {
143141
return false
144142
}
@@ -147,19 +145,14 @@ extension GutenbergMediaPickerHelper: WPMediaPickerViewControllerDelegate {
147145
}
148146

149147
func mediaPickerControllerReferenceSize(forCustomHeaderView picker: WPMediaPickerViewController) -> CGSize {
150-
guard #available(iOS 14.0, *) else {
151-
return .zero
152-
}
153-
154148
let header = DeviceMediaPermissionsHeader()
155149
header.translatesAutoresizingMaskIntoConstraints = false
156150

157151
return header.referenceSizeInView(picker.view)
158152
}
159153

160154
func mediaPickerController(_ picker: WPMediaPickerViewController, configureCustomHeaderView headerView: UICollectionReusableView) {
161-
guard #available(iOS 14.0, *),
162-
let headerView = headerView as? DeviceMediaPermissionsHeader else {
155+
guard let headerView = headerView as? DeviceMediaPermissionsHeader else {
163156
return
164157
}
165158

WordPress/Classes/ViewRelated/Gutenberg/GutenbergViewController+MoreActions.swift

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,7 @@ extension GutenbergViewController {
7373
ActionDispatcher.dispatch(NoticeAction.unlock)
7474
}
7575

76-
if #available(iOS 14.0, *),
77-
let button = navigationBarManager.moreBarButtonItem.customView {
76+
if let button = navigationBarManager.moreBarButtonItem.customView {
7877
// Required to work around an issue present in iOS 14 beta 2
7978
// https://github.com/wordpress-mobile/WordPress-iOS/issues/14460
8079
alert.popoverPresentationController?.sourceRect = button.convert(button.bounds, to: navigationController?.navigationBar)

0 commit comments

Comments
 (0)