Hello,

Thanks for such nice module.
I have question/promlem. I want to add GET parameter to the url of the path for one breadcrumb link.
I do something like

browse-shop/all?field_material2_tid=%nid:field-material2:0:tid

but as a result I got this

/browse-shop/all%3Ffield_material2_tid%3D1966

Of course I can change %3F to "?" and %3D to "=" with jQuery, but maybe you have better solution?

Thank you!
Denis

CommentFileSizeAuthor
#3 path_breadcrumbs-arg_from_get-2140497-2.patch3.57 KBskorzh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kalabro’s picture

kalabro
she/her
CreditAttribution: kalabro commented
Status: Active » Postponed (maintainer needs more info)

Thank you for your request, Denis!
Unfortunately I can't reproduce the bug on 7.x-3.0-beta6. Are you sure you are using latest release and there are no errors on admin/reports/status page?
Please, provide more info about Ctools, Token, Entity Token versions. Attach your PB export here.

You also can test latest dev release where GET params support was improved.

skorzh’s picture

skorzh
Primary language Russian
Location Belarus
CreditAttribution: skorzh commented

I created a patch that allows you to use arguments from GET params, for example you can use "node/add/article?og_group_ref=%node" path and at arguments tab you should to see "node" argument, and you can use this argument at breadcrumbs tab.

skorzh’s picture

skorzh
Primary language Russian
Location Belarus
CreditAttribution: skorzh commented
FileSize
path_breadcrumbs-arg_from_get-2140497-2.patch3.57 KB
Spleshka’s picture

Spleshka
he/him
Primary language English
Location UAE
CreditAttribution: Spleshka commented
Status: Postponed (maintainer needs more info) » Needs review
skorzh’s picture

skorzh
Primary language Russian
Location Belarus
CreditAttribution: skorzh commented

Any updates here?

Spleshka’s picture

Spleshka
he/him
Primary language English
Location UAE
CreditAttribution: Spleshka commented

Let me return from a vocation, i will review and commit this :)

skorzh’s picture

skorzh
Primary language Russian
Location Belarus
CreditAttribution: skorzh commented

I'm really looking forward this Ж)

kalabro’s picture

kalabro
she/her
CreditAttribution: kalabro commented
Version: 7.x-3.0-beta6 » 7.x-3.x-dev
Component: User interface » Code
Category: Bug report » Feature request

Looks like feature request.

Spleshka’s picture

Spleshka
he/him
Primary language English
Location UAE
CreditAttribution: Spleshka commented

I've reviewed the patch, but I am not sure if it is stable. I mean that we with @kalabro not sure that nothing will break if we commit it. So we will take some time for manual testing.

milos.kroulik’s picture

milos.kroulik CreditAttribution: milos.kroulik commented

Unfortunately, patch from #3 no longer applies on version 3.0. The error from git apply was:

error: patch failed: path_breadcrumbs.module:82
error: path_breadcrumbs.module: patch does not apply
ArtuDrop’s picture

ArtuDrop CreditAttribution: ArtuDrop as a volunteer commented

Hello,

Thank you for the module.
Now, I feel myself better with breadcrumbs.

I have the same issue.
GET parameters do not work in my case.
For example "booking-webform/event=%event"
where %event is Node ID.

Artu

edutrul’s picture

edutrul CreditAttribution: edutrul as a volunteer and at Anexus commented
Status: Needs review » Needs work
liezie_D’s picture

liezie_D CreditAttribution: liezie_D commented

I have the same issue.
My path is: http://www.example.com/productSearch?searchTerm=blablabla

I created a new breadcrumb path on productSearch?searchTerm=%myterm
In the configuration %myterm is not accesible.
Also, when i surf to http://www.example.com/productSearch?searchTerm=blablabla the configured breadcrumb path is not shown. instead i just get "home".

How can i fix this?

UPDATE: i was able to trigger the breadcrumb by using path "productSearch". Which improves my situation. But I am still unable to use the info from the searchterm.