Merge branch 'release/2.2.0'

Bean OSX Static Library/Bean OSX LibraryTests/Resources/Firmware Images/Bean+/version.txt

@@ -1 +1 @@
-201605240000
+201607260001

Bean OSX Static Library/Bean OSX LibraryTests/Resources/Firmware Images/Bean/version.txt

@@ -1 +1 @@
-201605240000
+201606030000

Bean-iOS-OSX-SDK.podspec

@@ -2,7 +2,7 @@
 Pod::Spec.new do |s|
 
   s.name         = "Bean-iOS-OSX-SDK"
-  s.version      = "2.1.0"
+  s.version      = "2.2.0"
   s.summary      = "Punch Through Design's SDK for speeding up development with the LightBlue Bean development platform"
   s.homepage     = "https://github.com/PunchThrough/Bean-iOS-OSX-SDK"
   s.license      = { :type => "MIT", :file => "LICENSE.txt" }

CHANGELOG.txt

@@ -7,6 +7,28 @@
            __/ |                                  SDK for iOS and OS X
           |___/
 
+---------------------------------
+2.2.0
+Released on 2016-08-11
+---------------------------------
+App Message Definitions: master/22a7dea (2016-03-28)
+Bean Test Firmware: 201606030000 (2016-06-03)
+Bean+ Test Firmware: 201607260001 (2016-07-26)
+---------------------------------
+
+FEATURES:
+* Updated firmware for Bean and Bean+
+* Updated App Message Definitions
+
+IMPROVEMENTS:
+* Better sketch upload error messages
+* Deprecated [PTDBean eraseSketchWithHandler:]; use 
+  [PTDBean setArduinoPowerState:] instead
+
+BUG FIXES:
+* Fixed spelling errors in logging messages from Bean Manager
+* Fixed improper handling of certain completion blocks
+
 ---------------------------------
 2.1.0
 Released on 2016-05-26

firmware/Bean+/version.txt

@@ -0,0 +1 @@
+201607260001

firmware/Bean/version.txt

@@ -0,0 +1 @@
+201606030000

source/PTDBean.m

@@ -31,6 +31,8 @@ @interface PTDBean () <CBPeripheralDelegate, AppMessagingLayerDelegate, OAD_Dele
 @property (nonatomic, readwrite) NSString *sketchName;
 @property (nonatomic, assign) NSInteger targetFirmwareVersion;
 @property (nonatomic, copy) void (^sketchErasedHandler)(BOOL sketchErased);
+@property (nonatomic, copy) void (^firmwareVersionAvailableHandler)(BOOL firmwareAvailable, NSError *error);
+@property (nonatomic, copy) void (^hardwareVersionAvailableHandler)(BOOL hardwareAvailable, NSError *error);
 
 @end
 
@@ -53,9 +55,6 @@ @implementation PTDBean
     BeanArduinoOADLocalState    localArduinoOADState;
     NSTimer*                    arduinoOADStateTimout;
     NSTimer*                    arduinoOADChunkSendTimer;
-
-    void (^firmwareVersionAvailableHandler)(BOOL firmwareAvailable, NSError *error);
-    void (^hardwareVersionAvailableHandler)(BOOL hardwareAvailable, NSError *error);
     NSDate*                     firmwareUpdateStartTime;
 
 }
