Basic report on wiki needs more code (I think)

[krypto]

The wiki at http://wiki.civicrm.org/confluence/display/CRMDOC/CiviReport+Reference gives an example of a very simple report, but as it stands you can't save instances without extra code. I don't want to update the wiki without checking I'm making sense, though. I made:

Code: [Select]

<?php
class CRM_Report_Form_FindBob extends CRM_Report_Form {

function postProcess( ) {
        $this->beginPostProcess( );
 
        // note1: your query
        $myQuery = "SELECT id from civicrm_contact where first_name like '%bob%'";
 
        // note2: register columns you want displayed-
        $this->_columnHeaders = array(
          'id'  => array('type'=>0,'title'=>'ID')
    );
  
        // note3: let report do the fetching of records for you
        $this->buildRows ( $myQuery, $rows );
 
        $this->formatDisplay($rows);
 
        $this->doTemplateAssignment( $rows );
        $this->endPostProcess( $rows );
     
     } // post process ends
 
} // class ends

which works as a report, but you can't save an instance. It tries to save it with a blank name - "" report has been successfully created" - and you get a smarty warning in the php error log: "missing 'var' parameter". The only way I could get it to work was to add a constructor with Contact Name in it:

Code: [Select]

  function __construct() {
    $this->_columns = array(
      'civicrm_contact' =>
      array(
        'dao' => 'CRM_Contact_DAO_Contact',
        'fields' =>
        array(
          'sort_name' =>
          array('title' => ts('Contact Name'),
            'required' => TRUE,
            'no_repeat' => TRUE,
          ),
        ),
      ),
    );
    parent::__construct();
  }

Does that seem like a sensible thing to do? Or is this a bug?

[Eileen]

Yes, that seems like a useful improvement on the wiki documentation.

I think there is an obscure setting you should use if you don't define any fields - I can't quite recall the use case although I'm pretty sure I have used it

  protected $_noFields = TRUE;

[krypto]

Aha - that works. Thank you! Will update the wiki.