Hi,

This issue is caused because the language prefix in the URL is doubled up by the sort link.

The attached patch fixes just that.

CommentFileSizeAuthor
#6 1661552-6.patch670 bytesAnonymous (not verified)
#5 1764886-5.patch666 bytesAnonymous (not verified)
0001-Fixed-sort-links-in-multi-language-sites.patch847 bytesFranciscoLuz

Comments

FranciscoLuz’s picture

FranciscoLuz commented
Title: Sort links in multi-language sites that has language prefix in the URL lead to 404 » Sort links lead to 404 in multi-language sites that has language prefix in the URL
FranciscoLuz’s picture

FranciscoLuz commented
Status: Active » Needs review
Anonymous’s picture

Anonymous (not verified) commented
Version: 7.x-1.2 » 7.x-1.x-dev

Thanks, will review this soon.

$_GET['q']; (from current_path()) may not be available when using pretty facet paths. So need to test that.

dydave’s picture

dydave commented

I agree with the previous comment (Note#3) and think this issue and http://drupal.org/node/1741434
have a relationship at some point and especially for testing.

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new 1764886-5.patch666 bytes

Pretty Paths now has this function: http://drupal.org/node/1661552#comment-6447350
Attached an improved patch with logic for pretty facet paths (also removing ?f[0] queries etc.)

This patch requires the latest facetapi_pretty_paths dev-x / beta

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new 1661552-6.patch670 bytes

The solution from facetapi_pretty_paths is not stable yet. Please, for now, try this patch instead.

dydave’s picture

dydave commented
Status: Needs review » Needs work

Sorry morningtime, but this patch still fails for me.
After testing I get a similar issue as previously reported
the localhost/foldername/lang part of the URL gets doubled.... so I'm getting:
localhost/foldername/lang/foldername/lang/path/facet1/value1?sort=created&order ....

I'm just thinking exactly the same as you on this:
http://drupal.org/node/1741434#comment-6451916

I'm guessing we have to continue posting in facetapi pretty path:
http://drupal.org/comment/reply/1717424/6326290

So I'll follow up there.

Feel free to keep us updated on the status of the integration with facet pretty path.
Thanks!

Anonymous’s picture

Anonymous (not verified) commented

What the problem is, is that we need the original $_GET['q'] before it was altered by Pretty Facet Paths. #1661552: Get pretty path on any page from any module

But currently I have no clue how to do it.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Fixed

I presume this is now fixed in #1741434: Doesn't work with Facetapi Pretty Paths // needs improved logic to get path

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

grammar fixing