CiviCRM Community Forums (archive)

*

News:

Have a question about CiviCRM?
Get it answered quickly at the new
CiviCRM Stack Exchange Q+A site

This forum was archived on 25 November 2017. Learn more.
How to get involved.
What to do if you think you've found a bug.



  • CiviCRM Community Forums (archive) »
  • Old sections (read-only, deprecated) »
  • Support »
  • Using CiviCRM »
  • Using Drupal Modules (Moderator: Donald Lobo) »
  • civicrm_member_roles_user_logout issue
Pages: [1]

Author Topic: civicrm_member_roles_user_logout issue  (Read 604 times)

chriscant

  • I post occasionally
  • **
  • Posts: 66
  • Karma: 4
    • PHDCC
  • CiviCRM version: 4.2.19
civicrm_member_roles_user_logout issue
December 06, 2011, 06:12:42 am
Using Drupal 7.09 and CiviCRM 4.07 this warning appears in the log when someone logs out:

Warning: Parameter 1 to civicrm_member_roles_user_logout() expected to be a reference, value given in module_invoke_all() (line 818 of /var/sites/wukdev7/includes/module.inc).

function civicrm_member_roles_user_logout( &$account ) {
    civicrm_member_roles_sync_user( $account );
}


The Drupal 7 spec for hook user_logout has no parameters.  Not sure what to suggest as a fix.  Any suggestions?

Thanks Chris

Donald Lobo

  • Administrator
  • I’m (like) Lobo ;)
  • *****
  • Posts: 15963
  • Karma: 470
    • CiviCRM site
  • CiviCRM version: 4.2+
  • CMS version: Drupal 7, Joomla 2.5+
  • MySQL version: 5.5.x
  • PHP version: 5.4.x
Re: civicrm_member_roles_user_logout issue
December 06, 2011, 09:06:04 am

checked the docs:

http://api.drupal.org/api/drupal/modules--user--user.api.php/function/hook_user_logout/7

it takes one parameter $account

lobo
A new CiviCRM Q&A resource needs YOUR help to get started. Visit our StackExchange proposed site, sign up and vote on 5 questions

chriscant

  • I post occasionally
  • **
  • Posts: 66
  • Karma: 4
    • PHDCC
  • CiviCRM version: 4.2.19
Re: civicrm_member_roles_user_logout issue
December 06, 2011, 09:25:26 am
Ah yes, I was looking in the wrong place.

Anyway, the civi ampersand & needs removing.

Ta Chris

Pages: [1]
  • CiviCRM Community Forums (archive) »
  • Old sections (read-only, deprecated) »
  • Support »
  • Using CiviCRM »
  • Using Drupal Modules (Moderator: Donald Lobo) »
  • civicrm_member_roles_user_logout issue

This forum was archived on 2017-11-26.