The function below was added in the most recent version. This appears to erase the node url for nodes of all types when it should (I assume?) just erase the node url for weblink nodes. This means that anywhere a node is displayed (for example, the front page of the site) the title just links back to the current page. Commenting out the unset line clears up the problem.
function weblinks_preprocess_node(&$variables) {
unset ($variables['node_url']);
}
Comments
Comment #1
obris commentedI can confirm this on my website :-( (http://www.drupeda.net)
I just searched the lines to uncomment but i could not find them. Can you please tell me where i can find them? Thank you very much.
Comment #2
nancydruI believe most, if not all of #582280: multiple issues after upgrade is also a duplicate of this.
That code was added because of #579208: Make fields available to CCK and I didn't see it doing what I wanted and should have removed it. However, now that you tell me what it is really doing, it opens an opportunity to restore an option that used to exist; I just have to check for node type properly. This is why we have -dev versions, but no one caught it, including me.
To get you going again, commenting that line (or the whole function) will do it.
@obris: search for the function name in weblinks.module and it will show up.
Comment #3
obris commentedThank you very much Nancy for your hint and abatishko for the fix. It works again without a problem.
Comment #4
nancydruI committed a change that not only fixes this problem but allows the node title to link to the URL (from another issue).
Comment #5
Anonymous (not verified) commentedWhich file needs to be edited to remove the code in #1
Gary
Comment #6
nancydruLook at the the last line in #2. Or download the -dev version, with the permanent fix.
Comment #7
Anonymous (not verified) commentedThanks NancyDru.
Gary
Comment #8
fenstratWow. Bit of a show stopper this one. Took 2 hours to track down why node links on all our site's home page's were all broken and pointing back at the homepage itself.
Can you package up a new release?
Comment #9
rmiddle commentedWe are planing a release very soon. Just need to do a little testing should have one released by this weekend.
Thanks
Robert
Comment #10
nancydruThe more testing that is done, the faster a new release goes out.
Comment #11
sinasalek commentedI've just tested the latest dev snapshot , i had problem with node title url and it's fixed now. everything else seems fine as well.
Thanks
Comment #12
nancydruThanks. Did you happen to test if setting the node title to go to the site works for you?
Comment #13
sinasalek commentedI just did, and it works
Comment #14
obris commentedI also made a test with -dev and it works. Tested on a spare installation with Drupal 6.14. I hope my feeback helps.
Comment #15
nancydruIncluded in 6.x-2.3.