From: Kirk Jackson <kirk@caat.org.uk>
Date: Tue, 2 Apr 2013 16:41:51 +0000 (+0100)
Subject: CRM-12133: Changed OptionValue's auto-increment function to use numeric comparison... 
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=0e9de3f57c7fec6726e4e914357805cbb4449ee8;p=civicrm-core.git

CRM-12133: Changed OptionValue's auto-increment function to use numeric comparison instead of string comparison.
---

diff --git a/CRM/Core/BAO/OptionValue.php b/CRM/Core/BAO/OptionValue.php
index edc771bf33..1aba33a618 100644
--- a/CRM/Core/BAO/OptionValue.php
+++ b/CRM/Core/BAO/OptionValue.php
@@ -111,7 +111,7 @@ class CRM_Core_BAO_OptionValue extends CRM_Core_DAO_OptionValue {
      }
      $bao->selectAdd();
      $bao->whereAdd("value REGEXP '^[0-9]+$'");
-     $bao->selectAdd('(ROUND(COALESCE(MAX(value),0)) +1) as nextvalue');
+     $bao->selectAdd('(ROUND(COALESCE(MAX(CONVERT(value, UNSIGNED)),0)) +1) as nextvalue');
      $bao->find(TRUE);
      return $bao->nextvalue;
   }