feat: add Ti.Platform.versionPatch and global.OS_VERSION_PATCH

android/modules/platform/src/java/ti/modules/titanium/platform/PlatformModule.java

@@ -73,6 +73,7 @@ public class PlatformModule extends KrollModule
 	private List<Processor> processors;
 	private int versionMajor;
 	private int versionMinor;
+	private int versionPatch;
 	protected int batteryState;
 	protected double batteryLevel;
 	protected boolean batteryStateReady;
@@ -92,6 +93,9 @@ public PlatformModule()
 			this.versionMajor = Integer.parseInt(versionComponents[0]);
 			if (versionComponents.length >= 2) {
 				this.versionMinor = Integer.parseInt(versionComponents[1]);
+				if (versionComponents.length >= 3) {
+					this.versionPatch = Integer.parseInt(versionComponents[2]);
+				}
 			}
 		} catch (Exception ex) {
 			Log.e(TAG, "Failed to parse OS version string.", ex);
@@ -163,6 +167,12 @@ public int getVersionMinor()
 		return this.versionMinor;
 	}
 
+	@Kroll.getProperty
+	public int getVersionPatch()
+	{
+		return this.versionPatch;
+	}
+
 	@Kroll.method
 	@Kroll.getProperty
 	public double getAvailableMemory()

apidoc/Global/Global.yml

@@ -345,6 +345,17 @@ properties:
     permission: read-only
     since: "9.2.0"
 
+  - name: OS_VERSION_PATCH
+    summary: The operating system's patch version number.
+    description: |
+        This returns the same value as the <Titanium.Platform.versionPatch> property.
+
+        Will return zero if the OS does not have a patch version.
+    type: Number
+    default: 0
+    permission: read-only
+    since: "9.2.0"
+
   - name: ENV_DEV
     summary: Alias for <ENV_DEVELOPMENT>
     type: Boolean

apidoc/Titanium/Platform/Platform.yml

@@ -380,6 +380,17 @@ properties:
     permission: read-only
     since: "9.2.0"
 
+  - name: versionPatch
+    summary: The operating system's patch version number.
+    description: |
+      This returns the same value as the [OS_VERSION_PATCH](Global.OS_VERSION_PATCH) constant.
+
+      Will return zero if the OS does not have a patch version.
+    type: Number
+    default: 0
+    permission: read-only
+    since: "9.2.0"
+
 examples:
   - title: Battery Event Example
     example: |

common/Resources/ti.internal/extensions/globals.js

@@ -12,5 +12,6 @@ Object.defineProperties(global, {
 	'OS_ANDROID': { value: OS_ANDROID, writable: false },
 	'OS_IOS': { value: OS_IOS, writable: false },
 	OS_VERSION_MAJOR: { value: Ti.Platform.versionMajor, writable: false },
-	OS_VERSION_MINOR: { value: Ti.Platform.versionMinor, writable: false }
+	OS_VERSION_MINOR: { value: Ti.Platform.versionMinor, writable: false },
+	OS_VERSION_PATCH: { value: Ti.Platform.versionPatch, writable: false }
 });

iphone/Classes/PlatformModule.h

@@ -48,6 +48,7 @@ READONLY_PROPERTY(NSString *, username, Username);
 READONLY_PROPERTY(NSString *, version, Version);
 READONLY_PROPERTY(NSNumber *, versionMajor, VersionMajor);
 READONLY_PROPERTY(NSNumber *, versionMinor, VersionMinor);
+READONLY_PROPERTY(NSNumber *, versionPatch, VersionPatch);
 
 // Methods
 - (BOOL)canOpenURL:(NSString *)url;

iphone/Classes/PlatformModule.m

@@ -29,24 +29,30 @@
 
 @implementation PlatformModule
 
