New warning check. Piece name is not unique.

(cherry picked from commit d5fd1380dc)
This commit is contained in:
Roman Telezhynskyi 2020-10-05 16:49:42 +03:00
parent 1483f7a0cc
commit 4a1d20810c

View File

@ -156,6 +156,30 @@ QVector<DetailForLayout> SortDetailsForLayout(const QHash<quint32, VPiece> *allD
return details; return details;
} }
//---------------------------------------------------------------------------------------------------------------------
void WarningNotUniquePieceName(const QHash<quint32, VPiece> *allDetails)
{
QHash<quint32, VPiece>::const_iterator i = allDetails->constBegin();
QSet<QString> uniqueNames;
while (i != allDetails->constEnd())
{
const QString pieceName = i.value().GetName();
if (not uniqueNames.contains(pieceName))
{
uniqueNames.insert(pieceName);
}
else
{
const QString errorMsg = QObject::tr("Piece name '%1' is not unique.").arg(pieceName);
qApp->IsPedantic() ? throw VException(errorMsg) :
qWarning() << VAbstractApplication::patternMessageSignature + errorMsg;
}
++i;
}
}
} // anonymous namespace } // anonymous namespace
//--------------------------------------------------------------------------------------------------------------------- //---------------------------------------------------------------------------------------------------------------------
@ -2780,6 +2804,8 @@ void MainWindow::ActionDetails(bool checked)
ui->dockWidgetToolOptions->setVisible(isDockToolOptionsVisible); ui->dockWidgetToolOptions->setVisible(isDockToolOptionsVisible);
m_statusLabel->setText(QString()); m_statusLabel->setText(QString());
WarningNotUniquePieceName(pattern->DataPieces());
} }
else else
{ {
@ -2826,6 +2852,7 @@ void MainWindow::ActionLayout(bool checked)
} }
else else
{ {
WarningNotUniquePieceName(allDetails);
details = SortDetailsForLayout(allDetails); details = SortDetailsForLayout(allDetails);
if (details.count() == 0) if (details.count() == 0)