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.

Comments

harking’s picture

StatusFileSize
new3.06 KB

Updated patch, previous version was broken.

hotspoons’s picture

Status: Needs review » Reviewed & tested by the community

I tested the patch, and it improves the default behavior of the module. Can we get a commit?

TheCrow’s picture

Status: Reviewed & tested by the community » Fixed

Reworked and added, thank you!

Status: Fixed » Closed (fixed)

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

jlea9378’s picture

Status: Closed (fixed) » Active

So 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.

TheCrow’s picture

Hi 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

jlea9378’s picture

I 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.

TheCrow’s picture

Status: Active » Closed (fixed)

ok, consider this issue fixed.

About the language prefix, if someone need clarify it, better open another issue