piotr/edgetx-gx12
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

check unique RX# based on protocol (#6781)

Browse Source
This commit is contained in:
Raphael Coeffic
2019-09-16 08:01:00 +02:00
committed by Bertrand Songis
parent 059ceee7e3
commit 7941c8b2b5
2 changed files with 13 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
companion/src/modelslist.cpp
+12 -11
View File
@@ -722,17 +722,17 @@ void TreeModel::refresh()
}
int protocol;
QString rxs;
for (unsigned j=0; j<CPN_MAX_MODULES; j++) {
protocol = model.moduleData[j].protocol;
for (auto const& moduleData: model.moduleData) {
protocol = moduleData.protocol;
// These are the only RXs that allow nominating RX # but changing RX or copying models can leave residual configuration which can cause issues
// if (protocol == PULSES_PXX_XJT_X16 || protocol == PULSES_PXX_XJT_LR12 || protocol == PULSES_PXX_R9M || protocol == PULSES_DSMX || protocol == PULSES_MULTIMODULE) {
if (protocol != PULSES_OFF && model.moduleData[j].modelId > 0) {
if (protocol != PULSES_OFF && moduleData.modelId > 0) {
if (!rxs.isEmpty()) {
rxs.append(", ");
}
unsigned mdlidx = model.moduleData[j].modelId;
unsigned mdlidx = moduleData.modelId;
rxs.append(QString("%1").arg(uint(mdlidx), 2, 10, QChar('0')));
if (!isModelIdUnique(mdlidx, j)) {
if (!isModelIdUnique(mdlidx, protocol)) {
current->setHighlightRX(true);
}
}
@@ -746,15 +746,16 @@ void TreeModel::refresh()
}
}
bool TreeModel::isModelIdUnique(unsigned modelIdx, unsigned module)
bool TreeModel::isModelIdUnique(unsigned modelIdx, unsigned protocol)
{
int cnt = 0;
for (unsigned i=0; i<radioData->models.size(); i++) {
ModelData & model = radioData->models[i];
for (auto const& model: radioData->models) {
if (!model.isEmpty()) {
if (model.moduleData[module].protocol != PULSES_OFF && model.moduleData[module].modelId == modelIdx) {
if (++cnt > 1) {
return false;
for (auto const& moduleData: model.moduleData) {
if (moduleData.protocol == protocol && moduleData.modelId == modelIdx) {
if (++cnt > 1) {
return false;
}
}
}
}
companion/src/modelslist.h
+1 -1
View File
@@ -149,7 +149,7 @@ class TreeModel : public QAbstractItemModel
private:
TreeItem * getItem(const QModelIndex & index) const;
bool isModelIdUnique(unsigned modelId, unsigned module);
bool isModelIdUnique(unsigned modelId, unsigned protocol);
TreeItem * rootItem;
RadioData * radioData;