I recently noticed that when you try to make a redirect for a bogus url (politics/fdjhafsdjjhdkfsalksdf) it returned this error. This is because Drupal will still load that url (as /politics) even though the path is not the correct path.
To fix this bug, I merely checked to ensure that the path provided by menu_get_item($element['#value']) was actually the value that was being passed.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | 648500-override-existing-paths.patch | 764 bytes | mrfelton |
| #4 | path_redirect-648500.patch | 807 bytes | kmonty |
| path_redirect-alias.patch | 525 bytes | kmonty |
Comments
Comment #1
kmontyfor the record, /politics was a view. This also happens at node/12/fjkdakjfdskjsdfakjsfda
Comment #2
dave reidCan you please re-upload your patch. I'm not sure if this would be fixed with #615008: "Forced" path redirect - override the existing path or alias.
Comment #3
kmontySorry, I don't know why that patch failed. This is a different issue than the force issue, although forcing would be a solution. The reason this is different is because I am attempting to redirect unregistered paths yet the module still denies me the privilege of redirecting them.
Additionally, this is an easy fix whereas the force option might not be available for some time.
Comment #4
kmontyWhoops
Comment #5
marcus0263 commentedThanks kmonty, this resolved my issue
http://drupal.org/node/647166
Comment #6
jim kirkpatrick commentedWorks for me too, updating.
Comment #7
alrueden commentedThis patch resolved my problem too (Path redirect would not recognize my number one 404, kept insisting it was valid already).
However, if you're using the most current update of this version, the relevant code is actually not in path_redirect.module. It's now in path_redirect.admin.inc. So you have to download path_redirect.admin.inc, change the one line, and re-upload it. Hopefully this will be fixed soon.
Comment #8
xsean commentedi tried to update the patch but i did not found function path_redirect_validate_source_f
or even the existing code in path_redirect.module and path_redirect.admin.inc
// Cannot create redirects for valid paths.
$menu_item = menu_get_item($element['#value']);
- if ($menu_item && $menu_item['page_callback'] != 'path_redirect_goto') {
i'm using version 6.x-1.0-beta5. is it only apply to certain version of path redirect?
Comment #9
chrisshattuck commentedI've created a version of this patch for 6.x-1.0-beta5 here: http://drupal.org/node/708484. Cheers!
Comment #10
mrfelton commentedUpdated patch applies to Drupal latest stable D6 version (none of the others applied for me)
Comment #11
yan commented#10 worked for me.
Comment #12
dave reidIs there a path in core (without having to install Views, etc) that can be tested to make sure this bug doesn't regress?
Comment #13
introfini commentedsubscribing
Comment #14
kmonty@David Reid
you can try to alias the path node/1/randomtextthatisneverused
Comment #15
jordanmagnuson commented#4 worked for me. Thanks. I would love for this to be rolled.
Comment #16
Rainman commentedGood fix, another worked for me... big issue with ubercart catalog views
Thanks!
Comment #17
Mac Clemmens commentedStrongly recommend committing this patch. Have upgraded this module a few times and keep having to re-patch it.
Comment #18
vood002 commentedI second committing this patch...I've experienced no bugs when using the patched module, and have had to repatch multiple times.
Comment #19
apshoemaker commentedJust wanted to confirm that #10 worked for me too. Please consider adding to next release.
Comment #20
dave reidThank you. Committed #10 to CVS.
http://drupal.org/cvs?commit=402790