diff --git a/glabels/TemplatePicker.cpp b/glabels/TemplatePicker.cpp index d7e2a9d..33bb14d 100644 --- a/glabels/TemplatePicker.cpp +++ b/glabels/TemplatePicker.cpp @@ -63,38 +63,39 @@ namespace glabels { foreach ( QListWidgetItem *item, findItems( "*", Qt::MatchWildcard ) ) { - auto *tItem = dynamic_cast(item); - - bool nameMask = tItem->tmplate()->name().contains( searchString, Qt::CaseInsensitive ); + if (auto *tItem = dynamic_cast(item)) + { + bool nameMask = tItem->tmplate()->name().contains( searchString, Qt::CaseInsensitive ); - bool sizeMask = - (isoMask && tItem->tmplate()->isSizeIso()) || - (usMask && tItem->tmplate()->isSizeUs()) || - (otherMask && tItem->tmplate()->isSizeOther()); + bool sizeMask = + (isoMask && tItem->tmplate()->isSizeIso()) || + (usMask && tItem->tmplate()->isSizeUs()) || + (otherMask && tItem->tmplate()->isSizeOther()); - bool categoryMask; - if ( anyCategory ) - { - categoryMask = true; - } - else - { - categoryMask = false; - foreach ( QString id, categoryIds ) + bool categoryMask; + if ( anyCategory ) { - categoryMask = categoryMask || tItem->tmplate()->hasCategory( id ); + categoryMask = true; + } + else + { + categoryMask = false; + foreach ( QString id, categoryIds ) + { + categoryMask = categoryMask || tItem->tmplate()->hasCategory( id ); + } } - } - if ( nameMask && sizeMask && categoryMask ) - { - item->setHidden( false ); - } - else - { - item->setHidden( true ); - item->setSelected( false ); + if ( nameMask && sizeMask && categoryMask ) + { + item->setHidden( false ); + } + else + { + item->setHidden( true ); + item->setSelected( false ); + } } } } @@ -107,26 +108,27 @@ namespace glabels { foreach ( QListWidgetItem *item, findItems( "*", Qt::MatchWildcard ) ) { - auto *tItem = dynamic_cast(item); - - bool match = false; - foreach ( QString name, names ) + if (auto *tItem = dynamic_cast(item)) { - if ( tItem->tmplate()->name() == name ) + bool match = false; + foreach ( QString name, names ) { - match = true; - break; + if ( tItem->tmplate()->name() == name ) + { + match = true; + break; + } } - } - if ( match ) - { - item->setHidden( false ); - } - else - { - item->setHidden( true ); - item->setSelected( false ); + if ( match ) + { + item->setHidden( false ); + } + else + { + item->setHidden( true ); + item->setSelected( false ); + } } } } @@ -138,15 +140,15 @@ namespace glabels const model::Template *TemplatePicker::selectedTemplate() { QList items = selectedItems(); - if ( items.isEmpty() ) + if ( !items.isEmpty() ) { - return nullptr; - } - else - { - auto *tItem = dynamic_cast(items.first()); - return tItem->tmplate(); + if (auto *tItem = dynamic_cast(items.first())) + { + return tItem->tmplate(); + } } + + return nullptr; } } // namespace glabels