Posted by irakli on March 18, 2010 at 12:12am
Attached file provides significant performance improvement and is a recommended patch for OP 1.7.1 installations.
Just overwrite the original noderelationshops.inc file under sites/all/modules/noderelationships with the one attached here. You will have to rename attached file, removing the extra "_.txt" at the end. Sorry, drupal.org security rules, made us add that.
Make sure to clear Drupal cache (from devel or via http://yourdomain.com/admin/content/clearcache) after you update the file, otherwise you will start getting some erros.
We will release an updated 1.7.1 build soon, too.
| Attachment | Size |
|---|---|
| noderelationships.inc_.txt | 46.22 KB |
Comments
Applied
Thanks. I've applied the patch.
sTattler.com
Applied but a problem
Thanks. Always love "significant" performance improvements. :) Just an FYI that I applied this, cleared cache several times but still got the follow when viewing individual nodes. I've backed this out. All contrib modules are updated to current versions via Drush earlier today. (That may be the issue.)
warning: Wrong parameter count for debug_backtrace() in /srv/www/xxxxxxxxxxxxx/public_html/sites/all/modules/noderelationships/noderelationships.inc on line 79.
warning: Invalid argument supplied for foreach() in /srv/www/xxxxxxxxxxxxxxx/public_html/sites/all/modules/noderelationships/noderelationships.inc on line 81.
user warning: Table 'c2do.noderelationships_default_settings' doesn't exist query: SELECT type_name, relation_type, related_type, field_name, settings, export_key FROM noderelationships_default_settings WHERE type_name = 'article' in /srv/www/xxxxxxxxxxxxxxx/public_html/sites/all/modules/noderelationships/noderelationships.inc on line 55.
As far as I guess updating
As far as I guess updating the modules to the latest version is the reason. In particular, feature module should not be upgraded.
sTattler.com
@tech4him, which version of
@tech4him,
which version of PHP are you running on the server?
.............................................
http://twitter.com/inadarei
Looks like: PHP
Looks like: PHP 5.2.4-2ubuntu5.10 with Suhosin-Patch 0.9.6.2 (cli) (built: Jan 6 2010 22:01:14)
Blessings!
It looks like, for some
It looks like, for some reason, PHP is complaining about the argument FALSE passed to debug_backtrace() call in noderelationships.inc on line 79. Which is weird, since PHP 5 should allow for the argument. Either way you can delete FALSE and pass no arguments if you wish. That argument is not critical at all.
In any case, we found more space for further performance improvements in the same module and will be releasing an updated patch later today or tomorrow (which will not have backtrace() call) so you can also roll back and wait for that, if you wish.
Thank you!
.............................................
http://twitter.com/inadarei
Thanks irakli! Sounds Good.
Thanks irakli! Sounds Good. I've already rolled back. No harm, no foul. Will try the improved version when it comes out.
Thanks for all your efforts!
I've been scratching my head
I've been scratching my head with this problem for the past two hours. I'm glad you're on top of these things. Thk.
EDIT: I just noticed the time you posted this. Well, somebody is burning the midnight oil :)
Marine job board with Drupal 7 at http://windwardjobs.com
I've updated to this version
I've updated to this version and looks ok if you change the thing that irakli says :)
So $dbt = debug_backtrace(FALSE); will be $dbt = debug_backtrace();
Aye, my head is spinning.
Aye, my head is spinning. Should I wait for a full version release? Or take my chances with the $dbt = debug_backtrace(FALSE); will be $dbt = debug_backtrace();
It's a coin toss at this point, eh? Advice is most welcome.
You should install the new
You should install the new patch: http://groups.drupal.org/node/57453
.............................................
http://twitter.com/inadarei
Applied the patch
I've applied the patch. Thanks for the update.
sTattler.com