From: Pradeep Nayak Date: Mon, 8 Feb 2016 14:59:29 +0000 (+0530) Subject: --CRM-16617, moved function to DAO X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=a00fe575163337d1aaf5ca6ec697fccf2ec31d44;p=civicrm-core.git --CRM-16617, moved function to DAO Conflicts: CRM/Core/DAO.php --- diff --git a/CRM/Core/DAO.php b/CRM/Core/DAO.php index 189b6c8d59..f2f4ba5f23 100644 --- a/CRM/Core/DAO.php +++ b/CRM/Core/DAO.php @@ -2559,4 +2559,31 @@ SELECT contact_id return $clauses; } + /** + * function to check valid db name + * + * @param $database + * @param $testDetails + * + * @return bool + */ + public static function requireValidDBName($database, $testDetails, $object = NULL) { + $matches = array(); + preg_match( + "/^[0-9]*[a-zA-Z_]+[a-zA-Z0-9_]*$/", + $database, + $matches + ); + if (empty($matches)) { + if ($object) { + $object->error($testDetails); + } + return FALSE; + } + if ($object) { + $object->testing($testDetails); + } + return TRUE; + } + } diff --git a/install/index.php b/install/index.php index 292879c52c..cd4091cba2 100644 --- a/install/index.php +++ b/install/index.php @@ -394,8 +394,9 @@ class InstallRequirements { ts("An auto_increment_increment value greater than 1 is not currently supported. Please see issue CRM-7923 for further details and potential workaround."), ) ); - if (!$this->requireValidDBName( + if (!CRM_Core_DAO::requireValidDBName( $databaseConfig['database'], + $this, array( ts("MySQL %1 Configuration", array(1 => $dbName)), ts("Is the provided database name valid?"), @@ -1283,27 +1284,6 @@ class InstallRequirements { return count($this->warnings); } - /** - * function to check valid db name - * - * @param $database - * @param $testDetails - */ - public function requireValidDBName($database, $testDetails) { - $matches = array(); - preg_match( - "/^[0-9]*[a-zA-Z_]+[a-zA-Z0-9_]*$/", - $database, - $matches - ); - if (empty($matches)) { - $this->error($testDetails); - return FALSE; - } - $this->testing($testDetails); - return TRUE; - } - } /**