diff --git a/src/CopyInputDialog.cpp b/src/CopyInputDialog.cpp --- a/src/CopyInputDialog.cpp +++ b/src/CopyInputDialog.cpp @@ -138,11 +138,7 @@ { QAbstractItemModel *model = _ui->sessionList->model(); QModelIndex index = model->index(row, _model->checkColumn()); - if (checked) { - model->setData(index, static_cast(Qt::Checked), Qt::CheckStateRole); - } else { - model->setData(index, static_cast(Qt::Unchecked), Qt::CheckStateRole); - } + model->setData(index, static_cast( checked ? Qt::Checked : Qt::Unchecked), Qt::CheckStateRole); } CheckableSessionModel::CheckableSessionModel(QObject *parent) : @@ -166,24 +162,19 @@ if (_fixedSessions.contains(session)) { return SessionListModel::flags(index) & ~Qt::ItemIsEnabled; - } else { - return SessionListModel::flags(index) | Qt::ItemIsUserCheckable; } + return SessionListModel::flags(index) | Qt::ItemIsUserCheckable; } QVariant CheckableSessionModel::data(const QModelIndex &index, int role) const { if (role == Qt::CheckStateRole && index.column() == _checkColumn) { auto *session = static_cast(index.internalPointer()); - - if (_checkedSessions.contains(session)) { - return QVariant::fromValue(static_cast(Qt::Checked)); - } else { - return QVariant::fromValue(static_cast(Qt::Unchecked)); - } - } else { - return SessionListModel::data(index, role); + return QVariant::fromValue(static_cast( + _checkedSessions.contains(session) ? Qt::Checked : Qt::Unchecked) + ); } + return SessionListModel::data(index, role); } bool CheckableSessionModel::setData(const QModelIndex &index, const QVariant &value, int role) @@ -203,9 +194,8 @@ emit dataChanged(index, index); return true; - } else { - return SessionListModel::setData(index, value, role); } + return SessionListModel::setData(index, value, role); } void CheckableSessionModel::setCheckedSessions(const QSet &sessions)