On the nodereferrer page, I get just one oval with the node name and not the referring node name, with this error:
warning: Invalid argument supplied for foreach() in /var/www/html/test/sites/all/modules/graphviz_noderef/graphviz_noderef.module on line 328.

the CCK export is attached, the error appears on Conference Proceedings type.

CommentFileSizeAuthor
cck.txt6.74 KBdeltab
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

naught101’s picture

naught101 CreditAttribution: naught101 commented
Priority: Normal » Critical

hrm, I just updated the module, and the line numbers have changed. Reckon you could either copy that line here, and/or re-checkout the module? there's a lot of foreaches around there.

naught101’s picture

naught101 CreditAttribution: naught101 commented

ahh, you need this patch: http://drupal.org/node/726450#comment-2668832 . if it works, please set that issue to "reviewed and tested by the community"

deltab’s picture

deltab CreditAttribution: deltab commented

No joy. I patched on nodereferrer DEV, and I checked out the new Graphviz Noderef. It is still line 328.

deltab’s picture

deltab CreditAttribution: deltab commented
Status: Active » Reviewed & tested by the community

Nodereferrer version 6.x-1.0-rc1 (2010-Feb-12) works, without the patch.

naught101’s picture

naught101 CreditAttribution: naught101 commented

That's strange, it shouldn't. I'm gonna leave this open for a while, see what's going on...

deltab’s picture

deltab CreditAttribution: deltab commented

Okay, the error is back, after I added more fields, so upgraded to -dev nodereferrer (the patch didn't apply on rc1), patched it.

Now I see the error three times if there are three nodereferences on the node, once if there is only one nodereference.

So I cleared the cache (admin/settings/performance) and no more error!

naught101’s picture

naught101 CreditAttribution: naught101 commented

Schweet. I'll leave this open until #726450: Find nodereference field name from nodereferrer field is sorted, in case others hit this problem.

deltab’s picture

deltab CreditAttribution: deltab commented

Okay, but only the admin can see the graph - how do I make it visible for the other users? Nothing suggests itself in the permissions UI.

naught101’s picture

naught101 CreditAttribution: naught101 commented

deltab: user permissions? admin/user/permissions Sorry, I just fixed that :)

Might not be in the latest dev release yet, try the latest CVS.

naught101’s picture

naught101 CreditAttribution: naught101 commented
Version: » 6.x-1.x-dev
Status: Reviewed & tested by the community » Fixed

Ok, andypost has commited the patch to nodereferrer. fixed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.