Merge pull request #54 from ParsePlatform/nlutsenko.locale

Unpublicize localeIdentifier property on PFInstallation.

Author: nlutsenko

Parse/Internal/Installation/PFInstallationPrivate.h

@@ -24,6 +24,5 @@
 @property (nonatomic, copy, readwrite) NSString *deviceType;
 @property (nonatomic, copy, readwrite) NSString *installationId;
 @property (nonatomic, copy, readwrite) NSString *timeZone;
-@property (nonatomic, copy, readwrite) NSString *localeIdentifier;
 
 @end

Parse/PFInstallation.h

@@ -79,11 +79,6 @@ PF_ASSUME_NONNULL_BEGIN
  */
 @property (PF_NULLABLE_PROPERTY nonatomic, copy, readonly) NSString *timeZone;
 
-/*!
- @abstract The localeIdentifier for the `PFInstallation` ([language code]-[COUNTRY CODE]).
- */
-@property (PF_NULLABLE_PROPERTY nonatomic, copy, readonly) NSString *localeIdentifier;
-
 /*!
  @abstract The channels for the `PFInstallation`.
  */

Parse/PFInstallation.m

@@ -108,7 +108,6 @@ @implementation PFInstallation
 @dynamic installationId;
 @dynamic deviceToken;
 @dynamic timeZone;
-@dynamic localeIdentifier;
 @dynamic channels;
 @dynamic badge;
 
@@ -336,8 +335,10 @@ - (void)_updateLocaleIdentifierFromDevice {
         localeIdentifier = language;
     }
 
-    if (localeIdentifier.length > 0 && ![localeIdentifier isEqualToString:self.localeIdentifier]) {
-        self.localeIdentifier = localeIdentifier;
+    NSString *currentLocaleIdentifier = self[PFInstallationKeyLocaleIdentifier];
+    if (localeIdentifier.length > 0 && ![localeIdentifier isEqualToString:currentLocaleIdentifier]) {
+        // Call into super to avoid checking on protected keys.
+        [super setObject:localeIdentifier forKey:PFInstallationKeyLocaleIdentifier];
     }
 }
 

Tests/Unit/InstallationUnitTests.m

@@ -31,6 +31,8 @@ - (void)testInstallationImmutableFieldsCannotBeChanged {
                                            @"Should throw an exception for trying to change deviceType.");
     PFAssertThrowsInvalidArgumentException(installation[@"installationId"] = @"a"
                                            @"Should throw an exception for trying to change installationId.");
+    PFAssertThrowsInvalidArgumentException(installation[@"localeIdentifier"] = @"a"
+                                           @"Should throw an exception for trying to change installationId.");
 }
 
 - (void)testInstallationImmutableFieldsCannotBeDeleted {
@@ -41,6 +43,8 @@ - (void)testInstallationImmutableFieldsCannotBeDeleted {
                                            @"Should throw an exception for trying to delete deviceType.");
     PFAssertThrowsInvalidArgumentException([installation removeObjectForKey:@"installationId"],
                                            @"Should throw an exception for trying to delete installationId.");
+    PFAssertThrowsInvalidArgumentException([installation removeObjectForKey:@"localeIdentifier"],
+                                           @"Should throw an exception for trying to delete installationId.");
 }
 
 @end