Separate Provisioning library from WiFi library to avoid unnecessary compilation while using WiFi library (#4550)

Author: sweetymhaiske

libraries/WiFiProv/examples/WiFiProv/WiFiProv.ino

@@ -1,3 +1,4 @@
+#include "WiFiProv.h"
 #include "WiFi.h"
 void SysProvEvent(arduino_event_t *sys_event)
 {
@@ -46,9 +47,9 @@ void setup() {
                    0xea, 0x4a, 0x82, 0x03, 0x04, 0x90, 0x1a, 0x02 };*/
   WiFi.onEvent(SysProvEvent);
 #if CONFIG_IDF_TARGET_ESP32 && CONFIG_BLUEDROID_ENABLED
-    WiFi.beginProvision(WIFI_PROV_SCHEME_BLE, WIFI_PROV_SCHEME_HANDLER_FREE_BTDM, WIFI_PROV_SECURITY_1, "abcd1234", "Prov_123");
+    WiFiProv.beginProvision(WIFI_PROV_SCHEME_BLE, WIFI_PROV_SCHEME_HANDLER_FREE_BTDM, WIFI_PROV_SECURITY_1, "abcd1234", "Prov_123");
 #else
-    WiFi.beginProvision(WIFI_PROV_SCHEME_SOFTAP, WIFI_PROV_SCHEME_HANDLER_NONE, WIFI_PROV_SECURITY_1, "abcd1234", "Prov_123");
+    WiFiProv.beginProvision(WIFI_PROV_SCHEME_SOFTAP, WIFI_PROV_SCHEME_HANDLER_NONE, WIFI_PROV_SECURITY_1, "abcd1234", "Prov_123");
 #endif
 }
 

libraries/WiFiProv/library.properties

@@ -4,6 +4,5 @@ author=Switi Mhaiske <[email protected]>
 maintainer=Hristo Gochkov <[email protected]>
 sentence=Enables provisioning.
 paragraph=With this library you can perform provisioning on esp32 via SoftAP or BLE.
-category=Communication
 url=
 architectures=esp32

libraries/WiFiProv/src/WiFiProv.cpp

@@ -34,7 +34,7 @@
 #include <wifi_provisioning/scheme_softap.h>
 #include <wifi_provisioning/manager.h>
 #undef IPADDR_NONE
-#include "WiFi.h"
+#include "WiFiProv.h"
 #if CONFIG_IDF_TARGET_ESP32
 #include "SimpleBLE.h"
 #endif
@@ -48,13 +48,6 @@ static const uint8_t custom_service_uuid[16] = {  0xb4, 0xdf, 0x5a, 0x1c, 0x3f,
 
 #define SERV_NAME_PREFIX_PROV "PROV_"
 
-bool WiFiProvClass::prov_enable = true;
-
-bool WiFiProvClass::isProvEnabled()
-{
-    return prov_enable;
-}
-
 static void get_device_service_name(prov_scheme_t prov_scheme, char *service_name, size_t max)
 {
     uint8_t eth_mac[6] = {0,0,0,0,0,0};
@@ -84,7 +77,6 @@ static esp_err_t custom_prov_data_handler(uint32_t session_id, const uint8_t *in
 
 void WiFiProvClass :: beginProvision(prov_scheme_t prov_scheme, scheme_handler_t scheme_handler, wifi_prov_security_t security, const char * pop, const char *service_name, const char *service_key, uint8_t * uuid)
 {
-    prov_enable = true;
     bool provisioned = false;
     static char service_name_temp[32];
 
@@ -175,3 +167,4 @@ void WiFiProvClass :: beginProvision(prov_scheme_t prov_scheme, scheme_handler_t
     }
 }
 
+WiFiProvClass WiFiProv;

libraries/WiFiProv/src/WiFiProv.h

@@ -40,16 +40,11 @@ typedef enum {
 
 //Provisioning class 
 class WiFiProvClass
-{
-    protected:
-        static bool prov_enable;
-  
+{  
     public:
-        WiFiProvClass() {
-            prov_enable = false;
-        }
 
-        bool isProvEnabled();
         void beginProvision(prov_scheme_t prov_scheme = WIFI_PROV_SCHEME_SOFTAP, scheme_handler_t scheme_handler = WIFI_PROV_SCHEME_HANDLER_NONE,
         		wifi_prov_security_t security = WIFI_PROV_SECURITY_1, const char * pop = "abcd1234", const char * service_name = NULL, const char * service_key = NULL, uint8_t *uuid = NULL);
 };
+
+extern WiFiProvClass WiFiProv;