• fixed NSString pointer reassignment in SPArrayAddition's method 'componentsJoinedAndBacktickQuoted'

• improved deletion of rows in tables which have no primary keys
- check for duplicated rows first; if no duplicates are found use all columns as 'primary keys' and delete the rows via … WHERE (a='…' AND b='…') OR (a='…' AND b='…') OR … 

1 files changed, 25 insertions, 1 deletions

diff --git a/Source/TableContent.m b/Source/TableContent.m
index 19f77db4..1c5551c6 100644
--- a/Source/TableContent.m
+++ b/Source/TableContent.m
@@ -1787,7 +1787,31 @@
 
 			NSArray *primaryKeyFieldNames = [tableDataInstance primaryKeyColumnNames];
 
-			// TODO tentative → || [primaryKeyFields count] != 1
+			// If no PRIMARY KEY is found and numberOfSelectedRows > 3 then
+			// check for uniqueness of rows via combining all column values;
+			// if unique then use the all columns as 'primary keys'
+			if([selectedRows count] >3 && primaryKeyFieldNames == nil) {
+				primaryKeyFieldNames = [tableDataInstance columnNames];
+
+				NSInteger numberOfRows = 0;
+				// Get the number of rows in the table
+				MCPResult *r;
+				r = [mySQLConnection queryString:[NSString stringWithFormat:@"SELECT COUNT(*) FROM %@", [selectedTable backtickQuotedString]]];
+				if ([[mySQLConnection getLastErrorMessage] isEqualToString:@""]) {
+					NSArray *a = [r fetchRowAsArray];
+					if([a count])
+						numberOfRows = [[a objectAtIndex:0] integerValue];
+				}
+				// Check for uniqueness via LIMIT numberOfRows-1,numberOfRows for speed
+				if(numberOfRows > 0) {
+					[mySQLConnection queryString:[NSString stringWithFormat:@"SELECT * FROM %@ GROUP BY %@ LIMIT %d,%d", [selectedTable backtickQuotedString], [primaryKeyFieldNames componentsJoinedAndBacktickQuoted], numberOfRows-1, numberOfRows]];
+					if([mySQLConnection affectedRows] == 0)
+						primaryKeyFieldNames = nil;
+				} else {
+					primaryKeyFieldNames = nil;
+				}
+			}
+
 			if(primaryKeyFieldNames == nil) {
 				// delete row by row
 				while (index != NSNotFound) {