mirror/sqlitebrowser
1
0
Fork 0
mirror of https://github.com/sqlitebrowser/sqlitebrowser.git synced 2026-01-20 11:00:44 -06:00
Code Issues Packages Projects Releases Wiki Activity

Improve wording in comment

Browse Source
This commit is contained in:
Martin Kleusberg
2017-04-10 14:17:01 +02:00
parent ec916226b0
commit f11c0181ca
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/EditTableDialog.cpp
View File

@@ -257,12 +257,12 @@ 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. There is one exception, however,
// to this rule: if the field that is about to be renamed the one we're finding here, this can only mean that the user is trying to
// rename the field to essentially the same name but with different case. Example: if I rename column 'COLUMN' to 'column', findField()
// is going to return the current field number because it's doing a case-independent search and it can't return another field number
// because SQLite prohibits duplicate field names (no matter the case). So when this happens we just allow the renaming because there's
// no harm to be expected from it.
// When a field of that name already exists, show a warning to the user and don't apply the new name. This is done by searching for an
// existing field with the new name. There is one exception, however, to this rule: if the field that we have found is the same field
// as the one we are currently trying to rename, this means the user is trying to rename the field to essentially the same name but
// with different case. Example: if I rename column 'COLUMN' to 'column', findField() is going to return the current field number
// because it's doing a case-independent search and it can't return another field number because SQLite prohibits duplicate field
// names (no matter the case). So when this happens we just allow the renaming because there's no harm to be expected from it.
int foundField = m_table.findField(item->text(column));
if(foundField != -1 && foundField != index)
{