@@ -322,7 +321,7 @@ - (void)checkFirmwareVersionAvailableWithHandler:(void (^)(BOOL firmwareAvailabl
     if ( [self firmwareVersion] ) {
         handler( YES, nil );
     } else {
-        firmwareVersionAvailableHandler = handler;   // Wait until device info is valid
+        self.firmwareVersionAvailableHandler = handler;   // Wait until device info is valid
     }
 }
 
@@ -331,7 +330,7 @@ - (void)checkHardwareVersionAvailableWithHandler:(void (^)(BOOL hardwareAvailabl
     if ( [self hardwareVersion] ) {
         handler( YES, nil );
     } else {
-        hardwareVersionAvailableHandler = handler;   // Wait until device info is valid
+        self.hardwareVersionAvailableHandler = handler;   // Wait until device info is valid
     }
 }
 
@@ -365,6 +364,7 @@ - (void)cancelFirmwareUpdate{
     }
 }
 
+// Deprecated
 - (void)eraseSketchWithHandler:(void (^)(BOOL sketchErased))handler{
 
     if([self.sketchName isEqualToString:@""]) {
@@ -450,7 +450,7 @@ -(void)__setArduinoOADTimeout:(NSTimeInterval)duration{
     arduinoOADStateTimout = [NSTimer scheduledTimerWithTimeInterval:duration target:self selector:@selector(__arduinoOADTimeout:) userInfo:nil repeats:NO];
 }
 -(void)__arduinoOADTimeout:(NSTimer*)timer{
-    NSError* error = [BEAN_Helper basicError:@"Sketch upload failed!" domain:NSStringFromClass([self class]) code:0];
+    NSError* error = [BEAN_Helper basicError:@"Sketch upload failed: Arduino communication timed out" domain:NSStringFromClass([self class]) code:0];
     if (self.uploadInProgress) {
         [self __alertDelegateOfArduinoOADCompletion:error];
     }
@@ -516,7 +516,7 @@ -(void)__handleArduinoOADRemoteStateChange:(BL_HL_STATE_T)state{
             break;
         case BL_HL_STATE_ERROR:
         {
-            NSError *error = [BEAN_Helper basicError:@"Sketch upload failed!" domain:NSStringFromClass([self class]) code:0];
+            NSError *error = [BEAN_Helper basicError:@"Sketch upload failed: Bootloader error" domain:NSStringFromClass([self class]) code:0];
             if (self.uploadInProgress) {
                 [self __alertDelegateOfArduinoOADCompletion:error];
             }
@@ -926,9 +926,9 @@ -(void)batteryProfileDidUpdate
 
 - (void)hardwareVersionDidUpdate
 {
-    if (hardwareVersionAvailableHandler){
-        [self checkHardwareVersionAvailableWithHandler:hardwareVersionAvailableHandler];
-        hardwareVersionAvailableHandler = nil;
+    if (self.hardwareVersionAvailableHandler){
+        [self checkHardwareVersionAvailableWithHandler:self.hardwareVersionAvailableHandler];
+        self.hardwareVersionAvailableHandler = nil;
     }
 }
 
@@ -940,9 +940,9 @@ - (void)firmwareVersionDidUpdate
     // Don't send firmware version back to handler when firmware update is still in progress
     if (self.updateInProgress) return;
 
-    if (firmwareVersionAvailableHandler) {
-        [self checkFirmwareVersionAvailableWithHandler:firmwareVersionAvailableHandler];
-        firmwareVersionAvailableHandler = nil;
+    if (self.firmwareVersionAvailableHandler) {
+        [self checkFirmwareVersionAvailableWithHandler:self.firmwareVersionAvailableHandler];
+        self.firmwareVersionAvailableHandler = nil;
     }
 }
 

source/PTDBeanManager.m

@@ -125,20 +125,20 @@ -(void)connectToBean:(PTDBean*)bean_ withOptions:(NSDictionary*)options error:(N
     PTDBean* bean = [beanRecords objectForKey:bean_.identifier];
     //If there is no such peripheral, return error
     if(!bean){
-        if(error) *error = [BEAN_Helper basicError:@"Attemp to connect to Bean failed. No peripheral discovered with the corresponding UUID." domain:NSStringFromClass([self class]) code:BeanErrors_NoPeriphealDiscovered];
+        if(error) *error = [BEAN_Helper basicError:@"Attempt to connect to Bean failed. No peripheral discovered with the corresponding UUID." domain:NSStringFromClass([self class]) code:BeanErrors_NoPeriphealDiscovered];
         return;
     }
     //Check if the device is already connected
     else if(bean.state == BeanState_ConnectedAndValidated){
-        if(error) *error = [BEAN_Helper basicError:@"Attemp to connect to Bean failed. A device with this UUID is already connected" domain:NSStringFromClass([self class]) code:BeanErrors_AlreadyConnected];
+        if(error) *error = [BEAN_Helper basicError:@"Attempt to connect to Bean failed. A device with this UUID is already connected" domain:NSStringFromClass([self class]) code:BeanErrors_AlreadyConnected];
         return;
     }
     //Check if the device is already in the middle of an attempted connected
     else if(bean.state == BeanState_AttemptingConnection || bean.state == BeanState_AttemptingValidation){
-        if(error) *error = [BEAN_Helper basicError:@"Attemp to connect to Bean failed. A device with this UUID is in the process of being connected to." domain:NSStringFromClass([self class]) code:BeanErrors_AlreadyConnecting];
+        if(error) *error = [BEAN_Helper basicError:@"Attempt to connect to Bean failed. A device with this UUID is in the process of being connected to." domain:NSStringFromClass([self class]) code:BeanErrors_AlreadyConnecting];
         return;
     }else if(bean.state != BeanState_Discovered){
-        if(error) *error = [BEAN_Helper basicError:@"Attemp to connect to Bean failed. The device's current state is not eligible for a connection attempt." domain:NSStringFromClass([self class]) code:BeanErrors_DeviceNotEligible];
+        if(error) *error = [BEAN_Helper basicError:@"Attempt to connect to Bean failed. The device's current state is not eligible for a connection attempt." domain:NSStringFromClass([self class]) code:BeanErrors_DeviceNotEligible];
         return;
     }
 
@@ -164,7 +164,7 @@ -(void)disconnectBean:(PTDBean*)bean_ error:(NSError**)error
     PTDBean* bean = [beanRecords objectForKey:bean_.identifier];
     //Check if the device isn't currently connected
     if(!bean){
-        if(error) *error = [BEAN_Helper basicError:@"Failed attemp to disconnect Bean. No device with this UUID is currently connected" domain:NSStringFromClass([self class]) code:BeanErrors_FailedDisconnect];
+        if(error) *error = [BEAN_Helper basicError:@"Failed attempt to disconnect Bean. No device with this UUID is currently connected" domain:NSStringFromClass([self class]) code:BeanErrors_FailedDisconnect];
         return;
     }
     if(bean.peripheral.state != CBPeripheralStateConnected
@@ -206,7 +206,7 @@ -(void)bean:(PTDBean*)device hasBeenValidated_error:(NSError*)error{
     PTDBean* bean = [beanRecords objectForKey:[device identifier]];
     //If there is no such peripheral, return error
     if(!bean){
-        localError = [BEAN_Helper basicError:@"Attemp to connect to Bean failed. No peripheral discovered with the corresponding UUID." domain:NSStringFromClass([self class]) code:BeanErrors_NoPeriphealDiscovered];
+        localError = [BEAN_Helper basicError:@"Attempt to connect to Bean failed. No peripheral discovered with the corresponding UUID." domain:NSStringFromClass([self class]) code:BeanErrors_NoPeriphealDiscovered];
     }else if (error){
         localError = error;
         bean.state = BeanState_Discovered; // Reset bean state to the default, ready to connect

source/Public/PTDBean.h

@@ -624,7 +624,7 @@ typedef NS_ENUM(NSUInteger, BeanBluetoothError) {
  *  Erases sketch with completion handler. Used to ensure sketch is cleared before updating from Sym. to Asym. FW
  *  @param The handler to run once the sketch name has been updated. The sketchErased bool indicates whether the erasure was successful or not.
  */
-- (void)eraseSketchWithHandler:(void (^)(BOOL sketchErased))handler;
+- (void)eraseSketchWithHandler:(void (^)(BOOL sketchErased))handler __attribute__((deprecated(("Use [setArduinoPowerState:] instead"))));
 
 @end