Support for iOS 11 #2
Conversation
Wi-Fi long press reference code: https://github.com/ioscreatix/WeatherVane
Bensge
left a comment
Bensge
left a comment
There was a problem hiding this comment.
Thanks for the PR. Please look at my comments. Definitely want to get it merged 👍
| } | ||
| } | ||
| else if (kCFCoreFoundationVersionNumber >= CoreFoundationiOS11) { | ||
| [[objc_getClass("WPScanRequest") alloc] setRssiThreshold:[NSNumber numberWithInt:-130]]; |
There was a problem hiding this comment.
Does this method actually affect the wifi scan initiated by the system or does this only change this instance of WPScanRequest? I would assume the latter
There was a problem hiding this comment.
There is no change on the normal Wi-Fi setting screen. Valid only when scanning with WiPi.
| if ([((__bridge NSDictionary *)userInfo)[NSLoadedClasses] containsObject:@"CCUIConnectivityAirDropViewController"]) { // The Network Bundle is Loaded | ||
| if (!didHook) { | ||
| didHook = YES; | ||
| %init(ControlCenter11); |
There was a problem hiding this comment.
Can you remove the notification observer afterwards? Just to clean things up and not leave the notification listener active forever
There was a problem hiding this comment.
You can delete it. It is necessary to prepare another method.
| } | ||
|
|
||
| BOOL shouldInit = NO; | ||
| if (!NSClassFromString(@"APTableCell")) { |
There was a problem hiding this comment.
Can you explain the logic here? This is not obvious to me. Why do you load the AirPort Settings bundle, if it succeeds, wait for the network bundle to be loaded?
There was a problem hiding this comment.
As with the notification method above, you need to prepare another method.
2 participants
No description provided.