Removed unnecessary parts of the Mysql regexes.
authorMattias Michaux <mattias.michaux@gmail.com>
Wed, 27 Apr 2016 12:26:50 +0000 (14:26 +0200)
committerMattias Michaux <mattias.michaux@gmail.com>
Fri, 29 Apr 2016 05:42:04 +0000 (07:42 +0200)
CRM/Utils/Rule.php
CRM/Utils/Type.php

index 73b978967bcb94416ca53dbca2333231bc5a4f73..eeb97a291404a33eeb29811ab94a96467e63a52d 100644 (file)
@@ -120,7 +120,7 @@ class CRM_Utils_Rule {
     //
     // MySQL permits column names that don't match this (eg containing spaces),
     // but CiviCRM won't create those ...
-    if (!preg_match('/^[\w_]+(\.[\w_]+)?$/i', $str)) {
+    if (!preg_match('/^[\w]+(\.[\w]+)?$/i', $str)) {
       return FALSE;
     }
 
@@ -154,7 +154,7 @@ class CRM_Utils_Rule {
     // at all, so we split and loop over.
     $parts = explode(',', $str);
     foreach ($parts as $part) {
-      if (!preg_match('/^(([\w_]+)((\.)([\w_]+))?( (asc|desc))?)$/i', trim($part))) {
+      if (!preg_match('/^(([\w]+)((\.)([\w]+))?( (asc|desc))?)$/i', trim($part))) {
         return FALSE;
       }
     }
index 7f9ddafddba463021860808c2f37e851c00bb6eb..8bbdb8cb6166ed02295771ba9b04255d716d1689 100644 (file)
@@ -296,7 +296,7 @@ class CRM_Utils_Type {
         if (CRM_Utils_Rule::mysqlOrderBy($data)) {
           $parts = explode(',', $data);
           foreach ($parts as &$part) {
-            $part = preg_replace_callback('/(?:([\w_]+)(?:(?:\.)([\w_]+))?(?: (asc|desc))?)/i', array('CRM_Utils_Type', 'mysqlOrderByCallback'), trim($part));
+            $part = preg_replace_callback('/(?:([\w]+)(?:(?:\.)([\w]+))?(?: (asc|desc))?)/i', array('CRM_Utils_Type', 'mysqlOrderByCallback'), trim($part));
           }
           return implode(', ', $parts);
         }