napcok/calamares
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[libcalamares] Reduce indentation-depth in apply() through early-return

Browse Source
This commit is contained in:
Adriaan de Groot 2021-03-15 21:11:01 +01:00
parent 33fec86ef6
commit a4c1f07521
1 changed files with 25 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
src/libcalamares/modulesystem/Config.cpp
View File

@ -94,26 +94,34 @@ Config::ApplyPresets::apply( const char* fieldName )
if ( !prop.isValid() ) if ( !prop.isValid() )
{ {
cWarning() << "Applying invalid property" << fieldName; cWarning() << "Applying invalid property" << fieldName;
return *this;
} }
else
{
const QString key( fieldName );
if ( !key.isEmpty() && m_c.d->m_presets->find( key ).isValid() )
{
cWarning() << "Applying duplicate property" << fieldName;
}
else if ( !key.isEmpty() && m_map.contains( key ) )
{
QVariantMap m = CalamaresUtils::getSubMap( m_map, key, m_bogus );
QVariant value = m[ "value" ];
bool editable = CalamaresUtils::getBool( m, "editable", true );
if ( value.isValid() ) const QString key( fieldName );
{ if ( key.isEmpty() )
m_c.setProperty( fieldName, value ); {
} cWarning() << "Applying empty field";
m_c.d->m_presets->append( PresetField { key, value, editable } ); return *this;
}
if ( m_c.d->m_presets->find( key ).isValid() )
{
cWarning() << "Applying duplicate property" << fieldName;
return *this;
}
if ( m_map.contains( key ) )
{
// Key has an explicit setting
QVariantMap m = CalamaresUtils::getSubMap( m_map, key, m_bogus );
QVariant value = m[ "value" ];
bool editable = CalamaresUtils::getBool( m, "editable", true );
if ( value.isValid() )
{
m_c.setProperty( fieldName, value );
} }
m_c.d->m_presets->append( PresetField { key, value, editable } );
} }
return *this; return *this;
} }