From d1e08fffb5b9de4690c3c875ad83a7bc73905bf6 Mon Sep 17 00:00:00 2001 From: eileen Date: Thu, 30 May 2019 10:27:38 +1200 Subject: [PATCH] dev/core#994 remove Thousands Separator can not have more than 1 character rule Per issue it seems unnecessarily opinionated to block longer strings & we not know of at least one case where it is inappropriate --- CRM/Admin/Form/Setting/Localization.php | 4 ---- 1 file changed, 4 deletions(-) diff --git a/CRM/Admin/Form/Setting/Localization.php b/CRM/Admin/Form/Setting/Localization.php index 2940f4136f..e87f459575 100644 --- a/CRM/Admin/Form/Setting/Localization.php +++ b/CRM/Admin/Form/Setting/Localization.php @@ -137,10 +137,6 @@ class CRM_Admin_Form_Setting_Localization extends CRM_Admin_Form_Setting { $errors['monetaryThousandSeparator'] = ts('Thousands Separator can not be empty. You can use a space character instead.'); } - if (strlen($fields['monetaryThousandSeparator']) > 1) { - $errors['monetaryThousandSeparator'] = ts('Thousands Separator can not have more than 1 character.'); - } - if (strlen($fields['monetaryDecimalPoint']) > 1) { $errors['monetaryDecimalPoint'] = ts('Decimal Delimiter can not have more than 1 character.'); } -- 2.25.1