[ESP8266 Bridge] [BUGFIX] Add check for WIFI params for ESP8266 bridge

[zdanek]

Description

When using WIFI ESP8266 bridge for the first time, QGC was not checking for existence of WIFI parameters. It resulted in SIGSEGV

FactPanelControllerLog: Missing parameter: "240:WIFI_PASSWORD1"
Signal: SIGSEGV (Segmentation fault)

Test Steps

Create fresh build, with fresh ini config, connect ESP8266 bridge to Ardupilot board.

This fix checks for existing WIFI params.

[DonLakeFlyer]

This seems a bit like fixing the symptom not the problem. How come these parameters are missing? Aren't they required?

[zdanek]

TBH, I don't know. Are those nested inside Ardupilot? Or in QGC? And, can QGC (GS in general) set new params? That is, send from external world and AP will save it? It's possible for LUA scripts to add new params, and they will be seen from now on in AP parameters list.

[DonLakeFlyer]

Those params should come from the device itself. When QGC does a request for parameters from the vehicle it should get these as well.

Somehow it got this:

            //-- Is there an ESP8266 Connected?
            if(_vehicle->parameterManager()->parameterExists(MAV_COMP_ID_UDP_BRIDGE, "SW_VER")) {
                _esp8266Component = new ESP8266Component(_vehicle, this);
                _esp8266Component->setupTriggerSignals();
                _components.append(QVariant::fromValue((VehicleComponent*)_esp8266Component));
            }

But not the other parameters. That is very odd.

Can you help clean this code to at least tell the user that the UI isn't going to work? Right now it's just gonna show nothing in the UI for the wifi ssid/password fields. And they won't work. There should be some sort of 'valid' bool on the controller or something which indicates the whole things is hosed up. And then the ui lets the user at least know.

[DonLakeFlyer]

When you run a build with your fix. Do you still get a parameters missing error when you go to the setup page?

[DonLakeFlyer]

@zdanek Any updates on this?