matthew/ostinato
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Feature (contd.): Variable Fields - fixed bug where the correct validation/masks were not setup when adding a new VF or switching between protocols

Browse Source
This commit is contained in:
Srivats P. 2015-05-31 21:21:46 +05:30
parent 022fac534c
commit 22d35f8a41
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
client/variablefieldswidget.cpp
View File

@ -160,6 +160,13 @@ void VariableFieldsWidget::on_protocolList_currentItemChanged(
variableFieldList->addItem(vfItem); variableFieldList->addItem(vfItem);
} }
// While switching protocols, we want to setup the attrib group
// validation/ranges/masks for the current protocol, which is done
// by the field/type signal handlers - so clear field/type index
// now so that signals are emitted when we add/select a VF
field->setCurrentIndex(-1);
type->setCurrentIndex(-1);
lastSelectedProtocolIndex_ = protocolList->currentRow(); lastSelectedProtocolIndex_ = protocolList->currentRow();
_exit: _exit:
@ -263,7 +270,7 @@ void VariableFieldsWidget::on_field_currentIndexChanged(int index)
void VariableFieldsWidget::on_type_currentIndexChanged(int index) void VariableFieldsWidget::on_type_currentIndexChanged(int index)
{ {
if (!protocolList->currentItem()) if ((index < 0) || !protocolList->currentItem())
return; return;
AbstractProtocol *proto = protocolList->currentItem()->data(Qt::UserRole) AbstractProtocol *proto = protocolList->currentItem()->data(Qt::UserRole)