Hello, wonderful module. I migrated over from Taxonomy Breadcrumb because of the new Views breadcrumbs.
But I've ran into a problem. Using Taxonomy Breadcrumb module I could add a path to a view with an exposed
filter selected. i.e
example.com/node?filter=1

but custom breadcrumbs is changing it to

example.com/node%3Ffilter%3D1

Is there a way I can change this behavior in the code? I searched but couldn't find it..

CommentFileSizeAuthor
#3 543968_parse_url_patch.diff2.12 KBMGN

Comments

drupalfo’s picture

I found this, not sure if it is related. But it is a patch for 5.x

http://drupal.org/node/217875

drupalfo’s picture

OK , the problem is using relative paths...

node?filter=1 becomes node%3Ffilter%3D1

but example.com/node?filter=1 works and does not convert the URL.

I also realized this is the ssame with Taxonomy Breadcrumb module.

However , I do think you should consider adding this as a feature because you should be able to link
to a view with something already selected.

MGN’s picture

Status: Active » Needs review
StatusFileSize
new2.12 KB

This patch implements parse_url (as originally suggested in #217875: Allow Query String in paths ) to properly handle all parts of the url. Can you try this and see if it solves the problem?

Thanks.

MGN’s picture

Title: Breadcrumb for Path changes URL with query unlike Taxonomy Breadcrumb module » Provide support for breadcrumbs with query strings
Component: custom_breadcrumbs_paths » Code
Category: support » feature
Status: Needs review » Fixed

Seems to work well on my test sites. Committed to 6.x-2.x-dev http://drupal.org/cvs?commit=254122. Reopen if this has not fixed the original problem.

Status: Fixed » Closed (fixed)

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

lotyrin’s picture

Version: 6.x-2.x-dev » 7.x-2.x-dev

This appears to have been broken in 7.x.

lotyrin’s picture

Status: Closed (fixed) » Active
lotyrin’s picture

Assigned: Unassigned » lotyrin
lotyrin’s picture

Status: Active » Closed (fixed)

Nevermind, it's 7.x-1.x that has the issue.

Any plans on releasing 7.x-2.x?