CiviCRM Community Forums (archive)

Using Drupal 6.6 and CiviCRM 2.1.1, here is my code:

function civiaddresslog_civicrm_pre( $op, $objectName, $objectId, &$objectRef ) {
    print_r ($op);
    echo "\n\n";
    print_r ($objectName);
    echo "\n\n";
    print_r ($objectId);
    echo "\n\n";
    print_r ($objectRef);
    echo "\n\n";
    print_r ($_POST);
    echo "\n\n";
   
    die;
}

I edit an individual and I hope to see the form data displayed via this code. Whereas this code works if I use hook_civicrm_post or hook_civicrm_validate, with the 'pre' hook the objectId and objectRef are empty. Of course $_POST shows all of the data submitted, but it lacks the ids of the objects involved.

To explain why I am doing this, the task at hand is to keep a log of changes to Contact data only, with time stamps. The reason for this is because the Contact data is duplicated by another organization and they send their updated data twice a year to us. This log will then be used later to determine which address data is newer and update only that data.

So while the post or validate hooks give me all the data I need to create the log, they do that after the database was updated so I can't determine whether or not the address data was changed.

Is there something wrong with this hook code or am I using it incorrectly?

Thank you.

in edit mode you should get the contact_id (according to the code, if not please check and debug: CRM/Contact/Form/Edit.php, line 620)

My hook is being called in edit mode but I do not get the contact_id. If I edit \CRM\Utils\Hook.php and make the following change (starting at line 50--I simply replaced one line):

    static function pre( $op, $objectName, $id, &$params ) {
        $config =& CRM_Core_Config::singleton( );
        require_once( str_replace( '_', DIRECTORY_SEPARATOR, $config->userHookClass ) . '.php' );
        return   
            eval( 'return ' .
                  $config->userHookClass .
            //    '::invoke( 4, $op, $objectName, $objectId, $objectRef, $op, \'civicrm_pre\' );' ); 
                  '::invoke( 4, $op, $objectName, $id, $objectRef, $params, \'civicrm_pre\' );' ); 
    }

then I do get that id. I changed $objectId to $id but the truth is that I don't see where $objectRef is defined either--it comes thru as null for me so I think that may also be an error.

Let me know what you think.

Thank you.

> We dont have the objectRef in the pre hook, so we send the params

Makes sense.

> I've fixed 2.1 for this

Meaning this change will go out in 2.1.2?

> thanx for doing the fix.

My pleasure.

H