[MOD-2286] Ti.Facebook iOS: Sending Ti.Blob's in requestWithGraphPath throws mutating-error
GitHub Issue | n/a |
---|---|
Type | Bug |
Priority | High |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2016-08-29T03:49:09.000+0000 |
Affected Version/s | Release 5.1.2, Release 5.3.0, Release 5.4.0, Release 5.2.1 |
Fix Version/s | Release 6.0.0 |
Components | |
Labels | facebook, graph, mutating |
Reporter | Hans Knöchel |
Assignee | Hans Knöchel |
Created | 2016-08-26T23:20:09.000+0000 |
Updated | 2018-08-06T17:49:12.000+0000 |
Description
When using the method
requestWithGraphPath
to send images to the graph, we unpack the blobs to NSData
types internally. So far so good, but while doing that, we fast-iterate through the dictionary of parameters and mutate the value if it is a blob. That needs to be fixed.
More infos: http://stackoverflow.com/questions/39152131/appcelerator-facebook-module-error-posting-photo-on-ios
*Expected behavior*: The requestWithGraphPath
method succeeds, a new photo is uploaded to Facebook
*Actual behavior*: The app fails with a *** Collection <__NSDictionaryM: 0x14ee27430> was mutated while being enumerated.
error
PR: https://github.com/appcelerator-modules/ti.facebook/pull/61 Demo:
A compiled version of the module is attached to the PR. [~cng] I would like to have this in 6.0.0, let me know if that's ok and I'll prepare the SDK-PR's.
CR and FT passed. APPROVED. (Good fix btw). Please back port for 6.0.0. And also indicate *Expected Result* and any additional helpful information in your test so that QE can verify easily later.
Information updated, new Ti.Facebook version released on Github, prepackaged module updated for master and 6_0_X.
Cleaning up older fixed issues. If this issue should not have been closed as fixed, please reopen.