WifiWizard2 icon indicating copy to clipboard operation
WifiWizard2 copied to clipboard

Method "resetBindAll" improvements

Open cloudsurfers-it opened this issue 5 years ago • 2 comments

Description of the Change

Now when call resetBindAll always force bindProcessToNetwork(null) without network connection connected ("desired" variable equal NULL).

Events such network receiver and network callback are correctly managed and unregistered before register new events. This fix try to avoid double invoking of the same event and cause problems when connecting to a network with bindAll parameter set to TRUE and subsequently disconnected.

Benefits

If you connect to a Wifi network without internet connection (bindAll parameter activated) and subsequently connect to a Wifi network with internet connection, second connection invoke bindProcessToNetwork(network) also without bindAll parameter. Now if you disconnect from network, resetBindAll don't work property and app doesn't access to mobile data connection for example.

Possible Drawbacks

From my tests and use case, nothing. I use this plugin to connect to a specific wifi network without internet connection and switching between different network configurations: mobile data, wifi with internet and wifi without internet access.

Applicable Issues

https://github.com/tripflex/WifiWizard2/issues/94

cloudsurfers-it avatar Feb 21 '20 13:02 cloudsurfers-it

@cloudsurfers-it thanks for submitting this, can anybody else validate if they have tested this at all?

tripflex avatar Oct 12 '20 15:10 tripflex

@tripflex I need this fix, but I am not sure how to verify it works.

siderisltd avatar Feb 15 '21 19:02 siderisltd