Use request directly instead of handling a full request

Siegrist → changed the visibility of the branch 3426428-use-request-directly to hidden.

Ok, this solution passes the test. Removing the unused code.

I hope I haven't deleted any functionality. Would be nice to get this committed. Thanks

Updating target version to 1.2.x as the current supported branch.

This looks like it conflicts with the 1.2.x branch.

I think this change is a really good idea.
The original MR 13 was done on the 1.1.x branch rather than an issue specific branch, so I've copied that modified 1.1.x branch to 3426428_v2 branch, merged in the 1.2.x branch and fixed the merge conflicts.

It looks like we do now have test failures to address before we can proceed.

eli-t → closed merge request !13

There are two issues with this proposal at the moment.

1) it doesn't take account of the use_relative_urls configuration setting for the module
2) it breaks when viewing the content through JSON:API; that is the breadcrumbs refer to the JSON:API path

1) is not surprising as the initial work on this issue was done before that option was merged.

However, 2) is more of a fundamental problem with this approach - this is a breaking change, and it is difficult to see how this would ever be the wanted behaviour when viewing content through JSON:API.

For example - here is the jsonapi output on the main branch for an example page using absolute links

breadcrumbs	
  0	
    uri	"https://recommended-project.ddev.site/"
    title	"Home"
    options	[]

With this change it looks like this

breadcrumbs	
  0	
    uri	"https://recommended-project.ddev.site/"
    title	"Home"
    options	[]
  1	
    uri	"https://recommended-project.ddev.site/jsonapi"
    title	"Jsonapi"
    options	[]