Posted by lejon on July 30, 2008 at 1:36pm
Jump to:
| Project: | Forward |
| Version: | 5.x-1.6 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (cannot reproduce) |
Issue Summary
Hi,
Forward Module was working before. I have recently updated a load of modules, including Listhandler as well as the Drupal 5.9 upgrade. Now when I click on the Forward link I get page not found.
Any ideas why?
Type page not found
Date Wednesday, July 30, 2008 - 15:20
User admin
Location http://www.gowithmebook.org/main/forward/1068%2526cid%3D69
Referrer http://www.gowithmebook.org/main/node/1068
Message 1068&cid=69
Severity warning
Comments
#1
Try execute the database update script. I also upgrade from previous modules and found 5.x-1.14 has new database schema.
#2
Just realised that this is only the case for 'Email this comment' links.
The 'Email this comment' link has a couple of % signs in it, which is not the case for 'Email this page' links:
http://www.gowithmebook.org/main/forward/992%2526cid%3D40
Which produces 'page not found'
Any ideas?
#3
Ouch, looks like this whole chunk of code was pretty badly broken (but you'd only see the problem on comment links with the option to show the node type in the link enabled). I've now got it setting the link correctly, but it loses the comment info on the form (it'll forward the whole page but not the comment). I'll have to fix the second half of this later, don't have time to do it today. You'll have to get the 5.x-dev version after Drupal updates the package - I won't do a new release until I get the other half of it fixed.
#4
#5
I'm having the same problem of getting Page Not Found's when forwarding nodes (not just comments), but I'm using the Share module and have it set with the Forward form as a tab in the Share popup. I get Page Not Found's when trying to use the form from teaser and full node view. The problem apparently started when I upgraded to 1.14 (because I have a dev site using 1.13 and it works fine), and is still there with 1.16.
Are there other configuration settings that would effect this?
#6
Could you please pot a link to an accessible page with the form on it? The link above gives access denied.
#7
seanr: I realize you were asking about the link above but in case it's helpful here is a link to a page with my example.
http://bitchmagazine.org/article/hard-times
Scroll to the bottom of the article and click on the Share link.
Thanks
#8
Any update on this?
#9
I'm having the same problem, and I'm using Share tabs. Any developments?
#10
I tried setting up Share but I can't get the share popup to work at all (the link shows, but clicking it does not invoke the popup and instead leads to a 404. I've made more changes to the module, so go ahead and test it again. If anyone has any insights into how to get the popup to work or what's breaking the link (assuming it's still broken in CVS DRUPAL-5), I'd appreciate some help here.
My sandbox is here, BTW: http://drupal5.ngphost.com/ - sharethis works, share doesn't.
#11
I've not got Share, but Forward submits to a Page Not Found and therefore does not send an email.
Updated to 5.x-1.17 and it appears to have done this from the update, was working fine beforehand (I think). I might roll back if there are no fixes, because at the moment it's just not working.
From having a quick look a the code, it appears the forward_form_submit() is running a menu_execute_active_handler() and applying the result to the body property of the $content object, this is returning not found, therefore on line 651 of forward.module, we are returned a page not found response...
http://api.drupal.org/api/function/menu_execute_active_handler/5 - menu_execute_active_handler appears to be causing the issue, I don't want to guess anymore so does anybody else have any ideas?
Thanks,
Luke
#12
i am not doing quite the same thing; but still hitting the same place in forward.module where it bails with 404. i am using latest Dd6 dev version of this module.
if i were to hazard a guess i would say this code is not pathauto friendly.
#13
I still can't reproduce this with any version. Please update to the latest code, make sure you're using Forward's automatically generated links and not any custom code in your theme or elsewhere, and try it again.
#15