-@synthesize architecture, availableMemory, model, name, osname, ostype, processorCount, totalMemory, uptime, username, version, versionMajor, versionMinor;
+@synthesize architecture, availableMemory, model, name, osname, ostype, processorCount, totalMemory, uptime, username, version, versionMajor, versionMinor, versionPatch;
 
 #pragma mark Internal
 
 - (id)init
 {
   if (self = [super init]) {
-    UIDevice *theDevice = [UIDevice currentDevice];
-    name = [[theDevice systemName] retain];
-    version = [[theDevice systemVersion] retain];
+    UIDevice *theDevice = UIDevice.currentDevice;
+    name = [theDevice.systemName retain];
+    version = [theDevice.systemVersion retain];
 
     // Extract "<major>.<minor>" integers from OS version string.
     NSArray *versionComponents = [version componentsSeparatedByString:@"."];
     versionMajor = [NSNumber numberWithInt:[versionComponents[0] intValue]];
     if ([versionComponents count] >= 2) {
       versionMinor = [NSNumber numberWithInt:[versionComponents[1] intValue]];
+      if ([versionComponents count] >= 3) {
+        versionPatch = [NSNumber numberWithInt:[versionComponents[2] intValue]];
+      } else {
+        versionPatch = @0;
+      }
     } else {
       versionMinor = @0;
+      versionPatch = @0;
     }
 
     // grab logical CPUs
@@ -58,7 +64,7 @@ - (id)init
     }
     processorCount = [[NSNumber numberWithInt:cores] retain];
 
-    username = [[theDevice name] retain];
+    username = [theDevice.name retain];
 #ifdef __LP64__
     ostype = [@"64bit" retain];
 #else
@@ -73,7 +79,7 @@ - (id)init
       osname = [@"iphone" retain];
     }
 
-    NSString *themodel = [theDevice model];
+    NSString *themodel = theDevice.model;
 
     // attempt to determine extended phone info
     struct utsname u;
@@ -88,7 +94,7 @@ - (id)init
     architecture = [[TiUtils currentArchitecture] retain];
 
     // needed for platform displayCaps orientation to be correct
-    [[UIDevice currentDevice] beginGeneratingDeviceOrientationNotifications];
+    [theDevice beginGeneratingDeviceOrientationNotifications];
   }
   return self;
 }
@@ -100,6 +106,7 @@ - (void)dealloc
   RELEASE_TO_NIL(version);
   RELEASE_TO_NIL(versionMajor);
   RELEASE_TO_NIL(versionMinor);
+  RELEASE_TO_NIL(versionPatch);
   RELEASE_TO_NIL(architecture);
   RELEASE_TO_NIL(processorCount);
   RELEASE_TO_NIL(username);

tests/Resources/ti.platform.test.js

@@ -4,7 +4,7 @@
  * Licensed under the terms of the Apache Public License
  * Please see the LICENSE included with this distribution for details.
  */
-/* global OS_VERSION_MAJOR, OS_VERSION_MINOR, OS_IOS */
+/* global OS_VERSION_MAJOR, OS_VERSION_MINOR, OS_VERSION_PATCH, OS_IOS */
 /* eslint-env mocha */
 /* eslint no-unused-expressions: "off" */
 'use strict';
@@ -308,6 +308,15 @@ describe('Titanium.Platform', function () {
 		should(Ti.Platform.versionMinor).be.eql(versionMinor);
 	});
 
+	it('.versionPatch', () => {
+		should(Ti.Platform).have.readOnlyProperty('versionPatch').which.is.a.Number();
+		should(Ti.Platform.versionPatch).be.eql(OS_VERSION_PATCH);
+
+		const versionComponents = Ti.Platform.version.split('.');
+		const versionPatch = (versionComponents.length >= 3) ? parseInt(versionComponents[2]) : 0;
+		should(Ti.Platform.versionPatch).be.eql(versionPatch);
+	});
+
 	it.ios('.identifierForVendor', () => {
 		should(Ti.Platform.identifierForVendor).be.a.String();
 		should(Ti.Platform.getIdentifierForVendor).be.a.Function();