The forward service should instead default to the current url that the user is viewing.
The attached patch modifies the forward service to do this.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | forward-services.relative-url.diff | 3.06 KB | harking |
| forward-services.module.diff | 632 bytes | harking |
Comments
Comment #1
harking commentedUpdated patch, previous version was broken.
Comment #2
hotspoons commentedI tested the patch, and it improves the default behavior of the module. Can we get a commit?
Comment #3
TheCrow commentedReworked and added, thank you!
Comment #5
jlea9378 commentedSo I just installed the dev version of this module in an effort to fix this problem and it partially solved the problem. In the stable release of the module it was doing a weird thing where it was prefixing the URL with "node". So "about-ccc/press-releases/solar-power-cccs-jerome-ave-campus" became "node/about-ccc/press-releases/solar-power-cccs-jerome-ave-campus" which of course didn't work. Now in the dev version the node prefix is gone but now it is taking me to the spanish version of the forward page: "/es/forward?path=about-ccc%2Fpress-releases%2Fsolar-power-cccs-jerome-ave-campus"
It's strange because I'm using the site in English. I noticed the Bookmark service link also has an /es/ on it.
Comment #6
TheCrow commentedHi Jlea9378
the patch attached here should be considered outdated and not valid because affected by the same problem you noticed (node/...), this bad behaviour has been fixed afterwards (#988632: Page not found because "node" added to URL).
About the language prefix tell me if this help you: #357185: template base_path() code - breaking il8n menu links
Comment #7
jlea9378 commentedI turned off the spanish language and all is well. We weren't using it anyways. I just wanted to bring it up in case it was a bug in this module. Thanks.
Comment #8
TheCrow commentedok, consider this issue fixed.
About the language prefix, if someone need clarify it, better open another issue