CRM-18193 logging schema: fix mismatch between comment & code convention
authoreileen <emcnaughton@wikimedia.org>
Mon, 25 Apr 2016 22:36:43 +0000 (10:36 +1200)
committereileen <emcnaughton@wikimedia.org>
Mon, 25 Apr 2016 22:37:18 +0000 (10:37 +1200)
Comment suggests 'c_' to identify a non-CRM connection + timestamp to the hour + connection_id but code used 'con_'

I think the shorter version in comment is better as there is more real information space

CRM/Logging/Schema.php

index 99dcc2d6454606e04e4378f013322c5fcd51f1a0..402a77df526a0adca8689c589355ec696a87270e 100644 (file)
@@ -874,7 +874,7 @@ COLS;
         // We tried setting the @uniqueID in the trigger but it was unreliable.
         // An external interaction could split over 2 connections & it seems worth blocking the revert on
         // these reports & adding extra permissioning to the api for this.
-        $connectionSQLString = "COALESCE(@uniqueID, LEFT(CONCAT('con_', unix_timestamp()/3600, CONNECTION_ID()), 17))";
+        $connectionSQLString = "COALESCE(@uniqueID, LEFT(CONCAT('c_', unix_timestamp()/3600, CONNECTION_ID()), 17))";
       }
       else {
         // The log tables have not yet been converted to have varchar(17) fields for log_conn_id.