Skip to content

Commit 5e154d1

Browse files
committed
Normalize requiredMinimumOSVersion of patch is .0
1 parent 5befe84 commit 5e154d1

File tree

4 files changed

+8
-7
lines changed

4 files changed

+8
-7
lines changed

Nudge/Preferences/DefaultPreferencesNudge.swift

+1
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ let majorUpgradeAppPathExists = FileManager.default.fileExists(atPath: majorUpgr
2828
let majorUpgradeBackupAppPathExists = FileManager.default.fileExists(atPath: Utils().getBackupMajorUpgradeAppPath())
2929
let requiredInstallationDate = osVersionRequirementsProfile?.requiredInstallationDate ?? osVersionRequirementsJSON?.requiredInstallationDate ?? Date(timeIntervalSince1970: 0)
3030
let requiredMinimumOSVersion = osVersionRequirementsProfile?.requiredMinimumOSVersion ?? osVersionRequirementsJSON?.requiredMinimumOSVersion ?? "0.0"
31+
let requiredMinimumOSVersionNormalized = try! OSVersion(requiredMinimumOSVersion).description
3132
let aboutUpdateURL = getAboutUpdateURL(OSVerReq: osVersionRequirementsProfile) ?? getAboutUpdateURL(OSVerReq: osVersionRequirementsJSON) ?? ""
3233

3334
// userExperience

Nudge/UI/StandardMode/LeftSide.swift

+1-1
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ struct StandardModeLeftSide: View {
4747
Text("Required OS Version:".localized(desiredLanguage: getDesiredLanguage()))
4848
.fontWeight(.bold)
4949
Spacer()
50-
Text(String(requiredMinimumOSVersion))
50+
Text(String(requiredMinimumOSVersionNormalized))
5151
.foregroundColor(.secondary)
5252
.fontWeight(.bold)
5353
}

Nudge/Utilities/SoftwareUpdate.swift

+1-1
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ class SoftwareUpdate {
7575
softwareupdateListLog.notice("\(msg, privacy: .public)")
7676
let task = Process()
7777
task.launchPath = "/usr/sbin/softwareupdate"
78-
task.arguments = ["--fetch-full-installer", "--full-installer-version", requiredMinimumOSVersion]
78+
task.arguments = ["--fetch-full-installer", "--full-installer-version", requiredMinimumOSVersionNormalized]
7979

8080
let outputPipe = Pipe()
8181
let errorPipe = Pipe()

Nudge/Utilities/Utils.swift

+5-5
Original file line numberDiff line numberDiff line change
@@ -139,9 +139,9 @@ struct Utils {
139139
}
140140

141141
func fullyUpdated() -> Bool {
142-
let fullyUpdated = versionGreaterThanOrEqual(currentVersion: currentOSVersion, newVersion: requiredMinimumOSVersion)
142+
let fullyUpdated = versionGreaterThanOrEqual(currentVersion: currentOSVersion, newVersion: requiredMinimumOSVersionNormalized)
143143
if fullyUpdated {
144-
let msg = "Current operating system (\(currentOSVersion)) is greater than or equal to required operating system (\(requiredMinimumOSVersion))"
144+
let msg = "Current operating system (\(currentOSVersion)) is greater than or equal to required operating system (\(requiredMinimumOSVersionNormalized))"
145145
utilsLog.notice("\(msg, privacy: .public)")
146146
return true
147147
} else {
@@ -228,7 +228,7 @@ struct Utils {
228228
}
229229

230230
func getMajorRequiredNudgeOSVersion() -> Int {
231-
let parts = requiredMinimumOSVersion.split(separator: ".", omittingEmptySubsequences: false)
231+
let parts = requiredMinimumOSVersionNormalized.split(separator: ".", omittingEmptySubsequences: false)
232232
let majorRequiredNudgeOSVersion = Int((parts[0]))!
233233
if !nudgePrimaryState.hasLoggedMajorRequiredOSVersion {
234234
nudgePrimaryState.hasLoggedMajorRequiredOSVersion = true
@@ -408,11 +408,11 @@ struct Utils {
408408
}
409409

410410
func logRequiredMinimumOSVersion() {
411-
nudgeDefaults.set(requiredMinimumOSVersion, forKey: "requiredMinimumOSVersion")
411+
nudgeDefaults.set(requiredMinimumOSVersionNormalized, forKey: "requiredMinimumOSVersion")
412412
}
413413

414414
func newNudgeEvent() -> Bool {
415-
versionGreaterThan(currentVersion: requiredMinimumOSVersion, newVersion: nudgePrimaryState.userRequiredMinimumOSVersion)
415+
versionGreaterThan(currentVersion: requiredMinimumOSVersionNormalized, newVersion: nudgePrimaryState.userRequiredMinimumOSVersion)
416416
}
417417

418418
func openMoreInfo() {

0 commit comments

Comments
 (0)