Merge pull request #63 from onderceylan/master

Updated iOS SDK and changed bash script's relative path in workspace

Author: sarriaroman

README.md

@@ -4,7 +4,7 @@ This is a [Cordova](http://cordova.apache.org/) plugin for [Fabric.io](https://w
 
 It requires Cordova 3.x or newer (tested on 5.4.0) and has APIs for iOS and Android.
 
-The iOS version uses Fabric SDK 1.6.5 and Crashlytics SDK 3.6.0 framework bundles which are located in `lib/ios`.
+The iOS version uses Fabric SDK 1.6.9 and Crashlytics SDK 3.8.2 framework bundles which are located in `lib/ios`.
 
 The Android version uses Gradle to get the Fabric SDK (`io.fabric.tools:gradle:1.+`) and the Crashlytics SDK (`com.crashlytics.sdk.android:crashlytics:2.5.5`) from Maven repositories when the plugin is added.
 

hooks/lib/ios-helper.js

@@ -31,9 +31,10 @@ module.exports = {
         // ../../plugins/plugin-id/lib/ios/Crashlytics.framework/run API_KEY SECRET_KEY
         var script = [
             "\"",
-            "../../plugins/",
+            utilities.getAppName(context),
+            "/Plugins/",
             utilities.getPluginId(),
-            "/lib/ios/Crashlytics.framework/run ",
+            "/Fabric.framework/run ",
             pluginConfig.apiKey,
             " ",
             pluginConfig.apiSecret,

hooks/lib/utilities.js

@@ -6,18 +6,6 @@
 var path = require("path");
 var fs = require("fs");
 
-/**
- * Used to get the name of the application as defined in the config.xml.
- *
- * @param {object} context - The Cordova context.
- * @returns {string} The value of the name element in config.xml.
- */
-function getAppName(context) {
-    var ConfigParser = context.requireCordovaModule("cordova-lib").configparser;
-    var config = new ConfigParser("config.xml");
-    return config.name();
-};
-
 /**
  * Used to get the path to the build.gradle file for the Android project.
  *
@@ -29,6 +17,18 @@ function getBuildGradlePath() {
 
 module.exports = {
 
+    /**
+     * Used to get the name of the application as defined in the config.xml.
+     * 
+     * @param {object} context - The Cordova context.
+     * @returns {string} The value of the name element in config.xml.
+     */
+    getAppName: function(context) {
+        var ConfigParser = context.requireCordovaModule("cordova-lib").configparser;
+        var config = new ConfigParser("config.xml");
+        return config.name();
+    },
+
     /**
      * The ID of the plugin; this should match the ID in plugin.xml.
      */
@@ -76,7 +76,7 @@ module.exports = {
      */
     getXcodeProjectPath: function(context) {
 
-        var appName = getAppName(context);
+        var appName = this.getAppName(context);
 
         return path.join("platforms", "ios", appName + ".xcodeproj", "project.pbxproj");
     },

lib/ios/Crashlytics.framework/Crashlytics

@@ -26,6 +26,7 @@ NS_ASSUME_NONNULL_BEGIN
 + (instancetype)stackFrameWithSymbol:(NSString *)symbol;
 
 @property (nonatomic, copy, nullable) NSString *symbol;
+@property (nonatomic, copy, nullable) NSString *rawSymbol;
 @property (nonatomic, copy, nullable) NSString *library;
 @property (nonatomic, copy, nullable) NSString *fileName;
 @property (nonatomic, assign) uint32_t lineNumber;

lib/ios/Crashlytics.framework/Headers/CLSStackFrame.h

@@ -18,7 +18,10 @@ NS_ASSUME_NONNULL_BEGIN
 @protocol CrashlyticsDelegate;
 
 /**
- *  Crashlytics. Handles configuration and initialization of Crashlytics.
+ * Crashlytics. Handles configuration and initialization of Crashlytics.
+ *
+ * Note: The Crashlytics class cannot be subclassed. If this is causing you pain for
+ * testing, we suggest using either a wrapper class or a protocol extension.
  */
 @interface Crashlytics : NSObject