[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    branches/koffice/1.6/koffice/kexi/kexidb
From:       Jaroslaw Staniek <js () iidea ! pl>
Date:       2007-01-23 13:03:59
Message-ID: 1169557439.649595.29498.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 626475 by staniek:

Table Designer
- fixed crash on saving design with lookup columns defined

2.0: merged



 M  +5 -3      alter.cpp  


--- branches/koffice/1.6/koffice/kexi/kexidb/alter.cpp #626474:626475
@@ -614,9 +614,10 @@
 		}
 		if (!values.isEmpty()) {
 			//update field, so it will be created as one step
-			KexiDB::Field f = field();
-			if (KexiDB::setFieldProperties( f, values )) {
-				field() = f;
+			KexiDB::Field *f = new KexiDB::Field( field() );
+			if (KexiDB::setFieldProperties( *f, values )) {
+				//field() = f;
+				setField( f );
 				field().debug();
 #ifdef KEXI_DEBUG_GUI
 				KexiUtils::addAlterTableActionDebug(
@@ -629,6 +630,7 @@
 					QString("** Failed to set properties for field ")+field().debugString(), 0);
 #endif
 				KexiDBWarn << "AlterTableHandler::InsertFieldAction::simplifyActions(): \
KexiDB::setFieldProperties() failed!" << endl; +				delete f;
 			}
 		}
 	}


[prev in list] [next in list] [prev in thread] [next in thread]