-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Warning: Call-time pass-by-reference has been deprecated #1
Comments
What version of PHP? That ampersand shouldn't be there. There's no need to pass by reference in PHP5 if the variable is an object. SilverStripe tips, tutorials, screencasts and more: http://www.leftandmain.com Le Thursday, August 25, 2011 à 2:42 PM, Tuckie a écrit :
|
Its php 5.2.14, and your ampersand: https://github.com/unclecheese/KickAssets/blob/master/code/KickAssetAdmin.php#L567 |
Fixed! |
(Not a hard bug to fix, but...) Much thanks! |
Umm I just downloaded the latest version and there's still an ampersand in there. The code looks like this: |
5.3 wants it, 5.2 doesn't. That version compare should say >= 0. Sent from my iPhone On Aug 29, 2011, at 8:41 AM, smurkasreply@reply.github.com wrote:
|
This causes a fatal error for me in php 5.4. Fatal error: Call-time pass-by-reference has been removed in /path/to/kickassets/code/KickAssetAdmin.php on line 684 |
My install doesn't seem too happy with
$file->updateCMSFields(&$fields);
Seems to popup everywhere
The text was updated successfully, but these errors were encountered: