[[ObjectMapper sharedInstance] dictionaryFromObject:object] doesn't work if the object comes from a framework #36
Comments
|
what do you mean by doesn't work? What's the behavior? Would it work if you update the code according to this PR? https://github.com/aryaxt/OCMapper/pull/34/files |
|
@aryaxt Hi aryaxt, thanks for the super quick reply. Yes I noticed the cause is definitely the bundle checking. ...
// For example when we are mapping an array of string, we shouldn't try to map the string objects inside the array
if ([NSBundle mainBundle] != [NSBundle bundleForClass:object.class] && [object class] != [NSArray class])
{
return object;
}I think the pull request might go some way to fixing it... I'll need to try that. |
|
@aryaxt I checked out the version from the pull request but it's not helped. I don't understand why bundle checking is necessary at all. It really shouldn't matter whether the class that I'm trying to convert belongs in one bundle or another — it's just a class, right :S? Plus, I'm using AppCode, not Xcode. When I debugged the comparison between mainBundlePath = {__NSCFString * | 0x7ff013d6eb20} "/Users/myusername/Library/Developer/CoreSimulator/Devices/4CCEBA75-D7CE-408F-8135-97927736A940/data/Containers/Bundle/Application/854F84EA-647C-4B99-A94B-E5E94186714B/MakeEatSeeRNUI_Example.app"
classBundlePath = {__NSCFString * | 0x7ff016005020} "/Users/myusername/Library/Caches/AppCode32/DerivedData/MakeEatSeeRNUI-8ccb4a77/Build/Products/Debug-iphonesimulator/MakeEatSeePresenters.framework"So yeah, there's just no way this comparison could possibly succeed. This causes |
|
yeah the code to detect project-specific classes need to be reworked, right now it only works if the models are in the main bundle To understand the reason behind that logic, you can pull my code, comment out the code, and run the unit tests. Fell free to open a PR if you find a solution. I'll look into it myself as well |
|
Hmmm. The relevant test in question is this: - (void)testShouldMapArrayOfStringFromObjectToDictionary
{
User *user = [[User alloc] init];
user.randomKeywords = @[@"keyword1", @2].mutableCopy;
NSDictionary *dictionary = [self.mapper dictionaryFromObject:user];
NSArray *array = [dictionary objectForKey:@"randomKeywords"];
XCTAssertTrue(array.count == 2);
XCTAssertTrue([array[0] isEqualToString:@"keyword1"]);
XCTAssertTrue([array[1] isEqualToNumber:@2]);
}:/ the implementation appears to make quite a lot of assumptions about what classes belong in bundles and which don't. I've got a wild idea: how about check for the |
|
Let me think about the NS prefix, maybe it could be in additions to bundle checking. Bundle checking is very reliable as long as your models are in the main bundle |
|
@aryaxt Yeah I gave just checking for the |
|
@fatuhoku Did you manage to find a solution? I haven't had time to look into this |
|
@aryaxt Yes — I just used HRCoder + AutoCoding to serialise the JSON out for React Native consumption instead! I'm not sure how they solve the cross-bundle issue. Probably worth checking the |
I know that
MESRecipePreviewRecipeIngredientViewModelcan be converted into a dictionary normally in a standard iOS app project. The moment I moved it to another private pod framework, I saw this error:I.e. I was trying to use
OCMapperto convert myViewModelclass into a dictionary that React Native can consume. RN is barfing because the object didn't get converted at allThe text was updated successfully, but these errors were encountered: