ashfurrow / C-41

C-41 is an application to help people develop film at home by providing a series of "recipes" for photographers to use.
MIT License
2.05k stars 368 forks source link

Cancelling the insertion processes crashes app #1

Closed ashfurrow closed 10 years ago

ashfurrow commented 10 years ago

I think it's due to this line: https://github.com/ReactiveCocoa/ReactiveCocoa/blob/master/ReactiveCocoaFramework/ReactiveCocoa/RACKVOChannel.m#L129

[object setValue:x ?: nilValue forKey:lastKeyPathComponent];

x appears to always be zero. I get the following error:

Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '[<ASHEditRecipeViewModel 0x9819ac0> setNilValueForKey]: could not set nil as the value for the key filmType.'

Ultimately caused by the following line in my view model:

RACChannelTo(self, filmType) = RACChannelTo(self.model, filmType);
ashfurrow commented 10 years ago

Fixed with help from @jspahrsummers