Hi, downloaded this mod to test it out and received the following error upon install:

Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of module_invoke_all(). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file in /modules/ubercart_marketplace/mp_products/mp_products.module on line 219

Thanks!

Comments

Turgrid’s picture

Turgrid commented

I have never seen this error before. Not sure what to tell you or do. I will take a look at the code in that vicinity, anyways. I guess one quick solution would be to set allow_call_time_pass_reference to true in your php.ini file.

cumber8’s picture

cumber8 commented

I'm getting the exact same error.

Turgrid’s picture

Turgrid commented

If your php config does not allow pass by reference (and/or you do not wish to enable in php.ini), you can safely remove line 219 from mp_products.module. This is a hook that has a potential future use to some individuals and its quality and removal will be assessed for the next Marketplace release.

drew.p’s picture

drew.p commented

I too am getting this error:

Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of module_invoke_all(). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file in /home/.testaroser/userinternal/drupalsite/sites/all/modules/contribs/_ubercart/ubercart/contrib/ubercart_marketplace/mp_products/mp_products.module on line 219

Is this related to PHP version?

Turgrid’s picture

Turgrid commented

The solution has already been posted (See #3):

"You can safely remove line 219 from mp_products.module. This is a hook that has a potential future use to some individuals and its quality and removal will be assessed for the next Marketplace release."

It is a PHP5 issue that will be corrected next release. For now remove the hook_invoke_all line 219.

syndicateStorm’s picture

syndicateStorm commented

Looks like this may have been resolved back in 2008. I will look into this to see if it is still applicable when I get a chance.

syndicateStorm’s picture

syndicateStorm commented
Status: Active » Closed (fixed)

The fix was committed November 8, 2008.