Commit 4e035193 authored by Jacob Walser's avatar Jacob Walser

Bugfix joystick input handling with multi-vehicle scenario

parent d7e1b707
...@@ -182,7 +182,8 @@ Vehicle::Vehicle(LinkInterface* link, ...@@ -182,7 +182,8 @@ Vehicle::Vehicle(LinkInterface* link,
{ {
_addLink(link); _addLink(link);
connect(_joystickManager, &JoystickManager::activeJoystickChanged, this, &Vehicle::_activeJoystickChanged); connect(_joystickManager, &JoystickManager::activeJoystickChanged, this, &Vehicle::_loadSettings);
connect(qgcApp()->toolbox()->multiVehicleManager(), &MultiVehicleManager::activeVehicleAvailableChanged, this, &Vehicle::_loadSettings);
_mavlink = _toolbox->mavlinkProtocol(); _mavlink = _toolbox->mavlinkProtocol();
...@@ -230,7 +231,6 @@ Vehicle::Vehicle(LinkInterface* link, ...@@ -230,7 +231,6 @@ Vehicle::Vehicle(LinkInterface* link,
connect(_mav, SIGNAL(attitudeChanged (UASInterface*,double,double,double,quint64)), this, SLOT(_updateAttitude(UASInterface*, double, double, double, quint64))); connect(_mav, SIGNAL(attitudeChanged (UASInterface*,double,double,double,quint64)), this, SLOT(_updateAttitude(UASInterface*, double, double, double, quint64)));
connect(_mav, SIGNAL(attitudeChanged (UASInterface*,int,double,double,double,quint64)), this, SLOT(_updateAttitude(UASInterface*,int,double, double, double, quint64))); connect(_mav, SIGNAL(attitudeChanged (UASInterface*,int,double,double,double,quint64)), this, SLOT(_updateAttitude(UASInterface*,int,double, double, double, quint64)));
_loadSettings();
// Ask the vehicle for protocol version info. // Ask the vehicle for protocol version info.
sendMavCommand(MAV_COMP_ID_ALL, // Don't know default component id yet. sendMavCommand(MAV_COMP_ID_ALL, // Don't know default component id yet.
...@@ -1611,6 +1611,10 @@ int Vehicle::manualControlReservedButtonCount(void) ...@@ -1611,6 +1611,10 @@ int Vehicle::manualControlReservedButtonCount(void)
void Vehicle::_loadSettings(void) void Vehicle::_loadSettings(void)
{ {
if (!_active) {
return;
}
QSettings settings; QSettings settings;
settings.beginGroup(QString(_settingsGroup).arg(_id)); settings.beginGroup(QString(_settingsGroup).arg(_id));
...@@ -1669,12 +1673,6 @@ QStringList Vehicle::joystickModes(void) ...@@ -1669,12 +1673,6 @@ QStringList Vehicle::joystickModes(void)
return list; return list;
} }
void Vehicle::_activeJoystickChanged(void)
{
_loadSettings();
_startJoystick(true);
}
bool Vehicle::joystickEnabled(void) bool Vehicle::joystickEnabled(void)
{ {
return _joystickEnabled; return _joystickEnabled;
...@@ -1713,8 +1711,6 @@ void Vehicle::setActive(bool active) ...@@ -1713,8 +1711,6 @@ void Vehicle::setActive(bool active)
_active = active; _active = active;
emit activeChanged(_active); emit activeChanged(_active);
} }
_startJoystick(_active);
} }
QGeoCoordinate Vehicle::homePosition(void) QGeoCoordinate Vehicle::homePosition(void)
......
...@@ -870,7 +870,6 @@ private slots: ...@@ -870,7 +870,6 @@ private slots:
void _geoFenceLoadComplete(void); void _geoFenceLoadComplete(void);
void _rallyPointLoadComplete(void); void _rallyPointLoadComplete(void);
void _sendMavCommandAgain(void); void _sendMavCommandAgain(void);
void _activeJoystickChanged(void);
void _clearTrajectoryPoints(void); void _clearTrajectoryPoints(void);
void _clearCameraTriggerPoints(void); void _clearCameraTriggerPoints(void);
void _updateDistanceToHome(void); void _updateDistanceToHome(void);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment