Fixed the problem discussed in issues 513

appsquickly / typhoon

Powerful dependency injection for Objective-C ✨✨ (https://PILGRIM.PH is the pure Swift successor to Typhoon!!)✨✨

https://pilgrim.ph

Apache License 2.0

2.7k stars 269 forks source link

Fixed the problem discussed in issues 513 #524

Closed CognitiveDisson closed 8 years ago

CognitiveDisson commented 8 years ago

Solution with a subclass does not work ( in the ios 10 no more ivar firstItem, secondItem). In the pull request, I implemented another more complicated solution. It works correctly on all operating system versions. I also added integration tests on the transfer of outlets. I hope you are satisfied with this solution, unfortunately more elegant and working solution could not be found. Even with the use swizzle (these solutions also tried).

etolstoy commented 8 years ago

@CognitiveDisson thanks for fixes - looks great for me!

vasilenkoigor commented 8 years ago

@CognitiveDisson Looks good for me too

alexgarbarev commented 8 years ago

Sorry for delay. Finally I did complex review of this PullRequest. This looks good - thanks for PR!