Posted by Beltanin on May 6, 2011 at 7:37pm
11 followers
| Project: | NodeStream |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | major |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
| Issue tags: | PHP 5.3 compatibility |
Issue Summary
From a clean install of nodestream... From login, create article and the following error occurs.
warning: Parameter 2 to ns_promo_crossclone_node_alter() expected to be a reference, value given in /var/aegir/platforms/ns/includes/common.inc on line 2892.
warning: Parameter 2 to ns_core_crossclone_node_alter() expected to be a reference, value given in /var/aegir/platforms/ns/includes/common.inc on line 2892.
When checking the promo, the node-reference to the article is missing. Or am I?
Comments
#1
I have exactly the same problem, with same error.
#2
yeah, i get the same error:
warning: Parameter 2 to ns_promo_crossclone_node_alter() expected to be a reference, value given in /srv/www/xxxxxx.com/nodestream/includes/common.inc on line 2892.
#3
I get the same error as well. The whole idea with the Promo feature goes away if this doesn't work. That feature, and Nodestream, seems like great products, so I hope this will be looked into.
#4
I'm also having the issue above. Has anyone figured it out?
#5
I am also having the same issue, and now I can no longer post an article (500 error). I thought perhaps the theme I added after it caused the article posting issue, but nope... I can create blogs and everything else, but not an article.
If anyone gets an update or has the same issue, please update everyone on here. I emailed the developers, so we'll see if they respond quickly.
#6
This might be an issue related to PHP version. What version of PHP are you running?
#7
PHP Version 5.3.2
I seriously doubt its a PHP issue, because everything works with the exception of creating an article or editing the single article I made. I can create blogs and everything else Nodestream has setup stock. Its not a permission issue, either, I looked into that.
#8
I was just informed by someone at Nodestream that this may actually be a PHP issue, as Nodestream is currently designed to work with PHP 5.2, not 5.3. I will see if it is possible to downgrade my PHP version to 5.2 and report back if that solves it.
#9
subscribing
#10
This is indeed a PHP 5.3 issue. We have been working really hard on getting the drupal 7 version out the door, that is why we haven't been able to work on this issues. I'm tagging this issue properly so that we can get to it.
#11
I spent more time than I expected tracking the error down. Turns out this issue existed with several modules and the issue is actually with the crossclone module. Some of the discussions and different fixes:
http://drupal.org/node/925580#comment-4777844
http://drupal.org/node/705512#comment-2586014
http://drupal.org/node/930992#comment-4066840
Those fixes deal with altering the $form_state before calling drupal_alter which is not exactly what the crossclone module is doing. I tried updating crossclone, but didn't have any luck. Maybe someone more familiar with how the values are being passed can figure that out.
This patch to common.inc is ugly, but solves the problem...
http://drupal.org/node/710892#comment-3884210
#12
We actually don't need to get these parameters by reference. This issue should be fixed here:
http://drupalcode.org/project/ns_core.git/commit/b149385
#13
Automatically closed -- issue fixed for 2 weeks with no activity.