code style and declare entity in parent class definition.
authorTim Mallezie <tim.mallezie@chiro.be>
Wed, 11 Mar 2015 09:26:50 +0000 (10:26 +0100)
committerTim Mallezie <tim.mallezie@chiro.be>
Wed, 11 Mar 2015 09:26:50 +0000 (10:26 +0100)
CRM/Core/Form.php
CRM/Event/Form/ManageEvent.php
CRM/Event/Form/ManageEvent/EventInfo.php

index 5add53d7b4e79d203e6490c9eb633b7fb87659f9..15810e6c0f9887caa18f06693caa4e22ac4ecdcc 100644 (file)
@@ -125,10 +125,10 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
    * @var CRM_Core_Controller
    */
   public $controller;
-  
+
   /**
    * Api entity name
-   * 
+   *
    * @var string
    */
   public $entityName;
@@ -1058,7 +1058,7 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
    */
   public function addSelect($name, $props = array(), $required = FALSE) {
     if (!isset($props['entity'])) {
-      if(isset($this->entityName)){
+      if (isset($this->entityName)) {
         $props['entity'] = $this->entityName;
       }
       else {
@@ -1859,7 +1859,7 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
             if (!$options) {
               $targetField->setAttribute('placeholder', $targetField->getAttribute('data-none-prompt'));
             }
-          } 
+          }
           else {
             $targetField->setAttribute('placeholder', $targetField->getAttribute('data-empty-prompt'));
             $targetField->setAttribute('disabled', 'disabled');
index ae92d3a3633835bd47065b1f01cbd9f35599a0eb..6e4f11ff6a43ad116aa6c861e8023e96017c1ecf 100644 (file)
@@ -86,6 +86,11 @@ class CRM_Event_Form_ManageEvent extends CRM_Core_Form {
    * Check if repeating event.
    */
   public $_isRepeatingEvent;
+  
+  /**
+   * Explicitly declare the entity api name.
+   */
+  public $entityName = 'Event';
 
   /**
    * Set variables up before form is built.
index bdb10a2cb4d6d0b0e8403fdfb4938210c26addce..89d27e45048b9c9a1fc7d46e5f3813d5d2eb10e7 100644 (file)
@@ -187,8 +187,6 @@ class CRM_Event_Form_ManageEvent_EventInfo extends CRM_Event_Form_ManageEvent {
     // add event title, make required if this is not a template
     $this->add('text', 'title', ts('Event Title'), $attributes['event_title'], !$this->_isTemplate);
 
-    // We must specify the entity api name when using an addSelect
-    $this->entityName = 'Event';
     $this->addSelect('event_type_id',
       array('onChange' => "CRM.buildCustomData( 'Event', this.value );"),
       TRUE