Merge pull request #61 from hansemannn/MOD-2286

[MOD-2286] iOS: Fix fast-iteration in graph-request
tidev · Aug 29, 2016 · 0bf731a · 0bf731a
2 parents f9c1cd0 + 54f2618
commit 0bf731a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/ios/Classes/FacebookModule.m b/ios/Classes/FacebookModule.m
@@ -776,13 +776,17 @@ -(void)requestWithGraphPath:(id)args
     id args3 = [args objectAtIndex:3];
     ENSURE_SINGLE_ARG(args3, KrollCallback);
     KrollCallback *callback = args3;
-    for(NSString *key in params) {
+
+    for (NSUInteger i = 0; i <  [[params allKeys] count]; i++) {
+        NSString *key = [[params allKeys] objectAtIndex:i];
         id value = [params objectForKey:key];
+
         if ([value isKindOfClass:[TiBlob class]]) {
             TiBlob *blob = (TiBlob*)value;
             [params setObject:[blob data] forKey:key];
         }
     }
+
     TiThreadPerformOnMainThread(^{
         if ([FBSDKAccessToken currentAccessToken]) {
             [[[FBSDKGraphRequest alloc] initWithGraphPath:path parameters:params HTTPMethod:httpMethod]

diff --git a/ios/manifest b/ios/manifest
@@ -2,7 +2,7 @@
 # this is your module manifest and used by Titanium
 # during compilation, packaging, distribution, etc.
 #
-version: 5.2.3
+version: 5.2.4
 apiversion: 2
 description: Allows Facebook integration for Titanium apps
 author: Mark Mokryn, Ng Chee Kiat and Hans Knoechel

diff --git a/ios/module.xcconfig b/ios/module.xcconfig
@@ -9,7 +9,7 @@
 
 //
 // How to add a Framework (example)
-FRAMEWORK_SEARCH_PATHS=$(SRCROOT)/../../modules/iphone/facebook/5.2.3/platform "~/Library/Application Support/Titanium/modules/iphone/facebook/5.2.3/platform"
+FRAMEWORK_SEARCH_PATHS=$(SRCROOT)/../../modules/iphone/facebook/5.2.4/platform "~/Library/Application Support/Titanium/modules/iphone/facebook/5.2.4/platform"
 OTHER_LDFLAGS=$(inherited) -framework Social -framework FBSDKCoreKit -framework FBSDKLoginKit -framework FBSDKShareKit -framework FBSDKMessengerShareKit -framework Bolts
 //
 // OTHER_LDFLAGS=$(inherited) -framework Foo