mirror of
https://github.com/sqlitebrowser/sqlitebrowser.git
synced 2026-01-20 11:00:44 -06:00
Remove useless function in Edit Table dialog
This commit is contained in:
@@ -257,7 +257,7 @@ void EditTableDialog::itemChanged(QTreeWidgetItem *item, int column)
|
||||
{
|
||||
case kName:
|
||||
// When a field of that name already exists, show a warning to the user and don't apply the new name
|
||||
if(fieldNameExists(item->text(column)))
|
||||
if(m_table.findField(item->text(column)))
|
||||
{
|
||||
QMessageBox::warning(this, qApp->applicationName(), tr("There already is a field with that name. Please rename it first or choose a different "
|
||||
"name for this field."));
|
||||
@@ -504,7 +504,7 @@ void EditTableDialog::addField()
|
||||
{
|
||||
field_name = "Field" + QString::number(field_number);
|
||||
field_number++;
|
||||
} while(fieldNameExists(field_name));
|
||||
} while(m_table.findField(field_name));
|
||||
tbitem->setText(kName, field_name);
|
||||
}
|
||||
|
||||
@@ -729,14 +729,3 @@ void EditTableDialog::setTemporary(bool is_temp)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
bool EditTableDialog::fieldNameExists(const QString& name)
|
||||
{
|
||||
foreach(const sqlb::FieldPtr& ptr, m_table.fields())
|
||||
{
|
||||
if(ptr->name() == name)
|
||||
return true;
|
||||
}
|
||||
|
||||
return false;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user