Better errors for core form actions
authorColeman Watts <coleman@civicrm.org>
Wed, 30 Dec 2015 18:54:58 +0000 (13:54 -0500)
committerColeman Watts <coleman@civicrm.org>
Wed, 30 Dec 2015 18:54:58 +0000 (13:54 -0500)
CRM/Core/Action.php
CRM/Core/Form.php

index ab1979db14099279156c55ec963a56245b3a0f46..6a82e0e81eaf5c08a8718afa718e76b205885a5b 100644 (file)
@@ -171,7 +171,7 @@ class CRM_Core_Action {
    *   the corresponding action description
    */
   public static function description($mask) {
-    if (!isset($_description)) {
+    if (!isset(self::$_description)) {
       self::$_description = array_flip(self::$_names);
     }
 
index 90e16dde383b4f00adf671f2cf46d03824fc9af1..c53091d56b224ccd472ad87db00abfdf7e8fe305 100644 (file)
@@ -1225,7 +1225,7 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
       return 'get';
     }
     // If you get this exception try adding more cases above.
-    throw new Exception("Cannot determine api action for " . get_class($this));
+    throw new Exception("Cannot determine api action for " . get_class($this) . '.' . 'CRM_Core_Action "' . CRM_Core_Action::description($action) . '" not recognized.');
   }
 
   /**
@@ -1233,7 +1233,7 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
    * @throws Exception
    */
   public function getDefaultEntity() {
-    throw new Exception("Cannot determine default entity. The form class should implement getDefaultEntity().");
+    throw new Exception("Cannot determine default entity. " . get_class($this) . " should implement getDefaultEntity().");
   }
 
   /**
@@ -1243,7 +1243,7 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
    * @throws Exception
    */
   public function getDefaultContext() {
-    throw new Exception("Cannot determine default context. The form class should implement getDefaultContext().");
+    throw new Exception("Cannot determine default context. " . get_class($this) . " should implement getDefaultContext().");
   }
 
   /**