Placeholder is only used for select elements.
authorTim Mallezie <tim.mallezie@chiro.be>
Mon, 23 Mar 2015 07:00:26 +0000 (08:00 +0100)
committerTim Mallezie <tim.mallezie@chiro.be>
Mon, 23 Mar 2015 07:00:26 +0000 (08:00 +0100)
CRM/Core/Form.php

index 1b0f8b301011ce2c27f224e18a70967c1c5185e7..ecb87b2ad3b3d8e5e697dde66899c9065ce4bef6 100644 (file)
@@ -1152,10 +1152,6 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
 
     $label = isset($props['label']) ? $props['label'] : $fieldSpec['title'];
 
-    if (!array_key_exists('placeholder', $props)) {
-      $props['placeholder'] = $required ? ts('- select -') : CRM_Utils_Array::value('context', $props) == 'search' ? ts('- any -') : ts('- none -');
-    }
-
     $widget = isset($props['type']) ? $props['type'] : $fieldSpec['html']['type'];
 
     if ($widget == 'TextArea' && CRM_Utils_Array::value('context', $props) == 'search') {
@@ -1175,7 +1171,7 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
     )));
 
     if ($isSelect) {
-      // Fetch options from the api unless passed explicitly
+      // Fetch options from the api unless passed explicitly.
       if (isset($props['options'])) {
         $options = $props['options'];
       }
@@ -1183,6 +1179,12 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
         $info = civicrm_api3($props['entity'], 'getoptions', $props);
         $options = $info['values'];
       }
+
+      // The placeholder is only used for select-elements.
+      if (!array_key_exists('placeholder', $props)) {
+        $props['placeholder'] = $required ? ts('- select -') : CRM_Utils_Array::value('context', $props) == 'search' ? ts('- any -') : ts('- none -');
+      }
+
       if (CRM_Utils_Array::value('context', $props) == 'search' || ($widget !== 'AdvMulti-Select' && strpos($widget, 'Select') !== FALSE)) {
         $widget = 'Select';
       }