Merge pull request #946 from dlobo/CRM-12750

CRM-12755

CRM/Admin/Form/Setting/Component.php

@@ -76,7 +76,7 @@ class CRM_Admin_Form_Setting_Component extends CRM_Admin_Form_Setting {
    * @access public
    * @static
    */
-  static function formRule($fields) {
+  static function formRule($fields, $files, $options) {
     $errors = array();
 
     if (is_array($fields['enableComponents'])) {

CRM/Admin/Form/Setting/Miscellaneous.php

@@ -99,9 +99,38 @@ class CRM_Admin_Form_Setting_Miscellaneous extends CRM_Admin_Form_Setting {
 
     $this->addRule('checksumTimeout', ts('Value should be a positive number'), 'positiveInteger');
 
+    $this->addFormRule(array('CRM_Admin_Form_Setting_Miscellaneous', 'formRule'), $this);
+
     parent::buildQuickForm();
   }
 
+  /**
+   * global form rule
+   *
+   * @param array $fields  the input form values
+   * @param array $files   the uploaded files if any
+   * @param array $options additional user data
+   *
+   * @return true if no errors, else array of errors
+   * @access public
+   * @static
+   */
+  static function formRule($fields, $files, $options) {
+    $errors = array();
+
+    if (!empty($fields['wkhtmltopdfPath'])) {
+      // check and ensure that thi leads to the wkhtmltopdf binary
+      // and it is a valid executable binary
+      if (
+        !file_exists($fields['wkhtmltopdfPath']) ||
+        !is_executable($fields['wkhtmltopdfPath'])
+      ) {
+        $errors['wkhtmltopdfPath'] = ts('The wkhtmltodfPath does not exist or is not valid');
+      }
+    }
+    return $errors;
+  }
+
   function setDefaultValues() {
     parent::setDefaultValues();