[netinstall] Simplify getItemPackages
- Use convenience predicate isPackage() - Name child->item(i) for brevity
This commit is contained in:
parent
464561b420
commit
14a3e10cc2
@ -203,18 +203,19 @@ PackageModel::getItemPackages( PackageTreeItem* item ) const
|
||||
PackageTreeItem::List selectedPackages;
|
||||
for ( int i = 0; i < item->childCount(); i++ )
|
||||
{
|
||||
if ( item->child( i )->isSelected() == Qt::Unchecked )
|
||||
auto* child = item->child( i );
|
||||
if ( child->isSelected() == Qt::Unchecked )
|
||||
{
|
||||
continue;
|
||||
}
|
||||
|
||||
if ( !item->child( i )->childCount() ) // package
|
||||
if ( child->isPackage() ) // package
|
||||
{
|
||||
selectedPackages.append( item->child( i ) );
|
||||
selectedPackages.append( child );
|
||||
}
|
||||
else
|
||||
{
|
||||
selectedPackages.append( getItemPackages( item->child( i ) ) );
|
||||
selectedPackages.append( getItemPackages( child ) );
|
||||
}
|
||||
}
|
||||
return selectedPackages;
|
||||
|
Loading…
Reference in New Issue
Block a user