Hi,
I have path /something/{my_entity} and I am creating alias /something/[my_entity:title]. If I run bulk generation I will get the "..parameter was not converted for path" error message from ParamConverterManager and I am wondering if this issue with Drupal core(and therefore it is ok) or if this comes from Pathauto?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

ivanjaros created an issue. See original summary.

juampynr’s picture

@ivanjaros, can you provide a list of steps to reproduce the issue? Then I could debug it.

Saphyel’s picture

@juampynr I have the same issue, The steps to replicate is configure pathauto for use the URL "/news/[node:title]" to article, after this create a view with contextual filters "Content: Has taxonomy term ID" and finally try to create add a node .. you'll get the error. I can share with you a profile ready for test this.

Dave Reid’s picture

Status: Active » Closed (duplicate)

This sounds very similar to #2655874: Improve error handling for "The given alias pattern always matches the route" - let's merge this with that ticket.

Dave Reid’s picture

danielbeeke’s picture

I think this issue should still be open.

After testing '8.x-1.0-alpha2+5-dev' on 14 march I get

Drupal\Core\Entity\EntityStorageException: The "contact_form" parameter was not converted for the path "/contact/{contact_form}" (route name: "entity.contact_form.canonical") in Drupal\Core\Entity\Sql\SqlContentEntityStorage->save() (line 757 of core/lib/Drupal/Core/Entity/Sql/SqlContentEntityStorage.php).

It get's triggered on bulk update but also on a node save via the edit form.

danielbeeke’s picture

Status: Closed (duplicate) » Needs work

I am debugging and will update if I find results.

danielbeeke’s picture

I have created a patch and would like to know if this is the right way to fix this.

When ParamNotConvertedException is thrown the function returns false.

danielbeeke’s picture

Status: Needs work » Needs review
jeroen_drenth’s picture

Status: Needs review » Reviewed & tested by the community

I've tested the patch. Look good and it solves the issue.

alexpott’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs tests

We should have some tests for this...

Berdir’s picture

Status: Needs work » Postponed (maintainer needs more info)

Have a look at #2718375: Route collision reported when alias matches a different language, I hope that will solve your problem as well. If that's the case, please close this as a duplicate.

danielbeeke’s picture

I think #6 comment and issue is a little bit different from http://drupal.org/node/2718375.

The patch here fixes my issue. @Berdir have you read the patch from this issue?
It is simple and fixes one thing, the bug I had and I thought this issue was about.

Do you think it needs a different way of solving?

To clarify what I think this issue is about, if you have a node with an alias "/contact/about-us" drupal core contact modules tries to load the contact form "about-us" and if that form does not exist it throws ParamNotConvertedException (parameter was not converted for path)

Berdir’s picture

The other issue fixes that problem and many others as well because it doesn't do route matching or upcasting anymore.

This just fixes one symptom, the other issue attempts to solve the underlying problem.

Saphyel’s picture

#14 if this issues is older, why is this one the duplicated?

danielbeeke’s picture

@Berdir, I will test the issue tomorrow and report back to this issue.

alexpott’s picture

Anonymous’s picture

Saphyel: that's quite common practice here. It takes credit from the original poster(s) but live discussion and patches have always priority.

Arla’s picture

I had the same problem, tested the patch in #2718375: Route collision reported when alias matches a different language successfully.

Berdir’s picture

Status: Postponed (maintainer needs more info) » Closed (duplicate)
Issue tags: -Needs tests

I've committed that issue now. Thanks everyone who tested, I'm pretty sure this fixed all the related problems around that error.